From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:35362 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751774AbdBEJcF (ORCPT ); Sun, 5 Feb 2017 04:32:05 -0500 Subject: Re: [PATCH] iio: adc: handle unknow of_device_id data To: Marek Vasut , Arnd Bergmann References: <20170203193756.3848151-1-arnd@arndb.de> <94e9f5e1-0892-428e-66ed-81d5dd5bf388@gmail.com> Cc: Marek Vasut , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , linux-iio@vger.kernel.org, Linux Kernel Mailing List From: Jonathan Cameron Message-ID: <1ab64563-e536-ab8a-5d1e-f77cbf519dda@kernel.org> Date: Sun, 5 Feb 2017 09:32:02 +0000 MIME-Version: 1.0 In-Reply-To: <94e9f5e1-0892-428e-66ed-81d5dd5bf388@gmail.com> Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org 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 :) 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