Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1488930 > unrolled thread
| Started by | Jagan Teki <jagannadh.teki@gmail.com> |
|---|---|
| First post | 2016-09-22 16:30 +0200 |
| Last post | 2016-09-22 18:00 +0200 |
| Articles | 2 — 2 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/6] mtd: spi-nor: Add quad page program support Jagan Teki <jagannadh.teki@gmail.com> - 2016-09-22 16:30 +0200
Re: [PATCH 1/6] mtd: spi-nor: Add quad page program support Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-22 18:00 +0200
| From | Jagan Teki <jagannadh.teki@gmail.com> |
|---|---|
| Date | 2016-09-22 16:30 +0200 |
| Subject | Re: [PATCH 1/6] mtd: spi-nor: Add quad page program support |
| Message-ID | <skdaF-7wd-3@gated-at.bofh.it> |
On Mon, Aug 29, 2016 at 6:54 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> On Wed, Aug 10, 2016 at 2:10 AM, Jagan Teki <jteki@openedev.com> wrote:
>> Add quad page program support with the use of nor->flags
>> and then controller will use 4 lines for data transmission
>> which is quite faster than page program(02h)
>>
>> Cc: Brian Norris <computersforpeace@gmail.com>
>> Signed-off-by: Jagan Teki <jteki@openedev.com>
>> ---
>> drivers/mtd/spi-nor/spi-nor.c | 13 +++++++++++++
>> include/linux/mtd/spi-nor.h | 2 ++
>> 2 files changed, 15 insertions(+)
>>
>> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
>> index d0fc165..f88bd7e 100644
>> --- a/drivers/mtd/spi-nor/spi-nor.c
>> +++ b/drivers/mtd/spi-nor/spi-nor.c
>> @@ -75,6 +75,7 @@ struct flash_info {
>> * bit. Must be used with
>> * SPI_NOR_HAS_LOCK.
>> */
>> +#define SPI_NOR_QUAD_WRITE BIT(10) /* Flash supports Quad Write */
>> };
>>
>> #define JEDEC_MFR(info) ((info)->id[0])
>> @@ -1306,6 +1307,7 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
>> struct device *dev = nor->dev;
>> struct mtd_info *mtd = &nor->mtd;
>> struct device_node *np = spi_nor_get_flash_node(nor);
>> + bool need_quad = false;
>> int ret;
>> int i;
>>
>> @@ -1441,6 +1443,7 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
>> return ret;
>> }
>> nor->flash_read = SPI_NOR_QUAD;
>> + need_quad = true;
>> } else if (mode == SPI_NOR_DUAL && info->flags & SPI_NOR_DUAL_READ) {
>> nor->flash_read = SPI_NOR_DUAL;
>> }
>> @@ -1465,6 +1468,16 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
>> }
>>
>> nor->program_opcode = SPINOR_OP_PP;
>> + if (nor->flags == SNOR_F_USE_QPP && info->flags & SPI_NOR_QUAD_WRITE) {
>> + if (!need_quad)
>> + ret = set_quad_mode(nor, info);
>> + if (ret) {
>> + dev_err(dev, "quad mode not supported\n");
>> + return ret;
>> + }
>> + }
>> + nor->program_opcode = SPINOR_OP_QPP;
>> + }
>>
>> if (info->addr_width)
>> nor->addr_width = info->addr_width;
>> diff --git a/include/linux/mtd/spi-nor.h b/include/linux/mtd/spi-nor.h
>> index c425c7b..aa0e6cd 100644
>> --- a/include/linux/mtd/spi-nor.h
>> +++ b/include/linux/mtd/spi-nor.h
>> @@ -46,6 +46,7 @@
>> #define SPINOR_OP_READ_1_1_2 0x3b /* Read data bytes (Dual SPI) */
>> #define SPINOR_OP_READ_1_1_4 0x6b /* Read data bytes (Quad SPI) */
>> #define SPINOR_OP_PP 0x02 /* Page program (up to 256 bytes) */
>> +#define SPINOR_OP_QPP 0x32 /* Quad Page program */
>> #define SPINOR_OP_BE_4K 0x20 /* Erase 4KiB block */
>> #define SPINOR_OP_BE_4K_PMC 0xd7 /* Erase 4KiB block on PMC chips */
>> #define SPINOR_OP_BE_32K 0x52 /* Erase 32KiB block */
>> @@ -119,6 +120,7 @@ enum spi_nor_ops {
>> enum spi_nor_option_flags {
>> SNOR_F_USE_FSR = BIT(0),
>> SNOR_F_HAS_SR_TB = BIT(1),
>> + SNOR_F_USE_QPP = BIT(2),
>> };
>>
>> /**
>> --
>> 2.7.4
>>
>
> Ping!
Any comments? will resend rebased v2 if it is OK?
thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
[toc] | [next] | [standalone]
| From | Boris Brezillon <boris.brezillon@free-electrons.com> |
|---|---|
| Date | 2016-09-22 18:00 +0200 |
| Message-ID | <skezM-8hf-7@gated-at.bofh.it> |
| In reply to | #1488930 |
On Thu, 22 Sep 2016 19:59:13 +0530
Jagan Teki <jagannadh.teki@gmail.com> wrote:
> On Mon, Aug 29, 2016 at 6:54 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> > On Wed, Aug 10, 2016 at 2:10 AM, Jagan Teki <jteki@openedev.com> wrote:
> >> Add quad page program support with the use of nor->flags
> >> and then controller will use 4 lines for data transmission
> >> which is quite faster than page program(02h)
> >>
> >> Cc: Brian Norris <computersforpeace@gmail.com>
> >> Signed-off-by: Jagan Teki <jteki@openedev.com>
> >> ---
> >> drivers/mtd/spi-nor/spi-nor.c | 13 +++++++++++++
> >> include/linux/mtd/spi-nor.h | 2 ++
> >> 2 files changed, 15 insertions(+)
> >>
> >> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> >> index d0fc165..f88bd7e 100644
> >> --- a/drivers/mtd/spi-nor/spi-nor.c
> >> +++ b/drivers/mtd/spi-nor/spi-nor.c
> >> @@ -75,6 +75,7 @@ struct flash_info {
> >> * bit. Must be used with
> >> * SPI_NOR_HAS_LOCK.
> >> */
> >> +#define SPI_NOR_QUAD_WRITE BIT(10) /* Flash supports Quad Write */
> >> };
> >>
> >> #define JEDEC_MFR(info) ((info)->id[0])
> >> @@ -1306,6 +1307,7 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
> >> struct device *dev = nor->dev;
> >> struct mtd_info *mtd = &nor->mtd;
> >> struct device_node *np = spi_nor_get_flash_node(nor);
> >> + bool need_quad = false;
> >> int ret;
> >> int i;
> >>
> >> @@ -1441,6 +1443,7 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
> >> return ret;
> >> }
> >> nor->flash_read = SPI_NOR_QUAD;
> >> + need_quad = true;
> >> } else if (mode == SPI_NOR_DUAL && info->flags & SPI_NOR_DUAL_READ) {
> >> nor->flash_read = SPI_NOR_DUAL;
> >> }
> >> @@ -1465,6 +1468,16 @@ int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode)
> >> }
> >>
> >> nor->program_opcode = SPINOR_OP_PP;
> >> + if (nor->flags == SNOR_F_USE_QPP && info->flags & SPI_NOR_QUAD_WRITE) {
> >> + if (!need_quad)
> >> + ret = set_quad_mode(nor, info);
> >> + if (ret) {
> >> + dev_err(dev, "quad mode not supported\n");
> >> + return ret;
> >> + }
> >> + }
> >> + nor->program_opcode = SPINOR_OP_QPP;
> >> + }
> >>
> >> if (info->addr_width)
> >> nor->addr_width = info->addr_width;
> >> diff --git a/include/linux/mtd/spi-nor.h b/include/linux/mtd/spi-nor.h
> >> index c425c7b..aa0e6cd 100644
> >> --- a/include/linux/mtd/spi-nor.h
> >> +++ b/include/linux/mtd/spi-nor.h
> >> @@ -46,6 +46,7 @@
> >> #define SPINOR_OP_READ_1_1_2 0x3b /* Read data bytes (Dual SPI) */
> >> #define SPINOR_OP_READ_1_1_4 0x6b /* Read data bytes (Quad SPI) */
> >> #define SPINOR_OP_PP 0x02 /* Page program (up to 256 bytes) */
> >> +#define SPINOR_OP_QPP 0x32 /* Quad Page program */
> >> #define SPINOR_OP_BE_4K 0x20 /* Erase 4KiB block */
> >> #define SPINOR_OP_BE_4K_PMC 0xd7 /* Erase 4KiB block on PMC chips */
> >> #define SPINOR_OP_BE_32K 0x52 /* Erase 32KiB block */
> >> @@ -119,6 +120,7 @@ enum spi_nor_ops {
> >> enum spi_nor_option_flags {
> >> SNOR_F_USE_FSR = BIT(0),
> >> SNOR_F_HAS_SR_TB = BIT(1),
> >> + SNOR_F_USE_QPP = BIT(2),
> >> };
> >>
> >> /**
> >> --
> >> 2.7.4
> >>
> >
> > Ping!
>
> Any comments? will resend rebased v2 if it is OK?
Did you look at Cyrille's work [1]?
It seems you're trying to support the same kind of features.
Can you sync with him?
[1]http://lkml.iu.edu/hypermail/linux/kernel/1606.2/03302.html
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web