Re: [PATCH v3 2/2] iio: adc: Add Spreadtrum SC27XX PMICs ADC support

[Jonathan Cameron <jic23@jic23.retrosnub.co.uk>]

On Sun, 24 Jun 2018 14:30:09 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> On Sun, 24 Jun 2018 17:13:00 +0800
> Baolin Wang <baolin.wang@linaro.org> wrote:
> 
> > Hi Jonathan,
> > 
> > On 22 June 2018 at 22:13, Jonathan Cameron <jonathan.cameron@huawei.com> wrote:  
> > > On Thu, 21 Jun 2018 11:14:05 +0800
> > > Baolin Wang <baolin.wang@linaro.org> wrote:
> > >    
> > >> From: Freeman Liu <freeman.liu@spreadtrum.com>
> > >>
> > >> The Spreadtrum SC27XX PMICs ADC controller contains 32 channels,
> > >> which is used to sample voltages with 12 bits conversion.
> > >>
> > >> [Baolin Wang did lots of improvements]
> > >>
> > >> Signed-off-by: Freeman Liu <freeman.liu@spreadtrum.com>
> > >> Signed-off-by: Baolin Wang <baolin.wang@linaro.org>    
> > >
> > > One trivial missed bit of cleanup inline.  I'll sort that
> > > when applying if no one else points anything out before I get back to my
> > > development machine.    
> > 
> > Thanks.  
> 
> Applied to the togreg branch of iio.git and pushed out as testing
> for the autobuilders to play with it.
Sorry, backed out for now.  My togreg tree is non rebasing and
this is dependant on some stuff that only went in during the merge
window.

I'll send a pull request to GregKH fairly soon and after that do
the merge back into my tree to pick that hwspin lock stuff up.

So this will be a week or two before I can apply it without issues.

Give me a poke if I seem to have forgotten about it.

Jonathan

> 
> Thanks,
> 
> Jonathan
> 
> >   
> > >> +
> > >> +     indio_dev->dev.parent = &pdev->dev;
> > >> +     indio_dev->name = dev_name(&pdev->dev);
> > >> +     indio_dev->modes = INDIO_DIRECT_MODE;
> > >> +     indio_dev->info = &sc27xx_info;
> > >> +     indio_dev->channels = sc27xx_channels;
> > >> +     indio_dev->num_channels = ARRAY_SIZE(sc27xx_channels);
> > >> +     ret = devm_iio_device_register(&pdev->dev, indio_dev);
> > >> +     if (ret) {
> > >> +             dev_err(&pdev->dev, "could not register iio (ADC)");
> > >> +             return ret;
> > >> +     }
> > >> +
> > >> +     platform_set_drvdata(pdev, indio_dev);    
> > >
> > > Why?  Looks like this is left over from when we had a remove.    
> > 
> > Sorry, I missed to remove this redundant code.
> >   
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html