Message ID | 1575886763-19089-1-git-send-email-eugen.hristev@microchip.com |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | [v3,1/4] dt-bindings: i2c: at91: cosmetic formatting of compatible list | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success |
On Mon, 9 Dec 2019 10:19:59 +0000, <Eugen.Hristev@microchip.com> wrote: > > From: Eugen Hristev <eugen.hristev@microchip.com> > > Format the list of compatibles with one compatible per line. > > Suggested-by: Peter Rosin <peda@axentia.se> > Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com> > Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com> > --- > Documentation/devicetree/bindings/i2c/i2c-at91.txt | 14 ++++++++++---- > 1 file changed, 10 insertions(+), 4 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Mon, Dec 09, 2019 at 10:20:02AM +0000, Eugen.Hristev@microchip.com wrote: > From: Eugen Hristev <eugen.hristev@microchip.com> > > In SAM9X60 datasheet, FLEX_TWI_CWGR register description mentions clock > offset of 3 cycles (compared to 4 in eg. SAMA5D3). > This is the same offset as in SAMA5D2. > > Fixes: b00277923743 ("i2c: at91: add new platform support for sam9x60") > Suggested-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> > Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com> > Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com> > Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> Applied to for-current, thanks!
On Mon, Dec 09, 2019 at 10:20:05AM +0000, Eugen.Hristev@microchip.com wrote: > From: Eugen Hristev <eugen.hristev@microchip.com> > > Platform data structs are initialized by default with zero values. > Thus it becomes redundant to initialize them manually to zero (false). > Remove extra false initialization for field members in structs. > > Suggested-by: Wolfram Sang <wsa@the-dreams.de> > Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com> > Reviewed-by: Ludovic Desroches <ludovic.desroches@microchip.com> Applied to for-next, thanks!
On Wed, Dec 18, 2019 at 03:37:16PM -0600, Rob Herring wrote: > On Mon, 9 Dec 2019 10:19:59 +0000, <Eugen.Hristev@microchip.com> wrote: > > > > From: Eugen Hristev <eugen.hristev@microchip.com> > > > > Format the list of compatibles with one compatible per line. > > > > Suggested-by: Peter Rosin <peda@axentia.se> > > Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com> > > Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com> Applied to for-next, thanks!
diff --git a/Documentation/devicetree/bindings/i2c/i2c-at91.txt b/Documentation/devicetree/bindings/i2c/i2c-at91.txt index 2210f43..d35cd63 100644 --- a/Documentation/devicetree/bindings/i2c/i2c-at91.txt +++ b/Documentation/devicetree/bindings/i2c/i2c-at91.txt @@ -1,10 +1,16 @@ I2C for Atmel platforms Required properties : -- compatible : Must be "atmel,at91rm9200-i2c", "atmel,at91sam9261-i2c", - "atmel,at91sam9260-i2c", "atmel,at91sam9g20-i2c", "atmel,at91sam9g10-i2c", - "atmel,at91sam9x5-i2c", "atmel,sama5d4-i2c", "atmel,sama5d2-i2c" or - "microchip,sam9x60-i2c" +- compatible : Must be one of: + "atmel,at91rm9200-i2c", + "atmel,at91sam9261-i2c", + "atmel,at91sam9260-i2c", + "atmel,at91sam9g20-i2c", + "atmel,at91sam9g10-i2c", + "atmel,at91sam9x5-i2c", + "atmel,sama5d4-i2c", + "atmel,sama5d2-i2c", + "microchip,sam9x60-i2c". - reg: physical base address of the controller and length of memory mapped region. - interrupts: interrupt number to the cpu.