Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1224397 > unrolled thread
| Started by | Mark Brown <broonie@kernel.org> |
|---|---|
| First post | 2015-09-14 20:40 +0200 |
| Last post | 2015-09-16 17:40 +0200 |
| Articles | 5 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices Mark Brown <broonie@kernel.org> - 2015-09-14 20:40 +0200
Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices Vignesh R <vigneshr@ti.com> - 2015-09-16 12:10 +0200
Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices Jagan Teki <jteki@openedev.com> - 2015-09-16 14:50 +0200
Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices Mark Brown <broonie@kernel.org> - 2015-09-16 17:30 +0200
Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices Michal Suchanek <hramrach@gmail.com> - 2015-09-16 17:40 +0200
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Date | 2015-09-14 20:40 +0200 |
| Subject | Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices |
| Message-ID | <q8GPv-5VR-3@gated-at.bofh.it> |
[Multipart message — attachments visible in raw view] — view raw
On Fri, Sep 04, 2015 at 04:55:33PM +0530, Jagan Teki wrote: > On 4 September 2015 at 13:59, Vignesh R <vigneshr@ti.com> wrote: > > + * @spi_mtd_mmap_read: some spi-controller hardwares provide memory > > + * mapped interface to communicate with mtd flashes. > > + * For this, spi controller needs to know flash > > + * memory settings like read command to use, dummy > > + * bytes and address width. Once these settings are > > + * populated in hardware registers, any read > > + * accesses to flash's memory map region(as defined > > + * by SoC) through memcpy or mem-to-mem DMA copy > > + * will be handled by controller hardware. The > > + * hardware will automatically generate spi signals > > + * required to read data from flash and present it > > + * to CPU or DMA. SPI master drivers can use this > > + * callback to implement memory mapped read > > + * interface. Flash driver (like m25p80) requests > > + * memory mapped read via this method. The interface > > + * should only be used mtd flashes and cannot be > > + * used with other spi devices. This comment is *way* too verbose - probably you just need up to the "Once" here. > > + int (*spi_mtd_mmap_read)(struct spi_device *spi, > > + loff_t from, size_t len, size_t *retlen, > > + u_char *buf, u8 read_opcode, > > + u8 addr_width, u8 dummy_bytes); > This looks un-manageable to know spi core or master knows what are the > command or opcode used by spi-nor flash and also I think mmap support > seems to be flash related or any justification this is spi bus > master/controller feature? There seem to be a reasonable number of SPI controllers out there which have as an extension the ability to do memory mapped reads but are otherwise perfectly normal SPI controllers and which rely on that for everything except reads.
[toc] | [next] | [standalone]
| From | Vignesh R <vigneshr@ti.com> |
|---|---|
| Date | 2015-09-16 12:10 +0200 |
| Message-ID | <q9hP4-1id-23@gated-at.bofh.it> |
| In reply to | #1224397 |
On 09/15/2015 12:05 AM, Mark Brown wrote: > On Fri, Sep 04, 2015 at 04:55:33PM +0530, Jagan Teki wrote: >> On 4 September 2015 at 13:59, Vignesh R <vigneshr@ti.com> wrote: > >>> + * @spi_mtd_mmap_read: some spi-controller hardwares provide memory >>> + * mapped interface to communicate with mtd flashes. >>> + * For this, spi controller needs to know flash >>> + * memory settings like read command to use, dummy >>> + * bytes and address width. Once these settings are >>> + * populated in hardware registers, any read >>> + * accesses to flash's memory map region(as defined >>> + * by SoC) through memcpy or mem-to-mem DMA copy >>> + * will be handled by controller hardware. The >>> + * hardware will automatically generate spi signals >>> + * required to read data from flash and present it >>> + * to CPU or DMA. SPI master drivers can use this >>> + * callback to implement memory mapped read >>> + * interface. Flash driver (like m25p80) requests >>> + * memory mapped read via this method. The interface >>> + * should only be used mtd flashes and cannot be >>> + * used with other spi devices. > > This comment is *way* too verbose - probably you just need up to the > "Once" here. > Ok, I will move the extra text to commit log. -- Thanks, Vignesh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Jagan Teki <jteki@openedev.com> |
|---|---|
| Date | 2015-09-16 14:50 +0200 |
| Subject | Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices |
| Message-ID | <q9kjT-4IQ-17@gated-at.bofh.it> |
| In reply to | #1224397 |
On 15 September 2015 at 00:05, Mark Brown <broonie@kernel.org> wrote: > On Fri, Sep 04, 2015 at 04:55:33PM +0530, Jagan Teki wrote: >> On 4 September 2015 at 13:59, Vignesh R <vigneshr@ti.com> wrote: > >> > + * @spi_mtd_mmap_read: some spi-controller hardwares provide memory >> > + * mapped interface to communicate with mtd flashes. >> > + * For this, spi controller needs to know flash >> > + * memory settings like read command to use, dummy >> > + * bytes and address width. Once these settings are >> > + * populated in hardware registers, any read >> > + * accesses to flash's memory map region(as defined >> > + * by SoC) through memcpy or mem-to-mem DMA copy >> > + * will be handled by controller hardware. The >> > + * hardware will automatically generate spi signals >> > + * required to read data from flash and present it >> > + * to CPU or DMA. SPI master drivers can use this >> > + * callback to implement memory mapped read >> > + * interface. Flash driver (like m25p80) requests >> > + * memory mapped read via this method. The interface >> > + * should only be used mtd flashes and cannot be >> > + * used with other spi devices. > > This comment is *way* too verbose - probably you just need up to the > "Once" here. > >> > + int (*spi_mtd_mmap_read)(struct spi_device *spi, >> > + loff_t from, size_t len, size_t *retlen, >> > + u_char *buf, u8 read_opcode, >> > + u8 addr_width, u8 dummy_bytes); > >> This looks un-manageable to know spi core or master knows what are the >> command or opcode used by spi-nor flash and also I think mmap support >> seems to be flash related or any justification this is spi bus >> master/controller feature? > > There seem to be a reasonable number of SPI controllers out there which > have as an extension the ability to do memory mapped reads but are > otherwise perfectly normal SPI controllers and which rely on that for > everything except reads. This is true, but exposing direct spi-nor arguments or features like read_opcode, addr_width or dummy_byte to spi layer isn't fine? because for spi layer there is a spi to flash transition mtd m25p80.c is there to handle is it? thanks! -- Jagan | openedev. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Date | 2015-09-16 17:30 +0200 |
| Message-ID | <q9mOJ-8oQ-13@gated-at.bofh.it> |
| In reply to | #1226056 |
[Multipart message — attachments visible in raw view] — view raw
On Wed, Sep 16, 2015 at 06:16:55PM +0530, Jagan Teki wrote: > On 15 September 2015 at 00:05, Mark Brown <broonie@kernel.org> wrote: > > There seem to be a reasonable number of SPI controllers out there which > > have as an extension the ability to do memory mapped reads but are > > otherwise perfectly normal SPI controllers and which rely on that for > > everything except reads. > This is true, but exposing direct spi-nor arguments or features like > read_opcode, addr_width or dummy_byte to spi layer isn't fine? because > for spi layer there is a spi to flash transition mtd m25p80.c is there > to handle is it? The m25p80 driver is a generic, SPI level piece of code not a driver for specific controller hardware. The controller hardware drivers live in the SPI subsystem. In order to support this memory mapped feature we need the controller drivers to expose an interface for controlling it.
[toc] | [prev] | [next] | [standalone]
| From | Michal Suchanek <hramrach@gmail.com> |
|---|---|
| Date | 2015-09-16 17:40 +0200 |
| Subject | Re: [PATCH 1/5] spi: introduce mmap read support for spi flash devices |
| Message-ID | <q9mYq-8y-11@gated-at.bofh.it> |
| In reply to | #1226056 |
On 16 September 2015 at 14:46, Jagan Teki <jteki@openedev.com> wrote: > On 15 September 2015 at 00:05, Mark Brown <broonie@kernel.org> wrote: >> On Fri, Sep 04, 2015 at 04:55:33PM +0530, Jagan Teki wrote: >>> On 4 September 2015 at 13:59, Vignesh R <vigneshr@ti.com> wrote: >> >>> > + * @spi_mtd_mmap_read: some spi-controller hardwares provide memory >>> > + * mapped interface to communicate with mtd flashes. >>> > + * For this, spi controller needs to know flash >>> > + * memory settings like read command to use, dummy >>> > + * bytes and address width. Once these settings are >>> > + * populated in hardware registers, any read >>> > + * accesses to flash's memory map region(as defined >>> > + * by SoC) through memcpy or mem-to-mem DMA copy >>> > + * will be handled by controller hardware. The >>> > + * hardware will automatically generate spi signals >>> > + * required to read data from flash and present it >>> > + * to CPU or DMA. SPI master drivers can use this >>> > + * callback to implement memory mapped read >>> > + * interface. Flash driver (like m25p80) requests >>> > + * memory mapped read via this method. The interface >>> > + * should only be used mtd flashes and cannot be >>> > + * used with other spi devices. >> >> This comment is *way* too verbose - probably you just need up to the >> "Once" here. >> >>> > + int (*spi_mtd_mmap_read)(struct spi_device *spi, >>> > + loff_t from, size_t len, size_t *retlen, >>> > + u_char *buf, u8 read_opcode, >>> > + u8 addr_width, u8 dummy_bytes); >> >>> This looks un-manageable to know spi core or master knows what are the >>> command or opcode used by spi-nor flash and also I think mmap support >>> seems to be flash related or any justification this is spi bus >>> master/controller feature? >> >> There seem to be a reasonable number of SPI controllers out there which >> have as an extension the ability to do memory mapped reads but are >> otherwise perfectly normal SPI controllers and which rely on that for >> everything except reads. > > This is true, but exposing direct spi-nor arguments or features like > read_opcode, addr_width or dummy_byte to spi layer isn't fine? because > for spi layer there is a spi to flash transition mtd m25p80.c is there > to handle is it? For the TI SPI controller this is programmable. So instead of composing a SPI message that sends the opcode (and address and dummy bytes) you set the opcode in a controller register and then perform the memory read which sends the configured opcode to the flash memory. So this needs to be exposed in an API which the m25p80 driver can use. On 16 September 2015 at 12:08, Vignesh R <vigneshr@ti.com> wrote: > > > On 09/15/2015 12:07 AM, Mark Brown wrote: >> On Fri, Sep 04, 2015 at 01:59:58PM +0530, Vignesh R wrote: >>> In addition to providing direct access to SPI bus, some spi controller >>> hardwares (like ti-qspi) provide special memory mapped port >>> to accesses SPI flash devices in order to increase read performance. >>> This means the controller can automatically send the SPI signals >>> required to read data from the SPI flash device. >> >> Sorry, also meant to say here: as I kind of indicated in response to the >> flash patch I'd expect to see the SPI core know something about this and >> export an API for this which is integrated with things like the existing >> message queue. >> > > > Adding an API to SPI core makes sense to me. This can take care of spi > bus locking and runtime pm. > But, I didn't get how to integrate with existing message queue. Memory > mapped read by-passes message queue of SPI core. Could you please > explain a bit more on integrating with message queue? Did you mean > locking the existing message queue when memory mapped read is being > requested? Presumably when you have a function in SPI core for this it takes the same lock transfer_one does so either you do mmap transfer or normal SPI transfer. You can probably manually sync on transfer completion in a driver that uses both normal SPI messages and mmap transfer - which m25p80 does anyway. Thanks Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web