Convert GPMC to driver
Subsystems (7)
v2 cover Convert GPMC to driver
Hi,
Convert GPCM code to be a driver. Existing GPMC NAND platform handling has been modified to work with the new GPMC driver (patches 2 & 3). Patch 5 is to test the driver in OMAP3EVM.
TODO
1. Let NAND driver deal with GPMC NAND block
2. Remove struct gpmc * stored as static
3. Convert all peripherals to use GPMC driver
4. Devise method to handle OneNAND cleanly
5. Handle acquiring CS# cases
6. Decide on where GPMC driver should live
7. Adapt to HWMOD, use RPM
8. GPMC driver cleanup
Regards
Afzal
Afzal Mohammed (5):
ARM: OMAP2+: gpmc: driver conversion
ARM: OMAP2+: nand: create platform data structure
ARM: OMAP2+: gpmc-nand: populate gpmc configs
mtd: nand: omap2: obtain memory from resource
OMAP3EVM: Test gpmc-nand
v2: Avoid code movement that kept similar code together (for easy review)
arch/arm/mach-omap2/board-devkit8000.c | 6 +- arch/arm/mach-omap2/board-flash.c | 63 ++--- arch/arm/mach-omap2/board-flash.h | 13 +- arch/arm/mach-omap2/board-ldp.c | 4 +- arch/arm/mach-omap2/board-omap3beagle.c | 6 +- arch/arm/mach-omap2/board-omap3evm.c | 82 +++++++ arch/arm/mach-omap2/board-omap3touchbook.c | 6 +- arch/arm/mach-omap2/board-overo.c | 5 +- arch/arm/mach-omap2/board-zoom.c | 5 +- arch/arm/mach-omap2/common-board-devices.c | 46 ---- arch/arm/mach-omap2/common-board-devices.h | 1 - arch/arm/mach-omap2/gpmc-nand.c | 88 +++---- arch/arm/mach-omap2/gpmc.c | 366 +++++++++++++++++++++------- arch/arm/plat-omap/include/plat/gpmc.h | 34 ++- arch/arm/plat-omap/include/plat/nand.h | 9 +- drivers/mtd/nand/omap2.c | 19 +- 16 files changed, 502 insertions(+), 251 deletions(-)
Signature · 1 line
1.7.9.3
RFC PATCH v2 1/5 ARM: OMAP2+: gpmc: driver conversion
Convert GPMC code to driver. Boards using GPMC should provide driver with type of configuration, timing, CS, GPMC address space details (if already configured, driver will retrieve, as is existing). Platform devices would the be created for each connected peripheral (details also to be passed by board so that it reaches respective driver). And GPMC driver would populate memory resource details for the driver of connected peripheral.
A peripheral connected to GPMC can have multiple address spaces using different chip select. Hence GPMC driver has been provided capability to distinguish this scenario, i.e. create platform devices only once for each connected peripheral, and not for each configured chip select. The peripheral that made it necessary was tusb6010.
Final destination for this driver is being investigated. Before moving to the new location, all existing GPMC users has to be converted to work with this driver.
NAND driver for NAND used via GPMC is tightly coupled with GPMC driver (GPMC has few blocks exclusively for NAND), while that is not the case for most of the other users (they need GPMCs help only for initial configuration). Currently NAND driver manage using exported GPMC symbols. This is being planned to remove later & would need informing NAND driver about GPMC NAND registers. This would help to have export symbol free GPMC driver, and probably
"mv omap2.c gpmc-nand.c" for OMAP NAND driver.
Thanks to Vaibhav Hiremath for his ideas on this.
Acquiring CS# for NAND is done on a few boards. It means, depending on bootloader to embed this information. Probably CS# being used can be set in the Kernel, and acquiring it can be removed. If ever this capbility is needed, GPMC driver has to be made aware of handling it.
OneNAND - as it may involve reconfiguring GPMC for synchronous may need a quirk to handle or driver has to be made more intelligent to handle it.
Code related to GPMC clock may have to continue live in platform folders (even if the driver is moved to MFD) as input clock is beyond the control of GPMC and calculating timing for the peripheral may need other helpers.
TODO
1. Let NAND driver deal with GPMC NAND block
2. Remove struct gpmc * stored as static
3. Convert all peripherals to use GPMC driver
4. Devise method to handle OneNAND cleanly
5. Handle acquiring CS# cases
6. Decide on where GPMC driver should live
7. Adapt to HWMOD, use RPM
8. GPMC driver cleanup
---
v2: Avoid code movement that kept similar code together
arch/arm/mach-omap2/gpmc.c | 366 ++++++++++++++++++++++++-------- arch/arm/plat-omap/include/plat/gpmc.h | 34 ++- 2 files changed, 305 insertions(+), 95 deletions(-)
arch/arm/mach-omap2/gpmc.c +273 -93
@@ -14,8 +14,11 @@*/ #undef DEBUG +#include <linux/platform_device.h> +#include <linux/irq.h> #include <linux/kernel.h> +#include <linux/slab.h> #include <linux/init.h> #include <linux/err.h> #include <linux/clk.h>
@@ -64,6 +67,35 @@#define ENABLE_PREFETCH (0x1 << 7)#define DMA_MPU_MODE 2 +#define DRIVER_NAME "omap-gpmc" + +struct gpmc_child { +char *name; +int id; +struct resource *res; +unsigned num_res; +struct resource gpmc_res[GPMC_CS_NUM ]; +unsigned gpmc_num_res; +void *pdata; +unsigned pdata_size; +}; + +struct gpmc { +struct device *dev; +unsigned long fclk_period; +void __iomem *io_base; +unsigned long phys_base; +u32 memsize; +unsigned cs_map; +int ecc_used; +spinlock_t mem_lock; +struct resource mem_root; +struct resource cs_mem[GPMC_CS_NUM ]; +struct gpmc_child child_device[GPMC_CS_NUM ]; +unsigned num_child; +unsigned num_device; +}; +/* Structure to save gpmc cs context */ struct gpmc_cs_config {u32 config1;
@@ -91,58 +123,50 @@ struct omap3_gpmc_regs {struct gpmc_cs_config cs_context[GPMC_CS_NUM ]; }; -static struct resource gpmc_mem_root; -static struct resource gpmc_cs_mem[GPMC_CS_NUM ]; -static DEFINE_SPINLOCK (gpmc_mem_lock); -static unsigned int gpmc_cs_map;/* flag for cs which are initialized */ -static int gpmc_ecc_used = -EINVAL ;/* cs using ecc engine */ - -static void __iomem *gpmc_base; -static struct clk *gpmc_l3_clk; -static irqreturn_t gpmc_handle_irq (int irq,void *dev); +static struct gpmc *gpmc;static void gpmc_write_reg (int idx,u32 val) { -__raw_writel (val, gpmc_base + idx); +writel (val, gpmc->io_base + idx); }static u32 gpmc_read_reg (int idx) { -return __raw_readl (gpmc_base + idx); +return readl (gpmc->io_base + idx); }static void gpmc_cs_write_byte (int cs,int idx,u8 val) {void __iomem *reg_addr; - reg_addr = gpmc_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; -__raw_writeb (val, reg_addr); + reg_addr = gpmc->io_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; +writeb (val, reg_addr); }static u8 gpmc_cs_read_byte (int cs,int idx) {void __iomem *reg_addr; - reg_addr = gpmc_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; -return __raw_readb (reg_addr); + reg_addr = gpmc->io_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; +return readb (reg_addr); }void gpmc_cs_write_reg (int cs,int idx,u32 val) {void __iomem *reg_addr; - reg_addr = gpmc_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; -__raw_writel (val, reg_addr); + reg_addr = gpmc->io_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; +writel (val, reg_addr); }u32 gpmc_cs_read_reg (int cs,int idx) {void __iomem *reg_addr; - reg_addr = gpmc_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; -return __raw_readl (reg_addr); + reg_addr = gpmc->io_base +GPMC_CS0_OFFSET + (cs *GPMC_CS_SIZE ) + idx; +return readl (reg_addr); }/* TODO: Add support for gpmc_fck to clock framework and use it */
@@ -332,7 +356,7 @@ static void gpmc_cs_disable_mem(int cs)gpmc_cs_write_reg (cs,GPMC_CS_CONFIG7 , l); } -static void gpmc_cs_get_memconf (int cs,u32 *base,u32 *size) +static __devinit void gpmc_cs_get_memconf (int cs,u32 *base,u32 *size) {u32 l;u32 mask;
@@ -351,23 +375,23 @@ static int gpmc_cs_mem_enabled(int cs)return l &GPMC_CONFIG7_CSVALID ; } -int gpmc_cs_set_reserved (int cs,int reserved) +static int gpmc_cs_set_reserved (int cs,int reserved) {if (cs >GPMC_CS_NUM )return -ENODEV ; - gpmc_cs_map &= ~(1 << cs); - gpmc_cs_map |= (reserved ?1 :0 ) << cs; + gpmc->cs_map &= ~(1 << cs); + gpmc->cs_map |= (reserved ?1 :0 ) << cs;return 0 ; } -int gpmc_cs_reserved (int cs) +static int gpmc_cs_reserved (int cs) {if (cs >GPMC_CS_NUM )return -ENODEV ; -return gpmc_cs_map & (1 << cs); +return gpmc->cs_map & (1 << cs); }static unsigned long gpmc_mem_align (unsigned long size)
@@ -384,24 +408,25 @@ static unsigned long gpmc_mem_align(unsigned long size)return size; } -static int gpmc_cs_insert_mem (int cs,unsigned long base,unsigned long size) +static __devinit +int gpmc_cs_insert_mem (int cs,unsigned long base,unsigned long size) { -struct resource *res = &gpmc_cs_mem[cs]; +struct resource *res = &gpmc->cs_mem[cs];int r; size =gpmc_mem_align (size); -spin_lock (&gpmc_mem_lock); +spin_lock (&gpmc->mem_lock); res->start = base; res->end = base + size -1 ; - r =request_resource (&gpmc_mem_root, res); -spin_unlock (&gpmc_mem_lock); + r =request_resource (&gpmc->mem_root, res); +spin_unlock (&gpmc->mem_lock);return r; }int gpmc_cs_request (int cs,unsigned long size,unsigned long *base) { -struct resource *res = &gpmc_cs_mem[cs]; +struct resource *res = &gpmc->cs_mem[cs];int r =-1 ;if (cs >GPMC_CS_NUM )
@@ -411,7 +436,7 @@ int gpmc_cs_request(int cs, unsigned long size, unsigned long *base)if (size > (1 <<GPMC_SECTION_SHIFT ))return -ENOMEM ; -spin_lock (&gpmc_mem_lock); +spin_lock (&gpmc->mem_lock);if (gpmc_cs_reserved (cs)) { r = -EBUSY ; goto out;
@@ -419,7 +444,7 @@ int gpmc_cs_request(int cs, unsigned long size, unsigned long *base)if (gpmc_cs_mem_enabled (cs)) r =adjust_resource (res, res->start & ~(size -1 ), size);if (r <0 ) - r =allocate_resource (&gpmc_mem_root, res, size,0 , ~0 , + r =allocate_resource (&gpmc->mem_root, res, size,0 , ~0 , size,NULL ,NULL );if (r <0 ) goto out;
@@ -428,24 +453,24 @@ int gpmc_cs_request(int cs, unsigned long size, unsigned long *base) *base = res->start;gpmc_cs_set_reserved (cs,1 ); out: -spin_unlock (&gpmc_mem_lock); +spin_unlock (&gpmc->mem_lock);return r; }EXPORT_SYMBOL (gpmc_cs_request);void gpmc_cs_free (int cs) { -spin_lock (&gpmc_mem_lock); +spin_lock (&gpmc->mem_lock);if (cs >=GPMC_CS_NUM || cs <0 || !gpmc_cs_reserved (cs)) {printk (KERN_ERR "Trying to free non-reserved GPMC CS%d\n" , cs);BUG (); -spin_unlock (&gpmc_mem_lock); +spin_unlock (&gpmc->mem_lock);return ; }gpmc_cs_disable_mem (cs); -release_resource (&gpmc_cs_mem[cs]); +release_resource (&gpmc->cs_mem[cs]);gpmc_cs_set_reserved (cs,0 ); -spin_unlock (&gpmc_mem_lock); +spin_unlock (&gpmc->mem_lock); }EXPORT_SYMBOL (gpmc_cs_free);
@@ -668,7 +693,7 @@ int gpmc_prefetch_reset(int cs) }EXPORT_SYMBOL (gpmc_prefetch_reset); -static void __init gpmc_mem_init (void ) +static __devinit void gpmc_mem_init (void ) {int cs;unsigned long boot_rom_space =0 ;
@@ -680,8 +705,8 @@ static void __init gpmc_mem_init(void)/* In apollon the CS0 is mapped as 0x0000 0000 */ if (machine_is_omap_apollon ()) boot_rom_space =0 ; - gpmc_mem_root.start =GPMC_MEM_START + boot_rom_space; - gpmc_mem_root.end =GPMC_MEM_END ; + gpmc->mem_root.start =GPMC_MEM_START + boot_rom_space; + gpmc->mem_root.end =GPMC_MEM_END ;/* Reserve all regions that has been set up by bootloader */ for (cs =0 ; cs <GPMC_CS_NUM ; cs++) {
@@ -697,27 +722,15 @@ static void __init gpmc_mem_init(void)static int __init gpmc_init (void ) { -u32 l, irq; -int cs, ret = -EINVAL ; -int gpmc_irq; +int ret = -EINVAL ;char *ck =NULL ; -if (cpu_is_omap24xx ()) { +if (cpu_is_omap24xx ()) ck ="core_l3_ck" ; -if (cpu_is_omap2420 ()) - l =OMAP2420_GPMC_BASE ; -else - l =OMAP34XX_GPMC_BASE ; - gpmc_irq =INT_34XX_GPMC_IRQ ; - }else if (cpu_is_omap34xx ()) { +else if (cpu_is_omap34xx ()) ck ="gpmc_fck" ; - l =OMAP34XX_GPMC_BASE ; - gpmc_irq =INT_34XX_GPMC_IRQ ; - }else if (cpu_is_omap44xx ()) { +else if (cpu_is_omap44xx ()) ck ="gpmc_ck" ; - l =OMAP44XX_GPMC_BASE ; - gpmc_irq =OMAP44XX_IRQ_GPMC ; - }if (WARN_ON (!ck))return ret;
@@ -728,54 +741,221 @@ static int __init gpmc_init(void)BUG (); } - gpmc_base =ioremap (l,SZ_4K ); -if (!gpmc_base) { -clk_put (gpmc_l3_clk); -printk (KERN_ERR "Could not get GPMC register memory\n" ); -BUG (); +clk_enable (gpmc_l3_clk); + +return 0 ; +} +postcore_initcall (gpmc_init); + +static inline int __devinit gpmc_find_next_child_slot (void ) +{ +return gpmc->num_child; +} + +static int __devinit gpmc_match_child (char *name,int id) +{ +int i; +struct gpmc_child *p; + +for (i =0 , p = gpmc->child_device; i < gpmc->num_child; i++, p++) +if (!strcmp (p->name, name) && (p->id == id)) +return i; + +return -ENOENT ; +} + +static __devinit int gpmc_setup_child (struct gpmc_device_pdata *gdp) +{ +struct gpmc_config *c; +int i, ret =0 ; +struct resource res; +unsigned long start; + + start = gdp->mem_start; + + ret =gpmc_cs_request (gdp->cs, gdp->mem_size, &start); +if (IS_ERR_VALUE (ret)) { +dev_err (gpmc->dev,"error: gpmc request on CS: %u\n" , gdp->cs); +return ret; } -clk_enable (gpmc_l3_clk); + c = gdp->config; +if (!c) { +dev_err (gpmc->dev,"config not present for CS: %u\n" , gdp->cs); +return -EINVAL ; + } + +for (i =0 ; i < gdp->num_config; c++, i++) { + ret =gpmc_cs_configure (gdp->cs, c->cmd, c->val); +if (IS_ERR_VALUE (ret)) { +dev_err (gpmc->dev,"invalid cmd or value on CS: \ +%u: cmd: %d value: %d\n" , gdp->cs, c->cmd, c->val); +return ret; + } + } + +if (gdp->timing) { + ret =gpmc_cs_set_timings (gdp->cs, gdp->timing); +if (IS_ERR_VALUE (ret)) { +dev_err (gpmc->dev,"error: setting timing on CS: %d\n" , + gdp->cs); +return ret; + } + } + + res.start = start + gdp->mem_offset; + res.end = res.start + gdp->mem_size -1 ; + res.flags =IORESOURCE_MEM ; + + i =gpmc_match_child (gdp->name, gdp->id); +/* i >= GPMC_CS_NUM can never happen, this is for compiler to shutup */ +if (i >=0 && i <GPMC_CS_NUM ) { +int j; + + j = gpmc->child_device[i].gpmc_num_res; + gpmc->child_device[i].gpmc_res[j] = res; + }else if (i == -ENOENT ) { + i =gpmc_find_next_child_slot (); +if (IS_ERR_VALUE (i)) { +dev_err (gpmc->dev,"error: childs exceeded\n" ); +return -ENODEV ; + } + gpmc->child_device[i].name = gdp->name; + gpmc->child_device[i].id = gdp->id; + gpmc->child_device[i].pdata = gdp->pdata; + gpmc->child_device[i].pdata_size = gdp->pdata_size; + gpmc->child_device[i].gpmc_res[0 ] = res; + }else { +/* should never come here */ +dev_err (gpmc->dev,"error: childs exceeded\n" ); +return -ENODEV ; + } + + gpmc->child_device[i].gpmc_num_res++; + +return ret; +} + +static __devinit int gpmc_create_child (int cnt) +{ +struct gpmc_child *p = gpmc->child_device + cnt; +int num = p->num_res + p->gpmc_num_res; +struct resource *res; + + res =kzalloc (sizeof (struct resource ) * num,GFP_KERNEL ); +if (!res) { +dev_err (gpmc->dev,"error: allocating memory\n" ); +return -ENOMEM ; + } + +memcpy ((char *)res, (const char *) p->gpmc_res, +sizeof (struct resource ) * p->gpmc_num_res); +memcpy ((char *)(res + p->gpmc_num_res), (const char *)p->res, +sizeof (struct resource ) * p->num_res); + +platform_device_register_resndata (gpmc->dev, p->name, p->id, res, + num, p->pdata, p->pdata_size); + +return 0 ; +} + +static __devinit int gpmc_probe (struct platform_device *pdev) +{ +u32 l; +int i; +int ret = -EINVAL ; +struct resource *res =NULL ; +struct gpmc_pdata *gp =dev_get_platdata (&pdev->dev); +struct gpmc_device_pdata *gdp =NULL ; + + gpmc =devm_kzalloc (&pdev->dev,sizeof (struct gpmc ),GFP_KERNEL ); +if (!gpmc) { +dev_err (&pdev->dev,"Failed to allocate memory\n" ); +return -ENOMEM ; + } + + gpmc->dev = &pdev->dev; + gpmc->fclk_period = gp->fclk_period; + gpmc->num_device = gp->num_device; + + res =platform_get_resource (pdev,IORESOURCE_MEM ,0 ); +if (!res) { + ret = -ENOENT ; +dev_err (gpmc->dev,"Failed to get resource: memory\n" ); + goto err_res; + } + gpmc->phys_base = res->start; + gpmc->memsize =resource_size (res); + +if (request_mem_region (gpmc->phys_base, + gpmc->memsize,DRIVER_NAME ) ==NULL ) { + ret = -ENOMEM ; +dev_err (gpmc->dev,"Failed to request memory region\n" ); + goto err_mem; + } + + gpmc->io_base =ioremap (gpmc->phys_base, gpmc->memsize); +if (!gpmc->io_base) { + ret = -ENOMEM ; +dev_err (gpmc->dev,"Failed to ioremap memory\n" ); + goto err_remap; + } + + gpmc->ecc_used = -EINVAL ; +spin_lock_init (&gpmc->mem_lock); +platform_set_drvdata (pdev, gpmc); l =gpmc_read_reg (GPMC_REVISION ); -printk (KERN_INFO "GPMC revision %d.%d\n" , (l >>4 ) &0x0f , l &0x0f ); -/* Set smart idle mode and automatic L3 clock gating */ - l =gpmc_read_reg (GPMC_SYSCONFIG ); - l &=0x03 <<3 ; - l |= (0x02 <<3 ) | (1 <<0 ); -gpmc_write_reg (GPMC_SYSCONFIG , l); +dev_info (gpmc->dev,"GPMC revision %d.%d\n" , (l >>4 ) &0x0f , l &0x0f ); +gpmc_mem_init (); -/* initalize the irq_chained */ - irq =OMAP_GPMC_IRQ_BASE ; -for (cs =0 ; cs <GPMC_CS_NUM ; cs++) { -irq_set_chip_and_handler (irq, &dummy_irq_chip, - handle_simple_irq); -set_irq_flags (irq,IRQF_VALID ); - irq++; +for (i =0 , gdp = gp->device_pdata; i < gp->num_device; gdp++, i++) { + ret =gpmc_setup_child (gdp); +if (IS_ERR_VALUE (ret)) +dev_err (gpmc->dev,"gpmc setup on CS: %u failed\n" , + gdp->cs); +else + gpmc->num_child++; } - ret =request_irq (gpmc_irq, - gpmc_handle_irq,IRQF_SHARED ,"gpmc" , gpmc_base); -if (ret) -pr_err ("gpmc: irq-%d could not claim: err %d\n" , - gpmc_irq, ret); +/* XXX: This would get modified once MFD */ +for (i =0 ; i < gpmc->num_child; i++) +gpmc_create_child (i); + +return ret; + +err_remap: +release_mem_region (gpmc->phys_base, gpmc->memsize); +err_mem: +err_res: +devm_kfree (&pdev->dev, gpmc);return ret; } -postcore_initcall (gpmc_init); -static irqreturn_t gpmc_handle_irq (int irq,void *dev) +static __devexit int gpmc_remove(struct platform_device *pdev) { -u8 cs; +struct gpmc *gpmc =platform_get_drvdata (pdev); -/* check cs to invoke the irq */ - cs = ((gpmc_read_reg (GPMC_PREFETCH_CONFIG1 )) >>CS_NUM_SHIFT ) &0x7 ; -if (OMAP_GPMC_IRQ_BASE +cs <=OMAP_GPMC_IRQ_END ) -generic_handle_irq (OMAP_GPMC_IRQ_BASE +cs); +platform_set_drvdata (pdev,NULL ); +iounmap (gpmc->io_base); +release_mem_region (gpmc->phys_base, gpmc->memsize); +devm_kfree (&pdev->dev, gpmc); -return IRQ_HANDLED ; +return 0 ; } +static struct platform_driver gpmc_driver = { + .probe = gpmc_probe, + .remove =__devexit_p (gpmc_remove), + .driver = { + .name =DRIVER_NAME , + .owner =THIS_MODULE , + }, +}; + +module_platform_driver (gpmc_driver); +#ifdef CONFIG_ARCH_OMAP3 static struct omap3_gpmc_regs gpmc_context;
@@ -855,10 +1035,10 @@ int gpmc_enable_hwecc(int cs, int mode, int dev_width, int ecc_size)unsigned int val;/* check if ecc module is in used */ -if (gpmc_ecc_used != -EINVAL ) +if (gpmc->ecc_used != -EINVAL )return -EINVAL ; - gpmc_ecc_used = cs; + gpmc->ecc_used = cs;/* clear ecc and enable bits */ val = ((0x00000001 <<8 ) |0x00000001 );
@@ -906,7 +1086,7 @@ int gpmc_calculate_ecc(int cs, const u_char *dat, u_char *ecc_code) {unsigned int val =0x0 ; -if (gpmc_ecc_used != cs) +if (gpmc->ecc_used != cs)return -EINVAL ;/* read ecc result */
@@ -916,7 +1096,7 @@ int gpmc_calculate_ecc(int cs, const u_char *dat, u_char *ecc_code)/* P2048o, P1024o, P512o, P256o, P2048e, P1024e, P512e, P256e */ *ecc_code++ = ((val >>8 ) &0x0f ) | ((val >>20 ) &0xf0 ); - gpmc_ecc_used = -EINVAL ; + gpmc->ecc_used = -EINVAL ;return 0 ; }EXPORT_SYMBOL_GPL (gpmc_calculate_ecc);
arch/arm/plat-omap/include/plat/gpmc.h +32 -2
@@ -131,6 +131,38 @@ struct gpmc_timings {u16 wr_data_mux_bus;/* WRDATAONADMUXBUS */ }; +struct gpmc_config { +int cmd; +int val; +}; + +struct gpmc_device_pdata { +/* connected peripheral specific */ +char *name; +int id; +/* resources configured via GPMC will be created by GPMC driver */ +struct resource *res; +unsigned num_res; +void *pdata; +unsigned pdata_size; + +/* GPMC specific */ +unsigned cs; +unsigned long mem_size; +unsigned long mem_start; +unsigned long mem_offset; +struct gpmc_config *config; +unsigned num_config; +struct gpmc_timings *timing; +}; + +struct gpmc_pdata { +/* GPMC_FCLK rate in picoseconds */ +unsigned long fclk_period; +struct gpmc_device_pdata *device_pdata; +unsigned num_device; +}; +extern unsigned int gpmc_ns_to_ticks (unsigned int time_ns);extern unsigned int gpmc_ps_to_ticks (unsigned int time_ps);extern unsigned int gpmc_ticks_to_ns (unsigned int ticks);
@@ -143,8 +175,6 @@ extern int gpmc_cs_calc_divider(int cs, unsigned int sync_clk);extern int gpmc_cs_set_timings (int cs,const struct gpmc_timings *t);extern int gpmc_cs_request (int cs,unsigned long size,unsigned long *base);extern void gpmc_cs_free (int cs); -extern int gpmc_cs_set_reserved (int cs,int reserved); -extern int gpmc_cs_reserved (int cs);extern int gpmc_prefetch_enable (int cs,int fifo_th,int dma_mode,unsigned int u32_count,int is_write);extern int gpmc_prefetch_reset (int cs);
Signature · 1 line
1.7.9.3
RFC PATCH v2 2/5 ARM: OMAP2+: nand: create platform data structure
In reply to Afzal MohammedNew API for updating nand platform data. This has been created by unifying the two existing ones and taking out gpmc hardware handling.
Quoted 1 line
From now on, platforms can call
omap_nand_initto
initialize platform nand structures, it's fields. Or can statically create the same.
Acquiring gpmc CS has been removed. Acquiring CS probably should be avoided, if ever required, do in GPMC driver.
--- arch/arm/mach-omap2/board-devkit8000.c | 6 ++- arch/arm/mach-omap2/board-flash.c | 63 ++++++++++++++-------------- arch/arm/mach-omap2/board-flash.h | 13 ++++-- arch/arm/mach-omap2/board-ldp.c | 4 +- arch/arm/mach-omap2/board-omap3beagle.c | 6 ++- arch/arm/mach-omap2/board-omap3touchbook.c | 6 ++- arch/arm/mach-omap2/board-overo.c | 5 ++- arch/arm/mach-omap2/board-zoom.c | 5 ++- arch/arm/mach-omap2/common-board-devices.c | 46 -------------------- arch/arm/mach-omap2/common-board-devices.h | 1 - 10 files changed, 61 insertions(+), 94 deletions(-)
arch/arm/mach-omap2/board-devkit8000.c +4 -2
@@ -59,6 +59,7 @@#include "mux.h" #include "hsmmc.h" +#include "board-flash.h" #include "common-board-devices.h" #define OMAP_DM9000_GPIO_IRQ 25
@@ -648,8 +649,9 @@ static void __init devkit8000_init(void)usb_musb_init (NULL );usbhs_init (&usbhs_bdata); -omap_nand_flash_init (NAND_BUSWIDTH_16 , devkit8000_nand_partitions, -ARRAY_SIZE (devkit8000_nand_partitions)); +omap_nand_init (devkit8000_nand_partitions, +ARRAY_SIZE (devkit8000_nand_partitions),GPMC_CS_NUM +1 , +NAND_BUSWIDTH_16 ,NULL );/* Ensure SDRC pins are mux'd for self-refresh */ omap_mux_init_signal ("sdrc_cke0" ,OMAP_PIN_OUTPUT );
arch/arm/mach-omap2/board-flash.c +32 -31
83 lines not on this page · raw message
arch/arm/mach-omap2/board-flash.h +9 -4
@@ -39,11 +39,16 @@ static inline void board_flash_init(struct flash_partitions part[],#if defined(CONFIG_MTD_NAND_OMAP2 ) || \ defined(CONFIG_MTD_NAND_OMAP2_MODULE ) -extern void board_nand_init (struct mtd_partition *nand_parts, -u8 nr_parts,u8 cs,int nand_type); +extern struct gpmc_timings nand_default_timings[]; +extern struct omap_nand_platform_data * +__init omap_nand_init (struct mtd_partition *nand_parts,u8 nr_parts,u8 cs, +int nand_type,struct gpmc_timings *gpmc_t);#else -static inline void board_nand_init (struct mtd_partition *nand_parts, -u8 nr_parts,u8 cs,int nand_type) +#define nand_default_timings NULL +static inline struct omap_nand_platform_data * +omap_nand_init (struct mtd_partition *nand_parts, +u8 nr_parts,u8 cs,int nand_type,struct gpmc_timings *gpmc_t) { +return NULL ; }#endif
arch/arm/mach-omap2/board-ldp.c +2 -2
@@ -420,8 +420,8 @@ static void __init omap_ldp_init(void)omap_serial_init ();omap_sdrc_init (NULL ,NULL );usb_musb_init (NULL ); -board_nand_init (ldp_nand_partitions, -ARRAY_SIZE (ldp_nand_partitions),ZOOM_NAND_CS ,0 ); +omap_nand_init (ldp_nand_partitions,ARRAY_SIZE (ldp_nand_partitions), +ZOOM_NAND_CS ,0 , nand_default_timings);omap_hsmmc_init (mmc);ldp_display_init ();
arch/arm/mach-omap2/board-omap3beagle.c +4 -2
@@ -51,6 +51,7 @@#include "mux.h" #include "hsmmc.h" #include "pm.h" +#include "board-flash.h" #include "common-board-devices.h" /*
@@ -542,8 +543,9 @@ static void __init omap3_beagle_init(void)usb_musb_init (NULL );usbhs_init (&usbhs_bdata); -omap_nand_flash_init (NAND_BUSWIDTH_16 , omap3beagle_nand_partitions, -ARRAY_SIZE (omap3beagle_nand_partitions)); +omap_nand_init (omap3beagle_nand_partitions, +ARRAY_SIZE (omap3beagle_nand_partitions),GPMC_CS_NUM +1 , +NAND_BUSWIDTH_16 ,NULL );/* Ensure msecure is mux'd to be able to set the RTC. */ omap_mux_init_signal ("sys_drm_msecure" ,OMAP_PIN_OFF_OUTPUT_HIGH );
arch/arm/mach-omap2/board-omap3touchbook.c +4 -2
@@ -51,6 +51,7 @@#include "mux.h" #include "hsmmc.h" +#include "board-flash.h" #include "common-board-devices.h" #include <asm/setup.h>
@@ -369,8 +370,9 @@ static void __init omap3_touchbook_init(void)omap_ads7846_init (4 ,OMAP3_TS_GPIO ,310 , &ads7846_pdata);usb_musb_init (NULL );usbhs_init (&usbhs_bdata); -omap_nand_flash_init (NAND_BUSWIDTH_16 , omap3touchbook_nand_partitions, -ARRAY_SIZE (omap3touchbook_nand_partitions)); +omap_nand_init (omap3touchbook_nand_partitions, +ARRAY_SIZE (omap3touchbook_nand_partitions),GPMC_CS_NUM +1 , +NAND_BUSWIDTH_16 ,NULL );/* Ensure SDRC pins are mux'd for self-refresh */ omap_mux_init_signal ("sdrc_cke0" ,OMAP_PIN_OUTPUT );
arch/arm/mach-omap2/board-overo.c +3 -2
@@ -57,6 +57,7 @@#include "mux.h" #include "sdram-micron-mt46h32m32lf-6.h" #include "hsmmc.h" +#include "board-flash.h" #include "common-board-devices.h" #define OVERO_GPIO_BT_XGATE 15
@@ -509,8 +510,8 @@ static void __init overo_init(void)omap_serial_init ();omap_sdrc_init (mt46h32m32lf6_sdrc_params, mt46h32m32lf6_sdrc_params); -omap_nand_flash_init (0 , overo_nand_partitions, -ARRAY_SIZE (overo_nand_partitions)); +omap_nand_init (overo_nand_partitions, +ARRAY_SIZE (overo_nand_partitions),GPMC_CS_NUM +1 ,0 ,NULL );usb_musb_init (NULL );usbhs_init (&usbhs_bdata);overo_spi_init ();
arch/arm/mach-omap2/board-zoom.c +3 -2
@@ -114,8 +114,9 @@ static void __init omap_zoom_init(void)usbhs_init (&usbhs_bdata); } -board_nand_init (zoom_nand_partitions,ARRAY_SIZE (zoom_nand_partitions), -ZOOM_NAND_CS ,NAND_BUSWIDTH_16 ); +omap_nand_init (zoom_nand_partitions, +ARRAY_SIZE (zoom_nand_partitions),ZOOM_NAND_CS , +NAND_BUSWIDTH_16 , nand_default_timings);zoom_debugboard_init ();zoom_peripherals_init ();
arch/arm/mach-omap2/common-board-devices.c +0 -46
49 lines not on this page · raw message
arch/arm/mach-omap2/common-board-devices.h +0 -1
@@ -10,6 +10,5 @@ struct ads7846_platform_data;void omap_ads7846_init (int bus_num,int gpio_pendown,int gpio_debounce,struct ads7846_platform_data *board_pdata); -void omap_nand_flash_init (int opts,struct mtd_partition *parts,int n_parts);#endif /* __OMAP_COMMON_BOARD_DEVICES__ */
Signature · 1 line
1.7.9.3
RFC PATCH v2 3/5 ARM: OMAP2+: gpmc-nand: populate gpmc configs
In reply to Afzal MohammedCurrently gpmc is configured in platform for nand. As configuring gpmc has been moved to gpmc driver, populate details needed for the driver to configure gpmc. gpmc driver would configure based on this information.
--- arch/arm/mach-omap2/gpmc-nand.c | 88 ++++++++++++-------------------- arch/arm/plat-omap/include/plat/nand.h | 8 +-- 2 files changed, 39 insertions(+), 57 deletions(-)
arch/arm/mach-omap2/gpmc-nand.c +34 -54
@@ -21,24 +21,29 @@#include <plat/board.h> #include <plat/gpmc.h> -static struct resource gpmc_nand_resource = { - .flags =IORESOURCE_MEM , + +#define GPMC_NAND_CONFIG_NUM 3 + +static struct gpmc_config gpmc_nand_config[GPMC_NAND_CONFIG_NUM ] = { + {GPMC_CONFIG_DEV_TYPE ,GPMC_DEVICETYPE_NAND }, }; -static struct platform_device gpmc_nand_device = { +static struct gpmc_device_pdata gpmc_nand_info = { .name ="omap2-nand" , .id =0 , - .num_resources =1 , - .resource = &gpmc_nand_resource, + .config = gpmc_nand_config, + .num_config =ARRAY_SIZE (gpmc_nand_config), }; -static int omap2_nand_gpmc_retime (struct omap_nand_platform_data *gpmc_nand_data) -{ -struct gpmc_timings t; -int err; +static struct gpmc_timings t; -if (!gpmc_nand_data->gpmc_t) +static struct gpmc_timings * +gpmc_nand_retime (struct omap_nand_platform_data *gpmc_nand_data) +{ +if (!gpmc_nand_data->gpmc_t) { +pr_warn ("gpmc timings not provided\n" );return 0 ; + }memset (&t,0 ,sizeof (t)); t.sync_clk = gpmc_nand_data->gpmc_t->sync_clk;
@@ -68,56 +73,31 @@ static int omap2_nand_gpmc_retime(struct omap_nand_platform_data *gpmc_nand_data t.cs_wr_off =gpmc_round_ns_to_ticks (gpmc_nand_data->gpmc_t->cs_wr_off); t.wr_cycle =gpmc_round_ns_to_ticks (gpmc_nand_data->gpmc_t->wr_cycle); -/* Configure GPMC */ -if (gpmc_nand_data->devsize ==NAND_BUSWIDTH_16 ) -gpmc_cs_configure (gpmc_nand_data->cs,GPMC_CONFIG_DEV_SIZE ,1 ); -else -gpmc_cs_configure (gpmc_nand_data->cs,GPMC_CONFIG_DEV_SIZE ,0 ); -gpmc_cs_configure (gpmc_nand_data->cs, -GPMC_CONFIG_DEV_TYPE ,GPMC_DEVICETYPE_NAND ); - err =gpmc_cs_set_timings (gpmc_nand_data->cs, &t); -if (err) -return err; - -return 0 ; +return &t; } -int __init gpmc_nand_init (struct omap_nand_platform_data *gpmc_nand_data) +struct gpmc_device_pdata * +__init gpmc_nand_init (struct omap_nand_platform_data *gpmc_nand_data) { -int err =0 ; -struct device *dev = &gpmc_nand_device.dev; + gpmc_nand_info.pdata = gpmc_nand_data; + gpmc_nand_info.pdata_size =sizeof (*gpmc_nand_data); - gpmc_nand_device.dev.platform_data = gpmc_nand_data; + gpmc_nand_info.cs = gpmc_nand_data->cs; + gpmc_nand_info.mem_size =NAND_IO_SIZE ; - err =gpmc_cs_request (gpmc_nand_data->cs,NAND_IO_SIZE , - &gpmc_nand_data->phys_base); -if (err <0 ) { -dev_err (dev,"Cannot request GPMC CS\n" ); -return err; - } + gpmc_nand_info.timing =gpmc_nand_retime (gpmc_nand_data); -/* Set timings in GPMC */ - err =omap2_nand_gpmc_retime (gpmc_nand_data); -if (err <0 ) { -dev_err (dev,"Unable to set gpmc timings: %d\n" , err); -return err; - } - -/* Enable RD PIN Monitoring Reg */ -if (gpmc_nand_data->dev_ready) { -gpmc_cs_configure (gpmc_nand_data->cs,GPMC_CONFIG_RDY_BSY ,1 ); - } - - err =platform_device_register (&gpmc_nand_device); -if (err <0 ) { -dev_err (dev,"Unable to register NAND device\n" ); - goto out_free_cs; - } - -return 0 ; + gpmc_nand_config[1 ].cmd =GPMC_CONFIG_DEV_SIZE ; +if (gpmc_nand_data->devsize ==NAND_BUSWIDTH_16 ) + gpmc_nand_config[1 ].val =1 ; +else + gpmc_nand_config[1 ].val =0 ; -out_free_cs: -gpmc_cs_free (gpmc_nand_data->cs); + gpmc_nand_config[2 ].cmd =GPMC_CONFIG_RDY_BSY ; +if (gpmc_nand_data->dev_ready) + gpmc_nand_config[2 ].val =1 ; +else + gpmc_nand_config[2 ].val =0 ; -return err; +return &gpmc_nand_info; }
arch/arm/plat-omap/include/plat/nand.h +5 -3
@@ -35,10 +35,12 @@ struct omap_nand_platform_data {#define NAND_IO_SIZE 4#if defined(CONFIG_MTD_NAND_OMAP2 ) || defined(CONFIG_MTD_NAND_OMAP2_MODULE ) -extern int gpmc_nand_init (struct omap_nand_platform_data *d); +extern struct gpmc_device_pdata * +gpmc_nand_init (struct omap_nand_platform_data *d);#else -static inline int gpmc_nand_init (struct omap_nand_platform_data *d) +static inline struct gpmc_device_pdata * +gpmc_nand_init (struct omap_nand_platform_data *d) { -return 0 ; +return NULL ; }#endif
Signature · 1 line
1.7.9.3
RFC PATCH v2 4/5 mtd: nand: omap2: obtain memory from resource
In reply to Afzal Mohammedgpmc being converted to driver, provides drivers
of peripheral connected memory space used by the
peripheral as memory resource.
Modify nand omap driver to obtain memory detials
from resource structure.
--- arch/arm/plat-omap/include/plat/nand.h | 1 - drivers/mtd/nand/omap2.c | 19 +++++++++++++++---- 2 files changed, 15 insertions(+), 5 deletions(-)
arch/arm/plat-omap/include/plat/nand.h +0 -1
@@ -26,7 +26,6 @@ struct omap_nand_platform_data {bool dev_ready;int gpmc_irq;enum nand_io xfer_type; -unsigned long phys_base;int devsize;enum omap_ecc ecc_opt; };
drivers/mtd/nand/omap2.c +15 -4
@@ -118,6 +118,7 @@ struct omap_nand_info {int gpmc_cs;unsigned long phys_base; +unsigned long mem_size;struct completion comp;int dma_ch;int gpmc_irq;
@@ -931,6 +932,7 @@ static int __devinit omap_nand_probe(struct platform_device *pdev)struct omap_nand_platform_data *pdata;int err;int i, offset; +struct resource *res; pdata = pdev->dev.platform_data;if (pdata ==NULL ) {
@@ -950,7 +952,6 @@ static int __devinit omap_nand_probe(struct platform_device *pdev) info->pdev = pdev; info->gpmc_cs = pdata->cs; - info->phys_base = pdata->phys_base; info->mtd.priv = &info->nand; info->mtd.name =dev_name (&pdev->dev);
@@ -962,13 +963,23 @@ static int __devinit omap_nand_probe(struct platform_device *pdev)/* NAND write protect off */ gpmc_cs_configure (info->gpmc_cs,GPMC_CONFIG_WP ,0 ); -if (!request_mem_region (info->phys_base,NAND_IO_SIZE , + res =platform_get_resource (pdev,IORESOURCE_MEM ,0 ); +if (res ==NULL ) { + err = -EINVAL ; +dev_err (&pdev->dev,"error getting memory resource\n" ); + goto out_free_info; + } + + info->phys_base = res->start; + info->mem_size =resource_size (res); + +if (!request_mem_region (info->phys_base, info->mem_size, pdev->dev.driver->name)) { err = -EBUSY ; goto out_free_info; } - info->nand.IO_ADDR_R =ioremap (info->phys_base,NAND_IO_SIZE ); + info->nand.IO_ADDR_R =ioremap (info->phys_base, info->mem_size);if (!info->nand.IO_ADDR_R) { err = -ENOMEM ; goto out_release_mem_region;
@@ -1109,7 +1120,7 @@ static int __devinit omap_nand_probe(struct platform_device *pdev)return 0 ; out_release_mem_region: -release_mem_region (info->phys_base,NAND_IO_SIZE ); +release_mem_region (info->phys_base, info->mem_size); out_free_info:kfree (info);
Signature · 1 line
1.7.9.3
PATCH v2 5/5 OMAP3EVM: Test gpmc-nand
In reply to Afzal Mohammed--- arch/arm/mach-omap2/board-omap3evm.c | 82 ++++++++++++++++++++++++++++++++++ 1 file changed, 82 insertions(+)
arch/arm/mach-omap2/board-omap3evm.c +82 -0
@@ -23,6 +23,7 @@#include <linux/input/matrix_keypad.h> #include <linux/leds.h> #include <linux/interrupt.h> +#include <linux/mtd/nand.h> #include <linux/spi/spi.h> #include <linux/spi/ads7846.h>
@@ -41,6 +42,7 @@#include <asm/mach/arch.h> #include <asm/mach/map.h> +#include <plat/nand.h> #include <plat/board.h> #include <plat/usb.h> #include "common.h"
@@ -52,6 +54,7 @@#include "sdram-micron-mt46h32m32lf-6.h" #include "hsmmc.h" #include "common-board-devices.h" +#include "board-flash.h" #define OMAP3_EVM_TS_GPIO 175#define OMAP3_EVM_EHCI_VBUS 22
@@ -102,6 +105,9 @@ static void __init omap3_evm_get_revision(void) } } +#undef CONFIG_SMSC911X +#undef CONFIG_SMSC911X_MODULE +#if defined(CONFIG_SMSC911X ) || defined(CONFIG_SMSC911X_MODULE )#include <plat/gpmc-smsc911x.h>
@@ -533,6 +539,41 @@ static struct usbhs_omap_board_data usbhs_bdata __initdata = { .reset_gpio_port[2] = -EINVAL }; +/* +* NAND +*/ +static struct mtd_partition omap3_evm_nand_partitions[] = { +/* All the partition sizes are listed in terms of NAND block size */ + { + .name ="X-Loader-NAND" , + .offset =0 , + .size =4 * (64 *2048 ), + .mask_flags =MTD_WRITEABLE ,/* force read-only */ + }, + { + .name ="U-Boot-NAND" , + .offset =MTDPART_OFS_APPEND ,/* Offset = 0x80000 */ + .size =10 * (64 *2048 ), + .mask_flags =MTD_WRITEABLE ,/* force read-only */ + }, + { + .name ="Boot Env-NAND" , + + .offset =MTDPART_OFS_APPEND ,/* Offset = 0x1c0000 */ + .size =6 * (64 *2048 ), + }, + { + .name ="Kernel-NAND" , + .offset =MTDPART_OFS_APPEND ,/* Offset = 0x280000 */ + .size =40 * (64 *2048 ), + }, + { + .name ="File System - NAND" , + .size =MTDPART_SIZ_FULL , + .offset =MTDPART_OFS_APPEND ,/* Offset = 0x780000 */ + }, +}; + #ifdef CONFIG_OMAP_MUX static struct omap_board_mux omap35x_board_mux[] __initdata = { OMAP3_MUX(SYS_NIRQ, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP |
@@ -632,8 +673,30 @@ static void __init omap3_evm_wl12xx_init(void)#endif } +static struct gpmc_pdata gpmc_data; + +static struct resource gpmc_resources[] = { + { + .start =OMAP34XX_GPMC_BASE , + .end =OMAP34XX_GPMC_BASE +SZ_4K -1 , + .flags =IORESOURCE_MEM , + }, +}; + +static struct platform_device gpmc_device = { + .name ="omap-gpmc" , + .id =-1 , + .num_resources =ARRAY_SIZE (gpmc_resources), + .resource = gpmc_resources, + .dev = { + .platform_data = &gpmc_data, + } +}; +static void __init omap3_evm_init (void ) { +struct omap_nand_platform_data *nand_data; +omap3_evm_get_revision ();if (cpu_is_omap3630 ())
@@ -684,6 +747,25 @@ static void __init omap3_evm_init(void)omap3evm_init_smsc911x ();omap3_evm_display_init ();omap3_evm_wl12xx_init (); +/* NAND */ + nand_data =omap_nand_init (omap3_evm_nand_partitions, +ARRAY_SIZE (omap3_evm_nand_partitions), +0 ,NAND_BUSWIDTH_16 , nand_default_timings); +if (nand_data ==NULL ) { +pr_err ("omap_nand_init() failed\n" ); +return ; + } + + gpmc_data.device_pdata =gpmc_nand_init (nand_data); +if (gpmc_data.device_pdata ==NULL ) { +pr_err ("gpmc_nand_init() failed\n" ); +return ; + } + + gpmc_data.num_device++; + gpmc_data.fclk_period =gpmc_get_fclk_period (); + +platform_device_register (&gpmc_device); }MACHINE_START (OMAP3EVM ,"OMAP3 EVM" )
Signature · 1 line
1.7.9.3