On Tue, Jun 1, 2021 at 6:26 PM Geert Uytterhoeven
<geert+renesas@xxxxxxxxx> wrote:
>
> The TI part is equivalent to the NXP part, and its compatible value is
> not documented in the DT bindings.
>
> Note that while the Linux driver DT match table does not contain the
> compatible value of the TI part, it could still match to this part, as
> i2c_device_id-based matching ignores the vendor part of the compatible
> value.
LGTM!
Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
Maybe at some point I will find the time to resurrect the CE4100 based
device that collects dirt on my desk...
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> v2:
> - New.
> ---
> arch/x86/platform/ce4100/falconfalls.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/x86/platform/ce4100/falconfalls.dts b/arch/x86/platform/ce4100/falconfalls.dts
> index 0ac3d43571361112..65fa3d866226ce97 100644
> --- a/arch/x86/platform/ce4100/falconfalls.dts
> +++ b/arch/x86/platform/ce4100/falconfalls.dts
> @@ -249,7 +249,7 @@ i2c@1 {
>
> gpio@26 {
> #gpio-cells = <2>;
> - compatible = "ti,pcf8575";
> + compatible = "nxp,pcf8575";
> reg = <0x26>;
> gpio-controller;
> };
> @@ -263,7 +263,7 @@ i2c@2 {
>
> gpio@26 {
> #gpio-cells = <2>;
> - compatible = "ti,pcf8575";
> + compatible = "nxp,pcf8575";
> reg = <0x26>;
> gpio-controller;
> };
> --
> 2.25.1
>
--
With Best Regards,
Andy Shevchenko
