| Message ID | 5236cf71d467bec862c4fa7849705caac195b23a.1622560799.git.geert+renesas@glider.be |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/4] dt-bindings: i2c: ce4100: Replace "ti,pcf8575" by "nxp,pcf8575" | expand |
On Tue, Jun 1, 2021 at 6:26 PM Geert Uytterhoeven <geert+renesas@glider.be> 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@gmail.com> 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@glider.be> > --- > 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 >
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; };
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. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- v2: - New. --- arch/x86/platform/ce4100/falconfalls.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)