diff mbox series

dt-bindings: eeprom: at24: Fix list of page sizes for writing

Message ID 20200512122450.20205-1-geert+renesas@glider.be
State Awaiting Upstream
Headers show
Series dt-bindings: eeprom: at24: Fix list of page sizes for writing | expand

Commit Message

Geert Uytterhoeven May 12, 2020, 12:24 p.m. UTC
"258" is an odd power-of-two ;-)
Obviously this is a typo, and the intended value is "256".

Fixes: 7f3bf4203774013b ("dt-bindings: at24: convert the binding document to yaml")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 Documentation/devicetree/bindings/eeprom/at24.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wolfram Sang May 12, 2020, 12:56 p.m. UTC | #1
On Tue, May 12, 2020 at 02:24:47PM +0200, Geert Uytterhoeven wrote:
> "258" is an odd power-of-two ;-)

Geez, you got two additional for free, Geert ;)

> Obviously this is a typo, and the intended value is "256".

Obviously.

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Wolfram Sang May 15, 2020, 9:27 a.m. UTC | #2
On Tue, May 12, 2020 at 02:24:47PM +0200, Geert Uytterhoeven wrote:
> "258" is an odd power-of-two ;-)
> Obviously this is a typo, and the intended value is "256".
> 
> Fixes: 7f3bf4203774013b ("dt-bindings: at24: convert the binding document to yaml")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Since I have a PR for Linus pending anyhow, I step ahead and apply it to
my tree. Hope this is fine for everyone. Applied to for-current, thanks!
Wolfram Sang May 15, 2020, 9:36 a.m. UTC | #3
On Fri, May 15, 2020 at 11:27:55AM +0200, Wolfram Sang wrote:
> On Tue, May 12, 2020 at 02:24:47PM +0200, Geert Uytterhoeven wrote:
> > "258" is an odd power-of-two ;-)
> > Obviously this is a typo, and the intended value is "256".
> > 
> > Fixes: 7f3bf4203774013b ("dt-bindings: at24: convert the binding document to yaml")
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Since I have a PR for Linus pending anyhow, I step ahead and apply it to
> my tree. Hope this is fine for everyone. Applied to for-current, thanks!

Brown paper bag, please. This is in -next only, so I rather leave it for
Rob because of the dependencies.
Rob Herring May 19, 2020, 6:52 p.m. UTC | #4
On Tue, 12 May 2020 14:24:47 +0200, Geert Uytterhoeven wrote:
> "258" is an odd power-of-two ;-)
> Obviously this is a typo, and the intended value is "256".
> 
> Fixes: 7f3bf4203774013b ("dt-bindings: at24: convert the binding document to yaml")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  Documentation/devicetree/bindings/eeprom/at24.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/eeprom/at24.yaml b/Documentation/devicetree/bindings/eeprom/at24.yaml
index d77bc475fd148d99..4cee72d5331877a8 100644
--- a/Documentation/devicetree/bindings/eeprom/at24.yaml
+++ b/Documentation/devicetree/bindings/eeprom/at24.yaml
@@ -124,7 +124,7 @@  properties:
       may result in data loss! If not specified, a safety value of
       '1' is used which will be very slow.
     $ref: /schemas/types.yaml#/definitions/uint32
-    enum: [1, 8, 16, 32, 64, 128, 258]
+    enum: [1, 8, 16, 32, 64, 128, 256]
     default: 1
 
   read-only: