Re: [PATCH] iio: adc: handle unknow of_device_id data

From: Marek Vasut
Date: Tue Feb 07 2017 - 20:22:56 EST


On 02/05/2017 10:32 AM, Jonathan Cameron wrote:
> On 04/02/17 21:47, Marek Vasut wrote:
>> On 02/04/2017 10:44 PM, Arnd Bergmann wrote:
>>> On Sat, Feb 4, 2017 at 10:11 PM, Marek Vasut <marek.vasut@xxxxxxxxx> wrote:
>>>> On 02/03/2017 06:01 PM, Arnd Bergmann wrote:
>>>>> diff --git a/drivers/iio/adc/rcar-gyroadc.c b/drivers/iio/adc/rcar-gyroadc.c
>>>>> index 0c44f72c32a8..331ff9a673be 100644
>>>>> --- a/drivers/iio/adc/rcar-gyroadc.c
>>>>> +++ b/drivers/iio/adc/rcar-gyroadc.c
>>>>> @@ -336,7 +336,7 @@ static int rcar_gyroadc_parse_subdevs(struct iio_dev *indio_dev)
>>>>> struct device_node *child;
>>>>> struct regulator *vref;
>>>>> unsigned int reg;
>>>>> - unsigned int adcmode, childmode;
>>>>> + unsigned int adcmode = -1, childmode;
>>>>> unsigned int sample_width;
>>>>> unsigned int num_channels;
>>>>> int ret, first = 1;
>>>>> @@ -366,6 +366,9 @@ static int rcar_gyroadc_parse_subdevs(struct iio_dev *indio_dev)
>>>>> channels = rcar_gyroadc_iio_channels_3;
>>>>> num_channels = ARRAY_SIZE(rcar_gyroadc_iio_channels_3);
>>>>> break;
>>>>> + default:
>>>>> + dev_err(dev, "unknown device type");
>>>>
>>>> Is this verbose output really needed ?
>>>
>>> I don't care much either way, I was just trying to follow what the function does
>>> for the other failure cases. We can probably drop it though, as this
>>> would indicate
>>> a bug in the driver implementation, while the others are about
>>> incorrect DT data.
>>
>> I don't really want to pester you too much with minor details, let's
>> wait for JIC's opinion. If you're resending, please add my Ack,
>> otherwise consider this patch:
>>
>> Acked-by: Marek Vasut <marek.vasut@xxxxxxxxx>
>>
> I've applied with the dev_err dropped. Didn't care that much either
> way :)

Noone seems to care much, but everyone is slightly opinionated, hehe :)

> Anyhow, applied to the fixes-togreg-post-rc1 branch of iio.git as
> I might not get another pull request out pre the merge window.
> (if I do I'll apply it with that).
>
> Thanks,
>
> Jonathan
>
Thanks

--
Best regards,
Marek Vasut