From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] iio: adc: handle unknow of_device_id data To: Jonathan Cameron , Arnd Bergmann References: <20170203193756.3848151-1-arnd@arndb.de> <94e9f5e1-0892-428e-66ed-81d5dd5bf388@gmail.com> <1ab64563-e536-ab8a-5d1e-f77cbf519dda@kernel.org> Cc: Marek Vasut , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , linux-iio@vger.kernel.org, Linux Kernel Mailing List From: Marek Vasut Message-ID: <6f13a6fe-028e-e9af-db16-4d77e6c40d82@gmail.com> Date: Wed, 8 Feb 2017 02:13:54 +0100 MIME-Version: 1.0 In-Reply-To: <1ab64563-e536-ab8a-5d1e-f77cbf519dda@kernel.org> Content-Type: text/plain; charset=utf-8 List-ID: 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 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 >> > 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