Mikkel Christensen <mlc@xxxxxx> writes:
> This patch creates the minimal OMAP3 Zoom2 board support.
>
> Signed-off-by: Mikkel Christensen <mlc@xxxxxx>
> Signed-off-by: Vikram Pandita <vikram.pandita@xxxxxx>
> ---
> arch/arm/mach-omap2/board-zoom2-debugboard.c | 150 ++++++++++++++++++++++++++
> arch/arm/mach-omap2/board-zoom2.c | 126 +++++++++++++++++++++
> 2 files changed, 276 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/mach-omap2/board-zoom2-debugboard.c
> create mode 100644 arch/arm/mach-omap2/board-zoom2.c
Any reason you decided on a separate board file for the debug board?
While not a big deal, this not a separate board and is an extention of
the zoom2.
I would rather see everything in board-zoom2.c along with a separate
init and platform_device setup/register for the debug board IFF it is
present.
I understand that u-boot has a way of detecting if the debug board is
present. The board init code should also detect if the board is
present and init/regsiter devices present only if present.
> diff --git a/arch/arm/mach-omap2/board-zoom2-debugboard.c b/arch/arm/mach-omap2/board-zoom2-debugboard.c
> new file mode 100644
> index 0000000..ae90f2b
> --- /dev/null
> +++ b/arch/arm/mach-omap2/board-zoom2-debugboard.c
> @@ -0,0 +1,150 @@
> +/*
> + * arch/arm/mach-omap2/board-zoom2-debugboard.c
> + *
> + * OMAP3 Zoom2 debug board support.
> + *
> + * Copyright (C) 2009 Texas Instruments Inc.
> + * Mikkel Christensen <mlc@xxxxxx>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/delay.h>
> +#include <linux/serial_8250.h>
> +#include <linux/serial_reg.h>
> +#include <linux/smsc911x.h>
> +#include <linux/io.h>
> +
> +#include <mach/common.h>
> +#include <mach/board.h>
> +#include <mach/gpio.h>
> +#include <mach/gpmc.h>
> +
> +#define ZOOM2_SMSC911X_CS 7
> +#define ZOOM2_SMSC911X_GPIO 158
> +#define ZOOM2_QUADUART_CS 3
> +#define ZOOM2_QUADUART_GPIO 102
> +#define QUART_CLK 1843200
> +#define DEBUG_BASE 0x08000000
> +#define ZOOM2_ETHR_START DEBUG_BASE
> +
> +static struct resource zoom2_smsc911x_resources[] = {
> + [0] = {
> + .start = ZOOM2_ETHR_START,
> + .end = ZOOM2_ETHR_START + SZ_4K,
> + .flags = IORESOURCE_MEM,
> + },
> + [1] = {
> + .start = 0,
> + .end = 0,
Just leave these out, you init the IRQ later at runtime. And FYI,
you don't need 'end' for an IRQ.
> + .flags = IORESOURCE_IRQ | IORESOURCE_IRQ_LOWLEVEL,
> + },
> +};
> +
> +static struct smsc911x_platform_config zoom2_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 zoom2_smsc911x_device = {
> + .name = "smsc911x",
> + .id = -1,
> + .num_resources = ARRAY_SIZE(zoom2_smsc911x_resources),
> + .resource = zoom2_smsc911x_resources,
> + .dev = {
> + .platform_data = &zoom2_smsc911x_config,
> + },
> +};
> +
> +static inline void __init zoom2_init_smsc911x(void)
> +{
> + int eth_cs;
> + unsigned long cs_mem_base;
> + int eth_gpio = 0;
> +
> + eth_cs = ZOOM2_SMSC911X_CS;
> +
> + if (gpmc_cs_request(eth_cs, SZ_16M, &cs_mem_base) < 0) {
> + printk(KERN_ERR "Failed to request GPMC mem for smsc911x\n");
> + return;
> + }
> +
> + zoom2_smsc911x_resources[0].start = cs_mem_base + 0x0;
> + zoom2_smsc911x_resources[0].end = cs_mem_base + 0xff;
> + udelay(100);
What's this delay for? I think it should be dropped.
> + eth_gpio = ZOOM2_SMSC911X_GPIO;
> +
> + zoom2_smsc911x_resources[1].start = OMAP_GPIO_IRQ(eth_gpio);
> +
> + if (gpio_request(eth_gpio, "smsc911x irq") < 0) {
> + printk(KERN_ERR "Failed to request GPIO%d for smsc911x IRQ\n",
> + eth_gpio);
> + return;
> + }
> + gpio_direction_input(eth_gpio);
> +}
> +
> +static struct plat_serial8250_port serial_platform_data[] = {
> + {
> + .membase = 0,
You don't need to set fields to zero. They will init to zero.
> + .mapbase = 0x10000000,
> + .irq = OMAP_GPIO_IRQ(102),
> + .flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
> + .iotype = UPIO_MEM,
> + .regshift = 1,
> + .uartclk = QUART_CLK,
> + }, {
> + .flags = 0
> + }
> +};
> +
> +static struct platform_device zoom2_debugboard_serial_device = {
> + .name = "serial8250",
> + .id = PLAT8250_DEV_PLATFORM1,
> + .dev = {
> + .platform_data = serial_platform_data,
> + },
> +};
> +
> +static inline void __init zoom2_init_quaduart(void)
> +{
> + int quart_cs;
> + unsigned long cs_mem_base;
> + int quart_gpio = 0;
> +
> + quart_cs = ZOOM2_QUADUART_CS;
> +
> + if (gpmc_cs_request(quart_cs, SZ_1M, &cs_mem_base) < 0) {
> + printk(KERN_ERR "Failed to request GPMC mem"
> + "for Quad UART(TL16CP754C)\n");
> + return;
> + }
> +
> + quart_gpio = ZOOM2_QUADUART_GPIO;
> +
> + if (gpio_request(quart_gpio, "TL16CP754C GPIO") < 0) {
> + printk(KERN_ERR "Failed to request GPIO%d for TL16CP754C\n",
> + quart_gpio);
> + return;
> + }
> + gpio_direction_input(quart_gpio);
> +}
> +
> +static struct platform_device *zoom2_devices[] __initdata = {
> + &zoom2_smsc911x_device,
> + &zoom2_debugboard_serial_device,
> +};
> +
> +static int __init omap_zoom2_debugboard_init(void)
> +{
This debug board init should include some form of probing for the
debug board, and only continue if it is present.
> + zoom2_init_smsc911x();
> + zoom2_init_quaduart();
> + return platform_add_devices(zoom2_devices, ARRAY_SIZE(zoom2_devices));
> +}
> +arch_initcall(omap_zoom2_debugboard_init);
> diff --git a/arch/arm/mach-omap2/board-zoom2.c b/arch/arm/mach-omap2/board-zoom2.c
> new file mode 100644
> index 0000000..f43809a
> --- /dev/null
> +++ b/arch/arm/mach-omap2/board-zoom2.c
> @@ -0,0 +1,126 @@
> +/*
> + * linux/arch/arm/mach-omap2/board-zoom2.c
> + *
> + * Copyright (C) 2009 Texas Instruments Inc.
> + * Mikkel Christensen <mlc@xxxxxx>
> + *
> + * Modified from mach-omap2/board-ldp.c
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/platform_device.h>
> +#include <linux/delay.h>
> +#include <linux/input.h>
> +#include <linux/workqueue.h>
> +#include <linux/err.h>
> +#include <linux/clk.h>
> +#include <linux/spi/spi.h>
> +#include <linux/spi/ads7846.h>
> +#include <linux/i2c/twl4030.h>
> +#include <linux/serial_8250.h>
> +#include <linux/io.h>
>
> +#include <mach/hardware.h>
> +#include <asm/mach-types.h>
> +#include <asm/mach/arch.h>
> +#include <asm/mach/map.h>
> +
> +#include <mach/mcspi.h>
> +#include <mach/gpio.h>
> +#include <mach/board.h>
> +#include <mach/common.h>
> +#include <mach/gpmc.h>
> +
> +#include <asm/delay.h>
> +#include <mach/control.h>
> +#include <mach/usb.h>
> +
> +#include "mmc-twl4030.h"
I think there are many headers above that are not needed. Please keep
only ones that are used.
> +
> +static void __init omap_zoom2_init_irq(void)
> +{
> + omap2_init_common_hw(NULL);
> + omap_init_irq();
> + omap_gpio_init();
> +}
> +
> +static struct omap_uart_config zoom2_uart_config __initdata = {
> + .enabled_uarts = ((1 << 0) | (1 << 1) | (1 << 2)),
> +};
> +
> +static struct omap_board_config_kernel zoom2_config[] __initdata = {
> + { OMAP_TAG_UART, &zoom2_uart_config },
> +};
> +
> +static struct twl4030_gpio_platform_data zoom2_gpio_data = {
> + .gpio_base = OMAP_MAX_GPIO_LINES,
> + .irq_base = TWL4030_GPIO_IRQ_BASE,
> + .irq_end = TWL4030_GPIO_IRQ_END,
> +};
> +
> +static struct twl4030_platform_data zoom2_twldata = {
> + .irq_base = TWL4030_IRQ_BASE,
> + .irq_end = TWL4030_IRQ_END,
> +
> + /* platform_data for children goes here */
> + .gpio = &zoom2_gpio_data,
> +};
> +
> +static struct i2c_board_info __initdata zoom2_i2c_boardinfo[] = {
> + {
> + I2C_BOARD_INFO("twl4030", 0x48),
> + .flags = I2C_CLIENT_WAKE,
> + .irq = INT_34XX_SYS_NIRQ,
> + .platform_data = &zoom2_twldata,
> + },
> +};
> +
> +static int __init omap_i2c_init(void)
> +{
> + omap_register_i2c_bus(1, 2600, zoom2_i2c_boardinfo,
> + ARRAY_SIZE(zoom2_i2c_boardinfo));
> + omap_register_i2c_bus(2, 400, NULL, 0);
> + omap_register_i2c_bus(3, 400, NULL, 0);
> + return 0;
> +}
> +
> +static struct twl4030_hsmmc_info mmc[] __initdata = {
> + {
> + .mmc = 1,
> + .wires = 4,
> + .gpio_cd = -EINVAL,
> + .gpio_wp = -EINVAL,
> + },
> + {} /* Terminator */
> +};
> +
> +static void __init omap_zoom2_init(void)
> +{
> + omap_i2c_init();
> + omap_board_config = zoom2_config;
> + omap_board_config_size = ARRAY_SIZE(zoom2_config);
> + omap_serial_init();
> + twl4030_mmc_init(mmc);
> + usb_musb_init();
> +}
> +
> +static void __init omap_zoom2_map_io(void)
> +{
> + omap2_set_globals_343x();
> + omap2_map_common_io();
> +}
> +
> +MACHINE_START(OMAP_ZOOM2, "OMAP Zoom2 board")
> + .phys_io = 0x48000000,
> + .io_pg_offst = ((0xd8000000) >> 18) & 0xfffc,
> + .boot_params = 0x80000100,
> + .map_io = omap_zoom2_map_io,
> + .init_irq = omap_zoom2_init_irq,
> + .init_machine = omap_zoom2_init,
> + .timer = &omap_timer,
> +MACHINE_END
> --
> 1.5.4.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
