* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B
@ 2013-01-23 6:30 Rajeshwari Shinde
2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde
` (3 more replies)
0 siblings, 4 replies; 15+ messages in thread
From: Rajeshwari Shinde @ 2013-01-23 6:30 UTC (permalink / raw)
To: u-boot
This patch set adds driver for Gigabyte device GD25LQ and GD25Q64B
required for Snow board and enables same in config file.
Based on following patches submitted by Simon Glass:
"sf: Add spi_flash_alloc() to create a new SPI flash struct"
"sf: Use spi_flash_alloc() in each SPI flash driver"
Changes in V2:
- Added U-Boot copyright header to gigadevice.c
- Removed unnecessary blank lines.
Rajeshwari Shinde (2):
SF: Add driver for Gigabyte device GD25LQ and GD25Q64B
SMDK5250: Enable SPI Gigabyte device.
drivers/mtd/spi/Makefile | 1 +
drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++
drivers/mtd/spi/spi_flash.c | 3 +
drivers/mtd/spi/spi_flash_internal.h | 1 +
include/configs/exynos5250-dt.h | 1 +
5 files changed, 87 insertions(+), 0 deletions(-)
create mode 100644 drivers/mtd/spi/gigadevice.c
--
1.7.4.4
^ permalink raw reply [flat|nested] 15+ messages in thread* [U-Boot] [PATCH 1/2 V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Shinde @ 2013-01-23 6:30 ` Rajeshwari Shinde 2013-01-25 21:45 ` Simon Glass ` (2 more replies) 2013-01-23 6:30 ` [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device Rajeshwari Shinde ` (2 subsequent siblings) 3 siblings, 3 replies; 15+ messages in thread From: Rajeshwari Shinde @ 2013-01-23 6:30 UTC (permalink / raw) To: u-boot This patch adds driver for the gigabyte devices GD25LQ and GD25Q64B required for Snow Board. Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> --- Changes in V2: - Added U-Boot copyright header to gigadevice.c - Removed unnecessary blank lines. drivers/mtd/spi/Makefile | 1 + drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ drivers/mtd/spi/spi_flash.c | 3 + drivers/mtd/spi/spi_flash_internal.h | 1 + 4 files changed, 86 insertions(+), 0 deletions(-) create mode 100644 drivers/mtd/spi/gigadevice.c diff --git a/drivers/mtd/spi/Makefile b/drivers/mtd/spi/Makefile index 90f8392..ecbb210 100644 --- a/drivers/mtd/spi/Makefile +++ b/drivers/mtd/spi/Makefile @@ -32,6 +32,7 @@ endif COBJS-$(CONFIG_SPI_FLASH) += spi_flash.o COBJS-$(CONFIG_SPI_FLASH_ATMEL) += atmel.o COBJS-$(CONFIG_SPI_FLASH_EON) += eon.o +COBJS-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.o COBJS-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.o COBJS-$(CONFIG_SPI_FLASH_SPANSION) += spansion.o COBJS-$(CONFIG_SPI_FLASH_SST) += sst.o diff --git a/drivers/mtd/spi/gigadevice.c b/drivers/mtd/spi/gigadevice.c new file mode 100644 index 0000000..b5e1ebe --- /dev/null +++ b/drivers/mtd/spi/gigadevice.c @@ -0,0 +1,81 @@ +/* + * Gigadevice SPI flash driver + * Copyright 2013, Samsung Electronics Co., Ltd. + * Author: Banajit Goswami <banajit.g@samsung.com> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, + * MA 02111-1307 USA + */ + +#include <common.h> +#include <malloc.h> +#include <spi_flash.h> + +#include "spi_flash_internal.h" + +struct gigadevice_spi_flash_params { + uint16_t id; + uint16_t nr_blocks; + const char *name; +}; + +static const struct gigadevice_spi_flash_params gigadevice_spi_flash_table[] = { + { + .id = 0x6016, + .nr_blocks = 64, + .name = "GD25LQ", + }, + { + .id = 0x4017, + .nr_blocks = 128, + .name = "GD25Q64B", + }, +}; + +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode) +{ + const struct gigadevice_spi_flash_params *params; + struct spi_flash *flash; + unsigned int i; + + for (i = 0; i < ARRAY_SIZE(gigadevice_spi_flash_table); i++) { + params = &gigadevice_spi_flash_table[i]; + if (params->id == ((idcode[1] << 8) | idcode[2])) + break; + } + + if (i == ARRAY_SIZE(gigadevice_spi_flash_table)) { + debug("SF: Unsupported Gigadevice ID %02x%02x\n", + idcode[1], idcode[2]); + return NULL; + } + + flash = spi_flash_alloc_base(spi, params->name); + if (!flash) { + debug("SF: Failed to allocate memory\n"); + return NULL; + } + /* page_size */ + flash->page_size = 256; + /* sector_size = page_size * pages_per_sector */ + flash->sector_size = flash->page_size * 16; + /* size = sector_size * sector_per_block * number of blocks */ + flash->size = flash->sector_size * 16 * params->nr_blocks; + + return flash; +} diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 17f3d3c..ee05171 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -305,6 +305,9 @@ static const struct { #ifdef CONFIG_SPI_FLASH_EON { 0, 0x1c, spi_flash_probe_eon, }, #endif +#ifdef CONFIG_SPI_FLASH_GIGADEVICE + { 0, 0xc8, spi_flash_probe_gigadevice, }, +#endif #ifdef CONFIG_SPI_FLASH_MACRONIX { 0, 0xc2, spi_flash_probe_macronix, }, #endif diff --git a/drivers/mtd/spi/spi_flash_internal.h b/drivers/mtd/spi/spi_flash_internal.h index 141cfa8..e0afbc3 100644 --- a/drivers/mtd/spi/spi_flash_internal.h +++ b/drivers/mtd/spi/spi_flash_internal.h @@ -106,3 +106,4 @@ struct spi_flash *spi_flash_probe_sst(struct spi_slave *spi, u8 *idcode); struct spi_flash *spi_flash_probe_stmicro(struct spi_slave *spi, u8 *idcode); struct spi_flash *spi_flash_probe_winbond(struct spi_slave *spi, u8 *idcode); struct spi_flash *spi_fram_probe_ramtron(struct spi_slave *spi, u8 *idcode); +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode); -- 1.7.4.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/2 V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde @ 2013-01-25 21:45 ` Simon Glass 2013-02-20 15:01 ` [U-Boot] [U-Boot, 1/2, " Tom Rini 2013-05-21 13:10 ` [U-Boot] [PATCH 1/2 " Jagan Teki 2 siblings, 0 replies; 15+ messages in thread From: Simon Glass @ 2013-01-25 21:45 UTC (permalink / raw) To: u-boot On Wed, Jan 23, 2013 at 7:30 PM, Rajeshwari Shinde <rajeshwari.s@samsung.com> wrote: > This patch adds driver for the gigabyte devices > GD25LQ and GD25Q64B required for Snow Board. > > Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> Acked-by: Simon Glass <sjg@chromium.org> > --- > Changes in V2: > - Added U-Boot copyright header to gigadevice.c > - Removed unnecessary blank lines. > drivers/mtd/spi/Makefile | 1 + > drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ > drivers/mtd/spi/spi_flash.c | 3 + > drivers/mtd/spi/spi_flash_internal.h | 1 + > 4 files changed, 86 insertions(+), 0 deletions(-) > create mode 100644 drivers/mtd/spi/gigadevice.c > > diff --git a/drivers/mtd/spi/Makefile b/drivers/mtd/spi/Makefile > index 90f8392..ecbb210 100644 > --- a/drivers/mtd/spi/Makefile > +++ b/drivers/mtd/spi/Makefile > @@ -32,6 +32,7 @@ endif > COBJS-$(CONFIG_SPI_FLASH) += spi_flash.o > COBJS-$(CONFIG_SPI_FLASH_ATMEL) += atmel.o > COBJS-$(CONFIG_SPI_FLASH_EON) += eon.o > +COBJS-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.o > COBJS-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.o > COBJS-$(CONFIG_SPI_FLASH_SPANSION) += spansion.o > COBJS-$(CONFIG_SPI_FLASH_SST) += sst.o > diff --git a/drivers/mtd/spi/gigadevice.c b/drivers/mtd/spi/gigadevice.c > new file mode 100644 > index 0000000..b5e1ebe > --- /dev/null > +++ b/drivers/mtd/spi/gigadevice.c > @@ -0,0 +1,81 @@ > +/* > + * Gigadevice SPI flash driver > + * Copyright 2013, Samsung Electronics Co., Ltd. > + * Author: Banajit Goswami <banajit.g@samsung.com> > + * > + * See file CREDITS for list of people who contributed to this > + * project. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as > + * published by the Free Software Foundation; either version 2 of > + * the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, > + * MA 02111-1307 USA > + */ > + > +#include <common.h> > +#include <malloc.h> > +#include <spi_flash.h> > + > +#include "spi_flash_internal.h" > + > +struct gigadevice_spi_flash_params { > + uint16_t id; > + uint16_t nr_blocks; > + const char *name; > +}; > + > +static const struct gigadevice_spi_flash_params gigadevice_spi_flash_table[] = { > + { > + .id = 0x6016, > + .nr_blocks = 64, > + .name = "GD25LQ", > + }, > + { > + .id = 0x4017, > + .nr_blocks = 128, > + .name = "GD25Q64B", > + }, > +}; > + > +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode) > +{ > + const struct gigadevice_spi_flash_params *params; > + struct spi_flash *flash; > + unsigned int i; > + > + for (i = 0; i < ARRAY_SIZE(gigadevice_spi_flash_table); i++) { > + params = &gigadevice_spi_flash_table[i]; > + if (params->id == ((idcode[1] << 8) | idcode[2])) > + break; > + } > + > + if (i == ARRAY_SIZE(gigadevice_spi_flash_table)) { > + debug("SF: Unsupported Gigadevice ID %02x%02x\n", > + idcode[1], idcode[2]); > + return NULL; > + } > + > + flash = spi_flash_alloc_base(spi, params->name); > + if (!flash) { > + debug("SF: Failed to allocate memory\n"); > + return NULL; > + } > + /* page_size */ > + flash->page_size = 256; > + /* sector_size = page_size * pages_per_sector */ > + flash->sector_size = flash->page_size * 16; > + /* size = sector_size * sector_per_block * number of blocks */ > + flash->size = flash->sector_size * 16 * params->nr_blocks; > + > + return flash; > +} > diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c > index 17f3d3c..ee05171 100644 > --- a/drivers/mtd/spi/spi_flash.c > +++ b/drivers/mtd/spi/spi_flash.c > @@ -305,6 +305,9 @@ static const struct { > #ifdef CONFIG_SPI_FLASH_EON > { 0, 0x1c, spi_flash_probe_eon, }, > #endif > +#ifdef CONFIG_SPI_FLASH_GIGADEVICE > + { 0, 0xc8, spi_flash_probe_gigadevice, }, > +#endif > #ifdef CONFIG_SPI_FLASH_MACRONIX > { 0, 0xc2, spi_flash_probe_macronix, }, > #endif > diff --git a/drivers/mtd/spi/spi_flash_internal.h b/drivers/mtd/spi/spi_flash_internal.h > index 141cfa8..e0afbc3 100644 > --- a/drivers/mtd/spi/spi_flash_internal.h > +++ b/drivers/mtd/spi/spi_flash_internal.h > @@ -106,3 +106,4 @@ struct spi_flash *spi_flash_probe_sst(struct spi_slave *spi, u8 *idcode); > struct spi_flash *spi_flash_probe_stmicro(struct spi_slave *spi, u8 *idcode); > struct spi_flash *spi_flash_probe_winbond(struct spi_slave *spi, u8 *idcode); > struct spi_flash *spi_fram_probe_ramtron(struct spi_slave *spi, u8 *idcode); > +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode); > -- > 1.7.4.4 > ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [U-Boot, 1/2, V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde 2013-01-25 21:45 ` Simon Glass @ 2013-02-20 15:01 ` Tom Rini 2013-02-22 6:14 ` Rajeshwari Birje 2013-05-21 13:10 ` [U-Boot] [PATCH 1/2 " Jagan Teki 2 siblings, 1 reply; 15+ messages in thread From: Tom Rini @ 2013-02-20 15:01 UTC (permalink / raw) To: u-boot On Tue, Jan 22, 2013 at 08:30:18PM -0000, Rajeshwari Shinde wrote: > This patch adds driver for the gigabyte devices > GD25LQ and GD25Q64B required for Snow Board. > > Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> > Acked-by: Simon Glass <sjg@chromium.org> Reviewed-by: Tom Rini <trini@ti.com> Both parts of the series can go via u-boot-samsung and then u-boot-arm, thanks. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130220/56d371d2/attachment.pgp> ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [U-Boot, 1/2, V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-02-20 15:01 ` [U-Boot] [U-Boot, 1/2, " Tom Rini @ 2013-02-22 6:14 ` Rajeshwari Birje 0 siblings, 0 replies; 15+ messages in thread From: Rajeshwari Birje @ 2013-02-22 6:14 UTC (permalink / raw) To: u-boot Hi Minkyu Kang, Can we get these patches merged or do let me know if you have any review comments. Regards, Rajeshwari Shinde. On Wed, Feb 20, 2013 at 8:31 PM, Tom Rini <trini@ti.com> wrote: > On Tue, Jan 22, 2013 at 08:30:18PM -0000, Rajeshwari Shinde wrote: > >> This patch adds driver for the gigabyte devices >> GD25LQ and GD25Q64B required for Snow Board. >> >> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> >> Acked-by: Simon Glass <sjg@chromium.org> > > Reviewed-by: Tom Rini <trini@ti.com> > > Both parts of the series can go via u-boot-samsung and then u-boot-arm, > thanks. > > -- > Tom > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > -- Regards, Rajeshwari Shinde ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/2 V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde 2013-01-25 21:45 ` Simon Glass 2013-02-20 15:01 ` [U-Boot] [U-Boot, 1/2, " Tom Rini @ 2013-05-21 13:10 ` Jagan Teki 2013-05-28 6:29 ` Jagan Teki 2 siblings, 1 reply; 15+ messages in thread From: Jagan Teki @ 2013-05-21 13:10 UTC (permalink / raw) To: u-boot Hi, I think this reviewed already, but have a very few comments. On Wed, Jan 23, 2013 at 12:00 PM, Rajeshwari Shinde <rajeshwari.s@samsung.com> wrote: > This patch adds driver for the gigabyte devices > GD25LQ and GD25Q64B required for Snow Board. > > Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> > --- > Changes in V2: > - Added U-Boot copyright header to gigadevice.c > - Removed unnecessary blank lines. > drivers/mtd/spi/Makefile | 1 + > drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ > drivers/mtd/spi/spi_flash.c | 3 + > drivers/mtd/spi/spi_flash_internal.h | 1 + > 4 files changed, 86 insertions(+), 0 deletions(-) > create mode 100644 drivers/mtd/spi/gigadevice.c > > diff --git a/drivers/mtd/spi/Makefile b/drivers/mtd/spi/Makefile > index 90f8392..ecbb210 100644 > --- a/drivers/mtd/spi/Makefile > +++ b/drivers/mtd/spi/Makefile > @@ -32,6 +32,7 @@ endif > COBJS-$(CONFIG_SPI_FLASH) += spi_flash.o > COBJS-$(CONFIG_SPI_FLASH_ATMEL) += atmel.o > COBJS-$(CONFIG_SPI_FLASH_EON) += eon.o > +COBJS-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.o > COBJS-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.o > COBJS-$(CONFIG_SPI_FLASH_SPANSION) += spansion.o > COBJS-$(CONFIG_SPI_FLASH_SST) += sst.o > diff --git a/drivers/mtd/spi/gigadevice.c b/drivers/mtd/spi/gigadevice.c > new file mode 100644 > index 0000000..b5e1ebe > --- /dev/null > +++ b/drivers/mtd/spi/gigadevice.c > @@ -0,0 +1,81 @@ > +/* > + * Gigadevice SPI flash driver > + * Copyright 2013, Samsung Electronics Co., Ltd. > + * Author: Banajit Goswami <banajit.g@samsung.com> > + * > + * See file CREDITS for list of people who contributed to this > + * project. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as > + * published by the Free Software Foundation; either version 2 of > + * the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, > + * MA 02111-1307 USA > + */ > + > +#include <common.h> > +#include <malloc.h> > +#include <spi_flash.h> > + > +#include "spi_flash_internal.h" > + > +struct gigadevice_spi_flash_params { > + uint16_t id; > + uint16_t nr_blocks; I think it's better to use u16 instead of uint16_t, uin16_t will get back to arch include from include/linux which does u16 for directly for first time. > + const char *name; > +}; > + > +static const struct gigadevice_spi_flash_params gigadevice_spi_flash_table[] = { > + { > + .id = 0x6016, > + .nr_blocks = 64, > + .name = "GD25LQ", > + }, > + { > + .id = 0x4017, > + .nr_blocks = 128, > + .name = "GD25Q64B", > + }, Better to use clean code shape like.. { .id = 0x60, .nr_blocks = 64, .name = "GD25LQ", } > +}; > + > +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode) > +{ > + const struct gigadevice_spi_flash_params *params; > + struct spi_flash *flash; > + unsigned int i; > + > + for (i = 0; i < ARRAY_SIZE(gigadevice_spi_flash_table); i++) { > + params = &gigadevice_spi_flash_table[i]; > + if (params->id == ((idcode[1] << 8) | idcode[2])) > + break; > + } > + > + if (i == ARRAY_SIZE(gigadevice_spi_flash_table)) { > + debug("SF: Unsupported Gigadevice ID %02x%02x\n", > + idcode[1], idcode[2]); > + return NULL; > + } > + > + flash = spi_flash_alloc_base(spi, params->name); > + if (!flash) { > + debug("SF: Failed to allocate memory\n"); > + return NULL; > + } better to add a space here > + /* page_size */ > + flash->page_size = 256; > + /* sector_size = page_size * pages_per_sector */ > + flash->sector_size = flash->page_size * 16; > + /* size = sector_size * sector_per_block * number of blocks */ > + flash->size = flash->sector_size * 16 * params->nr_blocks; comments on above size calculations are good, but not that much important i guess. And also please provide a stand' link for flash part data sheet on commit message, if possible. I thought it's a good to refers don't no this sounds valid.. Thanks, Jagan. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/2 V2] SF: Add driver for Gigabyte device GD25LQ and GD25Q64B 2013-05-21 13:10 ` [U-Boot] [PATCH 1/2 " Jagan Teki @ 2013-05-28 6:29 ` Jagan Teki 0 siblings, 0 replies; 15+ messages in thread From: Jagan Teki @ 2013-05-28 6:29 UTC (permalink / raw) To: u-boot Any update on this. Was this patch refereed for denx tree? Thanks, Jagan. On Tue, May 21, 2013 at 6:40 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote: > Hi, > > I think this reviewed already, but have a very few comments. > > On Wed, Jan 23, 2013 at 12:00 PM, Rajeshwari Shinde > <rajeshwari.s@samsung.com> wrote: >> This patch adds driver for the gigabyte devices >> GD25LQ and GD25Q64B required for Snow Board. >> >> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> >> --- >> Changes in V2: >> - Added U-Boot copyright header to gigadevice.c >> - Removed unnecessary blank lines. >> drivers/mtd/spi/Makefile | 1 + >> drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ >> drivers/mtd/spi/spi_flash.c | 3 + >> drivers/mtd/spi/spi_flash_internal.h | 1 + >> 4 files changed, 86 insertions(+), 0 deletions(-) >> create mode 100644 drivers/mtd/spi/gigadevice.c >> >> diff --git a/drivers/mtd/spi/Makefile b/drivers/mtd/spi/Makefile >> index 90f8392..ecbb210 100644 >> --- a/drivers/mtd/spi/Makefile >> +++ b/drivers/mtd/spi/Makefile >> @@ -32,6 +32,7 @@ endif >> COBJS-$(CONFIG_SPI_FLASH) += spi_flash.o >> COBJS-$(CONFIG_SPI_FLASH_ATMEL) += atmel.o >> COBJS-$(CONFIG_SPI_FLASH_EON) += eon.o >> +COBJS-$(CONFIG_SPI_FLASH_GIGADEVICE) += gigadevice.o >> COBJS-$(CONFIG_SPI_FLASH_MACRONIX) += macronix.o >> COBJS-$(CONFIG_SPI_FLASH_SPANSION) += spansion.o >> COBJS-$(CONFIG_SPI_FLASH_SST) += sst.o >> diff --git a/drivers/mtd/spi/gigadevice.c b/drivers/mtd/spi/gigadevice.c >> new file mode 100644 >> index 0000000..b5e1ebe >> --- /dev/null >> +++ b/drivers/mtd/spi/gigadevice.c >> @@ -0,0 +1,81 @@ >> +/* >> + * Gigadevice SPI flash driver >> + * Copyright 2013, Samsung Electronics Co., Ltd. >> + * Author: Banajit Goswami <banajit.g@samsung.com> >> + * >> + * See file CREDITS for list of people who contributed to this >> + * project. >> + * >> + * This program is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU General Public License as >> + * published by the Free Software Foundation; either version 2 of >> + * the License, or (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License >> + * along with this program; if not, write to the Free Software >> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, >> + * MA 02111-1307 USA >> + */ >> + >> +#include <common.h> >> +#include <malloc.h> >> +#include <spi_flash.h> >> + >> +#include "spi_flash_internal.h" >> + >> +struct gigadevice_spi_flash_params { >> + uint16_t id; >> + uint16_t nr_blocks; > > I think it's better to use u16 instead of uint16_t, uin16_t will get > back to arch include from include/linux which does u16 for directly > for first time. > >> + const char *name; >> +}; >> + >> +static const struct gigadevice_spi_flash_params gigadevice_spi_flash_table[] = { >> + { >> + .id = 0x6016, >> + .nr_blocks = 64, >> + .name = "GD25LQ", >> + }, >> + { >> + .id = 0x4017, >> + .nr_blocks = 128, >> + .name = "GD25Q64B", >> + }, > > Better to use clean code shape like.. > { > .id = 0x60, > .nr_blocks = 64, > .name = "GD25LQ", > } > >> +}; >> + >> +struct spi_flash *spi_flash_probe_gigadevice(struct spi_slave *spi, u8 *idcode) >> +{ >> + const struct gigadevice_spi_flash_params *params; >> + struct spi_flash *flash; >> + unsigned int i; >> + >> + for (i = 0; i < ARRAY_SIZE(gigadevice_spi_flash_table); i++) { >> + params = &gigadevice_spi_flash_table[i]; >> + if (params->id == ((idcode[1] << 8) | idcode[2])) >> + break; >> + } >> + >> + if (i == ARRAY_SIZE(gigadevice_spi_flash_table)) { >> + debug("SF: Unsupported Gigadevice ID %02x%02x\n", >> + idcode[1], idcode[2]); >> + return NULL; >> + } >> + >> + flash = spi_flash_alloc_base(spi, params->name); >> + if (!flash) { >> + debug("SF: Failed to allocate memory\n"); >> + return NULL; >> + } > > better to add a space here > >> + /* page_size */ >> + flash->page_size = 256; >> + /* sector_size = page_size * pages_per_sector */ >> + flash->sector_size = flash->page_size * 16; >> + /* size = sector_size * sector_per_block * number of blocks */ >> + flash->size = flash->sector_size * 16 * params->nr_blocks; > > comments on above size calculations are good, but not that much > important i guess. > > And also please provide a stand' link for flash part data sheet on > commit message, if possible. > I thought it's a good to refers don't no this sounds valid.. > > Thanks, > Jagan. -- -- Thanks, Jagan. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device. 2013-01-23 6:30 [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Shinde 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde @ 2013-01-23 6:30 ` Rajeshwari Shinde 2013-01-23 6:27 ` Rajeshwari Birje 2013-02-13 5:23 ` [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Birje 2013-03-08 13:28 ` Minkyu Kang 3 siblings, 1 reply; 15+ messages in thread From: Rajeshwari Shinde @ 2013-01-23 6:30 UTC (permalink / raw) To: u-boot This patch enables gigabyte device for SMDK5250. Acked-by: Simon Glass <sjg@chromium.org> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> --- Changes in V2: - None include/configs/exynos5250-dt.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h index cabd2f2..a01fb96 100644 --- a/include/configs/exynos5250-dt.h +++ b/include/configs/exynos5250-dt.h @@ -265,6 +265,7 @@ #define CONFIG_CMD_SF #define CONFIG_CMD_SPI #define CONFIG_SPI_FLASH_WINBOND +#define CONFIG_SPI_FLASH_GIGADEVICE #define CONFIG_SF_DEFAULT_MODE SPI_MODE_0 #define CONFIG_SF_DEFAULT_SPEED 50000000 #define EXYNOS5_SPI_NUM_CONTROLLERS 5 -- 1.7.4.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device. 2013-01-23 6:30 ` [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device Rajeshwari Shinde @ 2013-01-23 6:27 ` Rajeshwari Birje 0 siblings, 0 replies; 15+ messages in thread From: Rajeshwari Birje @ 2013-01-23 6:27 UTC (permalink / raw) To: u-boot Hi All, Kindly ignore this patch as I have forgot to put the version V2. I have corrected the same and resent. Sorry for inconvenience. On Wed, Jan 23, 2013 at 12:00 PM, Rajeshwari Shinde <rajeshwari.s@samsung.com> wrote: > This patch enables gigabyte device for SMDK5250. > > Acked-by: Simon Glass <sjg@chromium.org> > Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com> > --- > Changes in V2: > - None > include/configs/exynos5250-dt.h | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h > index cabd2f2..a01fb96 100644 > --- a/include/configs/exynos5250-dt.h > +++ b/include/configs/exynos5250-dt.h > @@ -265,6 +265,7 @@ > #define CONFIG_CMD_SF > #define CONFIG_CMD_SPI > #define CONFIG_SPI_FLASH_WINBOND > +#define CONFIG_SPI_FLASH_GIGADEVICE > #define CONFIG_SF_DEFAULT_MODE SPI_MODE_0 > #define CONFIG_SF_DEFAULT_SPEED 50000000 > #define EXYNOS5_SPI_NUM_CONTROLLERS 5 > -- > 1.7.4.4 > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot -- Regards, Rajeshwari Shinde ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Shinde 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde 2013-01-23 6:30 ` [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device Rajeshwari Shinde @ 2013-02-13 5:23 ` Rajeshwari Birje 2013-03-08 13:28 ` Minkyu Kang 3 siblings, 0 replies; 15+ messages in thread From: Rajeshwari Birje @ 2013-02-13 5:23 UTC (permalink / raw) To: u-boot Hi, Please do let me know if any comments on this patch set. -- Regards, Rajeshwari Shinde On Wed, Jan 23, 2013 at 12:00 PM, Rajeshwari Shinde <rajeshwari.s@samsung.com> wrote: > This patch set adds driver for Gigabyte device GD25LQ and GD25Q64B > required for Snow board and enables same in config file. > > Based on following patches submitted by Simon Glass: > "sf: Add spi_flash_alloc() to create a new SPI flash struct" > "sf: Use spi_flash_alloc() in each SPI flash driver" > > Changes in V2: > - Added U-Boot copyright header to gigadevice.c > - Removed unnecessary blank lines. > > Rajeshwari Shinde (2): > SF: Add driver for Gigabyte device GD25LQ and GD25Q64B > SMDK5250: Enable SPI Gigabyte device. > > drivers/mtd/spi/Makefile | 1 + > drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ > drivers/mtd/spi/spi_flash.c | 3 + > drivers/mtd/spi/spi_flash_internal.h | 1 + > include/configs/exynos5250-dt.h | 1 + > 5 files changed, 87 insertions(+), 0 deletions(-) > create mode 100644 drivers/mtd/spi/gigadevice.c > > -- > 1.7.4.4 > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-01-23 6:30 [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Shinde ` (2 preceding siblings ...) 2013-02-13 5:23 ` [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Birje @ 2013-03-08 13:28 ` Minkyu Kang 2013-03-12 6:13 ` Rajeshwari Birje 3 siblings, 1 reply; 15+ messages in thread From: Minkyu Kang @ 2013-03-08 13:28 UTC (permalink / raw) To: u-boot Dear Rajeshwari, On 23/01/13 15:30, Rajeshwari Shinde wrote: > This patch set adds driver for Gigabyte device GD25LQ and GD25Q64B > required for Snow board and enables same in config file. > > Based on following patches submitted by Simon Glass: > "sf: Add spi_flash_alloc() to create a new SPI flash struct" > "sf: Use spi_flash_alloc() in each SPI flash driver" > > Changes in V2: > - Added U-Boot copyright header to gigadevice.c > - Removed unnecessary blank lines. > > Rajeshwari Shinde (2): > SF: Add driver for Gigabyte device GD25LQ and GD25Q64B > SMDK5250: Enable SPI Gigabyte device. > > drivers/mtd/spi/Makefile | 1 + > drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ > drivers/mtd/spi/spi_flash.c | 3 + > drivers/mtd/spi/spi_flash_internal.h | 1 + > include/configs/exynos5250-dt.h | 1 + > 5 files changed, 87 insertions(+), 0 deletions(-) > create mode 100644 drivers/mtd/spi/gigadevice.c > I've got compiler warning and error on this patch. gigadevice.c: In function 'spi_flash_probe_gigadevice': gigadevice.c:68:2: warning: implicit declaration of function 'spi_flash_alloc_base' [-Wimplicit-function-declaration] gigadevice.c:68:8: warning: assignment makes pointer from integer without a cast [enabled by default] drivers/mtd/spi/libspi_flash.o: In function `spi_flash_probe_gigadevice': /home/share/Work/u-boot-samsung/drivers/mtd/spi/gigadevice.c:68: undefined reference to `spi_flash_alloc_base' There is any dependency with other patches? then, please let me know. Thanks, Minkyu Kang. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-03-08 13:28 ` Minkyu Kang @ 2013-03-12 6:13 ` Rajeshwari Birje 2013-03-27 6:38 ` Minkyu Kang 0 siblings, 1 reply; 15+ messages in thread From: Rajeshwari Birje @ 2013-03-12 6:13 UTC (permalink / raw) To: u-boot Hi Minkyu, Yes it is based on the following patches submitted by Simon Glass and link for the same. "sf: Add spi_flash_alloc() to create a new SPI flash struct" http://patchwork.ozlabs.org/patch/208228/ "sf: Use spi_flash_alloc() in each SPI flash driver": http://patchwork.ozlabs.org/patch/226582/ -- Regards, Rajeshwari Shinde On Fri, Mar 8, 2013 at 6:58 PM, Minkyu Kang <mk7.kang@samsung.com> wrote: > Dear Rajeshwari, > > On 23/01/13 15:30, Rajeshwari Shinde wrote: >> This patch set adds driver for Gigabyte device GD25LQ and GD25Q64B >> required for Snow board and enables same in config file. >> >> Based on following patches submitted by Simon Glass: >> "sf: Add spi_flash_alloc() to create a new SPI flash struct" >> "sf: Use spi_flash_alloc() in each SPI flash driver" >> >> Changes in V2: >> - Added U-Boot copyright header to gigadevice.c >> - Removed unnecessary blank lines. >> >> Rajeshwari Shinde (2): >> SF: Add driver for Gigabyte device GD25LQ and GD25Q64B >> SMDK5250: Enable SPI Gigabyte device. >>Based on following patches submitted by Simon Glass: >> drivers/mtd/spi/Makefile | 1 + >> drivers/mtd/spi/gigadevice.c | 81 ++++++++++++++++++++++++++++++++++ >> drivers/mtd/spi/spi_flash.c | 3 + >> drivers/mtd/spi/spi_flash_internal.h | 1 + >> include/configs/exynos5250-dt.h | 1 + >> 5 files changed, 87 insertions(+), 0 deletions(-) >> create mode 100644 drivers/mtd/spi/gigadevice.c >> > > I've got compiler warning and error on this patch. > > gigadevice.c: In function 'spi_flash_probe_gigadevice': > gigadevice.c:68:2: warning: implicit declaration of function 'spi_flash_alloc_base' [-Wimplicit-function-declaration] > gigadevice.c:68:8: warning: assignment makes pointer from integer without a cast [enabled by default] > drivers/mtd/spi/libspi_flash.o: In function `spi_flash_probe_gigadevice': > /home/share/Work/u-boot-samsung/drivers/mtd/spi/gigadevice.c:68: undefined reference to `spi_flash_alloc_base' > > There is any dependency with other patches? > then, please let me know. > > Thanks, > Minkyu Kang. > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-03-12 6:13 ` Rajeshwari Birje @ 2013-03-27 6:38 ` Minkyu Kang 2013-03-27 16:42 ` Simon Glass 0 siblings, 1 reply; 15+ messages in thread From: Minkyu Kang @ 2013-03-27 6:38 UTC (permalink / raw) To: u-boot Dear Rajeshwari and Simon, On 12/03/13 15:13, Rajeshwari Birje wrote: > Hi Minkyu, > > Yes it is based on the following patches submitted by Simon Glass and > link for the same. > > "sf: Add spi_flash_alloc() to create a new SPI flash struct" > http://patchwork.ozlabs.org/patch/208228/ > > "sf: Use spi_flash_alloc() in each SPI flash driver": > http://patchwork.ozlabs.org/patch/226582/ > Those patches are merged? Thanks, Minkyu Kang. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-03-27 6:38 ` Minkyu Kang @ 2013-03-27 16:42 ` Simon Glass 2013-05-20 10:23 ` Minkyu Kang 0 siblings, 1 reply; 15+ messages in thread From: Simon Glass @ 2013-03-27 16:42 UTC (permalink / raw) To: u-boot Hi Minkyu, On Tue, Mar 26, 2013 at 11:38 PM, Minkyu Kang <mk7.kang@samsung.com> wrote: > Dear Rajeshwari and Simon, > > On 12/03/13 15:13, Rajeshwari Birje wrote: > > Hi Minkyu, > > > > Yes it is based on the following patches submitted by Simon Glass and > > link for the same. > > > > "sf: Add spi_flash_alloc() to create a new SPI flash struct" > > http://patchwork.ozlabs.org/patch/208228/ > > > > "sf: Use spi_flash_alloc() in each SPI flash driver": > > http://patchwork.ozlabs.org/patch/226582/ > > > > Those patches are merged? > Yes they are in mainline now. Regards, Simon ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B 2013-03-27 16:42 ` Simon Glass @ 2013-05-20 10:23 ` Minkyu Kang 0 siblings, 0 replies; 15+ messages in thread From: Minkyu Kang @ 2013-05-20 10:23 UTC (permalink / raw) To: u-boot Dear Rajeshwari, On 28/03/13 01:42, Simon Glass wrote: > Hi Minkyu, > > > On Tue, Mar 26, 2013 at 11:38 PM, Minkyu Kang <mk7.kang at samsung.com <mailto:mk7.kang@samsung.com>> wrote: > > Dear Rajeshwari and Simon, > > On 12/03/13 15:13, Rajeshwari Birje wrote: > > Hi Minkyu, > > > > Yes it is based on the following patches submitted by Simon Glass and > > link for the same. > > > > "sf: Add spi_flash_alloc() to create a new SPI flash struct" > > http://patchwork.ozlabs.org/patch/208228/ > > > > "sf: Use spi_flash_alloc() in each SPI flash driver": > > http://patchwork.ozlabs.org/patch/226582/ > > > > Those patches are merged? > > > Yes they are in mainline now. > > Regards, > Simon > applied to u-boot-samsung. Thanks, Minkyu Kang. ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-05-28 6:29 UTC | newest] Thread overview: 15+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-01-23 6:30 [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Shinde 2013-01-23 6:30 ` [U-Boot] [PATCH 1/2 V2] SF: Add driver for " Rajeshwari Shinde 2013-01-25 21:45 ` Simon Glass 2013-02-20 15:01 ` [U-Boot] [U-Boot, 1/2, " Tom Rini 2013-02-22 6:14 ` Rajeshwari Birje 2013-05-21 13:10 ` [U-Boot] [PATCH 1/2 " Jagan Teki 2013-05-28 6:29 ` Jagan Teki 2013-01-23 6:30 ` [U-Boot] [PATCH 2/2] SMDK5250: Enable SPI Gigabyte device Rajeshwari Shinde 2013-01-23 6:27 ` Rajeshwari Birje 2013-02-13 5:23 ` [U-Boot] [PATCH 0/2 V2] EXYNOS5: Enable Gigabyte device GD25LQ and GD25Q64B Rajeshwari Birje 2013-03-08 13:28 ` Minkyu Kang 2013-03-12 6:13 ` Rajeshwari Birje 2013-03-27 6:38 ` Minkyu Kang 2013-03-27 16:42 ` Simon Glass 2013-05-20 10:23 ` Minkyu Kang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox