* [bug report] regulator: da9121: Add device variant regmaps
@ 2020-12-04 15:04 Dan Carpenter
2020-12-07 17:08 ` Adam Ward
0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2020-12-04 15:04 UTC (permalink / raw)
To: kernel-janitors
Hello Adam Ward,
The patch c860476b9e3a: "regulator: da9121: Add device variant
regmaps" from Nov 30, 2020, leads to the following static checker
warning:
drivers/regulator/da9121-regulator.c:869 da9121_check_device_type()
error: uninitialized symbol 'name'.
drivers/regulator/da9121-regulator.c
811 static int da9121_check_device_type(struct i2c_client *i2c, struct da9121 *chip)
812 {
813 u32 device_id;
814 u8 chip_id = chip->variant_id;
815 u32 variant_id;
816 u8 variant_mrc, variant_vrc;
817 char *type;
818 const char *name;
^^^^^^^^^^^^^^^^
819 bool config_match = false;
820 int ret = 0;
821
822 ret = regmap_read(chip->regmap, DA9121_REG_OTP_DEVICE_ID, &device_id);
823 if (ret < 0) {
824 dev_err(chip->dev, "Cannot read device ID: %d\n", ret);
825 goto error;
826 }
827
828 ret = regmap_read(chip->regmap, DA9121_REG_OTP_VARIANT_ID, &variant_id);
829 if (ret < 0) {
830 dev_err(chip->dev, "Cannot read variant ID: %d\n", ret);
831 goto error;
832 }
833
834 if (device_id != DA9121_DEVICE_ID) {
835 dev_err(chip->dev, "Invalid device ID: 0x%02x\n", device_id);
836 ret = -ENODEV;
837 goto error;
838 }
839
840 variant_vrc = variant_id & DA9121_MASK_OTP_VARIANT_ID_VRC;
841
842 switch (variant_vrc) {
843 case DA9121_VARIANT_VRC:
844 type = "DA9121/DA9130";
845 config_match = (chip_id = DA9121_TYPE_DA9121_DA9130);
846 break;
847 case DA9220_VARIANT_VRC:
848 type = "DA9220/DA9132";
849 config_match = (chip_id = DA9121_TYPE_DA9220_DA9132);
850 break;
851 case DA9122_VARIANT_VRC:
852 type = "DA9122/DA9131";
853 config_match = (chip_id = DA9121_TYPE_DA9122_DA9131);
854 break;
855 case DA9217_VARIANT_VRC:
856 type = "DA9217";
857 config_match = (chip_id = DA9121_TYPE_DA9217);
858 break;
859 default:
860 type = "Unknown";
861 break;
862 }
863
864 dev_info(chip->dev,
865 "Device detected (device-ID: 0x%02X, var-ID: 0x%02X, %s)\n",
866 device_id, variant_id, type);
867
868 if (!config_match) {
869 dev_err(chip->dev, "Device tree configuration '%s' does not match detected device.\n", name);
^^^^
"name" is never set. What did you want it to be?
870 ret = -EINVAL;
871 goto error;
872 }
873
874 variant_mrc = (variant_id & DA9121_MASK_OTP_VARIANT_ID_MRC)
875 >> DA9121_SHIFT_OTP_VARIANT_ID_MRC;
876
877 if ((device_id = DA9121_DEVICE_ID) &&
878 (variant_mrc < DA9121_VARIANT_MRC_BASE)) {
879 dev_err(chip->dev,
880 "Cannot support variant MRC: 0x%02X\n", variant_mrc);
881 ret = -EINVAL;
882 }
883 error:
884 return ret;
885 }
regards,
dan carpenter
^ permalink raw reply [flat|nested] 2+ messages in thread
* RE: [bug report] regulator: da9121: Add device variant regmaps
2020-12-04 15:04 [bug report] regulator: da9121: Add device variant regmaps Dan Carpenter
@ 2020-12-07 17:08 ` Adam Ward
0 siblings, 0 replies; 2+ messages in thread
From: Adam Ward @ 2020-12-07 17:08 UTC (permalink / raw)
To: kernel-janitors
On Fri, 4 Dec 2020 15:05, Dan Carpenter wrote:
> 818 const char *name;
> ^^^^^^^^^^^^^^^^
> 868 if (!config_match) {
> 869 dev_err(chip->dev, "Device tree configuration '%s' does not
> match detected device.\n", name);
> ^^^^
> "name" is never set. What did you want it to be?
Ahh! Thought I'd removed that - thank you, another fine catch :-)
Regards,
Adam
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-07 17:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-04 15:04 [bug report] regulator: da9121: Add device variant regmaps Dan Carpenter
2020-12-07 17:08 ` Adam Ward
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.