Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 17:40:44 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 4:52 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 16:06:57 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi again Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 17:40:44 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 4:52 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 16:06:57 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi again Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 16:06:57 +0200 Ricardo Ribalda Delgado wrote: > Hi again Boris > > > On Fri, Oct 5, 2018 at 2:10 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 14:04:52 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi B

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 16:06:57 +0200 Ricardo Ribalda Delgado wrote: > Hi again Boris > > > On Fri, Oct 5, 2018 at 2:10 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 14:04:52 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi B

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 14:04:52 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Fri, Oct 5, 2018 at 12:12 PM Boris Brezillon > wrote: > > > > On Fri, 5 Oct 2018 11:54:18 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Boris > > &g

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 11:54:18 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > Just seen that you already did the rebase at > https://github.com/bbrezillon/linux-0day/commits/mtd/physmap-cleanup > > Thanks for that. > > I am about to test it in real hw (unless you want me wait) Sure, go

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 11:54:18 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > Just seen that you already did the rebase at > https://github.com/bbrezillon/linux-0day/commits/mtd/physmap-cleanup > > Thanks for that. > > I am about to test it in real hw (unless you want me wait) Sure, go

Re: [PATCH v2 2/3] mtd: rawnand: stm32_fmc2: add STM32 FMC2 NAND flash controller driver

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 11:41:59 +0200 wrote: > +struct stm32_fmc2 { You should inherit from nand_controller even if the nand_chip already embeds a dummy nand controller object. struct nand_controller base; > + struct stm32_fmc2_nand nand; > + struct device *dev; > + void

Re: [PATCH v2 2/3] mtd: rawnand: stm32_fmc2: add STM32 FMC2 NAND flash controller driver

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 11:41:59 +0200 wrote: > +struct stm32_fmc2 { You should inherit from nand_controller even if the nand_chip already embeds a dummy nand controller object. struct nand_controller base; > + struct stm32_fmc2_nand nand; > + struct device *dev; > + void

Re: [PATCH v2 1/3] dt-bindings: mtd: stm32_fmc2: add STM32 FMC2 NAND controller documentation

2018-10-05 Thread Boris Brezillon
ns 5 to 7 contain the same areas for CS1. Maybe you could use reg-names here ("nfc", "csX-data", "csX-cmd", "csX-addr"). Anyway, even without this Reviewed-by: Boris Brezillon > +- interrupts: The interrupt number > +- pinctrl-0: Standard Pinctr

Re: [PATCH v2 1/3] dt-bindings: mtd: stm32_fmc2: add STM32 FMC2 NAND controller documentation

2018-10-05 Thread Boris Brezillon
ns 5 to 7 contain the same areas for CS1. Maybe you could use reg-names here ("nfc", "csX-data", "csX-cmd", "csX-addr"). Anyway, even without this Reviewed-by: Boris Brezillon > +- interrupts: The interrupt number > +- pinctrl-0: Standard Pinctr

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 10:10:22 +0200 Ricardo Ribalda Delgado wrote: > On Fri, Oct 5, 2018 at 9:08 AM Boris Brezillon > wrote: > > > > Hi Ricardo, > > > > On Fri, 5 Oct 2018 08:31:35 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Bori

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
On Fri, 5 Oct 2018 10:10:22 +0200 Ricardo Ribalda Delgado wrote: > On Fri, Oct 5, 2018 at 9:08 AM Boris Brezillon > wrote: > > > > Hi Ricardo, > > > > On Fri, 5 Oct 2018 08:31:35 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Hi Bori

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
Hi Ricardo, On Fri, 5 Oct 2018 08:31:35 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > > On Fri, Oct 5, 2018 at 12:21 AM Boris Brezillon > wrote: > > > > Hi Ricardo, > > > > On Thu, 4 Oct 2018 16:29:42 +0200 > > Ricardo Ribalda Delgado wrot

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-05 Thread Boris Brezillon
Hi Ricardo, On Fri, 5 Oct 2018 08:31:35 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > > On Fri, Oct 5, 2018 at 12:21 AM Boris Brezillon > wrote: > > > > Hi Ricardo, > > > > On Thu, 4 Oct 2018 16:29:42 +0200 > > Ricardo Ribalda Delgado wrot

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-04 Thread Boris Brezillon
Hi Ricardo, On Thu, 4 Oct 2018 16:29:42 +0200 Ricardo Ribalda Delgado wrote: > Allow creating gpio-addr-flash via device-tree and not just via platform > data. > > Mimic what physmap_of_versatile and physmap_of_gemini does to reduce > code duplicity. > > Signed-off-by: Ricardo Ribalda

Re: [PATCH v10 10/10] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-04 Thread Boris Brezillon
Hi Ricardo, On Thu, 4 Oct 2018 16:29:42 +0200 Ricardo Ribalda Delgado wrote: > Allow creating gpio-addr-flash via device-tree and not just via platform > data. > > Mimic what physmap_of_versatile and physmap_of_gemini does to reduce > code duplicity. > > Signed-off-by: Ricardo Ribalda

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-04 Thread Boris Brezillon
On Thu, 04 Oct 2018 16:11:42 +0200 Janusz Krzysztofik wrote: > > Legacy nand_wait_ready() uses a hardcoded timeout value of 400 ms. Should > we > follow the same approach in nand_gpio_waitrdy(), or should we rather let > drivers pass the timeout value, like in case of nand_soft_waitrdy()?

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-04 Thread Boris Brezillon
On Thu, 04 Oct 2018 16:11:42 +0200 Janusz Krzysztofik wrote: > > Legacy nand_wait_ready() uses a hardcoded timeout value of 400 ms. Should > we > follow the same approach in nand_gpio_waitrdy(), or should we rather let > drivers pass the timeout value, like in case of nand_soft_waitrdy()?

Re: [PATCH v6 09/10] dt-binding: mtd: Document gpio-addr-flash

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 15:01:09 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [PATCH v6 09/10] dt-binding: mtd: Document gpio-addr-flash

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 15:01:09 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-04 Thread Boris Brezillon
On Thu, 04 Oct 2018 15:52:57 +0200 Janusz Krzysztofik wrote: > Hi Boris, > > On Wednesday, October 3, 2018 4:06:34 PM CEST Boris Brezillon wrote: > > On Wed, 03 Oct 2018 15:55:25 +0200 > > Janusz Krzysztofik wrote: > > > > > > > > > > &

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-04 Thread Boris Brezillon
On Thu, 04 Oct 2018 15:52:57 +0200 Janusz Krzysztofik wrote: > Hi Boris, > > On Wednesday, October 3, 2018 4:06:34 PM CEST Boris Brezillon wrote: > > On Wed, 03 Oct 2018 15:55:25 +0200 > > Janusz Krzysztofik wrote: > > > > > > > > > > &

Re: [PATCH v6 02/10] mtd: physmap_of: Remove unused struct of_device_id

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 15:01:02 +0200 Ricardo Ribalda Delgado wrote: > This struct does not seem to be used anywhere on the code > > Signed-off-by: Ricardo Ribalda Delgado Already queued that one. > --- > drivers/mtd/maps/physmap_of_gemini.c | 5 - > 1 file changed, 5 deletions(-) > >

Re: [PATCH v6 02/10] mtd: physmap_of: Remove unused struct of_device_id

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 15:01:02 +0200 Ricardo Ribalda Delgado wrote: > This struct does not seem to be used anywhere on the code > > Signed-off-by: Ricardo Ribalda Delgado Already queued that one. > --- > drivers/mtd/maps/physmap_of_gemini.c | 5 - > 1 file changed, 5 deletions(-) > >

Re: [PATCH v6 01/10] mtd: physmap_of: Release resources on error

2018-10-04 Thread Boris Brezillon
Hi Ricardo, On Thu, 4 Oct 2018 15:01:01 +0200 Ricardo Ribalda Delgado wrote: > During probe, if there was an error the memory region and the memory > map were not properly released. > > This can lead a system unusable if deferred probe is in use. > > Signed-off-by: Ricardo Ribalda Delgado >

Re: [PATCH v6 01/10] mtd: physmap_of: Release resources on error

2018-10-04 Thread Boris Brezillon
Hi Ricardo, On Thu, 4 Oct 2018 15:01:01 +0200 Ricardo Ribalda Delgado wrote: > During probe, if there was an error the memory region and the memory > map were not properly released. > > This can lead a system unusable if deferred probe is in use. > > Signed-off-by: Ricardo Ribalda Delgado >

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 12:07:17 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Yogesh Narayan Gaur > > Sent: Thursday, October 4, 2018 2:56 PM > > To: 'Boris Brezillon' > > Cc: linux-...@lists.infradead.org; marek.va...@g

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 12:07:17 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Yogesh Narayan Gaur > > Sent: Thursday, October 4, 2018 2:56 PM > > To: 'Boris Brezillon' > > Cc: linux-...@lists.infradead.org; marek.va...@g

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 16:42:22 +0530 Vignesh R wrote: > On Thursday 04 October 2018 03:15 PM, Boris Brezillon wrote: > > On Wed, 3 Oct 2018 22:26:01 +0530 > > Vignesh R wrote: > > > >> Micron's mt35xu512aba flash is an Octal flash that has x8 IO lines. It >

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 16:42:22 +0530 Vignesh R wrote: > On Thursday 04 October 2018 03:15 PM, Boris Brezillon wrote: > > On Wed, 3 Oct 2018 22:26:01 +0530 > > Vignesh R wrote: > > > >> Micron's mt35xu512aba flash is an Octal flash that has x8 IO lines. It >

Re: [PATCH 0/3] spi-nor: Add Octal SPI support

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 16:05:36 +0530 Vignesh R wrote: > >> > >> .../devicetree/bindings/mtd/cadence-quadspi.txt | 1 + > >> drivers/mtd/spi-nor/cadence-quadspi.c | 9 + > > > > On a slightly different topic, do you plan to convert the Cadence > > driver to

Re: [PATCH 0/3] spi-nor: Add Octal SPI support

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 16:05:36 +0530 Vignesh R wrote: > >> > >> .../devicetree/bindings/mtd/cadence-quadspi.txt | 1 + > >> drivers/mtd/spi-nor/cadence-quadspi.c | 9 + > > > > On a slightly different topic, do you plan to convert the Cadence > > driver to

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Wed, 3 Oct 2018 22:26:01 +0530 Vignesh R wrote: > Micron's mt35xu512aba flash is an Octal flash that has x8 IO lines. It > supports read/write over 8 IO lines simulatenously. Add support for > Octal read mode for Micron mt35xu512aba. > Unfortunately, this flash is only complaint to SFDP

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Wed, 3 Oct 2018 22:26:01 +0530 Vignesh R wrote: > Micron's mt35xu512aba flash is an Octal flash that has x8 IO lines. It > supports read/write over 8 IO lines simulatenously. Add support for > Octal read mode for Micron mt35xu512aba. > Unfortunately, this flash is only complaint to SFDP

Re: [PATCH 4/4] arm64: dts: lx2160a: update fspi node

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 09:24:57 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Boris Brezillon [mailto:boris.brezil...@bootlin.com] > > Sent: Thursday, October 4, 2018 2:48 PM > > To: Yogesh Narayan Gaur > > Cc: lin

Re: [PATCH 4/4] arm64: dts: lx2160a: update fspi node

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 09:24:57 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Boris Brezillon [mailto:boris.brezil...@bootlin.com] > > Sent: Thursday, October 4, 2018 2:48 PM > > To: Yogesh Narayan Gaur > > Cc: lin

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 09:14:36 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Boris Brezillon [mailto:boris.brezil...@bootlin.com] > > Sent: Thursday, October 4, 2018 2:35 PM > > To: Yogesh Narayan Gaur > > Cc: lin

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 09:14:36 + Yogesh Narayan Gaur wrote: > Hi Boris, > > > -Original Message- > > From: Boris Brezillon [mailto:boris.brezil...@bootlin.com] > > Sent: Thursday, October 4, 2018 2:35 PM > > To: Yogesh Narayan Gaur > > Cc: lin

Re: [PATCH 4/4] arm64: dts: lx2160a: update fspi node

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 14:18:40 +0530 Yogesh Gaur wrote: > Flash mt35xu512aba connected to FlexSPI controller supports > 1-1-8 protocol. > Added flag spi-rx-bus-width and spi-tx-bus-width with values as > 8 and 1 respectively for both flashes connected at CS0 and CS1. > > Signed-off-by: Yogesh

Re: [PATCH 4/4] arm64: dts: lx2160a: update fspi node

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 14:18:40 +0530 Yogesh Gaur wrote: > Flash mt35xu512aba connected to FlexSPI controller supports > 1-1-8 protocol. > Added flag spi-rx-bus-width and spi-tx-bus-width with values as > 8 and 1 respectively for both flashes connected at CS0 and CS1. > > Signed-off-by: Yogesh

Re: [PATCH 2/4] mtd: spi-nor: add support for octal mode data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 14:18:38 +0530 Yogesh Gaur wrote: > Add support for octal mode data transfer for Micron mt35xu512aba. > > Unfortunately, this flash is only complaint to SFDP JESD216B and does > not seem to support newer JESD216C standard that provides auto detection > of Octal mode

Re: [PATCH 2/4] mtd: spi-nor: add support for octal mode data transfer

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 14:18:38 +0530 Yogesh Gaur wrote: > Add support for octal mode data transfer for Micron mt35xu512aba. > > Unfortunately, this flash is only complaint to SFDP JESD216B and does > not seem to support newer JESD216C standard that provides auto detection > of Octal mode

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 08:47:33 + Yogesh Narayan Gaur wrote: > > > > Yogesh, you already sent "spi: add flags for octal I/O data > > transfer" [3] which is only adding the new OCTAL flags but is not > > patching spi.c and spi-mem.c to take those new flags into account. > > Here is my version of

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
On Thu, 4 Oct 2018 08:47:33 + Yogesh Narayan Gaur wrote: > > > > Yogesh, you already sent "spi: add flags for octal I/O data > > transfer" [3] which is only adding the new OCTAL flags but is not > > patching spi.c and spi-mem.c to take those new flags into account. > > Here is my version of

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
Hi Yogesh, On Thu, 4 Oct 2018 14:18:37 +0530 Yogesh Gaur wrote: > Add flags for Octal I/O data transfer > Required for the SPI controller which can do the data transfer (TX/RX) > on 8 data lines e.g. NXP FlexSPI controller. > SPI_TX_OCTAL: transmit with 8 wires > SPI_RX_OCTAL: receive with 8

Re: [PATCH 1/4] spi: add support for octal I/O data transfer

2018-10-04 Thread Boris Brezillon
Hi Yogesh, On Thu, 4 Oct 2018 14:18:37 +0530 Yogesh Gaur wrote: > Add flags for Octal I/O data transfer > Required for the SPI controller which can do the data transfer (TX/RX) > on 8 data lines e.g. NXP FlexSPI controller. > SPI_TX_OCTAL: transmit with 8 wires > SPI_RX_OCTAL: receive with 8

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
+Julien, Zhengxunli and Mason from Macronix Hi Yogesh, On Thu, 4 Oct 2018 06:51:41 + Yogesh Narayan Gaur wrote: > Hi Vignesh, > > > -Original Message- > > From: Vignesh R [mailto:vigne...@ti.com] > > Sent: Wednesday, October 3, 2018 10:26 PM > > To:

Re: [PATCH 1/3] mtd: spi-nor: Add Octal mode support for mt35xu512aba

2018-10-04 Thread Boris Brezillon
+Julien, Zhengxunli and Mason from Macronix Hi Yogesh, On Thu, 4 Oct 2018 06:51:41 + Yogesh Narayan Gaur wrote: > Hi Vignesh, > > > -Original Message- > > From: Vignesh R [mailto:vigne...@ti.com] > > Sent: Wednesday, October 3, 2018 10:26 PM > > To:

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Thu, 4 Oct 2018 00:14:15 +0200 Boris Brezillon wrote: > On Wed, 3 Oct 2018 23:53:27 +0200 > Ricardo Ribalda Delgado wrote: > > > Hi Boris > > On Wed, Oct 3, 2018 at 11:27 PM Boris Brezillon > > wrote: > > > > > > On Wed, 3 Oct 2018 21:38:

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Thu, 4 Oct 2018 00:14:15 +0200 Boris Brezillon wrote: > On Wed, 3 Oct 2018 23:53:27 +0200 > Ricardo Ribalda Delgado wrote: > > > Hi Boris > > On Wed, Oct 3, 2018 at 11:27 PM Boris Brezillon > > wrote: > > > > > > On Wed, 3 Oct 2018 21:38:

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 23:53:27 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Wed, Oct 3, 2018 at 11:27 PM Boris Brezillon > wrote: > > > > On Wed, 3 Oct 2018 21:38:58 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Add documentation for gp

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 23:53:27 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Wed, Oct 3, 2018 at 11:27 PM Boris Brezillon > wrote: > > > > On Wed, 3 Oct 2018 21:38:58 +0200 > > Ricardo Ribalda Delgado wrote: > > > > > Add documentation for gp

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:58 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:58 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [PATCH v5 8/8] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:59 +0200 Ricardo Ribalda Delgado wrote: > Allow creating gpio-addr-flash via device-tree and not just via platform > data. > > Option parsing has been moved to separated functions. > > Signed-off-by: Ricardo Ribalda Delgado > --- > drivers/mtd/maps/gpio-addr-flash.c

Re: [PATCH v5 8/8] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:59 +0200 Ricardo Ribalda Delgado wrote: > Allow creating gpio-addr-flash via device-tree and not just via platform > data. > > Option parsing has been moved to separated functions. > > Signed-off-by: Ricardo Ribalda Delgado > --- > drivers/mtd/maps/gpio-addr-flash.c

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:58 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [PATCH v5 7/8] dt-binding: mtd: Document gpio-addr-flash

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 21:38:58 +0200 Ricardo Ribalda Delgado wrote: > Add documentation for gpio-addr-flash. This binding allow creating > flash devices that are paged using GPIOs. > > Cc: devicet...@vger.kernel.org > Reviewed-by: Rob Herring > Signed-off-by: Ricardo Ribalda Delgado > --- >

Re: [LINUX PATCH v11 3/3] mtd: rawnand: arasan: Add support for Arasan NAND Flash Controller

2018-10-03 Thread Boris Brezillon
On Tue, 25 Sep 2018 17:50:31 +0530 Naga Sureshkumar Relli wrote: > +static int anfc_zero_len_page_write_type_exec(struct nand_chip *chip, > + const struct nand_subop *subop) > +{ > + const struct nand_op_instr *instr; > + struct anfc_nand_chip

Re: [LINUX PATCH v11 3/3] mtd: rawnand: arasan: Add support for Arasan NAND Flash Controller

2018-10-03 Thread Boris Brezillon
On Tue, 25 Sep 2018 17:50:31 +0530 Naga Sureshkumar Relli wrote: > +static int anfc_zero_len_page_write_type_exec(struct nand_chip *chip, > + const struct nand_subop *subop) > +{ > + const struct nand_op_instr *instr; > + struct anfc_nand_chip

Re: [LINUX PATCH v11 3/3] mtd: rawnand: arasan: Add support for Arasan NAND Flash Controller

2018-10-03 Thread Boris Brezillon
Hi Naga, On Tue, 25 Sep 2018 17:50:31 +0530 Naga Sureshkumar Relli wrote: > +static int anfc_read_param_get_feature_sp_read_type_exec(struct nand_chip > *chip, > + const struct nand_subop > +

Re: [LINUX PATCH v11 3/3] mtd: rawnand: arasan: Add support for Arasan NAND Flash Controller

2018-10-03 Thread Boris Brezillon
Hi Naga, On Tue, 25 Sep 2018 17:50:31 +0530 Naga Sureshkumar Relli wrote: > +static int anfc_read_param_get_feature_sp_read_type_exec(struct nand_chip > *chip, > + const struct nand_subop > +

Re: [LINUX PATCH v11 2/3] mtd: rawnand: Add an option to get sdr timing mode number

2018-10-03 Thread Boris Brezillon
mode: sdr timing mode value @mode: ONFI timing mode > */ > struct nand_sdr_timings { > u64 tBERS_max; > @@ -763,6 +764,7 @@ struct nand_sdr_timings { > u32 tWHR_min; > u32 tWP_min; > u32 tWW_min; > + u8 mode; > }; > > /** With this addressed, you can add Reviewed-by: Boris Brezillon

Re: [LINUX PATCH v11 2/3] mtd: rawnand: Add an option to get sdr timing mode number

2018-10-03 Thread Boris Brezillon
mode: sdr timing mode value @mode: ONFI timing mode > */ > struct nand_sdr_timings { > u64 tBERS_max; > @@ -763,6 +764,7 @@ struct nand_sdr_timings { > u32 tWHR_min; > u32 tWP_min; > u32 tWW_min; > + u8 mode; > }; > > /** With this addressed, you can add Reviewed-by: Boris Brezillon

Re: [PATCH 0/3] spi-nor: Add Octal SPI support

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 22:26:00 +0530 Vignesh R wrote: > This series adds support for octal mode of mt35x flash. Also, adds > support for OSPI version of Cadence QSPI controller. > > Based on top of patches adding basic support for mt35xu512aba here: > https://patchwork.ozlabs.org/cover/971437/ >

Re: [PATCH 0/3] spi-nor: Add Octal SPI support

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 22:26:00 +0530 Vignesh R wrote: > This series adds support for octal mode of mt35x flash. Also, adds > support for OSPI version of Cadence QSPI controller. > > Based on top of patches adding basic support for mt35xu512aba here: > https://patchwork.ozlabs.org/cover/971437/ >

Re: [PATCH v4 8/8] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-03 Thread Boris Brezillon
On Mon, 1 Oct 2018 14:43:51 +0200 Ricardo Ribalda Delgado wrote: > +static int gpio_flash_probe_gpios(struct platform_device *pdev, > + struct async_state *state) > +{ > + struct physmap_flash_data *pdata; > + struct device_node *dn; > + struct resource

Re: [PATCH v4 8/8] mtd: maps: gpio-addr-flash: Add support for device-tree devices

2018-10-03 Thread Boris Brezillon
On Mon, 1 Oct 2018 14:43:51 +0200 Ricardo Ribalda Delgado wrote: > +static int gpio_flash_probe_gpios(struct platform_device *pdev, > + struct async_state *state) > +{ > + struct physmap_flash_data *pdata; > + struct device_node *dn; > + struct resource

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
Hi Ricardo, On Wed, 3 Oct 2018 17:56:03 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Wed, Oct 3, 2018 at 5:17 PM Boris Brezillon > wrote: > > > > On Wed, 3 Oct 2018 17:11:14 +0200 > > Boris Brezillon wrote: > > > > > Hi Ricardo, >

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
Hi Ricardo, On Wed, 3 Oct 2018 17:56:03 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > On Wed, Oct 3, 2018 at 5:17 PM Boris Brezillon > wrote: > > > > On Wed, 3 Oct 2018 17:11:14 +0200 > > Boris Brezillon wrote: > > > > > Hi Ricardo, >

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 17:11:14 +0200 Boris Brezillon wrote: > Hi Ricardo, > > On Mon, 1 Oct 2018 14:43:49 +0200 > Ricardo Ribalda Delgado wrote: > > > @@ -248,14 +252,19 @@ static int gpio_flash_probe(struct platform_device > > *pdev) >

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
On Wed, 3 Oct 2018 17:11:14 +0200 Boris Brezillon wrote: > Hi Ricardo, > > On Mon, 1 Oct 2018 14:43:49 +0200 > Ricardo Ribalda Delgado wrote: > > > @@ -248,14 +252,19 @@ static int gpio_flash_probe(struct platform_device > > *pdev) >

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
Hi Ricardo, On Mon, 1 Oct 2018 14:43:49 +0200 Ricardo Ribalda Delgado wrote: > @@ -248,14 +252,19 @@ static int gpio_flash_probe(struct platform_device > *pdev) > > i = 0; > do { > - if (devm_gpio_request(>dev, state->gpio_addrs[i], > -

Re: [PATCH v4 6/8] mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-03 Thread Boris Brezillon
Hi Ricardo, On Mon, 1 Oct 2018 14:43:49 +0200 Ricardo Ribalda Delgado wrote: > @@ -248,14 +252,19 @@ static int gpio_flash_probe(struct platform_device > *pdev) > > i = 0; > do { > - if (devm_gpio_request(>dev, state->gpio_addrs[i], > -

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-03 Thread Boris Brezillon
On Wed, 03 Oct 2018 15:55:25 +0200 Janusz Krzysztofik wrote: > > > > > > Implementation of NAND_OP_WAITRDY_INSTR has been based on legacy > > > nand_wait_ready(), > > > > I don't remember what the ams-delta ->dev_ready()/->waitfunc() hooks > > are doing, but is shouldn't be too hard to

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-03 Thread Boris Brezillon
On Wed, 03 Oct 2018 15:55:25 +0200 Janusz Krzysztofik wrote: > > > > > > Implementation of NAND_OP_WAITRDY_INSTR has been based on legacy > > > nand_wait_ready(), > > > > I don't remember what the ams-delta ->dev_ready()/->waitfunc() hooks > > are doing, but is shouldn't be too hard to

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-03 Thread Boris Brezillon
ams_delta private struct so that this driver no longer uses the ->IO_ADDR_R/W fields? > > Suggested-by: Boris Brezillon > Signed-off-by: Janusz Krzysztofik > --- > Hi, > > I've not tested the change on hardware yet as I'm not sure if: > - handling of NCE limited to that insi

Re: [RFC PATCH] mtd: rawnand: ams-delta: use ->exec_op()

2018-10-03 Thread Boris Brezillon
ams_delta private struct so that this driver no longer uses the ->IO_ADDR_R/W fields? > > Suggested-by: Boris Brezillon > Signed-off-by: Janusz Krzysztofik > --- > Hi, > > I've not tested the change on hardware yet as I'm not sure if: > - handling of NCE limited to that insi

Re: [PATCH] mtd: spi-nor: Support controllers with limited TX FIFO size

2018-10-03 Thread Boris Brezillon
On Wed, 13 Jun 2018 11:38:12 +0530 Yogesh Gaur wrote: > Some SPI controllers can't write nor->page_size bytes in a single > step because their TX FIFO is too small. > > Allow nor->write() to return a size that is smaller than the requested > write size to gracefully handle this case. > >

Re: [PATCH] mtd: spi-nor: Support controllers with limited TX FIFO size

2018-10-03 Thread Boris Brezillon
On Wed, 13 Jun 2018 11:38:12 +0530 Yogesh Gaur wrote: > Some SPI controllers can't write nor->page_size bytes in a single > step because their TX FIFO is too small. > > Allow nor->write() to return a size that is smaller than the requested > write size to gracefully handle this case. > >

Re: [PATCH] mtd: devices: m25p80: Make sure WRITE_EN is issued before each write

2018-10-03 Thread Boris Brezillon
On Wed, 13 Jun 2018 11:39:18 +0530 Yogesh Gaur wrote: > Some SPI controllers can't write nor->page_size bytes in a single step > because their TX FIFO is too small, but when that happens we should > make sure a WRITE_EN command before each write access and READ_SR command > after each write

Re: [PATCH] mtd: devices: m25p80: Make sure WRITE_EN is issued before each write

2018-10-03 Thread Boris Brezillon
On Wed, 13 Jun 2018 11:39:18 +0530 Yogesh Gaur wrote: > Some SPI controllers can't write nor->page_size bytes in a single step > because their TX FIFO is too small, but when that happens we should > make sure a WRITE_EN command before each write access and READ_SR command > after each write

Re: [PATCH v2] mtd: rawnand: denali: set SPARE_AREA_SKIP_BYTES register to 8 if unset

2018-10-02 Thread Boris Brezillon
3] was suggested > as a counter-implementation. > (https://lore.kernel.org/patchwork/patch/983055/) > > The default value 8 was chosen to match to the boot ROM of the UniPhier > platform. The preferred value may vary by platform. If so, please > trade up to a different solution.

Re: [PATCH v2] mtd: rawnand: denali: set SPARE_AREA_SKIP_BYTES register to 8 if unset

2018-10-02 Thread Boris Brezillon
3] was suggested > as a counter-implementation. > (https://lore.kernel.org/patchwork/patch/983055/) > > The default value 8 was chosen to match to the boot ROM of the UniPhier > platform. The preferred value may vary by platform. If so, please > trade up to a different solution.

Re: [PATCH v2 5/8] mtd: maps: gpio-addr-flash: Replace array with an integer

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 14:10:03 +0200 Ricardo Ribalda Delgado wrote: > > > + return; > > > + > > > + for (i = 0; i < state->gpio_count; i++) { > > > + if ((ofs & BIT(i)) == (state->gpio_values & BIT(i))) > > > > Parens around the xx & BIT(i) operations are unneeded. >

Re: [PATCH v2 5/8] mtd: maps: gpio-addr-flash: Replace array with an integer

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 14:10:03 +0200 Ricardo Ribalda Delgado wrote: > > > + return; > > > + > > > + for (i = 0; i < state->gpio_count; i++) { > > > + if ((ofs & BIT(i)) == (state->gpio_values & BIT(i))) > > > > Parens around the xx & BIT(i) operations are unneeded. >

Re: [PATCH v2 6/8] mtd: maps: gpio-addr-flash: Split allocation in two

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 11:58:49 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > PLEASE do not apply this one! Okay, should I wait for a v6 of the whole series, or do you still want me to apply patches 1 to 5?

Re: [PATCH v2 6/8] mtd: maps: gpio-addr-flash: Split allocation in two

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 11:58:49 +0200 Ricardo Ribalda Delgado wrote: > Hi Boris > > PLEASE do not apply this one! Okay, should I wait for a v6 of the whole series, or do you still want me to apply patches 1 to 5?

Re: [PATCH v3 1/5] spi: spi-mem: Add driver for NXP FlexSPI controller

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 09:02:32 + Yogesh Narayan Gaur wrote: > > > static void fspi_writel(struct nxp_fspi *f, u32 val, void __iomem > > > *addr) { > > > if (f->big_endian) > > > iowrite32be(val, addr); > > > else > > > iowrite32(val, addr); > > > } > > > > > > > > >

Re: [PATCH v3 1/5] spi: spi-mem: Add driver for NXP FlexSPI controller

2018-10-01 Thread Boris Brezillon
On Mon, 1 Oct 2018 09:02:32 + Yogesh Narayan Gaur wrote: > > > static void fspi_writel(struct nxp_fspi *f, u32 val, void __iomem > > > *addr) { > > > if (f->big_endian) > > > iowrite32be(val, addr); > > > else > > > iowrite32(val, addr); > > > } > > > > > > > > >

Re: [PATCH v2 2/4] spi: spi-fsl-dspi: Fix delete the processing of undefined bitmask for rxdata

2018-09-30 Thread Boris Brezillon
On Sun, 30 Sep 2018 12:37:38 +0200 Esben Haabendal wrote: > Boris Brezillon writes: > > > On Sun, 30 Sep 2018 10:10:14 + > > Chuanhua Han wrote: > > > >> > -Original Message- > >> > From: Boris Brezillon > >> > Sen

Re: [PATCH v2 2/4] spi: spi-fsl-dspi: Fix delete the processing of undefined bitmask for rxdata

2018-09-30 Thread Boris Brezillon
On Sun, 30 Sep 2018 12:37:38 +0200 Esben Haabendal wrote: > Boris Brezillon writes: > > > On Sun, 30 Sep 2018 10:10:14 + > > Chuanhua Han wrote: > > > >> > -Original Message- > >> > From: Boris Brezillon > >> > Sen

Re: [PATCH v2 1/4] spi: spi-mem: Add the spi_set_xfer_bpw function

2018-09-30 Thread Boris Brezillon
On Sun, 30 Sep 2018 10:18:18 + Chuanhua Han wrote: > > -Original Message- > > From: Boris Brezillon > > Sent: 2018年9月30日 18:04 > > To: Chuanhua Han > > Cc: broo...@kernel.org; linux-...@vger.kernel.org; > > linux-kernel@vger.kernel.org; e...@dei

Re: [PATCH v2 1/4] spi: spi-mem: Add the spi_set_xfer_bpw function

2018-09-30 Thread Boris Brezillon
On Sun, 30 Sep 2018 10:18:18 + Chuanhua Han wrote: > > -Original Message- > > From: Boris Brezillon > > Sent: 2018年9月30日 18:04 > > To: Chuanhua Han > > Cc: broo...@kernel.org; linux-...@vger.kernel.org; > > linux-kernel@vger.kernel.org; e...@dei

<    5   6   7   8   9   10   11   12   13   14   >