Subject: [PATCH] davinci: Initial support for MityDSP-L138/MityARM-1808

From: [email-masked] (Kevin Hilman)
Date: Mon, 30 Aug 2010 08:05:59 -0700

Michael Williamson <[email-masked]> writes:

> This patch adds initial support for the MityDSP-L138 and MityDSP-1808 system
> on Module (SOM) under the machine name "mityomapl138".  These SOMs are based
> on the da850 davinci CPU architecture.  Information on these SOMs may be 
> found at http://www.mitydsp.com.
>
> Basic support for the console UART, NAND, and EMAC (MII interface) is 
> included in this patch. 
>
> Signed-off-by: Michael Williamson <[email-masked]>
> ---
> Notes:
>   1) Patch is against 0a50e05b20f3c6af67656303bdb3661a2541ce03 of Kevin's 
>      tree (2.6.36-rc2).
>   2) I did not include a defconfig update in this patch.  Until the regulator
>      support is added back in (planned subsequent patch), it cannot be added
>      to da8xx_omapl_defconfig as the CONFIG_REGULATOR option doesn't play
>      nice with platforms not defining a regulator.  If a defconfig support
>      is required, this platform should go in da8xx_omapl_defconfig.  We'd
>      prefer to just manage/supply a .config file to folks who needed one
>      outside of the mainline via the web.  Is that acceptable?  Users should
>      be able to use the da8xx_omapl_defconfig as a starting point if they
>      want to reference only the mainline software.
>
>  arch/arm/mach-davinci/Kconfig                   |    8 +
>  arch/arm/mach-davinci/Makefile                  |    1 +
>  arch/arm/mach-davinci/board-mityomapl138.c      |  216 +++++++++++++++++++++++
>  arch/arm/mach-davinci/include/mach/uncompress.h |    1 +
>  4 files changed, 226 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/Kconfig b/arch/arm/mach-davinci/Kconfig
> index 2bf03e9..9aca60c 100644
> --- a/arch/arm/mach-davinci/Kconfig
> +++ b/arch/arm/mach-davinci/Kconfig
> @@ -185,6 +185,14 @@ config MACH_TNETV107X
>  	help
>  	  Say Y here to select the TI TNETV107X Evaluation Module.
>  
> +config MACH_MITYOMAPL138
> +	bool "Critical Link MityDSP-L138/MityARM-1808 SoM"
> +	depends on ARCH_DAVINCI_DA850
> +	help
> +	  Say Y here to select the Critical Link MityDSP-L138/MityARM-1808
> +	  System on Module.  Information on this SoM may be found at
> +	  http://www.mitydsp.com
> +
>  config DAVINCI_MUX
>  	bool "DAVINCI multiplexing support"
>  	depends on ARCH_DAVINCI
> diff --git a/arch/arm/mach-davinci/Makefile b/arch/arm/mach-davinci/Makefile
> index eab4c0f..3e966e8 100644
> --- a/arch/arm/mach-davinci/Makefile
> +++ b/arch/arm/mach-davinci/Makefile
> @@ -33,6 +33,7 @@ obj-$(CONFIG_MACH_DAVINCI_DM365_EVM)	+= board-dm365-evm.o
>  obj-$(CONFIG_MACH_DAVINCI_DA830_EVM)	+= board-da830-evm.o
>  obj-$(CONFIG_MACH_DAVINCI_DA850_EVM)	+= board-da850-evm.o
>  obj-$(CONFIG_MACH_TNETV107X)		+= board-tnetv107x-evm.o
> +obj-$(CONFIG_MACH_MITYOMAPL138)		+= board-mityomapl138.o
>  
>  # Power Management
>  obj-$(CONFIG_CPU_FREQ)			+= cpufreq.o
> diff --git a/arch/arm/mach-davinci/board-mityomapl138.c b/arch/arm/mach-davinci/board-mityomapl138.c
> new file mode 100644
> index 0000000..3e141cf
> --- /dev/null
> +++ b/arch/arm/mach-davinci/board-mityomapl138.c
> @@ -0,0 +1,216 @@
> +/*
> + * Critical Link MityOMAP-L138 SoM
> + *
> + * Copyright (C) 2010 Critical Link LLC - http://www.criticallink.com
> + *
> + * This file is licensed under the terms of the GNU General Public License
> + * version 2. This program is licensed "as is" without any warranty of
> + * any kind, whether express or implied.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/console.h>
> +#include <linux/platform_device.h>
> +#include <linux/mtd/partitions.h>
> +
> +#include <asm/mach-types.h>
> +#include <asm/mach/arch.h>
> +#include <mach/common.h>
> +#include <mach/cp_intc.h>
> +#include <mach/da8xx.h>
> +#include <mach/nand.h>
> +#include <mach/mux.h>
> +
> +#define MITYOMAPL138_PHY_MASK		0x08 /* hardcoded for now */
> +#define MITYOMAPL138_MDIO_FREQUENCY	(2200000) /* PHY bus frequency */
> +
> +/* MityDSP-L138 includes a 256 MByte large-page NAND flash
> + * (128K blocks).
> + */

minor nit: please fix multi-line comment style.  search for 'multi-line'
in Documentation/CodingStyle for details.

> +struct mtd_partition mityomapl138_nandflash_partition[] = {
> +	{
> +		.name		= "rootfs",
> +		.offset		= 0,
> +		.size		= SZ_128M,
> +		.mask_flags	= 0, /* MTD_WRITEABLE, */
> +	},
> +	{
> +		.name		= "homefs",
> +		.offset		= MTDPART_OFS_APPEND,
> +		.size		= MTDPART_SIZ_FULL,
> +		.mask_flags	= 0,
> +	},
> +};

It's your board, you can have your partitions however you choose, but
just curious... Is your bootloader somehwhere in NAND too?  Also 128M
seems pretty small for a root filesystem.

> +static struct davinci_nand_pdata mityomapl138_nandflash_data = {
> +	.parts		= mityomapl138_nandflash_partition,
> +	.nr_parts	= ARRAY_SIZE(mityomapl138_nandflash_partition),
> +	.ecc_mode	= NAND_ECC_HW,
> +	.options	= NAND_USE_FLASH_BBT | NAND_BUSWIDTH_16,
> +	.ecc_bits   = 1, /* 4 bit mode is not supported with 16 bit NAND */
> +};
> +
> +static struct resource mityomapl138_nandflash_resource[] = {
> +	{
> +		.start	= DA8XX_AEMIF_CS3_BASE,
> +		.end	= DA8XX_AEMIF_CS3_BASE + SZ_512K + 2 * SZ_1K - 1,
> +		.flags	= IORESOURCE_MEM,
> +	},
> +	{
> +		.start	= DA8XX_AEMIF_CTL_BASE,
> +		.end	= DA8XX_AEMIF_CTL_BASE + SZ_32K - 1,
> +		.flags	= IORESOURCE_MEM,
> +	},
> +};
> +
> +static struct platform_device mityomapl138_nandflash_device = {
> +	.name		= "davinci_nand",
> +	.id		= 0,
> +	.dev		= {
> +		.platform_data	= &mityomapl138_nandflash_data,
> +	},
> +	.num_resources	= ARRAY_SIZE(mityomapl138_nandflash_resource),
> +	.resource	= mityomapl138_nandflash_resource,
> +};
> +
> +static struct platform_device *mityomapl138_devices[] __initdata = {
> +	&mityomapl138_nandflash_device,
> +};
> +
> +static void __init mityomapl138_setup_nand(void)
> +{
> +	platform_add_devices(mityomapl138_devices,
> +				 ARRAY_SIZE(mityomapl138_devices));
> +}
> +
> +static struct davinci_uart_config mityomapl138_uart_config __initdata = {
> +	.enabled_uarts = 0x7,
> +};
> +
> +static const short mityomap_mii_pins[] = {
> +	DA850_MII_TXEN, DA850_MII_TXCLK, DA850_MII_COL, DA850_MII_TXD_3,
> +	DA850_MII_TXD_2, DA850_MII_TXD_1, DA850_MII_TXD_0, DA850_MII_RXER,
> +	DA850_MII_CRS, DA850_MII_RXCLK, DA850_MII_RXDV, DA850_MII_RXD_3,
> +	DA850_MII_RXD_2, DA850_MII_RXD_1, DA850_MII_RXD_0, DA850_MDIO_CLK,
> +	DA850_MDIO_D,
> +	-1
> +};
> +
> +static const short mityomap_rmii_pins[] = {
> +	DA850_RMII_TXD_0, DA850_RMII_TXD_1, DA850_RMII_TXEN,
> +	DA850_RMII_CRS_DV, DA850_RMII_RXD_0, DA850_RMII_RXD_1,
> +	DA850_RMII_RXER, DA850_RMII_MHZ_50_CLK, DA850_MDIO_CLK,
> +	DA850_MDIO_D,
> +	-1
> +};
> +
> +static void __init mityomapl138_config_emac(void)
> +{
> +	void __iomem *cfg_chip3_base;
> +	int ret;
> +	u32 val;
> +	struct davinci_soc_info *soc_info = &davinci_soc_info;
> +	soc_info->emac_pdata->rmii_en = 0; /* hardcoded for now */
> +
> +	cfg_chip3_base = DA8XX_SYSCFG0_VIRT(DA8XX_CFGCHIP3_REG);
> +	val = __raw_readl(cfg_chip3_base);
> +
> +	if (soc_info->emac_pdata->rmii_en) {
> +		val |= BIT(8);
> +		ret = davinci_cfg_reg_list(mityomap_rmii_pins);
> +		pr_info("RMII PHY configured\n");
> +	} else {
> +		val &= ~BIT(8);
> +		ret = davinci_cfg_reg_list(mityomap_mii_pins);
> +		pr_info("MII PHY configured\n");
> +	}
> +
> +	if (ret) {
> +		pr_warning("mii/rmii mux setup failed: %d\n", ret);
> +		return;
> +	}
> +
> +	/* configure the CFGCHIP3 register for RMII or MII */
> +	__raw_writel(val, cfg_chip3_base);
> +
> +	soc_info->emac_pdata->phy_mask = MITYOMAPL138_PHY_MASK;
> +	pr_info("setting phy_mask to %x\n", soc_info->emac_pdata->phy_mask);
> +	soc_info->emac_pdata->mdio_max_freq = MITYOMAPL138_MDIO_FREQUENCY;
> +
> +	ret = da8xx_register_emac();
> +	if (ret)
> +		pr_warning("emac registration failed: %d\n", ret);
> +}
> +
> +static struct davinci_pm_config da850_pm_pdata = {
> +	.sleepcount = 128,
> +};
> +
> +static struct platform_device da850_pm_device = {
> +	.name	= "pm-davinci",
> +	.dev = {
> +		.platform_data  = &da850_pm_pdata,
> +	},
> +	.id	= -1,
> +};
> +
> +static void __init mityomapl138_init(void)
> +{
> +	int ret;
> +
> +	/* for now, no special EDMA channels are reserved */
> +	ret = da850_register_edma(NULL);
> +	if (ret)
> +		pr_warning("edma registration failed: %d\n", ret);
> +
> +	ret = da8xx_register_watchdog();
> +	if (ret)
> +		pr_warning("watchdog registration failed: %d\n", ret);
> +
> +	davinci_serial_init(&mityomapl138_uart_config);
> +
> +	mityomapl138_setup_nand();
> +
> +	mityomapl138_config_emac();
> +
> +	ret = da8xx_register_rtc();
> +	if (ret)
> +		pr_warning("rtc setup failed: %d\n", ret);
> +
> +	ret = da850_register_cpufreq("pll0_sysclk3");
> +	if (ret)
> +		pr_warning("cpufreq registration failed: %d\n", ret);
> +
> +	ret = da8xx_register_cpuidle();
> +	if (ret)
> +		pr_warning("cpuidle registration failed: %d\n", ret);
> +
> +	ret = da850_register_pm(&da850_pm_device);
> +	if (ret)
> +		pr_warning("da850_evm_init: suspend registration failed: %d\n",
> +				ret);
> +}
> +
> +#ifdef CONFIG_SERIAL_8250_CONSOLE
> +static int __init mityomapl138_console_init(void)
> +{

You need a machine_is_* check here since this will be run whenever this
is compiled in.  E.g., if a kernel is built to support all da8xx platforms,

In fact, the da850-evm board file should have this too since when both
this board and the EVM are compiled in, they will both be setting a
different preferred console.

> +	return add_preferred_console("ttyS", 1, "115200");
> +}
> +console_initcall(mityomapl138_console_init);
> +#endif
> +
> +static void __init mityomapl138_map_io(void)
> +{
> +	da850_init();
> +}
> +
> +MACHINE_START(MITYOMAPL138, "MityDSP-L138/MityARM-1808")
> +	.phys_io	= IO_PHYS,
> +	.io_pg_offst	= (__IO_ADDRESS(IO_PHYS) >> 18) & 0xfffc,
> +	.boot_params	= (DA8XX_DDR_BASE + 0x100),
> +	.map_io		= mityomapl138_map_io,
> +	.init_irq	= cp_intc_init,
> +	.timer		= &davinci_timer,
> +	.init_machine	= mityomapl138_init,
> +MACHINE_END
> diff --git a/arch/arm/mach-davinci/include/mach/uncompress.h b/arch/arm/mach-davinci/include/mach/uncompress.h
> index 15a6192..d370391 100644
> --- a/arch/arm/mach-davinci/include/mach/uncompress.h
> +++ b/arch/arm/mach-davinci/include/mach/uncompress.h
> @@ -88,6 +88,7 @@ static inline void __arch_decomp_setup(unsigned long arch_id)
>  		/* DA8xx boards */
>  		DEBUG_LL_DA8XX(davinci_da830_evm,	2);
>  		DEBUG_LL_DA8XX(davinci_da850_evm,	2);
> +		DEBUG_LL_DA8XX(mityomapl138,		1);
>  
>  		/* TNETV107x boards */
>  		DEBUG_LL_TNETV107X(tnetv107x,		1);

Kevin