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

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

 



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.

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



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux