diff mbox series

[v2,3/4] x86: ce4100: Replace "ti,pcf8575" by "nxp,pcf8575"

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

Commit Message

Geert Uytterhoeven June 1, 2021, 3:25 p.m. UTC
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(-)

Comments

Andy Shevchenko June 1, 2021, 3:30 p.m. UTC | #1
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 mbox series

Patch

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;
 						};