Re: [PATCH v2] ARM: dts: dra7xx: Fix compatible string for PCF8575 chip

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




* Roger Quadros <rogerq@xxxxxx> [160413 00:23]:
> On 13/04/16 10:16, Sebastian Andrzej Siewior wrote:
> > On 04/13/2016 09:11 AM, Roger Quadros wrote:
> >> On 13/04/16 09:59, Sebastian Andrzej Siewior wrote:
> >>> On 04/12/2016 11:04 PM, Tony Lindgren wrote:
> >>>>> While at it, do you want to fix up (+Seb):
> >>>>> a) Documentation/devicetree/bindings/i2c/i2c-pxa-pci-ce4100.txt which
> >>>>> also seems to refer to ti,pcf in example (introduced by df2634f43f51
> >>>>> x86: dtb: Add a device tree for CE4100 )
> >>>>> b) arch/x86/platform/ce4100/falconfalls.dts as well?
> >>>>
> >>>> Applying this one with Roger's ack, seems like those should be
> >>>> additional patches.
> >>>
> >>> Okay. I looked it up on the board and it says NXP PCF8575 and not TI.
> >>> TI's was probably the first hit while I was looking for it (I think
> >>> CE4100 manual mentioned just PCF8575).
> >>>
> >>> But if it would have a TI branding on it I would keep the TI prefix in
> >>> front and add the NXP prefix. As of today we might not know a
> >>> functional difference between those two but this does not mean we might
> >>> find one in the future.
> >>> That said, are you people sure that the DRA7 has a TI or a NXP branding
> >>> on it?
> >>
> >> Board has a Texas Instruments part: PCF8575PWR TSSOP24.
> >> http://www.ti.com/product/PCF8575
> > 
> > So this sounds like
> > 
> > 	compatible = "ti,pcf8575", "nxp,pcf8575";
> > 
> > then.
> 
> Agree with you.

Can you please do an additional patch against omap-for-v4.7/dt
for this one please? And also update the related binding documentation.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux