* [PATCH] dt-bindings: at24: add microchip,24lc[0-9]+ to the compatible pattern
@ 2020-04-22 14:18 Masahiro Yamada
2020-05-11 19:14 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2020-04-22 14:18 UTC (permalink / raw)
To: Bartosz Golaszewski, linux-i2c
Cc: Masahiro Yamada, Rob Herring, devicetree, linux-kernel
arch/arm/boot/dts/uniphier-ref-daughter.dtsi has
compatible = "microchip,24lc128", "atmel,24c128";
and 'make ARCH=arm dtbs_check' warns this:
eeprom@50: compatible: ['microchip,24lc128', 'atmel,24c128'] is not valid under any of the given schemas (Possible causes of the failure)
Microchip 24LC128 is the device used on this board, and I see it in
https://www.microchip.com/wwwproducts/en/24LC128
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---
Documentation/devicetree/bindings/eeprom/at24.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
index a15787e504f0..2cd7a04cab7c 100644
--- a/Documentation/devicetree/bindings/eeprom/at24.yaml
+++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
@@ -34,7 +34,7 @@ properties:
- minItems: 1
maxItems: 2
items:
- - pattern: "^(atmel|catalyst|microchip|nxp|ramtron|renesas|rohm|st),(24(c|cs|mac)[0-9]+|spd)$"
+ - pattern: "^(atmel|catalyst|microchip|nxp|ramtron|renesas|rohm|st),(24(c|cs|lc|mac)[0-9]+|spd)$"
- pattern: "^atmel,(24(c|cs|mac)[0-9]+|spd)$"
- oneOf:
- items:
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: at24: add microchip,24lc[0-9]+ to the compatible pattern
2020-04-22 14:18 [PATCH] dt-bindings: at24: add microchip,24lc[0-9]+ to the compatible pattern Masahiro Yamada
@ 2020-05-11 19:14 ` Rob Herring
2020-05-13 18:33 ` Masahiro Yamada
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2020-05-11 19:14 UTC (permalink / raw)
To: Masahiro Yamada
Cc: devicetree, linux-kernel, linux-i2c, Bartosz Golaszewski, Rob Herring
On Wed, 22 Apr 2020 23:18:36 +0900, Masahiro Yamada wrote:
> arch/arm/boot/dts/uniphier-ref-daughter.dtsi has
>
> compatible = "microchip,24lc128", "atmel,24c128";
>
> and 'make ARCH=arm dtbs_check' warns this:
>
> eeprom@50: compatible: ['microchip,24lc128', 'atmel,24c128'] is not valid under any of the given schemas (Possible causes of the failure)
>
> Microchip 24LC128 is the device used on this board, and I see it in
> https://www.microchip.com/wwwproducts/en/24LC128
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
> Documentation/devicetree/bindings/eeprom/at24.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: at24: add microchip,24lc[0-9]+ to the compatible pattern
2020-05-11 19:14 ` Rob Herring
@ 2020-05-13 18:33 ` Masahiro Yamada
0 siblings, 0 replies; 3+ messages in thread
From: Masahiro Yamada @ 2020-05-13 18:33 UTC (permalink / raw)
To: Rob Herring
Cc: DTML, Linux Kernel Mailing List, linux-i2c, Bartosz Golaszewski,
Rob Herring
Hi Rob,
On Tue, May 12, 2020 at 4:14 AM Rob Herring <robh@kernel.org> wrote:
>
> On Wed, 22 Apr 2020 23:18:36 +0900, Masahiro Yamada wrote:
> > arch/arm/boot/dts/uniphier-ref-daughter.dtsi has
> >
> > compatible = "microchip,24lc128", "atmel,24c128";
> >
> > and 'make ARCH=arm dtbs_check' warns this:
> >
> > eeprom@50: compatible: ['microchip,24lc128', 'atmel,24c128'] is not valid under any of the given schemas (Possible causes of the failure)
> >
> > Microchip 24LC128 is the device used on this board, and I see it in
> > https://www.microchip.com/wwwproducts/en/24LC128
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> > ---
> >
> > Documentation/devicetree/bindings/eeprom/at24.yaml | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Applied, thanks!
If it is not too late, can you correct the patch subject?
See the applied commit in linux-next.
Author: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Wed Apr 22 23:18:36 2020 +0900
dt-bindings: at24: add microchip, 24lc[0-9]+ to the compatible pattern
Please notice a space after 'microchip,'
And this is the patch I submitted:
https://lore.kernel.org/patchwork/patch/1229480/
No space after 'microchip,'
This is a patchwork bug.
It automatically inserts a space after a comma.
Patchwork adds various unpleasant modification,
and breaks the commit subject and log.
I reported this bug to patchwork ML, but
still not fixed...
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-05-13 18:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-22 14:18 [PATCH] dt-bindings: at24: add microchip,24lc[0-9]+ to the compatible pattern Masahiro Yamada
2020-05-11 19:14 ` Rob Herring
2020-05-13 18:33 ` Masahiro Yamada
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).