From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH v2] ARM: dts: dra7xx: Fix compatible string for PCF8575 chip Date: Wed, 13 Apr 2016 08:17:00 -0700 Message-ID: <20160413151700.GP5995@atomide.com> References: <1458147150.20441.2.camel@codethink.co.uk> <56E9DED9.6020909@ti.com> <20160412210438.GA5995@atomide.com> <570DEE5B.1010001@linutronix.de> <570DF110.3070805@ti.com> <570DF235.6040705@linutronix.de> <570DF38C.8000600@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <570DF38C.8000600-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Roger Quadros Cc: Sebastian Andrzej Siewior , Nishanth Menon , Ben Hutchings , linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, CT kernel , Grygorii Strashko , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-omap@vger.kernel.org * Roger Quadros [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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html