* [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples
@ 2021-05-28 11:10 Geert Uytterhoeven
2021-05-28 14:07 ` Wolfram Sang
2021-06-04 20:41 ` Wolfram Sang
0 siblings, 2 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2021-05-28 11:10 UTC (permalink / raw)
To: Rob Herring, Peter Rosin, Maxime Ripard
Cc: linux-i2c, devicetree, Geert Uytterhoeven
The "reset-active-low" property was removed from the ssd1307fb bindings
in commit 519b4dba586198ee ("fbdev: ssd1307fb: Remove reset-active-low
from the DT binding document") and from the ssd1307fb binding examples
in commit 7d7e58d30e046d34 ("dt-bindings: display: ssd1307fb: Remove
reset-active-low from examples").
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt | 1 -
Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt | 1 -
2 files changed, 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt b/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
index 21da3ecbb3700c06..1bf267302251bb63 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
+++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
@@ -62,7 +62,6 @@ Example:
reg = <0x3c>;
pwms = <&pwm 4 3000>;
reset-gpios = <&gpio2 7 1>;
- reset-active-low;
};
};
diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
index 8b444b94e92fde83..51356fdf2becc3f6 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
+++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
@@ -80,7 +80,6 @@ Example:
reg = <0x3c>;
pwms = <&pwm 4 3000>;
reset-gpios = <&gpio2 7 1>;
- reset-active-low;
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples
2021-05-28 11:10 [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples Geert Uytterhoeven
@ 2021-05-28 14:07 ` Wolfram Sang
2021-05-28 14:25 ` Geert Uytterhoeven
2021-06-04 20:41 ` Wolfram Sang
1 sibling, 1 reply; 5+ messages in thread
From: Wolfram Sang @ 2021-05-28 14:07 UTC (permalink / raw)
To: Geert Uytterhoeven, Rob Herring
Cc: Peter Rosin, Maxime Ripard, linux-i2c, devicetree
[-- Attachment #1: Type: text/plain, Size: 1887 bytes --]
On Fri, May 28, 2021 at 01:10:49PM +0200, Geert Uytterhoeven wrote:
> The "reset-active-low" property was removed from the ssd1307fb bindings
> in commit 519b4dba586198ee ("fbdev: ssd1307fb: Remove reset-active-low
> from the DT binding document") and from the ssd1307fb binding examples
> in commit 7d7e58d30e046d34 ("dt-bindings: display: ssd1307fb: Remove
> reset-active-low from examples").
>
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Now we have a conflict with this series from Rob: "[PATCH v2 0/6]
dt-bindings: Convert mux bindings to schema"
My suggestion would be that Rob adds this on top of his series?
Or he provides an immutable branch for me, so I can apply this patch on
top then?
> ---
> Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt | 1 -
> Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt b/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
> index 21da3ecbb3700c06..1bf267302251bb63 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
> @@ -62,7 +62,6 @@ Example:
> reg = <0x3c>;
> pwms = <&pwm 4 3000>;
> reset-gpios = <&gpio2 7 1>;
> - reset-active-low;
> };
> };
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
> index 8b444b94e92fde83..51356fdf2becc3f6 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
> @@ -80,7 +80,6 @@ Example:
> reg = <0x3c>;
> pwms = <&pwm 4 3000>;
> reset-gpios = <&gpio2 7 1>;
> - reset-active-low;
> };
> };
>
> --
> 2.25.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples
2021-05-28 14:07 ` Wolfram Sang
@ 2021-05-28 14:25 ` Geert Uytterhoeven
2021-05-28 14:51 ` Wolfram Sang
0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2021-05-28 14:25 UTC (permalink / raw)
To: Wolfram Sang
Cc: Peter Rosin, Maxime Ripard, Linux I2C,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Rob Herring
Hi Wolfram,
On Fri, May 28, 2021 at 4:07 PM Wolfram Sang <wsa@kernel.org> wrote:
> On Fri, May 28, 2021 at 01:10:49PM +0200, Geert Uytterhoeven wrote:
> > The "reset-active-low" property was removed from the ssd1307fb bindings
> > in commit 519b4dba586198ee ("fbdev: ssd1307fb: Remove reset-active-low
> > from the DT binding document") and from the ssd1307fb binding examples
> > in commit 7d7e58d30e046d34 ("dt-bindings: display: ssd1307fb: Remove
> > reset-active-low from examples").
> >
> > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
>
> Now we have a conflict with this series from Rob: "[PATCH v2 0/6]
> dt-bindings: Convert mux bindings to schema"
Sorry, I had checked that my patch applies to today's linux-next,
and wasn't aware of Rob's series.
> My suggestion would be that Rob adds this on top of his series?
> Or he provides an immutable branch for me, so I can apply this patch on
> top then?
Alternatively, can you just drop the second chunk?
> > --- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
> > +++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpio.txt
> > @@ -62,7 +62,6 @@ Example:
> > reg = <0x3c>;
> > pwms = <&pwm 4 3000>;
> > reset-gpios = <&gpio2 7 1>;
> > - reset-active-low;
> > };
> > };
This one is still valid.
> > --- a/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
> > +++ b/Documentation/devicetree/bindings/i2c/i2c-mux-gpmux.txt
> > @@ -80,7 +80,6 @@ Example:
> > reg = <0x3c>;
> > pwms = <&pwm 4 3000>;
> > reset-gpios = <&gpio2 7 1>;
> > - reset-active-low;
> > };
> > };
This one no longer is, as Rob dropped the example during the conversion.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples
2021-05-28 14:25 ` Geert Uytterhoeven
@ 2021-05-28 14:51 ` Wolfram Sang
0 siblings, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2021-05-28 14:51 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Peter Rosin, Maxime Ripard, Linux I2C,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Rob Herring
[-- Attachment #1: Type: text/plain, Size: 377 bytes --]
> > Now we have a conflict with this series from Rob: "[PATCH v2 0/6]
> > dt-bindings: Convert mux bindings to schema"
>
> Sorry, I had checked that my patch applies to today's linux-next,
> and wasn't aware of Rob's series.
No worries. That's what maintainers are primarily for :)
> Alternatively, can you just drop the second chunk?
That I can do as well.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples
2021-05-28 11:10 [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples Geert Uytterhoeven
2021-05-28 14:07 ` Wolfram Sang
@ 2021-06-04 20:41 ` Wolfram Sang
1 sibling, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2021-06-04 20:41 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Rob Herring, Peter Rosin, Maxime Ripard, linux-i2c, devicetree
[-- Attachment #1: Type: text/plain, Size: 515 bytes --]
On Fri, May 28, 2021 at 01:10:49PM +0200, Geert Uytterhoeven wrote:
> The "reset-active-low" property was removed from the ssd1307fb bindings
> in commit 519b4dba586198ee ("fbdev: ssd1307fb: Remove reset-active-low
> from the DT binding document") and from the ssd1307fb binding examples
> in commit 7d7e58d30e046d34 ("dt-bindings: display: ssd1307fb: Remove
> reset-active-low from examples").
>
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Applied the first chunk to for-next, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-06-04 20:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-28 11:10 [PATCH] dt-bindings: i2c: i2c-mux: Remove reset-active-low from ssd1307fb examples Geert Uytterhoeven
2021-05-28 14:07 ` Wolfram Sang
2021-05-28 14:25 ` Geert Uytterhoeven
2021-05-28 14:51 ` Wolfram Sang
2021-06-04 20:41 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).