Hi, Fabio
Obviously, some of the dts files (such as
arch/arm/boot/dts/exynos5420-peach-pit.dts) using "vcc" as isl29018's power
supply name, they are NOT matched with datasheet, so should we update those dts
files with "vdd" as well or just using the "vcc" in isl29018 driver? Which
solution is better?
Best Regards!
Anson Huang
> -----Original Message-----
> From: Rob Herring [mailto:[email protected]]
> Sent: 2018年12月11日 23:19
> To: Anson Huang <[email protected]>
> Cc: Jonathan Cameron <[email protected]>; Hartmut Knaack
> <[email protected]>; Lars-Peter Clausen <[email protected]>; Peter Meerwald
> <[email protected]>; Mark Rutland <[email protected]>;
> [email protected]; [email protected];
> [email protected]; dl-linux-imx <[email protected]>
> Subject: Re: [PATCH V3 1/2] dt-bindings: iio: light: isl29018: update power
> supply name
>
> On Mon, Dec 10, 2018 at 7:40 PM Anson Huang <[email protected]>
> wrote:
> >
> > Hi, Rob
> >
> > Best Regards!
> > Anson Huang
> >
> > > -----Original Message-----
> > > From: Rob Herring [mailto:[email protected]]
> > > Sent: 2018年12月11日 7:24
> > > To: Anson Huang <[email protected]>
> > > Cc: [email protected]; [email protected]; [email protected];
> > > [email protected]; [email protected]; [email protected];
> > > [email protected]; [email protected];
> > > dl-linux-imx <[email protected]>
> > > Subject: Re: [PATCH V3 1/2] dt-bindings: iio: light: isl29018:
> > > update power supply name
> > >
> > > On Mon, Dec 10, 2018 at 07:11:19AM +0000, Anson Huang wrote:
> > > > According to datasheet, the isl29018 has "vddd/vdda" power supply,
> > > > and
> > > > isl29023/isl29035 ONLY has "vdd" power supply, update the power
> > > > supply name with "vdd" and "vdda" according to datasheet to cover
> > > > all devices and avoid confusion.
> > > >
> > > > Signed-off-by: Anson Huang <[email protected]>
> > > > ---
> > > > Documentation/devicetree/bindings/iio/light/isl29018.txt | 4 +++-
> > > > 1 file changed, 3 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git
> > > > a/Documentation/devicetree/bindings/iio/light/isl29018.txt
> > > > b/Documentation/devicetree/bindings/iio/light/isl29018.txt
> > > > index b9bbde3..36f737d 100644
> > > > --- a/Documentation/devicetree/bindings/iio/light/isl29018.txt
> > > > +++ b/Documentation/devicetree/bindings/iio/light/isl29018.txt
> > > > @@ -15,7 +15,9 @@ Optional properties:
> > > > Refer to interrupt-controller/interrupts.txt for generic interrupt
> > > > client
> > > > node bindings.
> > > >
> > > > - - vcc-supply: phandle to the regulator that provides power to the
> sensor.
> > > > + - vdd-supply: phandle to the regulator that provides vdd power
> > > > + to the
> > > sensor.
> > > > +
> > > > + - vdda-supply: phandle to the regulator that provides vdda
> > > > + power to the
> > > sensor.
> > >
> > > Is this in use? You can't just change things if it is.
> >
> > I did NOT see any "vcc" in folder drivers/iio/light/, so I think it is
> > NOT used at all, so I take this chance to update it according to datasheet.
> Thanks.
>
> arch/arm/boot/dts/exynos5420-peach-pit.dts-629- light-sensor@44 {
> arch/arm/boot/dts/exynos5420-peach-pit.dts:630: compatible =
> "isil,isl29018";
> arch/arm/boot/dts/exynos5420-peach-pit.dts-631- reg = <0x44>;
> arch/arm/boot/dts/exynos5420-peach-pit.dts-632- vcc-supply =
> <&tps65090_fet5>;
> arch/arm/boot/dts/exynos5420-peach-pit.dts-633- };
>
> arch/arm/boot/dts/exynos5800-peach-pi.dts-629- light-sensor@44 {
> arch/arm/boot/dts/exynos5800-peach-pi.dts:630: compatible =
> "isil,isl29018";
> arch/arm/boot/dts/exynos5800-peach-pi.dts-631- reg = <0x44>;
> arch/arm/boot/dts/exynos5800-peach-pi.dts-632- vcc-supply =
> <&tps65090_fet5>;
> arch/arm/boot/dts/exynos5800-peach-pi.dts-633- };
>
> The rest of the dts files using this don't have a supply it seems. So you need
> permission from the Exynos folks if you want to just drop this. And also
> update
> their dts files.
>
> Rob
