* [PATCH] docs: mtd: Update spi nor reference driver
@ 2019-08-06 16:06 ` John Garry
0 siblings, 0 replies; 24+ messages in thread
From: John Garry @ 2019-08-06 16:06 UTC (permalink / raw)
To: corbet, mchehab+samsung, linux-mtd
Cc: linux-doc, linux-kernel, frieder.schrempf, John Garry
The reference driver no longer exists since commit 50f1242c6742 ("mtd:
fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c").
Update reference to spi-fsl-qspi.c driver.
Signed-off-by: John Garry <john.garry@huawei.com>
diff --git a/Documentation/driver-api/mtd/spi-nor.rst b/Documentation/driver-api/mtd/spi-nor.rst
index f5333e3bf486..1f0437676762 100644
--- a/Documentation/driver-api/mtd/spi-nor.rst
+++ b/Documentation/driver-api/mtd/spi-nor.rst
@@ -59,7 +59,7 @@ Part III - How can drivers use the framework?
The main API is spi_nor_scan(). Before you call the hook, a driver should
initialize the necessary fields for spi_nor{}. Please see
-drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to fsl-quadspi.c
+drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to spi-fsl-qspi.c
when you want to write a new driver for a SPI NOR controller.
Another API is spi_nor_restore(), this is used to restore the status of SPI
flash chip such as addressing mode. Call it whenever detach the driver from
--
2.17.1
^ permalink raw reply related [flat|nested] 24+ messages in thread* [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-06 16:06 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-06 16:06 UTC (permalink / raw) To: corbet, mchehab+samsung, linux-mtd Cc: John Garry, linux-kernel, frieder.schrempf, linux-doc The reference driver no longer exists since commit 50f1242c6742 ("mtd: fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). Update reference to spi-fsl-qspi.c driver. Signed-off-by: John Garry <john.garry@huawei.com> diff --git a/Documentation/driver-api/mtd/spi-nor.rst b/Documentation/driver-api/mtd/spi-nor.rst index f5333e3bf486..1f0437676762 100644 --- a/Documentation/driver-api/mtd/spi-nor.rst +++ b/Documentation/driver-api/mtd/spi-nor.rst @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? The main API is spi_nor_scan(). Before you call the hook, a driver should initialize the necessary fields for spi_nor{}. Please see -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to fsl-quadspi.c +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to spi-fsl-qspi.c when you want to write a new driver for a SPI NOR controller. Another API is spi_nor_restore(), this is used to restore the status of SPI flash chip such as addressing mode. Call it whenever detach the driver from -- 2.17.1 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-06 16:06 ` John Garry @ 2019-08-06 16:35 ` John Garry -1 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-06 16:35 UTC (permalink / raw) To: corbet, mchehab+samsung, linux-mtd Cc: linux-doc, linux-kernel, frieder.schrempf On 06/08/2019 17:06, John Garry wrote: > The reference driver no longer exists since commit 50f1242c6742 ("mtd: > fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). > > Update reference to spi-fsl-qspi.c driver. > > Signed-off-by: John Garry <john.garry@huawei.com> > > diff --git a/Documentation/driver-api/mtd/spi-nor.rst b/Documentation/driver-api/mtd/spi-nor.rst > index f5333e3bf486..1f0437676762 100644 > --- a/Documentation/driver-api/mtd/spi-nor.rst > +++ b/Documentation/driver-api/mtd/spi-nor.rst In fact this document has many references to Freescale QuadSPI - could someone kindly review this complete document for up-to-date accuracy? Thanks, John > @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? > > The main API is spi_nor_scan(). Before you call the hook, a driver should > initialize the necessary fields for spi_nor{}. Please see > -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to fsl-quadspi.c > +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to spi-fsl-qspi.c > when you want to write a new driver for a SPI NOR controller. > Another API is spi_nor_restore(), this is used to restore the status of SPI > flash chip such as addressing mode. Call it whenever detach the driver from > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-06 16:35 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-06 16:35 UTC (permalink / raw) To: corbet, mchehab+samsung, linux-mtd Cc: linux-kernel, frieder.schrempf, linux-doc On 06/08/2019 17:06, John Garry wrote: > The reference driver no longer exists since commit 50f1242c6742 ("mtd: > fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). > > Update reference to spi-fsl-qspi.c driver. > > Signed-off-by: John Garry <john.garry@huawei.com> > > diff --git a/Documentation/driver-api/mtd/spi-nor.rst b/Documentation/driver-api/mtd/spi-nor.rst > index f5333e3bf486..1f0437676762 100644 > --- a/Documentation/driver-api/mtd/spi-nor.rst > +++ b/Documentation/driver-api/mtd/spi-nor.rst In fact this document has many references to Freescale QuadSPI - could someone kindly review this complete document for up-to-date accuracy? Thanks, John > @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? > > The main API is spi_nor_scan(). Before you call the hook, a driver should > initialize the necessary fields for spi_nor{}. Please see > -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to fsl-quadspi.c > +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to spi-fsl-qspi.c > when you want to write a new driver for a SPI NOR controller. > Another API is spi_nor_restore(), this is used to restore the status of SPI > flash chip such as addressing mode. Call it whenever detach the driver from > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-06 16:35 ` John Garry @ 2019-08-06 16:40 ` Schrempf Frieder -1 siblings, 0 replies; 24+ messages in thread From: Schrempf Frieder @ 2019-08-06 16:40 UTC (permalink / raw) To: John Garry, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, broonie@kernel.org, miquel.raynal@bootlin.com, richard@nod.at, vigneshr@ti.com Cc: +MTD/SPI-NOR/SPI maintainers Hi John, On 06.08.19 18:35, John Garry wrote: > On 06/08/2019 17:06, John Garry wrote: >> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >> >> Update reference to spi-fsl-qspi.c driver. >> >> Signed-off-by: John Garry <john.garry@huawei.com> >> >> diff --git a/Documentation/driver-api/mtd/spi-nor.rst >> b/Documentation/driver-api/mtd/spi-nor.rst >> index f5333e3bf486..1f0437676762 100644 >> --- a/Documentation/driver-api/mtd/spi-nor.rst >> +++ b/Documentation/driver-api/mtd/spi-nor.rst > > In fact this document has many references to Freescale QuadSPI - could > someone kindly review this complete document for up-to-date accuracy? The new driver spi-fsl-qspi.c is not a SPI NOR controller driver anymore. It is now a SPI controller driver that uses the SPI MEM API, so referencing it here is obsolete. Actually it seems like the whole file is obsolete and needs to be removed or replaced by proper documentation of the SPI MEM API. @Maintainers: Maybe the docs under Documentation/driver-api/mtd should be officially maintained by the MTD subsystem (and added to MAINTAINERS). And if there will be some driver API docs for SPI MEM it should probably live in Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as spi-mem.c itself is in drivers/spi. Regards, Frieder > > Thanks, > John > >> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >> >> The main API is spi_nor_scan(). Before you call the hook, a driver >> should >> initialize the necessary fields for spi_nor{}. Please see >> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >> fsl-quadspi.c >> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >> spi-fsl-qspi.c >> when you want to write a new driver for a SPI NOR controller. >> Another API is spi_nor_restore(), this is used to restore the status >> of SPI >> flash chip such as addressing mode. Call it whenever detach the >> driver from >> > > > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-06 16:40 ` Schrempf Frieder 0 siblings, 0 replies; 24+ messages in thread From: Schrempf Frieder @ 2019-08-06 16:40 UTC (permalink / raw) To: John Garry, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: vigneshr@ti.com, tudor.ambarus@microchip.com, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, marek.vasut@gmail.com, broonie@kernel.org, miquel.raynal@bootlin.com Cc: +MTD/SPI-NOR/SPI maintainers Hi John, On 06.08.19 18:35, John Garry wrote: > On 06/08/2019 17:06, John Garry wrote: >> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >> >> Update reference to spi-fsl-qspi.c driver. >> >> Signed-off-by: John Garry <john.garry@huawei.com> >> >> diff --git a/Documentation/driver-api/mtd/spi-nor.rst >> b/Documentation/driver-api/mtd/spi-nor.rst >> index f5333e3bf486..1f0437676762 100644 >> --- a/Documentation/driver-api/mtd/spi-nor.rst >> +++ b/Documentation/driver-api/mtd/spi-nor.rst > > In fact this document has many references to Freescale QuadSPI - could > someone kindly review this complete document for up-to-date accuracy? The new driver spi-fsl-qspi.c is not a SPI NOR controller driver anymore. It is now a SPI controller driver that uses the SPI MEM API, so referencing it here is obsolete. Actually it seems like the whole file is obsolete and needs to be removed or replaced by proper documentation of the SPI MEM API. @Maintainers: Maybe the docs under Documentation/driver-api/mtd should be officially maintained by the MTD subsystem (and added to MAINTAINERS). And if there will be some driver API docs for SPI MEM it should probably live in Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as spi-mem.c itself is in drivers/spi. Regards, Frieder > > Thanks, > John > >> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >> >> The main API is spi_nor_scan(). Before you call the hook, a driver >> should >> initialize the necessary fields for spi_nor{}. Please see >> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >> fsl-quadspi.c >> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >> spi-fsl-qspi.c >> when you want to write a new driver for a SPI NOR controller. >> Another API is spi_nor_restore(), this is used to restore the status >> of SPI >> flash chip such as addressing mode. Call it whenever detach the >> driver from >> > > > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-06 16:40 ` Schrempf Frieder @ 2019-08-16 10:20 ` John Garry -1 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-16 10:20 UTC (permalink / raw) To: Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, broonie@kernel.org, miquel.raynal@bootlin.com, richard@nod.at, vigneshr@ti.com, wanghuiqiang On 06/08/2019 17:40, Schrempf Frieder wrote: > Cc: +MTD/SPI-NOR/SPI maintainers > > Hi John, > > On 06.08.19 18:35, John Garry wrote: >> On 06/08/2019 17:06, John Garry wrote: >>> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >>> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >>> >>> Update reference to spi-fsl-qspi.c driver. >>> >>> Signed-off-by: John Garry <john.garry@huawei.com> >>> >>> diff --git a/Documentation/driver-api/mtd/spi-nor.rst >>> b/Documentation/driver-api/mtd/spi-nor.rst >>> index f5333e3bf486..1f0437676762 100644 >>> --- a/Documentation/driver-api/mtd/spi-nor.rst >>> +++ b/Documentation/driver-api/mtd/spi-nor.rst >> >> In fact this document has many references to Freescale QuadSPI - could >> someone kindly review this complete document for up-to-date accuracy? > > The new driver spi-fsl-qspi.c is not a SPI NOR controller driver > anymore. It is now a SPI controller driver that uses the SPI MEM API, so > referencing it here is obsolete. > > Actually it seems like the whole file is obsolete and needs to be > removed or replaced by proper documentation of the SPI MEM API. Hi, Could someone kindly advise on the following: I am looking at ACPI support only for an mtd spi nor driver we're targeting for mainline support. So for the host, I could use a proprietary HID in the DSDT for matching in the kernel driver. About the child spi flash devices, is the recommendation to just use PRP0001 HID and "jedec,spi-nor" compatible? thanks, John > > @Maintainers: > Maybe the docs under Documentation/driver-api/mtd should be officially > maintained by the MTD subsystem (and added to MAINTAINERS). And if there > will be some driver API docs for SPI MEM it should probably live in > Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as > spi-mem.c itself is in drivers/spi. > > Regards, > Frieder > >> >> Thanks, >> John >> >>> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >>> >>> The main API is spi_nor_scan(). Before you call the hook, a driver >>> should >>> initialize the necessary fields for spi_nor{}. Please see >>> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>> fsl-quadspi.c >>> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>> spi-fsl-qspi.c >>> when you want to write a new driver for a SPI NOR controller. >>> Another API is spi_nor_restore(), this is used to restore the status >>> of SPI >>> flash chip such as addressing mode. Call it whenever detach the >>> driver from >>> >> >> >> >> ______________________________________________________ >> Linux MTD discussion mailing list >> http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-16 10:20 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-16 10:20 UTC (permalink / raw) To: Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: vigneshr@ti.com, tudor.ambarus@microchip.com, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, marek.vasut@gmail.com, broonie@kernel.org, miquel.raynal@bootlin.com, wanghuiqiang On 06/08/2019 17:40, Schrempf Frieder wrote: > Cc: +MTD/SPI-NOR/SPI maintainers > > Hi John, > > On 06.08.19 18:35, John Garry wrote: >> On 06/08/2019 17:06, John Garry wrote: >>> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >>> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >>> >>> Update reference to spi-fsl-qspi.c driver. >>> >>> Signed-off-by: John Garry <john.garry@huawei.com> >>> >>> diff --git a/Documentation/driver-api/mtd/spi-nor.rst >>> b/Documentation/driver-api/mtd/spi-nor.rst >>> index f5333e3bf486..1f0437676762 100644 >>> --- a/Documentation/driver-api/mtd/spi-nor.rst >>> +++ b/Documentation/driver-api/mtd/spi-nor.rst >> >> In fact this document has many references to Freescale QuadSPI - could >> someone kindly review this complete document for up-to-date accuracy? > > The new driver spi-fsl-qspi.c is not a SPI NOR controller driver > anymore. It is now a SPI controller driver that uses the SPI MEM API, so > referencing it here is obsolete. > > Actually it seems like the whole file is obsolete and needs to be > removed or replaced by proper documentation of the SPI MEM API. Hi, Could someone kindly advise on the following: I am looking at ACPI support only for an mtd spi nor driver we're targeting for mainline support. So for the host, I could use a proprietary HID in the DSDT for matching in the kernel driver. About the child spi flash devices, is the recommendation to just use PRP0001 HID and "jedec,spi-nor" compatible? thanks, John > > @Maintainers: > Maybe the docs under Documentation/driver-api/mtd should be officially > maintained by the MTD subsystem (and added to MAINTAINERS). And if there > will be some driver API docs for SPI MEM it should probably live in > Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as > spi-mem.c itself is in drivers/spi. > > Regards, > Frieder > >> >> Thanks, >> John >> >>> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >>> >>> The main API is spi_nor_scan(). Before you call the hook, a driver >>> should >>> initialize the necessary fields for spi_nor{}. Please see >>> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>> fsl-quadspi.c >>> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>> spi-fsl-qspi.c >>> when you want to write a new driver for a SPI NOR controller. >>> Another API is spi_nor_restore(), this is used to restore the status >>> of SPI >>> flash chip such as addressing mode. Call it whenever detach the >>> driver from >>> >> >> >> >> ______________________________________________________ >> Linux MTD discussion mailing list >> http://lists.infradead.org/mailman/listinfo/linux-mtd/ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-16 10:20 ` John Garry @ 2019-08-19 4:39 ` Vignesh Raghavendra -1 siblings, 0 replies; 24+ messages in thread From: Vignesh Raghavendra @ 2019-08-19 4:39 UTC (permalink / raw) To: John Garry, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, broonie@kernel.org, miquel.raynal@bootlin.com, richard@nod.at, wanghuiqiang Hi, On 16/08/19 3:50 PM, John Garry wrote: > On 06/08/2019 17:40, Schrempf Frieder wrote: [...] > > Hi, > > Could someone kindly advise on the following: > > I am looking at ACPI support only for an mtd spi nor driver we're > targeting for mainline support. > If its a new driver, please add it under drivers/spi implementing SPI MEM framework. There are few drivers under drivers/spi that can be used as example. (Search for "spi_mem_ops") > So for the host, I could use a proprietary HID in the DSDT for matching > in the kernel driver. > > About the child spi flash devices, is the recommendation to just use > PRP0001 HID and "jedec,spi-nor" compatible? > I am not quite familiar with ACPI systems, but child flash device should use "jedec,spi-nor" as compatible. Regards Vignesh > thanks, > John > > >> >> @Maintainers: >> Maybe the docs under Documentation/driver-api/mtd should be officially >> maintained by the MTD subsystem (and added to MAINTAINERS). And if there >> will be some driver API docs for SPI MEM it should probably live in >> Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as >> spi-mem.c itself is in drivers/spi. >> >> Regards, >> Frieder >> >>> >>> Thanks, >>> John >>> >>>> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >>>> >>>> The main API is spi_nor_scan(). Before you call the hook, a driver >>>> should >>>> initialize the necessary fields for spi_nor{}. Please see >>>> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>>> fsl-quadspi.c >>>> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>>> spi-fsl-qspi.c >>>> when you want to write a new driver for a SPI NOR controller. >>>> Another API is spi_nor_restore(), this is used to restore the status >>>> of SPI >>>> flash chip such as addressing mode. Call it whenever detach the >>>> driver from >>>> >>> >>> >>> >>> ______________________________________________________ >>> Linux MTD discussion mailing list >>> http://lists.infradead.org/mailman/listinfo/linux-mtd/ > > -- Regards Vignesh ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-19 4:39 ` Vignesh Raghavendra 0 siblings, 0 replies; 24+ messages in thread From: Vignesh Raghavendra @ 2019-08-19 4:39 UTC (permalink / raw) To: John Garry, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: tudor.ambarus@microchip.com, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, marek.vasut@gmail.com, broonie@kernel.org, miquel.raynal@bootlin.com, wanghuiqiang Hi, On 16/08/19 3:50 PM, John Garry wrote: > On 06/08/2019 17:40, Schrempf Frieder wrote: [...] > > Hi, > > Could someone kindly advise on the following: > > I am looking at ACPI support only for an mtd spi nor driver we're > targeting for mainline support. > If its a new driver, please add it under drivers/spi implementing SPI MEM framework. There are few drivers under drivers/spi that can be used as example. (Search for "spi_mem_ops") > So for the host, I could use a proprietary HID in the DSDT for matching > in the kernel driver. > > About the child spi flash devices, is the recommendation to just use > PRP0001 HID and "jedec,spi-nor" compatible? > I am not quite familiar with ACPI systems, but child flash device should use "jedec,spi-nor" as compatible. Regards Vignesh > thanks, > John > > >> >> @Maintainers: >> Maybe the docs under Documentation/driver-api/mtd should be officially >> maintained by the MTD subsystem (and added to MAINTAINERS). And if there >> will be some driver API docs for SPI MEM it should probably live in >> Documentation/driver-api/spi instead of Documentation/driver-api/mtd, as >> spi-mem.c itself is in drivers/spi. >> >> Regards, >> Frieder >> >>> >>> Thanks, >>> John >>> >>>> @@ -59,7 +59,7 @@ Part III - How can drivers use the framework? >>>> >>>> The main API is spi_nor_scan(). Before you call the hook, a driver >>>> should >>>> initialize the necessary fields for spi_nor{}. Please see >>>> -drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>>> fsl-quadspi.c >>>> +drivers/mtd/spi-nor/spi-nor.c for detail. Please also refer to >>>> spi-fsl-qspi.c >>>> when you want to write a new driver for a SPI NOR controller. >>>> Another API is spi_nor_restore(), this is used to restore the status >>>> of SPI >>>> flash chip such as addressing mode. Call it whenever detach the >>>> driver from >>>> >>> >>> >>> >>> ______________________________________________________ >>> Linux MTD discussion mailing list >>> http://lists.infradead.org/mailman/listinfo/linux-mtd/ > > -- Regards Vignesh ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-19 4:39 ` Vignesh Raghavendra @ 2019-08-20 14:09 ` John Garry -1 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-20 14:09 UTC (permalink / raw) To: Vignesh Raghavendra, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, broonie@kernel.org, miquel.raynal@bootlin.com, richard@nod.at, wanghuiqiang On 19/08/2019 05:39, Vignesh Raghavendra wrote: > Hi, > > On 16/08/19 3:50 PM, John Garry wrote: >> On 06/08/2019 17:40, Schrempf Frieder wrote: > [...] >> >> Hi, >> >> Could someone kindly advise on the following: >> Hi Vignesh, >> I am looking at ACPI support only for an mtd spi nor driver we're >> targeting for mainline support. >> > > If its a new driver, please add it under drivers/spi implementing SPI > MEM framework. > There are few drivers under drivers/spi that can be used as example. > (Search for "spi_mem_ops" Ok, fine. I note that in doing this I would still be using the spi nor framework indirectly through the m25p80 driver. >> So for the host, I could use a proprietary HID in the DSDT for matching >> in the kernel driver. >> >> About the child spi flash devices, is the recommendation to just use >> PRP0001 HID and "jedec,spi-nor" compatible? >> > > I am not quite familiar with ACPI systems, but child flash device should > use "jedec,spi-nor" as compatible. Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 and "jedec,spi-nor" as compatible. My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other OS can understand this. All the best, John > > Regards > Vignesh > >> thanks, >> John >> >> ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-20 14:09 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-20 14:09 UTC (permalink / raw) To: Vignesh Raghavendra, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org Cc: tudor.ambarus@microchip.com, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, marek.vasut@gmail.com, broonie@kernel.org, miquel.raynal@bootlin.com, wanghuiqiang On 19/08/2019 05:39, Vignesh Raghavendra wrote: > Hi, > > On 16/08/19 3:50 PM, John Garry wrote: >> On 06/08/2019 17:40, Schrempf Frieder wrote: > [...] >> >> Hi, >> >> Could someone kindly advise on the following: >> Hi Vignesh, >> I am looking at ACPI support only for an mtd spi nor driver we're >> targeting for mainline support. >> > > If its a new driver, please add it under drivers/spi implementing SPI > MEM framework. > There are few drivers under drivers/spi that can be used as example. > (Search for "spi_mem_ops" Ok, fine. I note that in doing this I would still be using the spi nor framework indirectly through the m25p80 driver. >> So for the host, I could use a proprietary HID in the DSDT for matching >> in the kernel driver. >> >> About the child spi flash devices, is the recommendation to just use >> PRP0001 HID and "jedec,spi-nor" compatible? >> > > I am not quite familiar with ACPI systems, but child flash device should > use "jedec,spi-nor" as compatible. Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 and "jedec,spi-nor" as compatible. My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other OS can understand this. All the best, John > > Regards > Vignesh > >> thanks, >> John >> >> ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-20 14:09 ` John Garry @ 2019-08-20 16:58 ` Mark Brown -1 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2019-08-20 16:58 UTC (permalink / raw) To: John Garry Cc: Vignesh Raghavendra, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, miquel.raynal@bootlin.com, richard@nod.at, wanghuiqiang [-- Attachment #1: Type: text/plain, Size: 919 bytes --] On Tue, Aug 20, 2019 at 03:09:15PM +0100, John Garry wrote: > On 19/08/2019 05:39, Vignesh Raghavendra wrote: > > On 16/08/19 3:50 PM, John Garry wrote: > > > About the child spi flash devices, is the recommendation to just use > > > PRP0001 HID and "jedec,spi-nor" compatible? > > I am not quite familiar with ACPI systems, but child flash device should > > use "jedec,spi-nor" as compatible. > Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 > and "jedec,spi-nor" as compatible. > My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other > OS can understand this. Last I heard Windows wasn't doing anything with PRP0001 but on the other hand the idiomatic way to handle this for ACPI is as far as I can tell to have what is essentially a board file loaded based on DMI information without any real enumerability so there's no real conflict between the two methods. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-20 16:58 ` Mark Brown 0 siblings, 0 replies; 24+ messages in thread From: Mark Brown @ 2019-08-20 16:58 UTC (permalink / raw) To: John Garry Cc: Vignesh Raghavendra, corbet@lwn.net, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Schrempf Frieder, marek.vasut@gmail.com, linux-mtd@lists.infradead.org, tudor.ambarus@microchip.com, miquel.raynal@bootlin.com, mchehab+samsung@kernel.org, wanghuiqiang [-- Attachment #1.1: Type: text/plain, Size: 919 bytes --] On Tue, Aug 20, 2019 at 03:09:15PM +0100, John Garry wrote: > On 19/08/2019 05:39, Vignesh Raghavendra wrote: > > On 16/08/19 3:50 PM, John Garry wrote: > > > About the child spi flash devices, is the recommendation to just use > > > PRP0001 HID and "jedec,spi-nor" compatible? > > I am not quite familiar with ACPI systems, but child flash device should > > use "jedec,spi-nor" as compatible. > Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 > and "jedec,spi-nor" as compatible. > My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other > OS can understand this. Last I heard Windows wasn't doing anything with PRP0001 but on the other hand the idiomatic way to handle this for ACPI is as far as I can tell to have what is essentially a board file loaded based on DMI information without any real enumerability so there's no real conflict between the two methods. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 144 bytes --] ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-20 16:58 ` Mark Brown @ 2019-08-21 13:39 ` John Garry -1 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-21 13:39 UTC (permalink / raw) To: Mark Brown Cc: Vignesh Raghavendra, Schrempf Frieder, corbet@lwn.net, mchehab+samsung@kernel.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, marek.vasut@gmail.com, tudor.ambarus@microchip.com, miquel.raynal@bootlin.com, richard@nod.at, wanghuiqiang On 20/08/2019 17:58, Mark Brown wrote: > On Tue, Aug 20, 2019 at 03:09:15PM +0100, John Garry wrote: >> On 19/08/2019 05:39, Vignesh Raghavendra wrote: >>> On 16/08/19 3:50 PM, John Garry wrote: > >>>> About the child spi flash devices, is the recommendation to just use >>>> PRP0001 HID and "jedec,spi-nor" compatible? > >>> I am not quite familiar with ACPI systems, but child flash device should >>> use "jedec,spi-nor" as compatible. > >> Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 >> and "jedec,spi-nor" as compatible. > >> My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other >> OS can understand this. > Hi Mark, > Last I heard Windows wasn't doing anything with PRP0001 but on the other > hand the idiomatic way to handle this for ACPI is as far as I can tell > to have what is essentially a board file loaded based on DMI information > without any real enumerability so there's no real conflict between the > two methods. Fine, I'll consider this alt method further. Thanks, John > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-08-21 13:39 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-08-21 13:39 UTC (permalink / raw) To: Mark Brown Cc: Vignesh Raghavendra, corbet@lwn.net, richard@nod.at, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Schrempf Frieder, marek.vasut@gmail.com, linux-mtd@lists.infradead.org, tudor.ambarus@microchip.com, miquel.raynal@bootlin.com, mchehab+samsung@kernel.org, wanghuiqiang On 20/08/2019 17:58, Mark Brown wrote: > On Tue, Aug 20, 2019 at 03:09:15PM +0100, John Garry wrote: >> On 19/08/2019 05:39, Vignesh Raghavendra wrote: >>> On 16/08/19 3:50 PM, John Garry wrote: > >>>> About the child spi flash devices, is the recommendation to just use >>>> PRP0001 HID and "jedec,spi-nor" compatible? > >>> I am not quite familiar with ACPI systems, but child flash device should >>> use "jedec,spi-nor" as compatible. > >> Right, so to use SPI MEM framework, it looks like I will have to use PRP0001 >> and "jedec,spi-nor" as compatible. > >> My reluctance in using PRP0001 and compatible "jedec,spi-nor" is how other >> OS can understand this. > Hi Mark, > Last I heard Windows wasn't doing anything with PRP0001 but on the other > hand the idiomatic way to handle this for ACPI is as far as I can tell > to have what is essentially a board file loaded based on DMI information > without any real enumerability so there's no real conflict between the > two methods. Fine, I'll consider this alt method further. Thanks, John > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-08-06 16:06 ` John Garry @ 2019-09-06 14:52 ` Jonathan Corbet -1 siblings, 0 replies; 24+ messages in thread From: Jonathan Corbet @ 2019-09-06 14:52 UTC (permalink / raw) To: John Garry Cc: mchehab+samsung, linux-mtd, linux-doc, linux-kernel, frieder.schrempf On Wed, 7 Aug 2019 00:06:23 +0800 John Garry <john.garry@huawei.com> wrote: > The reference driver no longer exists since commit 50f1242c6742 ("mtd: > fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). > > Update reference to spi-fsl-qspi.c driver. > > Signed-off-by: John Garry <john.garry@huawei.com> So this appears to have languished for a month...applied now, sorry for the delay. Thanks, jon ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-09-06 14:52 ` Jonathan Corbet 0 siblings, 0 replies; 24+ messages in thread From: Jonathan Corbet @ 2019-09-06 14:52 UTC (permalink / raw) To: John Garry Cc: mchehab+samsung, linux-mtd, linux-kernel, frieder.schrempf, linux-doc On Wed, 7 Aug 2019 00:06:23 +0800 John Garry <john.garry@huawei.com> wrote: > The reference driver no longer exists since commit 50f1242c6742 ("mtd: > fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). > > Update reference to spi-fsl-qspi.c driver. > > Signed-off-by: John Garry <john.garry@huawei.com> So this appears to have languished for a month...applied now, sorry for the delay. Thanks, jon ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-09-06 14:52 ` Jonathan Corbet @ 2019-09-06 14:58 ` John Garry -1 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-09-06 14:58 UTC (permalink / raw) To: Jonathan Corbet Cc: mchehab+samsung, linux-mtd, linux-doc, linux-kernel, frieder.schrempf On 06/09/2019 15:52, Jonathan Corbet wrote: > On Wed, 7 Aug 2019 00:06:23 +0800 > John Garry <john.garry@huawei.com> wrote: > >> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >> >> Update reference to spi-fsl-qspi.c driver. >> >> Signed-off-by: John Garry <john.garry@huawei.com> > > So this appears to have languished for a month...applied now, sorry for > the delay. > > Thanks, > > jon > > . > Hi Jon, I don't think that it was appropriate to apply this patch in the end - maybe this could have been communicated better. If you check the subsequent discussion in this thread, it seems that completely new documentation is required: "Actually it seems like the whole file is obsolete and needs to be removed or replaced by proper documentation of the SPI MEM API." But nothing seems to be happening there... Thanks, John ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-09-06 14:58 ` John Garry 0 siblings, 0 replies; 24+ messages in thread From: John Garry @ 2019-09-06 14:58 UTC (permalink / raw) To: Jonathan Corbet Cc: mchehab+samsung, linux-mtd, linux-kernel, frieder.schrempf, linux-doc On 06/09/2019 15:52, Jonathan Corbet wrote: > On Wed, 7 Aug 2019 00:06:23 +0800 > John Garry <john.garry@huawei.com> wrote: > >> The reference driver no longer exists since commit 50f1242c6742 ("mtd: >> fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c"). >> >> Update reference to spi-fsl-qspi.c driver. >> >> Signed-off-by: John Garry <john.garry@huawei.com> > > So this appears to have languished for a month...applied now, sorry for > the delay. > > Thanks, > > jon > > . > Hi Jon, I don't think that it was appropriate to apply this patch in the end - maybe this could have been communicated better. If you check the subsequent discussion in this thread, it seems that completely new documentation is required: "Actually it seems like the whole file is obsolete and needs to be removed or replaced by proper documentation of the SPI MEM API." But nothing seems to be happening there... Thanks, John ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-09-06 14:58 ` John Garry @ 2019-09-06 15:03 ` Jonathan Corbet -1 siblings, 0 replies; 24+ messages in thread From: Jonathan Corbet @ 2019-09-06 15:03 UTC (permalink / raw) To: John Garry Cc: mchehab+samsung, linux-mtd, linux-doc, linux-kernel, frieder.schrempf On Fri, 6 Sep 2019 15:58:41 +0100 John Garry <john.garry@huawei.com> wrote: > I don't think that it was appropriate to apply this patch in the end - > maybe this could have been communicated better. If you check the > subsequent discussion in this thread, it seems that completely new > documentation is required: > > "Actually it seems like the whole file is obsolete and needs to be > removed or replaced by proper documentation of the SPI MEM API." > > But nothing seems to be happening there... Ah, OK, I lost track of that somewhere along the way. Unless you object I'll leave it applied, it's not going to do any harm, methinks. jon ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-09-06 15:03 ` Jonathan Corbet 0 siblings, 0 replies; 24+ messages in thread From: Jonathan Corbet @ 2019-09-06 15:03 UTC (permalink / raw) To: John Garry Cc: mchehab+samsung, linux-mtd, linux-kernel, frieder.schrempf, linux-doc On Fri, 6 Sep 2019 15:58:41 +0100 John Garry <john.garry@huawei.com> wrote: > I don't think that it was appropriate to apply this patch in the end - > maybe this could have been communicated better. If you check the > subsequent discussion in this thread, it seems that completely new > documentation is required: > > "Actually it seems like the whole file is obsolete and needs to be > removed or replaced by proper documentation of the SPI MEM API." > > But nothing seems to be happening there... Ah, OK, I lost track of that somewhere along the way. Unless you object I'll leave it applied, it's not going to do any harm, methinks. jon ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver 2019-09-06 15:03 ` Jonathan Corbet @ 2019-09-06 15:10 ` Tudor.Ambarus -1 siblings, 0 replies; 24+ messages in thread From: Tudor.Ambarus @ 2019-09-06 15:10 UTC (permalink / raw) To: corbet, john.garry Cc: mchehab+samsung, linux-mtd, linux-doc, linux-kernel, frieder.schrempf On 09/06/2019 06:03 PM, Jonathan Corbet wrote: > On Fri, 6 Sep 2019 15:58:41 +0100 > John Garry <john.garry@huawei.com> wrote: > >> I don't think that it was appropriate to apply this patch in the end - >> maybe this could have been communicated better. If you check the >> subsequent discussion in this thread, it seems that completely new >> documentation is required: >> >> "Actually it seems like the whole file is obsolete and needs to be >> removed or replaced by proper documentation of the SPI MEM API." >> >> But nothing seems to be happening there... > > Ah, OK, I lost track of that somewhere along the way. > > Unless you object I'll leave it applied, it's not going to do any harm, > methinks. > We can rework the documentation on top of this, no worries. Cheers, ta ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] docs: mtd: Update spi nor reference driver @ 2019-09-06 15:10 ` Tudor.Ambarus 0 siblings, 0 replies; 24+ messages in thread From: Tudor.Ambarus @ 2019-09-06 15:10 UTC (permalink / raw) To: corbet, john.garry Cc: mchehab+samsung, linux-mtd, linux-kernel, frieder.schrempf, linux-doc On 09/06/2019 06:03 PM, Jonathan Corbet wrote: > On Fri, 6 Sep 2019 15:58:41 +0100 > John Garry <john.garry@huawei.com> wrote: > >> I don't think that it was appropriate to apply this patch in the end - >> maybe this could have been communicated better. If you check the >> subsequent discussion in this thread, it seems that completely new >> documentation is required: >> >> "Actually it seems like the whole file is obsolete and needs to be >> removed or replaced by proper documentation of the SPI MEM API." >> >> But nothing seems to be happening there... > > Ah, OK, I lost track of that somewhere along the way. > > Unless you object I'll leave it applied, it's not going to do any harm, > methinks. > We can rework the documentation on top of this, no worries. Cheers, ta ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2019-09-06 15:10 UTC | newest] Thread overview: 24+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-08-06 16:06 [PATCH] docs: mtd: Update spi nor reference driver John Garry 2019-08-06 16:06 ` John Garry 2019-08-06 16:35 ` John Garry 2019-08-06 16:35 ` John Garry 2019-08-06 16:40 ` Schrempf Frieder 2019-08-06 16:40 ` Schrempf Frieder 2019-08-16 10:20 ` John Garry 2019-08-16 10:20 ` John Garry 2019-08-19 4:39 ` Vignesh Raghavendra 2019-08-19 4:39 ` Vignesh Raghavendra 2019-08-20 14:09 ` John Garry 2019-08-20 14:09 ` John Garry 2019-08-20 16:58 ` Mark Brown 2019-08-20 16:58 ` Mark Brown 2019-08-21 13:39 ` John Garry 2019-08-21 13:39 ` John Garry 2019-09-06 14:52 ` Jonathan Corbet 2019-09-06 14:52 ` Jonathan Corbet 2019-09-06 14:58 ` John Garry 2019-09-06 14:58 ` John Garry 2019-09-06 15:03 ` Jonathan Corbet 2019-09-06 15:03 ` Jonathan Corbet 2019-09-06 15:10 ` Tudor.Ambarus 2019-09-06 15:10 ` Tudor.Ambarus
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.