* [RFC] ARM: Add SMSC911X support to Overo platform
@ 2009-01-30 20:28 Steve Sakoman
2009-01-30 20:43 ` Steve Sakoman
2009-01-30 22:18 ` Kevin Hilman
0 siblings, 2 replies; 3+ messages in thread
From: Steve Sakoman @ 2009-01-30 20:28 UTC (permalink / raw)
To: linux-omap, netdev
Gumstix will soon be shipping a variant of their Summit board that
includes an SMSC9221 ethernet interface. This patch is intended to
provide support via the smsc911x driver when enabled in defconfig.
The final submission of course will also include an updated defconfig.
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
diff --git a/arch/arm/mach-omap2/board-overo.c
b/arch/arm/mach-omap2/board-overo.c
index 9995ac2..97d95a4 100644
--- a/arch/arm/mach-omap2/board-overo.c
+++ b/arch/arm/mach-omap2/board-overo.c
@@ -55,6 +55,63 @@
#define GPMC_CS0_BASE 0x60
#define GPMC_CS_SIZE 0x30
+#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
+
+#include <linux/smsc911x.h>
+
+static struct resource overo_smsc911x_resources[] = {
+ {
+ .name = "smsc911x-memory",
+ .flags = IORESOURCE_MEM,
+ },
+ {
+ .flags = IORESOURCE_IRQ | IORESOURCE_IRQ_LOWLEVEL,
+ },
+};
+
+static struct smsc911x_platform_config overo_smsc911x_config = {
+ .irq_polarity = SMSC911X_IRQ_POLARITY_ACTIVE_LOW,
+ .irq_type = SMSC911X_IRQ_TYPE_OPEN_DRAIN,
+ .flags = SMSC911X_USE_32BIT ,
+ .phy_interface = PHY_INTERFACE_MODE_MII,
+};
+
+static struct platform_device overo_smsc911x_device = {
+ .name = "smsc911x",
+ .id = -1,
+ .num_resources = ARRAY_SIZE(overo_smsc911x_resources),
+ .resource = &overo_smsc911x_resources,
+ .dev = {
+ .platform_data = &overo_smsc911x_config,
+ },
+};
+
+static inline void __init overo_init_smsc911x(void)
+{
+ unsigned long cs_mem_base;
+
+ if (gpmc_cs_request(OVERO_SMSC911X_CS, SZ_16M, &cs_mem_base) < 0) {
+ printk(KERN_ERR "Failed request for GPMC mem for smsc911x\n");
+ return;
+ }
+
+ overo_smsc911x_resources[0].start = cs_mem_base + 0x0;
+ overo_smsc911x_resources[0].end = cs_mem_base + 0xff;
+
+ if ((gpio_request(OVERO_SMSC911X_GPIO, "SMSC911X IRQ") == 0) &&
+ (gpio_direction_input(OVERO_SMSC911X_GPIO) == 0)) {
+ gpio_export(OVERO_SMSC911X_GPIO, 0);
+ } else {
+ printk(KERN_ERR "could not obtain gpio for SMSC911X IRQ\n");
+ return;
+ }
+
+ overo_smsc911x_resources[1].start = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
+ overo_smsc911x_resources[1].end = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
+}
+
+#endif
+
static struct mtd_partition overo_nand_partitions[] = {
{
.name = "xloader",
@@ -187,6 +244,9 @@ static void __init overo_init_irq(void)
omap2_init_common_hw(mt46h32m32lf6_sdrc_params);
omap_init_irq();
omap_gpio_init();
+#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
+ overo_init_smsc911x();
+#endif
}
static struct platform_device overo_lcd_device = {
@@ -204,6 +264,9 @@ static struct omap_board_config_kernel
overo_config[] __initdata = {
};
static struct platform_device *overo_devices[] __initdata = {
+#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
+ &overo_smsc911x_device,
+#endif
&overo_lcd_device,
};
diff --git a/arch/arm/plat-omap/include/mach/board-overo.h
b/arch/arm/plat-omap/include/mach/board-overo.h
index 7ecae66..8635171 100644
--- a/arch/arm/plat-omap/include/mach/board-overo.h
+++ b/arch/arm/plat-omap/include/mach/board-overo.h
@@ -22,5 +22,8 @@
#define OVERO_GPIO_USBH_CPEN 168
#define OVERO_GPIO_USBH_NRESET 183
+#define OVERO_SMSC911X_CS 5
+#define OVERO_SMSC911X_GPIO 176
+
#endif /* ____ASM_ARCH_OVERO_H */
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [RFC] ARM: Add SMSC911X support to Overo platform
2009-01-30 20:28 [RFC] ARM: Add SMSC911X support to Overo platform Steve Sakoman
@ 2009-01-30 20:43 ` Steve Sakoman
2009-01-30 22:18 ` Kevin Hilman
1 sibling, 0 replies; 3+ messages in thread
From: Steve Sakoman @ 2009-01-30 20:43 UTC (permalink / raw)
To: linux-omap, netdev
On Fri, Jan 30, 2009 at 12:28 PM, Steve Sakoman <sakoman@gmail.com> wrote:
> Gumstix will soon be shipping a variant of their Summit board that
> includes an SMSC9221 ethernet interface. This patch is intended to
> provide support via the smsc911x driver when enabled in defconfig.
> The final submission of course will also include an updated defconfig.
>
> Signed-off-by: Steve Sakoman <steve@sakoman.com>
> ---
> diff --git a/arch/arm/mach-omap2/board-overo.c
> b/arch/arm/mach-omap2/board-overo.c
> index 9995ac2..97d95a4 100644
> --- a/arch/arm/mach-omap2/board-overo.c
> +++ b/arch/arm/mach-omap2/board-overo.c
> @@ -55,6 +55,63 @@
> #define GPMC_CS0_BASE 0x60
> #define GPMC_CS_SIZE 0x30
>
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> +
> +#include <linux/smsc911x.h>
> +
> +static struct resource overo_smsc911x_resources[] = {
> + {
> + .name = "smsc911x-memory",
> + .flags = IORESOURCE_MEM,
> + },
> + {
> + .flags = IORESOURCE_IRQ | IORESOURCE_IRQ_LOWLEVEL,
> + },
> +};
> +
> +static struct smsc911x_platform_config overo_smsc911x_config = {
> + .irq_polarity = SMSC911X_IRQ_POLARITY_ACTIVE_LOW,
> + .irq_type = SMSC911X_IRQ_TYPE_OPEN_DRAIN,
> + .flags = SMSC911X_USE_32BIT ,
> + .phy_interface = PHY_INTERFACE_MODE_MII,
> +};
> +
> +static struct platform_device overo_smsc911x_device = {
> + .name = "smsc911x",
> + .id = -1,
> + .num_resources = ARRAY_SIZE(overo_smsc911x_resources),
> + .resource = &overo_smsc911x_resources,
> + .dev = {
> + .platform_data = &overo_smsc911x_config,
> + },
> +};
> +
> +static inline void __init overo_init_smsc911x(void)
> +{
> + unsigned long cs_mem_base;
> +
> + if (gpmc_cs_request(OVERO_SMSC911X_CS, SZ_16M, &cs_mem_base) < 0) {
> + printk(KERN_ERR "Failed request for GPMC mem for smsc911x\n");
> + return;
> + }
> +
> + overo_smsc911x_resources[0].start = cs_mem_base + 0x0;
> + overo_smsc911x_resources[0].end = cs_mem_base + 0xff;
> +
> + if ((gpio_request(OVERO_SMSC911X_GPIO, "SMSC911X IRQ") == 0) &&
> + (gpio_direction_input(OVERO_SMSC911X_GPIO) == 0)) {
> + gpio_export(OVERO_SMSC911X_GPIO, 0);
> + } else {
> + printk(KERN_ERR "could not obtain gpio for SMSC911X IRQ\n");
> + return;
> + }
> +
> + overo_smsc911x_resources[1].start = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
> + overo_smsc911x_resources[1].end = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
Just to see if anyone was paying attention, I pasted the version with
the stupid typos :-)
The two lines above were meant to end with semicolons rather than commas!
> +}
> +
> +#endif
> +
> static struct mtd_partition overo_nand_partitions[] = {
> {
> .name = "xloader",
> @@ -187,6 +244,9 @@ static void __init overo_init_irq(void)
> omap2_init_common_hw(mt46h32m32lf6_sdrc_params);
> omap_init_irq();
> omap_gpio_init();
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> + overo_init_smsc911x();
> +#endif
> }
>
> static struct platform_device overo_lcd_device = {
> @@ -204,6 +264,9 @@ static struct omap_board_config_kernel
> overo_config[] __initdata = {
> };
>
> static struct platform_device *overo_devices[] __initdata = {
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> + &overo_smsc911x_device,
> +#endif
> &overo_lcd_device,
> };
>
> diff --git a/arch/arm/plat-omap/include/mach/board-overo.h
> b/arch/arm/plat-omap/include/mach/board-overo.h
> index 7ecae66..8635171 100644
> --- a/arch/arm/plat-omap/include/mach/board-overo.h
> +++ b/arch/arm/plat-omap/include/mach/board-overo.h
> @@ -22,5 +22,8 @@
> #define OVERO_GPIO_USBH_CPEN 168
> #define OVERO_GPIO_USBH_NRESET 183
>
> +#define OVERO_SMSC911X_CS 5
> +#define OVERO_SMSC911X_GPIO 176
> +
> #endif /* ____ASM_ARCH_OVERO_H */
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [RFC] ARM: Add SMSC911X support to Overo platform
2009-01-30 20:28 [RFC] ARM: Add SMSC911X support to Overo platform Steve Sakoman
2009-01-30 20:43 ` Steve Sakoman
@ 2009-01-30 22:18 ` Kevin Hilman
1 sibling, 0 replies; 3+ messages in thread
From: Kevin Hilman @ 2009-01-30 22:18 UTC (permalink / raw)
To: Steve Sakoman; +Cc: linux-omap, netdev
Steve Sakoman <sakoman@gmail.com> writes:
> Gumstix will soon be shipping a variant of their Summit board that
> includes an SMSC9221 ethernet interface. This patch is intended to
> provide support via the smsc911x driver when enabled in defconfig.
> The final submission of course will also include an updated defconfig.
>
> Signed-off-by: Steve Sakoman <steve@sakoman.com>
> ---
> diff --git a/arch/arm/mach-omap2/board-overo.c
> b/arch/arm/mach-omap2/board-overo.c
> index 9995ac2..97d95a4 100644
> --- a/arch/arm/mach-omap2/board-overo.c
> +++ b/arch/arm/mach-omap2/board-overo.c
> @@ -55,6 +55,63 @@
> #define GPMC_CS0_BASE 0x60
> #define GPMC_CS_SIZE 0x30
>
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> +
> +#include <linux/smsc911x.h>
> +
> +static struct resource overo_smsc911x_resources[] = {
> + {
> + .name = "smsc911x-memory",
> + .flags = IORESOURCE_MEM,
> + },
> + {
> + .flags = IORESOURCE_IRQ | IORESOURCE_IRQ_LOWLEVEL,
> + },
> +};
> +
> +static struct smsc911x_platform_config overo_smsc911x_config = {
> + .irq_polarity = SMSC911X_IRQ_POLARITY_ACTIVE_LOW,
> + .irq_type = SMSC911X_IRQ_TYPE_OPEN_DRAIN,
> + .flags = SMSC911X_USE_32BIT ,
> + .phy_interface = PHY_INTERFACE_MODE_MII,
> +};
> +
> +static struct platform_device overo_smsc911x_device = {
> + .name = "smsc911x",
> + .id = -1,
> + .num_resources = ARRAY_SIZE(overo_smsc911x_resources),
> + .resource = &overo_smsc911x_resources,
> + .dev = {
> + .platform_data = &overo_smsc911x_config,
> + },
> +};
> +
> +static inline void __init overo_init_smsc911x(void)
> +{
> + unsigned long cs_mem_base;
> +
> + if (gpmc_cs_request(OVERO_SMSC911X_CS, SZ_16M, &cs_mem_base) < 0) {
> + printk(KERN_ERR "Failed request for GPMC mem for smsc911x\n");
> + return;
> + }
> +
> + overo_smsc911x_resources[0].start = cs_mem_base + 0x0;
> + overo_smsc911x_resources[0].end = cs_mem_base + 0xff;
> +
> + if ((gpio_request(OVERO_SMSC911X_GPIO, "SMSC911X IRQ") == 0) &&
> + (gpio_direction_input(OVERO_SMSC911X_GPIO) == 0)) {
> + gpio_export(OVERO_SMSC911X_GPIO, 0);
> + } else {
> + printk(KERN_ERR "could not obtain gpio for SMSC911X IRQ\n");
> + return;
> + }
> +
> + overo_smsc911x_resources[1].start = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
> + overo_smsc911x_resources[1].end = OMAP_GPIO_IRQ(OVERO_SMSC911X_GPIO),
I would put the 'platform_device_register(&overo_smsc911x_device)' here.
> +}
> +
Then here, add:
#else
static inline void __init overo_init_smsc911x(void) { return; }
This allows you to drop the #ifdefs below and have it look a little
cleaner.
Other than that, looks OK to me.
Kevin
> +#endif
> +
> static struct mtd_partition overo_nand_partitions[] = {
> {
> .name = "xloader",
> @@ -187,6 +244,9 @@ static void __init overo_init_irq(void)
> omap2_init_common_hw(mt46h32m32lf6_sdrc_params);
> omap_init_irq();
> omap_gpio_init();
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> + overo_init_smsc911x();
> +#endif
> }
>
> static struct platform_device overo_lcd_device = {
> @@ -204,6 +264,9 @@ static struct omap_board_config_kernel
> overo_config[] __initdata = {
> };
>
> static struct platform_device *overo_devices[] __initdata = {
> +#if defined(CONFIG_SMSC911X) || defined(CONFIG_SMSC911X_MODULE)
> + &overo_smsc911x_device,
> +#endif
> &overo_lcd_device,
> };
>
> diff --git a/arch/arm/plat-omap/include/mach/board-overo.h
> b/arch/arm/plat-omap/include/mach/board-overo.h
> index 7ecae66..8635171 100644
> --- a/arch/arm/plat-omap/include/mach/board-overo.h
> +++ b/arch/arm/plat-omap/include/mach/board-overo.h
> @@ -22,5 +22,8 @@
> #define OVERO_GPIO_USBH_CPEN 168
> #define OVERO_GPIO_USBH_NRESET 183
>
> +#define OVERO_SMSC911X_CS 5
> +#define OVERO_SMSC911X_GPIO 176
> +
> #endif /* ____ASM_ARCH_OVERO_H */
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-01-30 22:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-30 20:28 [RFC] ARM: Add SMSC911X support to Overo platform Steve Sakoman
2009-01-30 20:43 ` Steve Sakoman
2009-01-30 22:18 ` Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).