Thread (1 message) 1 message, 1 author, 2012-09-05

Re: patch "misc/at25, dt: Improve at25 SPI eeprom device tree bindings." added to char-misc tree

From: David Daney <hidden>
Date: 2012-09-05 21:33:58

On 09/05/2012 02:10 PM, gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org wrote:
This is a note to let you know that I've just added the patch titled

     misc/at25, dt: Improve at25 SPI eeprom device tree bindings.
It is perhaps obvious that I think the patch should be merged (as I 
wrote it and sent it to you).  However, I think there is a bit of 
process problem here.  Patches that affect device tree bindings are part 
of the de facto kernel ABI, and as such, need proper vetting.  It is my 
opinion that neither the original patch, nor this one correcting it, 
should have been committed without either Grant or Rob giving an Acked-by.

David Daney.

to my char-misc git tree which can be found at
     git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
in the char-misc-next branch.

The patch will show up in the next release of the linux-next tree
(usually sometime within the next 24 hours during the week.)

The patch will also be merged in the next major kernel release
during the merge window.

If you have any questions about this process, please let me know.


 From d6ae0d578d24303941c1424b049d2cae28277666 Mon Sep 17 00:00:00 2001
From: David Daney <david.daney-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org>
Date: Wed, 22 Aug 2012 12:03:57 -0700
Subject: misc/at25, dt: Improve at25 SPI eeprom device tree bindings.

Commit 002176db (misc: at25: Parse dt settings) added device tree
bindings the differ significantly in style from the I2C EEPROM
bindings and don't seem well vetted.  Here I deprecate (but still
support) the "at25,*" properties, and add what I hope is a better
alternative.  These new bindings also happen to be deployed in the
field and were previously submitted for consideration here:

https://lists.ozlabs.org/pipermail/devicetree-discuss/2012-May/015556.html

The advantages of the new bindings are that they are similar to the
I2C EEPROMs and they don't conflate read-only and the address width
modes in a binary encoded blob.

Signed-off-by: David Daney <david.daney-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org>
Cc: Alexandre Pereira da Silva <redacted>
Cc: Greg Kroah-Hartman <redacted>
Cc: Michael Hennerich <redacted>
Cc: Axel Lin <redacted>
Signed-off-by: Greg Kroah-Hartman <redacted>
---
  Documentation/devicetree/bindings/misc/at25.txt | 34 +++++++---
  drivers/misc/eeprom/at25.c                      | 83 +++++++++++++++++--------
  2 files changed, 82 insertions(+), 35 deletions(-)
[...]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help