* [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1272659448-23302-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> @ 2010-04-30 20:30 ` Anatolij Gustschin [not found] ` <1272659448-23302-2-git-send-email-agust-ynQEQJNshbs@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Anatolij Gustschin @ 2010-04-30 20:30 UTC (permalink / raw) To: linuxppc-dev-mnsaURCQ41sdnm+yROfE0A Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, Wolfgang Grandegger Adds IFM PDM360NG device tree, and platform code. Currently following is supported: - Spansion S29GL512P 256 MB NOR flash - ST Micro NAND 1 GiB flash - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" at the kernel command line to enable PrimeView PM070WL3 Display support. - FEC - I2C - RTC, EEPROM - MSCAN - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" on the kernel command line. - USB0/1 Host - USB0 OTG Host/Device - VIU, Overlay support Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> --- This patch is intended to be used on top of MPC5121 DIU support patch series: http://thread.gmane.org/gmane.linux.ports.ppc.embedded/34806 I post it for review now, too. arch/powerpc/boot/dts/pdm360ng.dts | 430 ++++++++++++++++++++++++++++++++ arch/powerpc/platforms/512x/Kconfig | 7 + arch/powerpc/platforms/512x/Makefile | 1 + arch/powerpc/platforms/512x/pdm360ng.c | 158 ++++++++++++ 4 files changed, 596 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts new file mode 100644 index 0000000..23bce6e --- /dev/null +++ b/arch/powerpc/boot/dts/pdm360ng.dts @@ -0,0 +1,430 @@ +/* + * Device Tree Source for IFM PDM360NG. + * + * Copyright 2009 - 2010 DENX Software Engineering. + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> + * + * Based on MPC5121E ADS dts. + * Copyright 2008 Freescale Semiconductor Inc. + * + * 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. + */ + +/dts-v1/; + +/ { + model = "pdm360ng"; + compatible = "ifm,pdm360ng"; + #address-cells = <1>; + #size-cells = <1>; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + PowerPC,5121@0 { + device_type = "cpu"; + reg = <0>; + d-cache-line-size = <0x20>; // 32 bytes + i-cache-line-size = <0x20>; // 32 bytes + d-cache-size = <0x8000>; // L1, 32K + i-cache-size = <0x8000>; // L1, 32K + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) + bus-frequency = <198000000>; // 198 MHz csb bus + clock-frequency = <396000000>; // 396 MHz ppc core + }; + }; + + memory { + device_type = "memory"; + reg = <0x00000000 0x20000000>; // 512MB at 0 + }; + + nfc@40000000 { + compatible = "fsl,mpc5121-nfc"; + reg = <0x40000000 0x100000>; + interrupts = <0x6 0x8>; + interrupt-parent = <0x1>; + #address-cells = <0x1>; + #size-cells = <0x1>; + bank-width = <0x1>; + chips = <0x1>; + + partition@0 { + label = "nand0"; + reg = <0x0 0x40000000>; + }; + }; + + sram@50000000 { + compatible = "fsl,mpc5121-sram"; + reg = <0x50000000 0x20000>; // 128K at 0x50000000 + }; + + localbus@80000020 { + compatible = "fsl,mpc5121-localbus"; + #address-cells = <2>; + #size-cells = <1>; + reg = <0x80000020 0x40>; + + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ + + flash@0,0 { + compatible = "amd,s29gl01gp", "cfi-flash"; + reg = <0 0x00000000 0x08000000 + 0 0x08000000 0x08000000>; + #address-cells = <1>; + #size-cells = <1>; + bank-width = <4>; + device-width = <2>; + + partition@0 { + label = "u-boot"; + reg = <0x00000000 0x00080000>; + read-only; + }; + partition@80000 { + label = "environment"; + reg = <0x00080000 0x00080000>; + read-only; + }; + partition@100000 { + label = "splash-image"; + reg = <0x00100000 0x00080000>; + read-only; + }; + partition@180000 { + label = "device-tree"; + reg = <0x00180000 0x00040000>; + }; + partition@1c0000 { + label = "kernel"; + reg = <0x001c0000 0x00500000>; + }; + partition@6c0000 { + label = "filesystem"; + reg = <0x006c0000 0x07940000>; + }; + }; + + mram0@2,0 { + compatible = "mtd-ram"; + reg = <2 0x00000 0x10000>; + bank-width = <2>; + }; + + mram1@2,10000 { + compatible = "mtd-ram"; + reg = <2 0x010000 0x10000>; + bank-width = <2>; + }; + }; + + soc@80000000 { + compatible = "fsl,mpc5121-immr"; + #address-cells = <1>; + #size-cells = <1>; + #interrupt-cells = <2>; + ranges = <0x0 0x80000000 0x400000>; + reg = <0x80000000 0x400000>; + bus-frequency = <66000000>; // 66 MHz ips bus + + // IPIC + // interrupts cell = <intr #, sense> + // sense values match linux IORESOURCE_IRQ_* defines: + // sense == 8: Level, low assertion + // sense == 2: Edge, high-to-low change + // + ipic: interrupt-controller@c00 { + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; + interrupt-controller; + #address-cells = <0>; + #interrupt-cells = <2>; + reg = <0xc00 0x100>; + }; + + rtc@a00 { // Real time clock + compatible = "fsl,mpc5121-rtc"; + reg = <0xa00 0x100>; + interrupts = <79 0x8 80 0x8>; + interrupt-parent = < &ipic >; + }; + + reset@e00 { // Reset module + compatible = "fsl,mpc5121-reset"; + reg = <0xe00 0x100>; + }; + + clock@f00 { // Clock control + compatible = "fsl,mpc5121-clock"; + reg = <0xf00 0x100>; + }; + + pmc@1000{ //Power Management Controller + compatible = "fsl,mpc5121-pmc"; + reg = <0x1000 0x100>; + interrupts = <83 0x2>; + interrupt-parent = < &ipic >; + }; + + gpio@1100 { + compatible = "fsl,mpc5121-gpio"; + reg = <0x1100 0x100>; + interrupts = <78 0x8>; + interrupt-parent = < &ipic >; + }; + + can@1300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <12 0x8>; + interrupt-parent = < &ipic >; + reg = <0x1300 0x80>; + }; + + can@1380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <13 0x8>; + interrupt-parent = < &ipic >; + reg = <0x1380 0x80>; + }; + + i2c@1700 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1700 0x20>; + interrupts = <0x9 0x8>; + interrupt-parent = < &ipic >; + fsl,preserve-clocking; + + eeprom@50 { + compatible = "at,24c01"; + reg = <0x50>; + }; + + rtc@68 { + compatible = "stm,m41t00"; + reg = <0x68>; + }; + }; + + i2c@1740 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1740 0x20>; + interrupts = <0xb 0x8>; + interrupt-parent = < &ipic >; + fsl,preserve-clocking; + }; + + i2ccontrol@1760 { + compatible = "fsl,mpc5121-i2c-ctrl"; + reg = <0x1760 0x8>; + }; + + axe@2000 { + compatible = "fsl,mpc5121-axe"; + reg = <0x2000 0x100>; + interrupts = <42 0x8>; + interrupt-parent = < &ipic >; + }; + + display@2100 { + compatible = "fsl,mpc5121-diu"; + reg = <0x2100 0x100>; + interrupts = <64 0x8>; + interrupt-parent = < &ipic >; + }; + + can@2300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <90 0x8>; + interrupt-parent = < &ipic >; + reg = <0x2300 0x80>; + }; + + can@2380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <91 0x8>; + interrupt-parent = < &ipic >; + reg = <0x2380 0x80>; + }; + + viu@2400 { + compatible = "fsl,mpc5121-viu"; + reg = <0x2400 0x400>; + interrupts = <67 0x8>; + interrupt-parent = < &ipic >; + }; + + mdio@2800 { + compatible = "fsl,mpc5121-fec-mdio"; + reg = <0x2800 0x200>; + #address-cells = <1>; + #size-cells = <0>; + phy: ethernet-phy@0 { + reg = <0x1f>; + device_type = "ethernet-phy"; + }; + }; + + ethernet@2800 { + device_type = "network"; + compatible = "fsl,mpc5121-fec"; + reg = <0x2800 0x200>; + local-mac-address = [ 00 00 00 00 00 00 ]; + interrupts = <4 0x8>; + interrupt-parent = < &ipic >; + phy-handle = < &phy >; + }; + + // USB1 using external ULPI PHY + usb@3000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x3000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = < &ipic >; + interrupts = <43 0x8>; + dr_mode = "host"; + phy_type = "ulpi"; + }; + + // USB0 using internal UTMI PHY + usb@4000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x4000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = < &ipic >; + interrupts = <44 0x8>; + dr_mode = "otg"; + phy_type = "utmi_wide"; + fsl,invert-pwr-fault; + }; + + // IO control + ioctl@a000 { + compatible = "fsl,mpc5121-ioctl"; + reg = <0xA000 0x1000>; + }; + + // 512x PSCs are not 52xx PSCs compatible + serial@11000 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <0>; + reg = <0x11000 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11100 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <1>; + reg = <0x11100 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11200 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <2>; + reg = <0x11200 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11300 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <3>; + reg = <0x11300 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11400 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <4>; + reg = <0x11400 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11600 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <6>; + reg = <0x11600 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11800 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <8>; + reg = <0x11800 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11B00 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <11>; + reg = <0x11B00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + pscfifo@11f00 { + compatible = "fsl,mpc5121-psc-fifo"; + reg = <0x11f00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + }; + + spi@11900 { + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; + cell-index = <9>; + reg = <0x11900 0x100>; + interrupts = <40 0x8>; + interrupt-parent = < &ipic >; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + + // 7845 touch screen controller + ts@0 { + compatible = "ti,ads7845"; + interrupt-parent = < &ipic >; + // pen irq is GPIO25 + interrupts = <78 0x8>; + }; + }; + + dma@14000 { + compatible = "fsl,mpc5121-dma"; + reg = <0x14000 0x1800>; + interrupts = <65 0x8>; + interrupt-parent = < &ipic >; + }; + }; +}; diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig index e9dca28..27b0651 100644 --- a/arch/powerpc/platforms/512x/Kconfig +++ b/arch/powerpc/platforms/512x/Kconfig @@ -25,3 +25,10 @@ config MPC5121_GENERIC Compatible boards include: Protonic LVT base boards (ZANMCU and VICVT2). + +config PDM360NG + bool "ifm PDM360NG board" + depends on PPC_MPC512x + select DEFAULT_UIMAGE + help + This option enables support for the PDM360NG board. diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile index 90be2f5..4efc1c4 100644 --- a/arch/powerpc/platforms/512x/Makefile +++ b/arch/powerpc/platforms/512x/Makefile @@ -4,3 +4,4 @@ obj-y += clock.o mpc512x_shared.o obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o +obj-$(CONFIG_PDM360NG) += pdm360ng.o diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c new file mode 100644 index 0000000..dc8f860 --- /dev/null +++ b/arch/powerpc/platforms/512x/pdm360ng.c @@ -0,0 +1,158 @@ +/* + * Copyright (C) 2010 DENX Software Engineering + * + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> + * + * PDM360NG board setup + * + * This 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. + * + */ + +#include <linux/kernel.h> +#include <linux/io.h> +#include <linux/of_platform.h> + +#include <asm/machdep.h> +#include <asm/ipic.h> +#include <asm/prom.h> + +#include "mpc512x.h" + +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) +#include <linux/fsl_devices.h> +#include <linux/interrupt.h> +#include <linux/spi/ads7846.h> +#include <linux/spi/spi.h> + +static void *pdm360ng_gpio_base; + +static int pdm360ng_get_pendown_state(void) +{ + u32 reg; + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); + if (reg & 0x40) + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); + + /* return 1 if pen is down */ + return reg & 0x40 ? 0 : 1; +} + +static struct ads7846_platform_data pdm360ng_ads7846_pdata __initdata = { + .model = 7845, + .get_pendown_state = pdm360ng_get_pendown_state, + /*.irq_trigger = IRQF_TRIGGER_LOW,*/ +}; + +static int __init pdm360ng_penirq_init(void) +{ + struct device_node *np; + struct resource r; + + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); + if (!np) { + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); + return -1; + } + + if (of_address_to_resource(np, 0, &r)) { + pr_err("%s: Invalid gpio address.\n", __func__); + of_node_put(np); + return -1; + } + of_node_put(np); + + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); + if (!pdm360ng_gpio_base) { + pr_err("%s: Can't map gpio regs.\n", __func__); + return -1; + } + out_be32((u32 *)pdm360ng_gpio_base + 0xc, 0xffffffff); + setbits32((u32 *)(pdm360ng_gpio_base + 0x18), 0x2000); + setbits32((u32 *)(pdm360ng_gpio_base + 0x10), 0x40); + + return 0; +} + +static int __init pdm360ng_touchscreen_init(void) +{ + struct device_node *np; + struct of_device *of_dev; + struct spi_board_info info; + const u32 *prop; + int bus_num = -1; + int len; + + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); + if (!np) + return -ENODEV; + + memset(&info, 0, sizeof(info)); + info.irq = irq_of_parse_and_map(np, 0); + if (info.irq == NO_IRQ) + return -ENODEV; + + info.platform_data = &pdm360ng_ads7846_pdata; + if (strlcpy(info.modalias, "ads7846", + SPI_NAME_SIZE) >= SPI_NAME_SIZE) + return -ENOMEM; + + np = of_get_next_parent(np); + if (!np) + return -ENODEV; + + prop = of_get_property(np, "cell-index", &len); + if (prop && len == 4) + bus_num = *prop; + + if (bus_num < 0 || bus_num > 11) + return -ENODEV; + + info.bus_num = bus_num; + + of_dev = of_find_device_by_node(np); + of_node_put(np); + if (of_dev) { + struct fsl_spi_platform_data *pdata; + + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); + if (pdata) { + pdata->bus_num = bus_num; + pdata->max_chipselect = 1; + of_dev->dev.platform_data = pdata; + } + } + + if (pdm360ng_penirq_init()) + return -ENODEV; + + return spi_register_board_info(&info, 1); +} +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); +#endif + +static int __init pdm360ng_probe(void) +{ + unsigned long root = of_get_flat_dt_root(); + + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); +} + +define_machine(pdm360ng) { + .name = "PDM360NG", + .probe = pdm360ng_probe, + .setup_arch = mpc512x_setup_diu, + .init = mpc512x_init, + .init_early = mpc512x_init_diu, + .init_IRQ = mpc512x_init_IRQ, + .get_irq = ipic_get_irq, + .calibrate_decr = generic_calibrate_decr, + .restart = mpc512x_restart, +}; -- 1.6.3.3 ^ permalink raw reply related [flat|nested] 16+ messages in thread
[parent not found: <1272659448-23302-2-git-send-email-agust-ynQEQJNshbs@public.gmane.org>]
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1272659448-23302-2-git-send-email-agust-ynQEQJNshbs@public.gmane.org> @ 2010-05-02 14:54 ` Grant Likely 2010-05-03 9:22 ` Anatolij Gustschin [not found] ` <g2tfa686aa41005020754t46934f13hbd5c75a6b7668a54-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2010-05-03 10:23 ` [PATCH v2 " Anatolij Gustschin 1 sibling, 2 replies; 16+ messages in thread From: Grant Likely @ 2010-05-02 14:54 UTC (permalink / raw) To: Anatolij Gustschin Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger Hi Anatolij, Comments below. On Fri, Apr 30, 2010 at 2:30 PM, Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> wrote: > Adds IFM PDM360NG device tree, and platform code. > > Currently following is supported: > - Spansion S29GL512P 256 MB NOR flash > - ST Micro NAND 1 GiB flash > - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" > at the kernel command line to enable PrimeView PM070WL3 > Display support. > - FEC > - I2C > - RTC, EEPROM > - MSCAN > - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" > on the kernel command line. > - USB0/1 Host > - USB0 OTG Host/Device > - VIU, Overlay support > > Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> > Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> > Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> > Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> > Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> > Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org > Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> > --- > This patch is intended to be used on top of MPC5121 DIU > support patch series: > http://thread.gmane.org/gmane.linux.ports.ppc.embedded/34806 > I post it for review now, too. > > arch/powerpc/boot/dts/pdm360ng.dts | 430 ++++++++++++++++++++++++++++++++ > arch/powerpc/platforms/512x/Kconfig | 7 + > arch/powerpc/platforms/512x/Makefile | 1 + > arch/powerpc/platforms/512x/pdm360ng.c | 158 ++++++++++++ > 4 files changed, 596 insertions(+), 0 deletions(-) > create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts > create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c > > diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts > new file mode 100644 > index 0000000..23bce6e > --- /dev/null > +++ b/arch/powerpc/boot/dts/pdm360ng.dts > @@ -0,0 +1,430 @@ > +/* > + * Device Tree Source for IFM PDM360NG. > + * > + * Copyright 2009 - 2010 DENX Software Engineering. > + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> > + * > + * Based on MPC5121E ADS dts. > + * Copyright 2008 Freescale Semiconductor Inc. > + * > + * 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. > + */ > + > +/dts-v1/; > + > +/ { > + model = "pdm360ng"; > + compatible = "ifm,pdm360ng"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + > + PowerPC,5121@0 { > + device_type = "cpu"; > + reg = <0>; > + d-cache-line-size = <0x20>; // 32 bytes > + i-cache-line-size = <0x20>; // 32 bytes > + d-cache-size = <0x8000>; // L1, 32K > + i-cache-size = <0x8000>; // L1, 32K > + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) > + bus-frequency = <198000000>; // 198 MHz csb bus > + clock-frequency = <396000000>; // 396 MHz ppc core > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x00000000 0x20000000>; // 512MB at 0 > + }; > + > + nfc@40000000 { > + compatible = "fsl,mpc5121-nfc"; > + reg = <0x40000000 0x100000>; > + interrupts = <0x6 0x8>; > + interrupt-parent = <0x1>; This looks wrong. Shouldn't this be interrupt-parent = <&ipic>? In fact, if the root node has interrupt-parent = <&ipic> then the interrupt-parent property can be omitted from all other nodes unless they need to override it. > + #address-cells = <0x1>; > + #size-cells = <0x1>; > + bank-width = <0x1>; > + chips = <0x1>; > + > + partition@0 { > + label = "nand0"; > + reg = <0x0 0x40000000>; > + }; > + }; > + > + sram@50000000 { > + compatible = "fsl,mpc5121-sram"; > + reg = <0x50000000 0x20000>; // 128K at 0x50000000 > + }; > + > + localbus@80000020 { > + compatible = "fsl,mpc5121-localbus"; > + #address-cells = <2>; > + #size-cells = <1>; > + reg = <0x80000020 0x40>; > + > + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ > + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ > + > + flash@0,0 { > + compatible = "amd,s29gl01gp", "cfi-flash"; > + reg = <0 0x00000000 0x08000000 > + 0 0x08000000 0x08000000>; > + #address-cells = <1>; > + #size-cells = <1>; > + bank-width = <4>; > + device-width = <2>; > + > + partition@0 { > + label = "u-boot"; > + reg = <0x00000000 0x00080000>; > + read-only; > + }; > + partition@80000 { > + label = "environment"; > + reg = <0x00080000 0x00080000>; > + read-only; > + }; > + partition@100000 { > + label = "splash-image"; > + reg = <0x00100000 0x00080000>; > + read-only; > + }; > + partition@180000 { > + label = "device-tree"; > + reg = <0x00180000 0x00040000>; > + }; > + partition@1c0000 { > + label = "kernel"; > + reg = <0x001c0000 0x00500000>; > + }; > + partition@6c0000 { > + label = "filesystem"; > + reg = <0x006c0000 0x07940000>; > + }; > + }; > + > + mram0@2,0 { > + compatible = "mtd-ram"; > + reg = <2 0x00000 0x10000>; > + bank-width = <2>; > + }; > + > + mram1@2,10000 { > + compatible = "mtd-ram"; > + reg = <2 0x010000 0x10000>; > + bank-width = <2>; > + }; > + }; > + > + soc@80000000 { > + compatible = "fsl,mpc5121-immr"; > + #address-cells = <1>; > + #size-cells = <1>; > + #interrupt-cells = <2>; > + ranges = <0x0 0x80000000 0x400000>; > + reg = <0x80000000 0x400000>; > + bus-frequency = <66000000>; // 66 MHz ips bus > + > + // IPIC > + // interrupts cell = <intr #, sense> > + // sense values match linux IORESOURCE_IRQ_* defines: > + // sense == 8: Level, low assertion > + // sense == 2: Edge, high-to-low change > + // > + ipic: interrupt-controller@c00 { > + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; > + interrupt-controller; > + #address-cells = <0>; Don't need #address-cells here > + #interrupt-cells = <2>; > + reg = <0xc00 0x100>; > + }; > + > + rtc@a00 { // Real time clock > + compatible = "fsl,mpc5121-rtc"; > + reg = <0xa00 0x100>; > + interrupts = <79 0x8 80 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + reset@e00 { // Reset module > + compatible = "fsl,mpc5121-reset"; > + reg = <0xe00 0x100>; > + }; > + > + clock@f00 { // Clock control > + compatible = "fsl,mpc5121-clock"; > + reg = <0xf00 0x100>; > + }; > + > + pmc@1000{ //Power Management Controller > + compatible = "fsl,mpc5121-pmc"; > + reg = <0x1000 0x100>; > + interrupts = <83 0x2>; > + interrupt-parent = < &ipic >; > + }; > + > + gpio@1100 { > + compatible = "fsl,mpc5121-gpio"; > + reg = <0x1100 0x100>; > + interrupts = <78 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + can@1300 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <12 0x8>; > + interrupt-parent = < &ipic >; > + reg = <0x1300 0x80>; > + }; > + > + can@1380 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <13 0x8>; > + interrupt-parent = < &ipic >; > + reg = <0x1380 0x80>; > + }; > + > + i2c@1700 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "fsl,mpc5121-i2c"; > + reg = <0x1700 0x20>; > + interrupts = <0x9 0x8>; > + interrupt-parent = < &ipic >; > + fsl,preserve-clocking; > + > + eeprom@50 { > + compatible = "at,24c01"; > + reg = <0x50>; > + }; > + > + rtc@68 { > + compatible = "stm,m41t00"; > + reg = <0x68>; > + }; > + }; > + > + i2c@1740 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "fsl,mpc5121-i2c"; > + reg = <0x1740 0x20>; > + interrupts = <0xb 0x8>; > + interrupt-parent = < &ipic >; > + fsl,preserve-clocking; > + }; > + > + i2ccontrol@1760 { > + compatible = "fsl,mpc5121-i2c-ctrl"; > + reg = <0x1760 0x8>; > + }; > + > + axe@2000 { > + compatible = "fsl,mpc5121-axe"; > + reg = <0x2000 0x100>; > + interrupts = <42 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + display@2100 { > + compatible = "fsl,mpc5121-diu"; > + reg = <0x2100 0x100>; > + interrupts = <64 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + can@2300 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <90 0x8>; > + interrupt-parent = < &ipic >; > + reg = <0x2300 0x80>; > + }; > + > + can@2380 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <91 0x8>; > + interrupt-parent = < &ipic >; > + reg = <0x2380 0x80>; > + }; > + > + viu@2400 { > + compatible = "fsl,mpc5121-viu"; > + reg = <0x2400 0x400>; > + interrupts = <67 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + mdio@2800 { > + compatible = "fsl,mpc5121-fec-mdio"; > + reg = <0x2800 0x200>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy: ethernet-phy@0 { > + reg = <0x1f>; > + device_type = "ethernet-phy"; Remove device_type > + }; > + }; > + > + ethernet@2800 { > + device_type = "network"; Ditto > + compatible = "fsl,mpc5121-fec"; > + reg = <0x2800 0x200>; > + local-mac-address = [ 00 00 00 00 00 00 ]; > + interrupts = <4 0x8>; > + interrupt-parent = < &ipic >; > + phy-handle = < &phy >; > + }; > + > + // USB1 using external ULPI PHY > + usb@3000 { > + compatible = "fsl,mpc5121-usb2-dr"; > + reg = <0x3000 0x600>; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupt-parent = < &ipic >; > + interrupts = <43 0x8>; > + dr_mode = "host"; > + phy_type = "ulpi"; > + }; > + > + // USB0 using internal UTMI PHY > + usb@4000 { > + compatible = "fsl,mpc5121-usb2-dr"; > + reg = <0x4000 0x600>; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupt-parent = < &ipic >; > + interrupts = <44 0x8>; > + dr_mode = "otg"; > + phy_type = "utmi_wide"; > + fsl,invert-pwr-fault; > + }; > + > + // IO control > + ioctl@a000 { > + compatible = "fsl,mpc5121-ioctl"; > + reg = <0xA000 0x1000>; > + }; > + > + // 512x PSCs are not 52xx PSCs compatible > + serial@11000 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <0>; > + reg = <0x11000 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11100 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <1>; > + reg = <0x11100 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11200 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <2>; > + reg = <0x11200 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11300 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <3>; > + reg = <0x11300 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11400 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <4>; > + reg = <0x11400 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11600 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <6>; > + reg = <0x11600 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11800 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <8>; > + reg = <0x11800 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11B00 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <11>; > + reg = <0x11B00 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + pscfifo@11f00 { > + compatible = "fsl,mpc5121-psc-fifo"; > + reg = <0x11f00 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + }; > + > + spi@11900 { > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; > + cell-index = <9>; > + reg = <0x11900 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = < &ipic >; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + > + // 7845 touch screen controller > + ts@0 { > + compatible = "ti,ads7845"; > + interrupt-parent = < &ipic >; > + // pen irq is GPIO25 > + interrupts = <78 0x8>; > + }; > + }; > + > + dma@14000 { > + compatible = "fsl,mpc5121-dma"; > + reg = <0x14000 0x1800>; > + interrupts = <65 0x8>; > + interrupt-parent = < &ipic >; > + }; > + }; > +}; > diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig > index e9dca28..27b0651 100644 > --- a/arch/powerpc/platforms/512x/Kconfig > +++ b/arch/powerpc/platforms/512x/Kconfig > @@ -25,3 +25,10 @@ config MPC5121_GENERIC > > Compatible boards include: Protonic LVT base boards (ZANMCU > and VICVT2). > + > +config PDM360NG > + bool "ifm PDM360NG board" > + depends on PPC_MPC512x > + select DEFAULT_UIMAGE > + help > + This option enables support for the PDM360NG board. > diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile > index 90be2f5..4efc1c4 100644 > --- a/arch/powerpc/platforms/512x/Makefile > +++ b/arch/powerpc/platforms/512x/Makefile > @@ -4,3 +4,4 @@ > obj-y += clock.o mpc512x_shared.o > obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o > obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o > +obj-$(CONFIG_PDM360NG) += pdm360ng.o > diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c > new file mode 100644 > index 0000000..dc8f860 > --- /dev/null > +++ b/arch/powerpc/platforms/512x/pdm360ng.c > @@ -0,0 +1,158 @@ > +/* > + * Copyright (C) 2010 DENX Software Engineering > + * > + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> > + * > + * PDM360NG board setup > + * > + * This 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. > + * > + */ > + > +#include <linux/kernel.h> > +#include <linux/io.h> > +#include <linux/of_platform.h> > + > +#include <asm/machdep.h> > +#include <asm/ipic.h> > +#include <asm/prom.h> > + > +#include "mpc512x.h" > + > +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ > + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) > +#include <linux/fsl_devices.h> > +#include <linux/interrupt.h> > +#include <linux/spi/ads7846.h> > +#include <linux/spi/spi.h> > + > +static void *pdm360ng_gpio_base; > + > +static int pdm360ng_get_pendown_state(void) > +{ > + u32 reg; > + > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); > + if (reg & 0x40) > + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); > + > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); > + > + /* return 1 if pen is down */ > + return reg & 0x40 ? 0 : 1; > +} > + > +static struct ads7846_platform_data pdm360ng_ads7846_pdata __initdata = { > + .model = 7845, > + .get_pendown_state = pdm360ng_get_pendown_state, > + /*.irq_trigger = IRQF_TRIGGER_LOW,*/ > +}; > + > +static int __init pdm360ng_penirq_init(void) > +{ > + struct device_node *np; > + struct resource r; > + > + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); > + if (!np) { > + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); > + return -1; > + } > + > + if (of_address_to_resource(np, 0, &r)) { > + pr_err("%s: Invalid gpio address.\n", __func__); > + of_node_put(np); > + return -1; > + } > + of_node_put(np); > + > + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); > + if (!pdm360ng_gpio_base) { > + pr_err("%s: Can't map gpio regs.\n", __func__); > + return -1; > + } > + out_be32((u32 *)pdm360ng_gpio_base + 0xc, 0xffffffff); > + setbits32((u32 *)(pdm360ng_gpio_base + 0x18), 0x2000); > + setbits32((u32 *)(pdm360ng_gpio_base + 0x10), 0x40); > + > + return 0; > +} > + > +static int __init pdm360ng_touchscreen_init(void) > +{ > + struct device_node *np; > + struct of_device *of_dev; > + struct spi_board_info info; > + const u32 *prop; > + int bus_num = -1; > + int len; > + > + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); > + if (!np) > + return -ENODEV; > + > + memset(&info, 0, sizeof(info)); > + info.irq = irq_of_parse_and_map(np, 0); > + if (info.irq == NO_IRQ) > + return -ENODEV; > + > + info.platform_data = &pdm360ng_ads7846_pdata; > + if (strlcpy(info.modalias, "ads7846", > + SPI_NAME_SIZE) >= SPI_NAME_SIZE) > + return -ENOMEM; > + > + np = of_get_next_parent(np); > + if (!np) > + return -ENODEV; > + > + prop = of_get_property(np, "cell-index", &len); > + if (prop && len == 4) > + bus_num = *prop; > + > + if (bus_num < 0 || bus_num > 11) > + return -ENODEV; > + > + info.bus_num = bus_num; > + > + of_dev = of_find_device_by_node(np); > + of_node_put(np); > + if (of_dev) { > + struct fsl_spi_platform_data *pdata; > + > + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); > + if (pdata) { > + pdata->bus_num = bus_num; > + pdata->max_chipselect = 1; > + of_dev->dev.platform_data = pdata; > + } > + } > + > + if (pdm360ng_penirq_init()) > + return -ENODEV; > + > + return spi_register_board_info(&info, 1); > +} > +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); Blech. We need a better way of doing this. It would be nicer if the core OF code provided pre-binding hooks or notifiers so that pdata could be added to the device. You don't need to change this code (yet), but there needs to be a better way of handling this. > +#endif > + > +static int __init pdm360ng_probe(void) > +{ > + unsigned long root = of_get_flat_dt_root(); > + > + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); > +} > + > +define_machine(pdm360ng) { > + .name = "PDM360NG", > + .probe = pdm360ng_probe, > + .setup_arch = mpc512x_setup_diu, > + .init = mpc512x_init, > + .init_early = mpc512x_init_diu, > + .init_IRQ = mpc512x_init_IRQ, > + .get_irq = ipic_get_irq, > + .calibrate_decr = generic_calibrate_decr, > + .restart = mpc512x_restart, > +}; > -- > 1.6.3.3 > > -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board 2010-05-02 14:54 ` Grant Likely @ 2010-05-03 9:22 ` Anatolij Gustschin [not found] ` <g2tfa686aa41005020754t46934f13hbd5c75a6b7668a54-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 1 sibling, 0 replies; 16+ messages in thread From: Anatolij Gustschin @ 2010-05-03 9:22 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss, Michael Weiss, linuxppc-dev, Wolfgang Grandegger Hi Grant, On Sun, 2 May 2010 08:54:26 -0600 Grant Likely <grant.likely@secretlab.ca> wrote: > Hi Anatolij, > > Comments below. > > On Fri, Apr 30, 2010 at 2:30 PM, Anatolij Gustschin <agust@denx.de> wrote: > > Adds IFM PDM360NG device tree, and platform code. ... > > + nfc@40000000 { > > + compatible = "fsl,mpc5121-nfc"; > > + reg = <0x40000000 0x100000>; > > + interrupts = <0x6 0x8>; > > + interrupt-parent = <0x1>; > > This looks wrong. Shouldn't this be interrupt-parent = <&ipic>? In > fact, if the root node has interrupt-parent = <&ipic> then the > interrupt-parent property can be omitted from all other nodes unless > they need to override it. Yes, it should be interrupt-parent = <&ipic>. I will fix it, thanks. ... > > + // IPIC > > + // interrupts cell = <intr #, sense> > > + // sense values match linux IORESOURCE_IRQ_* defines: > > + // sense == 8: Level, low assertion > > + // sense == 2: Edge, high-to-low change > > + // > > + ipic: interrupt-controller@c00 { > > + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; > > + interrupt-controller; > > + #address-cells = <0>; > > Don't need #address-cells here Ok, will remove. ... > > + mdio@2800 { > > + compatible = "fsl,mpc5121-fec-mdio"; > > + reg = <0x2800 0x200>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + phy: ethernet-phy@0 { > > + reg = <0x1f>; > > + device_type = "ethernet-phy"; > > Remove device_type Ok. > > + ethernet@2800 { > > + device_type = "network"; > > Ditto Removing this property results in not functional ethernet, in particular, the MAC address won't be set by U-Boot. I need to add aliases node for ethernet0 to fix this. Ok, will remove this device_type, too. Thanks, Anatolij ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <g2tfa686aa41005020754t46934f13hbd5c75a6b7668a54-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <g2tfa686aa41005020754t46934f13hbd5c75a6b7668a54-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2010-05-03 16:34 ` Scott Wood [not found] ` <4BDEFB13.5060407-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Scott Wood @ 2010-05-03 16:34 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger Grant Likely wrote: >> + // IPIC >> + // interrupts cell = <intr #, sense> >> + // sense values match linux IORESOURCE_IRQ_* defines: >> + // sense == 8: Level, low assertion >> + // sense == 2: Edge, high-to-low change >> + // >> + ipic: interrupt-controller@c00 { >> + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; >> + interrupt-controller; >> + #address-cells = <0>; > > Don't need #address-cells here #address-cells is required by ePAPR for interrupt controllers if an interrupt-map is used. While there isn't one in this tree, it's best to just always include #address-cells in interrupt controller nodes. -Scott ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <4BDEFB13.5060407-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <4BDEFB13.5060407-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2010-05-19 21:27 ` Grant Likely 2010-05-19 21:37 ` Scott Wood 0 siblings, 1 reply; 16+ messages in thread From: Grant Likely @ 2010-05-19 21:27 UTC (permalink / raw) To: Scott Wood Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Mon, May 3, 2010 at 10:34 AM, Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: > Grant Likely wrote: >>> >>> + // IPIC >>> + // interrupts cell = <intr #, sense> >>> + // sense values match linux IORESOURCE_IRQ_* defines: >>> + // sense == 8: Level, low assertion >>> + // sense == 2: Edge, high-to-low change >>> + // >>> + ipic: interrupt-controller@c00 { >>> + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; >>> + interrupt-controller; >>> + #address-cells = <0>; >> >> Don't need #address-cells here > > #address-cells is required by ePAPR for interrupt controllers if an > interrupt-map is used. Why? /me is too lazy to dig out ePAPR and look. g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board 2010-05-19 21:27 ` Grant Likely @ 2010-05-19 21:37 ` Scott Wood [not found] ` <4BF45A1F.70100-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Scott Wood @ 2010-05-19 21:37 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss, Michael Weiss, linuxppc-dev, Anatolij Gustschin, Wolfgang Grandegger On 05/19/2010 04:27 PM, Grant Likely wrote: > On Mon, May 3, 2010 at 10:34 AM, Scott Wood<scottwood@freescale.com> wrote: >> Grant Likely wrote: >>>> >>>> + // IPIC >>>> + // interrupts cell =<intr #, sense> >>>> + // sense values match linux IORESOURCE_IRQ_* defines: >>>> + // sense == 8: Level, low assertion >>>> + // sense == 2: Edge, high-to-low change >>>> + // >>>> + ipic: interrupt-controller@c00 { >>>> + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; >>>> + interrupt-controller; >>>> + #address-cells =<0>; >>> >>> Don't need #address-cells here >> >> #address-cells is required by ePAPR for interrupt controllers if an >> interrupt-map is used. > > Why? > > /me is too lazy to dig out ePAPR and look. Address cells are part of the interrupt identification. Typically with interrupt maps this is only used on the child end (e.g. to select a particular PCI slot), but if the parent interrupt controller's address cells are non-zero it will be expected in the parent interrupt specifier as well. I believe the only part of this that is new with ePAPR is that it asks that the interrupt controller address cells be explicitly specified, as it's a bit icky for it to default to 2 in some contexts and 0 in others. -Scott ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <4BF45A1F.70100-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <4BF45A1F.70100-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2010-05-19 21:47 ` Grant Likely [not found] ` <AANLkTinxuFz3SSLZtF3yGYtRN8nsW8_6EDiqzmJilYE9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Grant Likely @ 2010-05-19 21:47 UTC (permalink / raw) To: Scott Wood Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Wed, May 19, 2010 at 3:37 PM, Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: > On 05/19/2010 04:27 PM, Grant Likely wrote: >> >> On Mon, May 3, 2010 at 10:34 AM, Scott Wood<scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >> wrote: >>> >>> Grant Likely wrote: >>>>> >>>>> + // IPIC >>>>> + // interrupts cell =<intr #, sense> >>>>> + // sense values match linux IORESOURCE_IRQ_* defines: >>>>> + // sense == 8: Level, low assertion >>>>> + // sense == 2: Edge, high-to-low change >>>>> + // >>>>> + ipic: interrupt-controller@c00 { >>>>> + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; >>>>> + interrupt-controller; >>>>> + #address-cells =<0>; >>>> >>>> Don't need #address-cells here >>> >>> #address-cells is required by ePAPR for interrupt controllers if an >>> interrupt-map is used. >> >> Why? >> >> /me is too lazy to dig out ePAPR and look. > > Address cells are part of the interrupt identification. Typically with > interrupt maps this is only used on the child end (e.g. to select a > particular PCI slot), but if the parent interrupt controller's address cells > are non-zero it will be expected in the parent interrupt specifier as well. > > I believe the only part of this that is new with ePAPR is that it asks that > the interrupt controller address cells be explicitly specified, as it's a > bit icky for it to default to 2 in some contexts and 0 in others. Hmmm. I've not seen that before. On the 5200 the value of #address-cells for interrupt controllers has apparently defaulted to <0> so I've never encountered or thought about it. I'm not even sure what #address-cells != 0 would mean in the context of interrupt mapping, or where it would be relevant. g. ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <AANLkTinxuFz3SSLZtF3yGYtRN8nsW8_6EDiqzmJilYE9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <AANLkTinxuFz3SSLZtF3yGYtRN8nsW8_6EDiqzmJilYE9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2010-06-22 21:39 ` Scott Wood 0 siblings, 0 replies; 16+ messages in thread From: Scott Wood @ 2010-06-22 21:39 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On 05/19/2010 04:47 PM, Grant Likely wrote: > On Wed, May 19, 2010 at 3:37 PM, Scott Wood<scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: >> I believe the only part of this that is new with ePAPR is that it asks that >> the interrupt controller address cells be explicitly specified, as it's a >> bit icky for it to default to 2 in some contexts and 0 in others. > > Hmmm. I've not seen that before. On the 5200 the value of > #address-cells for interrupt controllers has apparently defaulted to > <0> so I've never encountered or thought about it. I'm not even sure > what #address-cells != 0 would mean in the context of interrupt > mapping, or where it would be relevant. The address component is mainly used in PCI interrupt mapping, where each slot has a distinct wiring. It would be weird to see it on a normal interrupt controller, but explicitly setting it to zero helps avoid someone deciding that an interrupt controller ought to have a child node for whatever reason (this was an issue with MPIC), setting address cells to something non-zero, and messing up interrupt maps. -Scott ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1272659448-23302-2-git-send-email-agust-ynQEQJNshbs@public.gmane.org> 2010-05-02 14:54 ` Grant Likely @ 2010-05-03 10:23 ` Anatolij Gustschin [not found] ` <1272882222-12253-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> 1 sibling, 1 reply; 16+ messages in thread From: Anatolij Gustschin @ 2010-05-03 10:23 UTC (permalink / raw) To: linuxppc-dev-mnsaURCQ41sdnm+yROfE0A Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, Wolfgang Grandegger Adds IFM PDM360NG device tree, and platform code. Currently following is supported: - Spansion S29GL512P 256 MB NOR flash - ST Micro NAND 1 GiB flash - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" at the kernel command line to enable PrimeView PM070WL3 Display support. - FEC - I2C - RTC, EEPROM - MSCAN - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" on the kernel command line. - USB0/1 Host - USB0 OTG Host/Device - VIU, Overlay support Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> --- This patch is intended to be used on top of MPC5121 DIU support patch series: http://thread.gmane.org/gmane.linux.ports.ppc.embedded/34806 v1 -> v2: - fix interrupt-parent property in nfc node - drop #address-cells in ipic node - remove device_type from ethernet-phy sub-node - remove device_type from ethernet node - add aliases node for eth0, needed for MAC address update by U-Boot - removed spaces around &ipic arch/powerpc/boot/dts/pdm360ng.dts | 431 ++++++++++++++++++++++++++++++++ arch/powerpc/platforms/512x/Kconfig | 7 + arch/powerpc/platforms/512x/Makefile | 1 + arch/powerpc/platforms/512x/pdm360ng.c | 158 ++++++++++++ 4 files changed, 597 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts new file mode 100644 index 0000000..3d96a09 --- /dev/null +++ b/arch/powerpc/boot/dts/pdm360ng.dts @@ -0,0 +1,431 @@ +/* + * Device Tree Source for IFM PDM360NG. + * + * Copyright 2009 - 2010 DENX Software Engineering. + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> + * + * Based on MPC5121E ADS dts. + * Copyright 2008 Freescale Semiconductor Inc. + * + * 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. + */ + +/dts-v1/; + +/ { + model = "pdm360ng"; + compatible = "ifm,pdm360ng"; + #address-cells = <1>; + #size-cells = <1>; + + aliases { + ethernet0 = ð0; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + PowerPC,5121@0 { + device_type = "cpu"; + reg = <0>; + d-cache-line-size = <0x20>; // 32 bytes + i-cache-line-size = <0x20>; // 32 bytes + d-cache-size = <0x8000>; // L1, 32K + i-cache-size = <0x8000>; // L1, 32K + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) + bus-frequency = <198000000>; // 198 MHz csb bus + clock-frequency = <396000000>; // 396 MHz ppc core + }; + }; + + memory { + device_type = "memory"; + reg = <0x00000000 0x20000000>; // 512MB at 0 + }; + + nfc@40000000 { + compatible = "fsl,mpc5121-nfc"; + reg = <0x40000000 0x100000>; + interrupts = <0x6 0x8>; + interrupt-parent = <&ipic>; + #address-cells = <0x1>; + #size-cells = <0x1>; + bank-width = <0x1>; + chips = <0x1>; + + partition@0 { + label = "nand0"; + reg = <0x0 0x40000000>; + }; + }; + + sram@50000000 { + compatible = "fsl,mpc5121-sram"; + reg = <0x50000000 0x20000>; // 128K at 0x50000000 + }; + + localbus@80000020 { + compatible = "fsl,mpc5121-localbus"; + #address-cells = <2>; + #size-cells = <1>; + reg = <0x80000020 0x40>; + + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ + + flash@0,0 { + compatible = "amd,s29gl01gp", "cfi-flash"; + reg = <0 0x00000000 0x08000000 + 0 0x08000000 0x08000000>; + #address-cells = <1>; + #size-cells = <1>; + bank-width = <4>; + device-width = <2>; + + partition@0 { + label = "u-boot"; + reg = <0x00000000 0x00080000>; + read-only; + }; + partition@80000 { + label = "environment"; + reg = <0x00080000 0x00080000>; + read-only; + }; + partition@100000 { + label = "splash-image"; + reg = <0x00100000 0x00080000>; + read-only; + }; + partition@180000 { + label = "device-tree"; + reg = <0x00180000 0x00040000>; + }; + partition@1c0000 { + label = "kernel"; + reg = <0x001c0000 0x00500000>; + }; + partition@6c0000 { + label = "filesystem"; + reg = <0x006c0000 0x07940000>; + }; + }; + + mram0@2,0 { + compatible = "mtd-ram"; + reg = <2 0x00000 0x10000>; + bank-width = <2>; + }; + + mram1@2,10000 { + compatible = "mtd-ram"; + reg = <2 0x010000 0x10000>; + bank-width = <2>; + }; + }; + + soc@80000000 { + compatible = "fsl,mpc5121-immr"; + #address-cells = <1>; + #size-cells = <1>; + #interrupt-cells = <2>; + ranges = <0x0 0x80000000 0x400000>; + reg = <0x80000000 0x400000>; + bus-frequency = <66000000>; // 66 MHz ips bus + + // IPIC + // interrupts cell = <intr #, sense> + // sense values match linux IORESOURCE_IRQ_* defines: + // sense == 8: Level, low assertion + // sense == 2: Edge, high-to-low change + // + ipic: interrupt-controller@c00 { + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; + interrupt-controller; + #interrupt-cells = <2>; + reg = <0xc00 0x100>; + }; + + rtc@a00 { // Real time clock + compatible = "fsl,mpc5121-rtc"; + reg = <0xa00 0x100>; + interrupts = <79 0x8 80 0x8>; + interrupt-parent = <&ipic>; + }; + + reset@e00 { // Reset module + compatible = "fsl,mpc5121-reset"; + reg = <0xe00 0x100>; + }; + + clock@f00 { // Clock control + compatible = "fsl,mpc5121-clock"; + reg = <0xf00 0x100>; + }; + + pmc@1000{ //Power Management Controller + compatible = "fsl,mpc5121-pmc"; + reg = <0x1000 0x100>; + interrupts = <83 0x2>; + interrupt-parent = <&ipic>; + }; + + gpio@1100 { + compatible = "fsl,mpc5121-gpio"; + reg = <0x1100 0x100>; + interrupts = <78 0x8>; + interrupt-parent = <&ipic>; + }; + + can@1300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <12 0x8>; + interrupt-parent = <&ipic>; + reg = <0x1300 0x80>; + }; + + can@1380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <13 0x8>; + interrupt-parent = <&ipic>; + reg = <0x1380 0x80>; + }; + + i2c@1700 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1700 0x20>; + interrupts = <0x9 0x8>; + interrupt-parent = <&ipic>; + fsl,preserve-clocking; + + eeprom@50 { + compatible = "at,24c01"; + reg = <0x50>; + }; + + rtc@68 { + compatible = "stm,m41t00"; + reg = <0x68>; + }; + }; + + i2c@1740 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1740 0x20>; + interrupts = <0xb 0x8>; + interrupt-parent = <&ipic>; + fsl,preserve-clocking; + }; + + i2ccontrol@1760 { + compatible = "fsl,mpc5121-i2c-ctrl"; + reg = <0x1760 0x8>; + }; + + axe@2000 { + compatible = "fsl,mpc5121-axe"; + reg = <0x2000 0x100>; + interrupts = <42 0x8>; + interrupt-parent = <&ipic>; + }; + + display@2100 { + compatible = "fsl,mpc5121-diu"; + reg = <0x2100 0x100>; + interrupts = <64 0x8>; + interrupt-parent = <&ipic>; + }; + + can@2300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <90 0x8>; + interrupt-parent = <&ipic>; + reg = <0x2300 0x80>; + }; + + can@2380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <91 0x8>; + interrupt-parent = <&ipic>; + reg = <0x2380 0x80>; + }; + + viu@2400 { + compatible = "fsl,mpc5121-viu"; + reg = <0x2400 0x400>; + interrupts = <67 0x8>; + interrupt-parent = <&ipic>; + }; + + mdio@2800 { + compatible = "fsl,mpc5121-fec-mdio"; + reg = <0x2800 0x200>; + #address-cells = <1>; + #size-cells = <0>; + phy: ethernet-phy@0 { + reg = <0x1f>; + }; + }; + + eth0: ethernet@2800 { + compatible = "fsl,mpc5121-fec"; + reg = <0x2800 0x200>; + local-mac-address = [ 00 00 00 00 00 00 ]; + interrupts = <4 0x8>; + interrupt-parent = <&ipic>; + phy-handle = < &phy >; + }; + + // USB1 using external ULPI PHY + usb@3000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x3000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = <&ipic>; + interrupts = <43 0x8>; + dr_mode = "host"; + phy_type = "ulpi"; + }; + + // USB0 using internal UTMI PHY + usb@4000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x4000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = <&ipic>; + interrupts = <44 0x8>; + dr_mode = "otg"; + phy_type = "utmi_wide"; + fsl,invert-pwr-fault; + }; + + // IO control + ioctl@a000 { + compatible = "fsl,mpc5121-ioctl"; + reg = <0xA000 0x1000>; + }; + + // 512x PSCs are not 52xx PSCs compatible + serial@11000 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <0>; + reg = <0x11000 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11100 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <1>; + reg = <0x11100 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11200 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <2>; + reg = <0x11200 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11300 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <3>; + reg = <0x11300 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11400 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <4>; + reg = <0x11400 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11600 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <6>; + reg = <0x11600 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11800 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <8>; + reg = <0x11800 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11B00 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <11>; + reg = <0x11B00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + pscfifo@11f00 { + compatible = "fsl,mpc5121-psc-fifo"; + reg = <0x11f00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + }; + + spi@11900 { + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; + cell-index = <9>; + reg = <0x11900 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + + // 7845 touch screen controller + ts@0 { + compatible = "ti,ads7845"; + interrupt-parent = <&ipic>; + // pen irq is GPIO25 + interrupts = <78 0x8>; + }; + }; + + dma@14000 { + compatible = "fsl,mpc5121-dma"; + reg = <0x14000 0x1800>; + interrupts = <65 0x8>; + interrupt-parent = <&ipic>; + }; + }; +}; diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig index e9dca28..27b0651 100644 --- a/arch/powerpc/platforms/512x/Kconfig +++ b/arch/powerpc/platforms/512x/Kconfig @@ -25,3 +25,10 @@ config MPC5121_GENERIC Compatible boards include: Protonic LVT base boards (ZANMCU and VICVT2). + +config PDM360NG + bool "ifm PDM360NG board" + depends on PPC_MPC512x + select DEFAULT_UIMAGE + help + This option enables support for the PDM360NG board. diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile index 90be2f5..4efc1c4 100644 --- a/arch/powerpc/platforms/512x/Makefile +++ b/arch/powerpc/platforms/512x/Makefile @@ -4,3 +4,4 @@ obj-y += clock.o mpc512x_shared.o obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o +obj-$(CONFIG_PDM360NG) += pdm360ng.o diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c new file mode 100644 index 0000000..dc8f860 --- /dev/null +++ b/arch/powerpc/platforms/512x/pdm360ng.c @@ -0,0 +1,158 @@ +/* + * Copyright (C) 2010 DENX Software Engineering + * + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> + * + * PDM360NG board setup + * + * This 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. + * + */ + +#include <linux/kernel.h> +#include <linux/io.h> +#include <linux/of_platform.h> + +#include <asm/machdep.h> +#include <asm/ipic.h> +#include <asm/prom.h> + +#include "mpc512x.h" + +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) +#include <linux/fsl_devices.h> +#include <linux/interrupt.h> +#include <linux/spi/ads7846.h> +#include <linux/spi/spi.h> + +static void *pdm360ng_gpio_base; + +static int pdm360ng_get_pendown_state(void) +{ + u32 reg; + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); + if (reg & 0x40) + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); + + /* return 1 if pen is down */ + return reg & 0x40 ? 0 : 1; +} + +static struct ads7846_platform_data pdm360ng_ads7846_pdata __initdata = { + .model = 7845, + .get_pendown_state = pdm360ng_get_pendown_state, + /*.irq_trigger = IRQF_TRIGGER_LOW,*/ +}; + +static int __init pdm360ng_penirq_init(void) +{ + struct device_node *np; + struct resource r; + + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); + if (!np) { + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); + return -1; + } + + if (of_address_to_resource(np, 0, &r)) { + pr_err("%s: Invalid gpio address.\n", __func__); + of_node_put(np); + return -1; + } + of_node_put(np); + + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); + if (!pdm360ng_gpio_base) { + pr_err("%s: Can't map gpio regs.\n", __func__); + return -1; + } + out_be32((u32 *)pdm360ng_gpio_base + 0xc, 0xffffffff); + setbits32((u32 *)(pdm360ng_gpio_base + 0x18), 0x2000); + setbits32((u32 *)(pdm360ng_gpio_base + 0x10), 0x40); + + return 0; +} + +static int __init pdm360ng_touchscreen_init(void) +{ + struct device_node *np; + struct of_device *of_dev; + struct spi_board_info info; + const u32 *prop; + int bus_num = -1; + int len; + + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); + if (!np) + return -ENODEV; + + memset(&info, 0, sizeof(info)); + info.irq = irq_of_parse_and_map(np, 0); + if (info.irq == NO_IRQ) + return -ENODEV; + + info.platform_data = &pdm360ng_ads7846_pdata; + if (strlcpy(info.modalias, "ads7846", + SPI_NAME_SIZE) >= SPI_NAME_SIZE) + return -ENOMEM; + + np = of_get_next_parent(np); + if (!np) + return -ENODEV; + + prop = of_get_property(np, "cell-index", &len); + if (prop && len == 4) + bus_num = *prop; + + if (bus_num < 0 || bus_num > 11) + return -ENODEV; + + info.bus_num = bus_num; + + of_dev = of_find_device_by_node(np); + of_node_put(np); + if (of_dev) { + struct fsl_spi_platform_data *pdata; + + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); + if (pdata) { + pdata->bus_num = bus_num; + pdata->max_chipselect = 1; + of_dev->dev.platform_data = pdata; + } + } + + if (pdm360ng_penirq_init()) + return -ENODEV; + + return spi_register_board_info(&info, 1); +} +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); +#endif + +static int __init pdm360ng_probe(void) +{ + unsigned long root = of_get_flat_dt_root(); + + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); +} + +define_machine(pdm360ng) { + .name = "PDM360NG", + .probe = pdm360ng_probe, + .setup_arch = mpc512x_setup_diu, + .init = mpc512x_init, + .init_early = mpc512x_init_diu, + .init_IRQ = mpc512x_init_IRQ, + .get_irq = ipic_get_irq, + .calibrate_decr = generic_calibrate_decr, + .restart = mpc512x_restart, +}; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
[parent not found: <1272882222-12253-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org>]
* [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1272882222-12253-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> @ 2010-07-23 13:49 ` Anatolij Gustschin [not found] ` <1279892973-24110-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Anatolij Gustschin @ 2010-07-23 13:49 UTC (permalink / raw) To: linuxppc-dev-mnsaURCQ41sdnm+yROfE0A Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, Wolfgang Grandegger Adds IFM PDM360NG device tree, and platform code. Currently following is supported: - Spansion S29GL512P 256 MB NOR flash - ST Micro NAND 1 GiB flash - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" at the kernel command line to enable PrimeView PM070WL3 Display support. - FEC - I2C - RTC, EEPROM - MSCAN - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" on the kernel command line. - SPI, ADS7845 Touchscreen - USB0/1 Host - USB0 OTG Host/Device - VIU, Overlay/Capture support Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> --- v3: - uncomment and correct .irq_flags field of touchscreen platform data struct as proposed extension to this data struct has been accepted and merged via input tree a short while ago v1 -> v2: - fix interrupt-parent property in nfc node - drop #address-cells in ipic node - remove device_type from ethernet-phy sub-node - remove device_type from ethernet node - add aliases node for eth0, needed for MAC address update by U-Boot - removed spaces around &ipic arch/powerpc/boot/dts/pdm360ng.dts | 432 ++++++++++++++++++++++++++++++++ arch/powerpc/platforms/512x/Kconfig | 7 + arch/powerpc/platforms/512x/Makefile | 1 + arch/powerpc/platforms/512x/pdm360ng.c | 158 ++++++++++++ 4 files changed, 598 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts new file mode 100644 index 0000000..7dc42e9 --- /dev/null +++ b/arch/powerpc/boot/dts/pdm360ng.dts @@ -0,0 +1,432 @@ +/* + * Device Tree Source for IFM PDM360NG. + * + * Copyright 2009 - 2010 DENX Software Engineering. + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> + * + * Based on MPC5121E ADS dts. + * Copyright 2008 Freescale Semiconductor Inc. + * + * 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. + */ + +/dts-v1/; + +/ { + model = "pdm360ng"; + compatible = "ifm,pdm360ng"; + #address-cells = <1>; + #size-cells = <1>; + + aliases { + ethernet0 = ð0; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + PowerPC,5121@0 { + device_type = "cpu"; + reg = <0>; + d-cache-line-size = <0x20>; // 32 bytes + i-cache-line-size = <0x20>; // 32 bytes + d-cache-size = <0x8000>; // L1, 32K + i-cache-size = <0x8000>; // L1, 32K + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) + bus-frequency = <198000000>; // 198 MHz csb bus + clock-frequency = <396000000>; // 396 MHz ppc core + }; + }; + + memory { + device_type = "memory"; + reg = <0x00000000 0x20000000>; // 512MB at 0 + }; + + nfc@40000000 { + compatible = "fsl,mpc5121-nfc"; + reg = <0x40000000 0x100000>; + interrupts = <0x6 0x8>; + interrupt-parent = <&ipic>; + #address-cells = <0x1>; + #size-cells = <0x1>; + bank-width = <0x1>; + chips = <0x1>; + + partition@0 { + label = "nand0"; + reg = <0x0 0x40000000>; + }; + }; + + sram@50000000 { + compatible = "fsl,mpc5121-sram"; + reg = <0x50000000 0x20000>; // 128K at 0x50000000 + }; + + localbus@80000020 { + compatible = "fsl,mpc5121-localbus"; + #address-cells = <2>; + #size-cells = <1>; + reg = <0x80000020 0x40>; + + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ + + flash@0,0 { + compatible = "amd,s29gl01gp", "cfi-flash"; + reg = <0 0x00000000 0x08000000 + 0 0x08000000 0x08000000>; + #address-cells = <1>; + #size-cells = <1>; + bank-width = <4>; + device-width = <2>; + + partition@0 { + label = "u-boot"; + reg = <0x00000000 0x00080000>; + read-only; + }; + partition@80000 { + label = "environment"; + reg = <0x00080000 0x00080000>; + read-only; + }; + partition@100000 { + label = "splash-image"; + reg = <0x00100000 0x00080000>; + read-only; + }; + partition@180000 { + label = "device-tree"; + reg = <0x00180000 0x00040000>; + }; + partition@1c0000 { + label = "kernel"; + reg = <0x001c0000 0x00500000>; + }; + partition@6c0000 { + label = "filesystem"; + reg = <0x006c0000 0x07940000>; + }; + }; + + mram0@2,0 { + compatible = "mtd-ram"; + reg = <2 0x00000 0x10000>; + bank-width = <2>; + }; + + mram1@2,10000 { + compatible = "mtd-ram"; + reg = <2 0x010000 0x10000>; + bank-width = <2>; + }; + }; + + soc@80000000 { + compatible = "fsl,mpc5121-immr"; + #address-cells = <1>; + #size-cells = <1>; + #interrupt-cells = <2>; + ranges = <0x0 0x80000000 0x400000>; + reg = <0x80000000 0x400000>; + bus-frequency = <66000000>; // 66 MHz ips bus + + // IPIC + // interrupts cell = <intr #, sense> + // sense values match linux IORESOURCE_IRQ_* defines: + // sense == 8: Level, low assertion + // sense == 2: Edge, high-to-low change + // + ipic: interrupt-controller@c00 { + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; + interrupt-controller; + #address-cells = <0>; + #interrupt-cells = <2>; + reg = <0xc00 0x100>; + }; + + rtc@a00 { // Real time clock + compatible = "fsl,mpc5121-rtc"; + reg = <0xa00 0x100>; + interrupts = <79 0x8 80 0x8>; + interrupt-parent = <&ipic>; + }; + + reset@e00 { // Reset module + compatible = "fsl,mpc5121-reset"; + reg = <0xe00 0x100>; + }; + + clock@f00 { // Clock control + compatible = "fsl,mpc5121-clock"; + reg = <0xf00 0x100>; + }; + + pmc@1000{ //Power Management Controller + compatible = "fsl,mpc5121-pmc"; + reg = <0x1000 0x100>; + interrupts = <83 0x2>; + interrupt-parent = <&ipic>; + }; + + gpio@1100 { + compatible = "fsl,mpc5121-gpio"; + reg = <0x1100 0x100>; + interrupts = <78 0x8>; + interrupt-parent = <&ipic>; + }; + + can@1300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <12 0x8>; + interrupt-parent = <&ipic>; + reg = <0x1300 0x80>; + }; + + can@1380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <13 0x8>; + interrupt-parent = <&ipic>; + reg = <0x1380 0x80>; + }; + + i2c@1700 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1700 0x20>; + interrupts = <0x9 0x8>; + interrupt-parent = <&ipic>; + fsl,preserve-clocking; + + eeprom@50 { + compatible = "at,24c01"; + reg = <0x50>; + }; + + rtc@68 { + compatible = "stm,m41t00"; + reg = <0x68>; + }; + }; + + i2c@1740 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1740 0x20>; + interrupts = <0xb 0x8>; + interrupt-parent = <&ipic>; + fsl,preserve-clocking; + }; + + i2ccontrol@1760 { + compatible = "fsl,mpc5121-i2c-ctrl"; + reg = <0x1760 0x8>; + }; + + axe@2000 { + compatible = "fsl,mpc5121-axe"; + reg = <0x2000 0x100>; + interrupts = <42 0x8>; + interrupt-parent = <&ipic>; + }; + + display@2100 { + compatible = "fsl,mpc5121-diu"; + reg = <0x2100 0x100>; + interrupts = <64 0x8>; + interrupt-parent = <&ipic>; + }; + + can@2300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <90 0x8>; + interrupt-parent = <&ipic>; + reg = <0x2300 0x80>; + }; + + can@2380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <91 0x8>; + interrupt-parent = <&ipic>; + reg = <0x2380 0x80>; + }; + + viu@2400 { + compatible = "fsl,mpc5121-viu"; + reg = <0x2400 0x400>; + interrupts = <67 0x8>; + interrupt-parent = <&ipic>; + }; + + mdio@2800 { + compatible = "fsl,mpc5121-fec-mdio"; + reg = <0x2800 0x200>; + #address-cells = <1>; + #size-cells = <0>; + phy: ethernet-phy@0 { + reg = <0x1f>; + }; + }; + + eth0: ethernet@2800 { + compatible = "fsl,mpc5121-fec"; + reg = <0x2800 0x200>; + local-mac-address = [ 00 00 00 00 00 00 ]; + interrupts = <4 0x8>; + interrupt-parent = <&ipic>; + phy-handle = < &phy >; + }; + + // USB1 using external ULPI PHY + usb@3000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x3000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = <&ipic>; + interrupts = <43 0x8>; + dr_mode = "host"; + phy_type = "ulpi"; + }; + + // USB0 using internal UTMI PHY + usb@4000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x4000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupt-parent = <&ipic>; + interrupts = <44 0x8>; + dr_mode = "otg"; + phy_type = "utmi_wide"; + fsl,invert-pwr-fault; + }; + + // IO control + ioctl@a000 { + compatible = "fsl,mpc5121-ioctl"; + reg = <0xA000 0x1000>; + }; + + // 512x PSCs are not 52xx PSCs compatible + serial@11000 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <0>; + reg = <0x11000 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11100 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <1>; + reg = <0x11100 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11200 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <2>; + reg = <0x11200 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11300 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <3>; + reg = <0x11300 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11400 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <4>; + reg = <0x11400 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11600 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <6>; + reg = <0x11600 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11800 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <8>; + reg = <0x11800 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11B00 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <11>; + reg = <0x11B00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + pscfifo@11f00 { + compatible = "fsl,mpc5121-psc-fifo"; + reg = <0x11f00 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + }; + + spi@11900 { + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; + cell-index = <9>; + reg = <0x11900 0x100>; + interrupts = <40 0x8>; + interrupt-parent = <&ipic>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + + // 7845 touch screen controller + ts@0 { + compatible = "ti,ads7845"; + interrupt-parent = <&ipic>; + // pen irq is GPIO25 + interrupts = <78 0x8>; + }; + }; + + dma@14000 { + compatible = "fsl,mpc5121-dma"; + reg = <0x14000 0x1800>; + interrupts = <65 0x8>; + interrupt-parent = <&ipic>; + }; + }; +}; diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig index e9dca28..27b0651 100644 --- a/arch/powerpc/platforms/512x/Kconfig +++ b/arch/powerpc/platforms/512x/Kconfig @@ -25,3 +25,10 @@ config MPC5121_GENERIC Compatible boards include: Protonic LVT base boards (ZANMCU and VICVT2). + +config PDM360NG + bool "ifm PDM360NG board" + depends on PPC_MPC512x + select DEFAULT_UIMAGE + help + This option enables support for the PDM360NG board. diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile index 90be2f5..4efc1c4 100644 --- a/arch/powerpc/platforms/512x/Makefile +++ b/arch/powerpc/platforms/512x/Makefile @@ -4,3 +4,4 @@ obj-y += clock.o mpc512x_shared.o obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o +obj-$(CONFIG_PDM360NG) += pdm360ng.o diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c new file mode 100644 index 0000000..01e6023 --- /dev/null +++ b/arch/powerpc/platforms/512x/pdm360ng.c @@ -0,0 +1,158 @@ +/* + * Copyright (C) 2010 DENX Software Engineering + * + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> + * + * PDM360NG board setup + * + * This 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. + * + */ + +#include <linux/kernel.h> +#include <linux/io.h> +#include <linux/of_platform.h> + +#include <asm/machdep.h> +#include <asm/ipic.h> +#include <asm/prom.h> + +#include "mpc512x.h" + +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) +#include <linux/fsl_devices.h> +#include <linux/interrupt.h> +#include <linux/spi/ads7846.h> +#include <linux/spi/spi.h> + +static void *pdm360ng_gpio_base; + +static int pdm360ng_get_pendown_state(void) +{ + u32 reg; + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); + if (reg & 0x40) + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); + + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); + + /* return 1 if pen is down */ + return reg & 0x40 ? 0 : 1; +} + +static struct ads7846_platform_data pdm360ng_ads7846_pdata __initdata = { + .model = 7845, + .get_pendown_state = pdm360ng_get_pendown_state, + .irq_flags = IRQF_TRIGGER_LOW, +}; + +static int __init pdm360ng_penirq_init(void) +{ + struct device_node *np; + struct resource r; + + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); + if (!np) { + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); + return -1; + } + + if (of_address_to_resource(np, 0, &r)) { + pr_err("%s: Invalid gpio address.\n", __func__); + of_node_put(np); + return -1; + } + of_node_put(np); + + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); + if (!pdm360ng_gpio_base) { + pr_err("%s: Can't map gpio regs.\n", __func__); + return -1; + } + out_be32((u32 *)pdm360ng_gpio_base + 0xc, 0xffffffff); + setbits32((u32 *)(pdm360ng_gpio_base + 0x18), 0x2000); + setbits32((u32 *)(pdm360ng_gpio_base + 0x10), 0x40); + + return 0; +} + +static int __init pdm360ng_touchscreen_init(void) +{ + struct device_node *np; + struct of_device *of_dev; + struct spi_board_info info; + const u32 *prop; + int bus_num = -1; + int len; + + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); + if (!np) + return -ENODEV; + + memset(&info, 0, sizeof(info)); + info.irq = irq_of_parse_and_map(np, 0); + if (info.irq == NO_IRQ) + return -ENODEV; + + info.platform_data = &pdm360ng_ads7846_pdata; + if (strlcpy(info.modalias, "ads7846", + SPI_NAME_SIZE) >= SPI_NAME_SIZE) + return -ENOMEM; + + np = of_get_next_parent(np); + if (!np) + return -ENODEV; + + prop = of_get_property(np, "cell-index", &len); + if (prop && len == 4) + bus_num = *prop; + + if (bus_num < 0 || bus_num > 11) + return -ENODEV; + + info.bus_num = bus_num; + + of_dev = of_find_device_by_node(np); + of_node_put(np); + if (of_dev) { + struct fsl_spi_platform_data *pdata; + + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); + if (pdata) { + pdata->bus_num = bus_num; + pdata->max_chipselect = 1; + of_dev->dev.platform_data = pdata; + } + } + + if (pdm360ng_penirq_init()) + return -ENODEV; + + return spi_register_board_info(&info, 1); +} +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); +#endif + +static int __init pdm360ng_probe(void) +{ + unsigned long root = of_get_flat_dt_root(); + + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); +} + +define_machine(pdm360ng) { + .name = "PDM360NG", + .probe = pdm360ng_probe, + .setup_arch = mpc512x_setup_diu, + .init = mpc512x_init, + .init_early = mpc512x_init_diu, + .init_IRQ = mpc512x_init_IRQ, + .get_irq = ipic_get_irq, + .calibrate_decr = generic_calibrate_decr, + .restart = mpc512x_restart, +}; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
[parent not found: <1279892973-24110-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org>]
* Re: [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1279892973-24110-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> @ 2010-07-25 7:42 ` Grant Likely [not found] ` <AANLkTi=Ee6HR+Ux0g4V1+81DXUmsO=UDhi6jWsDgGchv-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2010-07-27 21:26 ` [PATCH v4 " Anatolij Gustschin 1 sibling, 1 reply; 16+ messages in thread From: Grant Likely @ 2010-07-25 7:42 UTC (permalink / raw) To: Anatolij Gustschin Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Fri, Jul 23, 2010 at 7:49 AM, Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> wrote: > Adds IFM PDM360NG device tree, and platform code. > > Currently following is supported: > - Spansion S29GL512P 256 MB NOR flash > - ST Micro NAND 1 GiB flash > - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" > at the kernel command line to enable PrimeView PM070WL3 > Display support. > - FEC > - I2C > - RTC, EEPROM > - MSCAN > - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" > on the kernel command line. > - SPI, ADS7845 Touchscreen > - USB0/1 Host > - USB0 OTG Host/Device > - VIU, Overlay/Capture support > > Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> > Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> > Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> > Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> > Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> > Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org > Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> Hi Anatolij, Finally got some time tonight to properly dig into this patch. Comments below. > --- > v3: > - uncomment and correct .irq_flags field of touchscreen > platform data struct as proposed extension to this > data struct has been accepted and merged via input tree > a short while ago > > v1 -> v2: > - fix interrupt-parent property in nfc node > - drop #address-cells in ipic node > - remove device_type from ethernet-phy sub-node > - remove device_type from ethernet node > - add aliases node for eth0, needed for MAC address > update by U-Boot > - removed spaces around &ipic > > arch/powerpc/boot/dts/pdm360ng.dts | 432 ++++++++++++++++++++++++++++++++ > arch/powerpc/platforms/512x/Kconfig | 7 + > arch/powerpc/platforms/512x/Makefile | 1 + > arch/powerpc/platforms/512x/pdm360ng.c | 158 ++++++++++++ > 4 files changed, 598 insertions(+), 0 deletions(-) > create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts > create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c > > diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts > new file mode 100644 > index 0000000..7dc42e9 > --- /dev/null > +++ b/arch/powerpc/boot/dts/pdm360ng.dts > @@ -0,0 +1,432 @@ > +/* > + * Device Tree Source for IFM PDM360NG. > + * > + * Copyright 2009 - 2010 DENX Software Engineering. > + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> > + * > + * Based on MPC5121E ADS dts. > + * Copyright 2008 Freescale Semiconductor Inc. > + * > + * 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. > + */ > + > +/dts-v1/; > + > +/ { > + model = "pdm360ng"; > + compatible = "ifm,pdm360ng"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + aliases { > + ethernet0 = ð0; > + }; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + > + PowerPC,5121@0 { > + device_type = "cpu"; > + reg = <0>; > + d-cache-line-size = <0x20>; // 32 bytes > + i-cache-line-size = <0x20>; // 32 bytes > + d-cache-size = <0x8000>; // L1, 32K > + i-cache-size = <0x8000>; // L1, 32K > + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) > + bus-frequency = <198000000>; // 198 MHz csb bus > + clock-frequency = <396000000>; // 396 MHz ppc core > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x00000000 0x20000000>; // 512MB at 0 > + }; > + > + nfc@40000000 { > + compatible = "fsl,mpc5121-nfc"; > + reg = <0x40000000 0x100000>; > + interrupts = <0x6 0x8>; > + interrupt-parent = <&ipic>; This device tree can be less verbose if you remove the interrupt-parent property from all the nodes, and have a single interrupt-parent = <&ipic>; in the root node. Nodes inherit the interrupt-parent from their (grand-)parents. > + #address-cells = <0x1>; > + #size-cells = <0x1>; > + bank-width = <0x1>; > + chips = <0x1>; > + > + partition@0 { > + label = "nand0"; > + reg = <0x0 0x40000000>; > + }; > + }; > + > + sram@50000000 { > + compatible = "fsl,mpc5121-sram"; > + reg = <0x50000000 0x20000>; // 128K at 0x50000000 > + }; > + > + localbus@80000020 { > + compatible = "fsl,mpc5121-localbus"; > + #address-cells = <2>; > + #size-cells = <1>; > + reg = <0x80000020 0x40>; > + > + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ > + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ > + > + flash@0,0 { > + compatible = "amd,s29gl01gp", "cfi-flash"; > + reg = <0 0x00000000 0x08000000 > + 0 0x08000000 0x08000000>; > + #address-cells = <1>; > + #size-cells = <1>; > + bank-width = <4>; > + device-width = <2>; > + > + partition@0 { > + label = "u-boot"; > + reg = <0x00000000 0x00080000>; > + read-only; > + }; > + partition@80000 { > + label = "environment"; > + reg = <0x00080000 0x00080000>; > + read-only; > + }; > + partition@100000 { > + label = "splash-image"; > + reg = <0x00100000 0x00080000>; > + read-only; > + }; > + partition@180000 { > + label = "device-tree"; > + reg = <0x00180000 0x00040000>; > + }; > + partition@1c0000 { > + label = "kernel"; > + reg = <0x001c0000 0x00500000>; > + }; > + partition@6c0000 { > + label = "filesystem"; > + reg = <0x006c0000 0x07940000>; > + }; > + }; > + > + mram0@2,0 { > + compatible = "mtd-ram"; > + reg = <2 0x00000 0x10000>; > + bank-width = <2>; > + }; > + > + mram1@2,10000 { > + compatible = "mtd-ram"; > + reg = <2 0x010000 0x10000>; > + bank-width = <2>; > + }; > + }; > + > + soc@80000000 { > + compatible = "fsl,mpc5121-immr"; > + #address-cells = <1>; > + #size-cells = <1>; > + #interrupt-cells = <2>; > + ranges = <0x0 0x80000000 0x400000>; > + reg = <0x80000000 0x400000>; > + bus-frequency = <66000000>; // 66 MHz ips bus > + > + // IPIC > + // interrupts cell = <intr #, sense> > + // sense values match linux IORESOURCE_IRQ_* defines: > + // sense == 8: Level, low assertion > + // sense == 2: Edge, high-to-low change > + // > + ipic: interrupt-controller@c00 { > + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; > + interrupt-controller; > + #address-cells = <0>; > + #interrupt-cells = <2>; > + reg = <0xc00 0x100>; > + }; > + > + rtc@a00 { // Real time clock > + compatible = "fsl,mpc5121-rtc"; > + reg = <0xa00 0x100>; > + interrupts = <79 0x8 80 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + reset@e00 { // Reset module > + compatible = "fsl,mpc5121-reset"; > + reg = <0xe00 0x100>; > + }; > + > + clock@f00 { // Clock control > + compatible = "fsl,mpc5121-clock"; > + reg = <0xf00 0x100>; > + }; > + > + pmc@1000{ //Power Management Controller > + compatible = "fsl,mpc5121-pmc"; > + reg = <0x1000 0x100>; > + interrupts = <83 0x2>; > + interrupt-parent = <&ipic>; > + }; > + > + gpio@1100 { > + compatible = "fsl,mpc5121-gpio"; > + reg = <0x1100 0x100>; > + interrupts = <78 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + can@1300 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <12 0x8>; > + interrupt-parent = <&ipic>; > + reg = <0x1300 0x80>; > + }; > + > + can@1380 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <13 0x8>; > + interrupt-parent = <&ipic>; > + reg = <0x1380 0x80>; > + }; > + > + i2c@1700 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "fsl,mpc5121-i2c"; > + reg = <0x1700 0x20>; > + interrupts = <0x9 0x8>; > + interrupt-parent = <&ipic>; > + fsl,preserve-clocking; > + > + eeprom@50 { > + compatible = "at,24c01"; > + reg = <0x50>; > + }; > + > + rtc@68 { > + compatible = "stm,m41t00"; > + reg = <0x68>; > + }; > + }; > + > + i2c@1740 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "fsl,mpc5121-i2c"; > + reg = <0x1740 0x20>; > + interrupts = <0xb 0x8>; > + interrupt-parent = <&ipic>; > + fsl,preserve-clocking; > + }; > + > + i2ccontrol@1760 { > + compatible = "fsl,mpc5121-i2c-ctrl"; > + reg = <0x1760 0x8>; > + }; > + > + axe@2000 { > + compatible = "fsl,mpc5121-axe"; > + reg = <0x2000 0x100>; > + interrupts = <42 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + display@2100 { > + compatible = "fsl,mpc5121-diu"; > + reg = <0x2100 0x100>; > + interrupts = <64 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + can@2300 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <90 0x8>; > + interrupt-parent = <&ipic>; > + reg = <0x2300 0x80>; > + }; > + > + can@2380 { > + compatible = "fsl,mpc5121-mscan"; > + interrupts = <91 0x8>; > + interrupt-parent = <&ipic>; > + reg = <0x2380 0x80>; > + }; > + > + viu@2400 { > + compatible = "fsl,mpc5121-viu"; > + reg = <0x2400 0x400>; > + interrupts = <67 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + mdio@2800 { > + compatible = "fsl,mpc5121-fec-mdio"; > + reg = <0x2800 0x200>; > + #address-cells = <1>; > + #size-cells = <0>; > + phy: ethernet-phy@0 { > + reg = <0x1f>; For completeness, phy should have a compatible property. > + }; > + }; > + > + eth0: ethernet@2800 { > + compatible = "fsl,mpc5121-fec"; > + reg = <0x2800 0x200>; > + local-mac-address = [ 00 00 00 00 00 00 ]; > + interrupts = <4 0x8>; > + interrupt-parent = <&ipic>; > + phy-handle = < &phy >; > + }; > + > + // USB1 using external ULPI PHY > + usb@3000 { > + compatible = "fsl,mpc5121-usb2-dr"; > + reg = <0x3000 0x600>; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupt-parent = <&ipic>; > + interrupts = <43 0x8>; > + dr_mode = "host"; > + phy_type = "ulpi"; > + }; > + > + // USB0 using internal UTMI PHY > + usb@4000 { > + compatible = "fsl,mpc5121-usb2-dr"; > + reg = <0x4000 0x600>; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupt-parent = <&ipic>; > + interrupts = <44 0x8>; > + dr_mode = "otg"; > + phy_type = "utmi_wide"; > + fsl,invert-pwr-fault; > + }; > + > + // IO control > + ioctl@a000 { > + compatible = "fsl,mpc5121-ioctl"; > + reg = <0xA000 0x1000>; > + }; > + > + // 512x PSCs are not 52xx PSCs compatible > + serial@11000 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <0>; > + reg = <0x11000 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11100 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <1>; > + reg = <0x11100 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11200 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <2>; > + reg = <0x11200 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11300 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <3>; > + reg = <0x11300 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11400 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <4>; > + reg = <0x11400 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11600 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <6>; > + reg = <0x11600 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11800 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <8>; > + reg = <0x11800 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + serial@11B00 { > + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; > + cell-index = <11>; > + reg = <0x11B00 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + }; > + > + pscfifo@11f00 { > + compatible = "fsl,mpc5121-psc-fifo"; > + reg = <0x11f00 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + }; > + > + spi@11900 { > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; > + cell-index = <9>; Try to drop the cell-index properties. They are almost always misused. > + reg = <0x11900 0x100>; > + interrupts = <40 0x8>; > + interrupt-parent = <&ipic>; > + fsl,rx-fifo-size = <16>; > + fsl,tx-fifo-size = <16>; > + > + // 7845 touch screen controller > + ts@0 { > + compatible = "ti,ads7845"; > + interrupt-parent = <&ipic>; > + // pen irq is GPIO25 > + interrupts = <78 0x8>; > + }; > + }; > + > + dma@14000 { > + compatible = "fsl,mpc5121-dma"; > + reg = <0x14000 0x1800>; > + interrupts = <65 0x8>; > + interrupt-parent = <&ipic>; > + }; > + }; > +}; > diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig > index e9dca28..27b0651 100644 > --- a/arch/powerpc/platforms/512x/Kconfig > +++ b/arch/powerpc/platforms/512x/Kconfig > @@ -25,3 +25,10 @@ config MPC5121_GENERIC > > Compatible boards include: Protonic LVT base boards (ZANMCU > and VICVT2). > + > +config PDM360NG > + bool "ifm PDM360NG board" > + depends on PPC_MPC512x > + select DEFAULT_UIMAGE > + help > + This option enables support for the PDM360NG board. > diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile > index 90be2f5..4efc1c4 100644 > --- a/arch/powerpc/platforms/512x/Makefile > +++ b/arch/powerpc/platforms/512x/Makefile > @@ -4,3 +4,4 @@ > obj-y += clock.o mpc512x_shared.o > obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o > obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o > +obj-$(CONFIG_PDM360NG) += pdm360ng.o > diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c > new file mode 100644 > index 0000000..01e6023 > --- /dev/null > +++ b/arch/powerpc/platforms/512x/pdm360ng.c > @@ -0,0 +1,158 @@ > +/* > + * Copyright (C) 2010 DENX Software Engineering > + * > + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> > + * > + * PDM360NG board setup > + * > + * This 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. > + * > + */ > + > +#include <linux/kernel.h> > +#include <linux/io.h> > +#include <linux/of_platform.h> > + > +#include <asm/machdep.h> > +#include <asm/ipic.h> > +#include <asm/prom.h> > + > +#include "mpc512x.h" > + > +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ > + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) > +#include <linux/fsl_devices.h> > +#include <linux/interrupt.h> > +#include <linux/spi/ads7846.h> > +#include <linux/spi/spi.h> > + > +static void *pdm360ng_gpio_base; > + > +static int pdm360ng_get_pendown_state(void) > +{ > + u32 reg; > + > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); > + if (reg & 0x40) > + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); > + > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); (u32*) casts are unnecessary and can be removed. > + > + /* return 1 if pen is down */ > + return reg & 0x40 ? 0 : 1; return reg & 0x40 == 0; > +} > + > +static struct ads7846_platform_data pdm360ng_ads7846_pdata __initdata = { > + .model = 7845, > + .get_pendown_state = pdm360ng_get_pendown_state, > + .irq_flags = IRQF_TRIGGER_LOW, > +}; > + > +static int __init pdm360ng_penirq_init(void) > +{ > + struct device_node *np; > + struct resource r; > + > + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); > + if (!np) { > + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); > + return -1; > + } return -ENODEV; > + > + if (of_address_to_resource(np, 0, &r)) { > + pr_err("%s: Invalid gpio address.\n", __func__); > + of_node_put(np); > + return -1; ditto > + } > + of_node_put(np); > + > + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); Or you could have simply used of_iomap() to eliminate some code. > + if (!pdm360ng_gpio_base) { > + pr_err("%s: Can't map gpio regs.\n", __func__); > + return -1; ditto > + } > + out_be32((u32 *)pdm360ng_gpio_base + 0xc, 0xffffffff); > + setbits32((u32 *)(pdm360ng_gpio_base + 0x18), 0x2000); > + setbits32((u32 *)(pdm360ng_gpio_base + 0x10), 0x40); > + > + return 0; > +} > + > +static int __init pdm360ng_touchscreen_init(void) > +{ > + struct device_node *np; > + struct of_device *of_dev; > + struct spi_board_info info; > + const u32 *prop; > + int bus_num = -1; > + int len; > + > + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); > + if (!np) > + return -ENODEV; > + > + memset(&info, 0, sizeof(info)); > + info.irq = irq_of_parse_and_map(np, 0); > + if (info.irq == NO_IRQ) > + return -ENODEV; > + > + info.platform_data = &pdm360ng_ads7846_pdata; > + if (strlcpy(info.modalias, "ads7846", > + SPI_NAME_SIZE) >= SPI_NAME_SIZE) > + return -ENOMEM; > + > + np = of_get_next_parent(np); > + if (!np) > + return -ENODEV; > + > + prop = of_get_property(np, "cell-index", &len); > + if (prop && len == 4) > + bus_num = *prop; Blech. Don't use cell-index for bus enumeration. In fact, none of this should be necessary at all (see below). > + > + if (bus_num < 0 || bus_num > 11) > + return -ENODEV; > + > + info.bus_num = bus_num; > + > + of_dev = of_find_device_by_node(np); > + of_node_put(np); > + if (of_dev) { > + struct fsl_spi_platform_data *pdata; > + > + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); > + if (pdata) { > + pdata->bus_num = bus_num; > + pdata->max_chipselect = 1; > + of_dev->dev.platform_data = pdata; > + } > + } > + > + if (pdm360ng_penirq_init()) > + return -ENODEV; > + > + return spi_register_board_info(&info, 1); This ends up being a lot of code simply to attach a pdata structure to an spi device. I've been thinking about this problem, and I think there is a better way. Instead, use a bus notifier attached to the SPI bus to wait for the spi_device to get registered, but before it gets bound to a driver. Then you can attach the pdata structure very simply. Something like this I think (completely untested, or even compiled. Details are left as an exercise to the developer): static int pdm360ng_touchscreen_notifier_call(struct notifier_block *nb, unsigned long event, void *__dev) { struct device *dev = __dev; if ((event == BUS_NOTIFIY_ADD_DEVICE) && (dev->of_node == [FOO])) dev->platform_data = [BAR]; } static struct notifier_block pdm360ng_touchscreen_nb = { notifier_call = pdm360ng_touchscreen_notifier_call; }; static int __init pdm360ng_touchscreen_init(void) { bus_register_notifier(&spi_bus_type, pdm360ng_touchscreen_nb); } > +} > +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); This code is *in the same file* as the board setup file. Don't use an initcall. Call it from the init callback instead. > +#endif > + > +static int __init pdm360ng_probe(void) > +{ > + unsigned long root = of_get_flat_dt_root(); > + > + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); > +} > + > +define_machine(pdm360ng) { > + .name = "PDM360NG", > + .probe = pdm360ng_probe, > + .setup_arch = mpc512x_setup_diu, > + .init = mpc512x_init, > + .init_early = mpc512x_init_diu, > + .init_IRQ = mpc512x_init_IRQ, > + .get_irq = ipic_get_irq, > + .calibrate_decr = generic_calibrate_decr, > + .restart = mpc512x_restart, > +}; > -- > 1.7.0.4 > > -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <AANLkTi=Ee6HR+Ux0g4V1+81DXUmsO=UDhi6jWsDgGchv-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <AANLkTi=Ee6HR+Ux0g4V1+81DXUmsO=UDhi6jWsDgGchv-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2010-07-27 10:36 ` Anatolij Gustschin 2010-07-27 16:58 ` Grant Likely 0 siblings, 1 reply; 16+ messages in thread From: Anatolij Gustschin @ 2010-07-27 10:36 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger Hi Grant, On Sun, 25 Jul 2010 01:42:23 -0600 Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> wrote: ... > Hi Anatolij, > > Finally got some time tonight to properly dig into this patch. Comments below. Thanks for review and comments! My reply below. ... > > + nfc@40000000 { > > + compatible = "fsl,mpc5121-nfc"; > > + reg = <0x40000000 0x100000>; > > + interrupts = <0x6 0x8>; > > + interrupt-parent = <&ipic>; > > This device tree can be less verbose if you remove the > interrupt-parent property from all the nodes, and have a single > interrupt-parent = <&ipic>; in the root node. Nodes inherit the > interrupt-parent from their (grand-)parents. Fixed in next patch version to use interrupt-parent in root node only. ... > > + mdio@2800 { > > + compatible = "fsl,mpc5121-fec-mdio"; > > + reg = <0x2800 0x200>; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + phy: ethernet-phy@0 { > > + reg = <0x1f>; > > For completeness, phy should have a compatible property. Added in next patch version. ... > > + spi@11900 { > > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; > > + cell-index = <9>; > > Try to drop the cell-index properties. They are almost always misused. Removing cell-index would require changing the spi driver's probe. Currently cell-index is used to set spi bus number. What could be used for bus enumeration instead? Is it okay to use part of the spi node address? e.g. obtaining the offset 0x11900, masking out the unrelated bits and shifting by 8 would deliver unique index 9 for PSC9 in SPI mode. This would work for all 12 PSC SPI controllers of mpc5121. ... > > +static int pdm360ng_get_pendown_state(void) > > +{ > > + u32 reg; > > + > > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0xc)); > > + if (reg & 0x40) > > + setbits32((u32 *)(pdm360ng_gpio_base + 0xc), 0x40); > > + > > + reg = in_be32((u32 *)(pdm360ng_gpio_base + 0x8)); > > (u32*) casts are unnecessary and can be removed. Fixed. > > + > > + /* return 1 if pen is down */ > > + return reg & 0x40 ? 0 : 1; > > return reg & 0x40 == 0; Fixed. ... > > +static int __init pdm360ng_penirq_init(void) > > +{ > > + struct device_node *np; > > + struct resource r; > > + > > + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); > > + if (!np) { > > + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); > > + return -1; > > + } > > return -ENODEV; Ok, fixed also. ... > > + pdm360ng_gpio_base = ioremap(r.start, resource_size(&r)); > > Or you could have simply used of_iomap() to eliminate some code. of_iomap() is used in next patch version. ... > > +static int __init pdm360ng_touchscreen_init(void) > > +{ > > + struct device_node *np; > > + struct of_device *of_dev; > > + struct spi_board_info info; > > + const u32 *prop; > > + int bus_num = -1; > > + int len; > > + > > + np = of_find_compatible_node(NULL, NULL, "ti,ads7845"); > > + if (!np) > > + return -ENODEV; > > + > > + memset(&info, 0, sizeof(info)); > > + info.irq = irq_of_parse_and_map(np, 0); > > + if (info.irq == NO_IRQ) > > + return -ENODEV; > > + > > + info.platform_data = &pdm360ng_ads7846_pdata; > > + if (strlcpy(info.modalias, "ads7846", > > + SPI_NAME_SIZE) >= SPI_NAME_SIZE) > > + return -ENOMEM; > > + > > + np = of_get_next_parent(np); > > + if (!np) > > + return -ENODEV; > > + > > + prop = of_get_property(np, "cell-index", &len); > > + if (prop && len == 4) > > + bus_num = *prop; > > Blech. Don't use cell-index for bus enumeration. In fact, none of > this should be necessary at all (see below). I dropped this code entirely in the next patch version. > > + > > + if (bus_num < 0 || bus_num > 11) > > + return -ENODEV; > > + > > + info.bus_num = bus_num; > > + > > + of_dev = of_find_device_by_node(np); > > + of_node_put(np); > > + if (of_dev) { > > + struct fsl_spi_platform_data *pdata; > > + > > + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); > > + if (pdata) { > > + pdata->bus_num = bus_num; > > + pdata->max_chipselect = 1; > > + of_dev->dev.platform_data = pdata; > > + } > > + } > > + > > + if (pdm360ng_penirq_init()) > > + return -ENODEV; > > + > > + return spi_register_board_info(&info, 1); > > This ends up being a lot of code simply to attach a pdata structure to > an spi device. I've been thinking about this problem, and I think > there is a better way. Instead, use a bus notifier attached to the > SPI bus to wait for the spi_device to get registered, but before it > gets bound to a driver. Then you can attach the pdata structure very > simply. Something like this I think (completely untested, or even > compiled. Details are left as an exercise to the developer): > > static int pdm360ng_touchscreen_notifier_call(struct notifier_block *nb, > unsigned long event, void *__dev) > { > struct device *dev = __dev; > > if ((event == BUS_NOTIFIY_ADD_DEVICE) && (dev->of_node == [FOO])) > dev->platform_data = [BAR]; > } > > static struct notifier_block pdm360ng_touchscreen_nb = { > notifier_call = pdm360ng_touchscreen_notifier_call; > }; > > static int __init pdm360ng_touchscreen_init(void) > { > bus_register_notifier(&spi_bus_type, pdm360ng_touchscreen_nb); > } Thanks! Bus notifier is now used for setting platform data in v4. This requires fixing the mpc5121 psc spi driver to create spi child nodes of the spi master node. I have already send the appropriate patch to spi-devel list, but it is not the right approach to call of_register_spi_devices() in each driver. Do you plan to fix it in core spi code in v2.6.36? ... > > +} > > +machine_device_initcall(pdm360ng, pdm360ng_touchscreen_init); > > This code is *in the same file* as the board setup file. Don't use an > initcall. Call it from the init callback instead. OK, initcall dropped. Thanks, Anatolij ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board 2010-07-27 10:36 ` Anatolij Gustschin @ 2010-07-27 16:58 ` Grant Likely [not found] ` <AANLkTika=KeKcaY_FF8tN5Fy-H+cozjZsGObXoQhKiVh-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 16+ messages in thread From: Grant Likely @ 2010-07-27 16:58 UTC (permalink / raw) To: Anatolij Gustschin Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Tue, Jul 27, 2010 at 4:36 AM, Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> wrote: > Hi Grant, > > On Sun, 25 Jul 2010 01:42:23 -0600 > Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> wrote: > ... >> Hi Anatolij, >> >> Finally got some time tonight to properly dig into this patch. Comments below. > > Thanks for review and comments! My reply below. Thanks Anatolij, replies below. >> > + spi@11900 { >> > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; >> > + cell-index = <9>; >> >> Try to drop the cell-index properties. They are almost always misused. > > Removing cell-index would require changing the spi driver's probe. > Currently cell-index is used to set spi bus number. What could be used > for bus enumeration instead? Is it okay to use part of the spi node > address? e.g. obtaining the offset 0x11900, masking out the unrelated > bits and shifting by 8 would deliver unique index 9 for PSC9 in SPI > mode. This would work for all 12 PSC SPI controllers of mpc5121. Does the spi bus number really matter? The device tree context gives you a firm association between spi masters and devices which doesn't require assigning a specific bus number. The core spi code can dynamically assign a bus number for the bus by setting bus_num to -1. >> > + >> > + if (bus_num < 0 || bus_num > 11) >> > + return -ENODEV; >> > + >> > + info.bus_num = bus_num; >> > + >> > + of_dev = of_find_device_by_node(np); >> > + of_node_put(np); >> > + if (of_dev) { >> > + struct fsl_spi_platform_data *pdata; >> > + >> > + pdata = kzalloc(sizeof(*pdata), GFP_KERNEL); >> > + if (pdata) { >> > + pdata->bus_num = bus_num; >> > + pdata->max_chipselect = 1; >> > + of_dev->dev.platform_data = pdata; >> > + } >> > + } >> > + >> > + if (pdm360ng_penirq_init()) >> > + return -ENODEV; >> > + >> > + return spi_register_board_info(&info, 1); >> >> This ends up being a lot of code simply to attach a pdata structure to >> an spi device. I've been thinking about this problem, and I think >> there is a better way. Instead, use a bus notifier attached to the >> SPI bus to wait for the spi_device to get registered, but before it >> gets bound to a driver. Then you can attach the pdata structure very >> simply. Something like this I think (completely untested, or even >> compiled. Details are left as an exercise to the developer): >> >> static int pdm360ng_touchscreen_notifier_call(struct notifier_block *nb, >> unsigned long event, void *__dev) >> { >> struct device *dev = __dev; >> >> if ((event == BUS_NOTIFIY_ADD_DEVICE) && (dev->of_node == [FOO])) >> dev->platform_data = [BAR]; >> } >> >> static struct notifier_block pdm360ng_touchscreen_nb = { >> notifier_call = pdm360ng_touchscreen_notifier_call; >> }; >> >> static int __init pdm360ng_touchscreen_init(void) >> { >> bus_register_notifier(&spi_bus_type, pdm360ng_touchscreen_nb); >> } > > Thanks! Bus notifier is now used for setting platform data in v4. Cool! > This requires fixing the mpc5121 psc spi driver to create spi child > nodes of the spi master node. I have already send the appropriate > patch to spi-devel list, but it is not the right approach to call > of_register_spi_devices() in each driver. It's not wrong; but it isn't ideal either. > Do you plan to fix it in > core spi code in v2.6.36? Apparently I no longer have to because you've gone ahead and done it for me anyway. :-) I'll take a look at that patch and send you my comments. Cheers, g. ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <AANLkTika=KeKcaY_FF8tN5Fy-H+cozjZsGObXoQhKiVh-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <AANLkTika=KeKcaY_FF8tN5Fy-H+cozjZsGObXoQhKiVh-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2010-07-27 17:28 ` Anatolij Gustschin 2010-07-27 17:43 ` Grant Likely 0 siblings, 1 reply; 16+ messages in thread From: Anatolij Gustschin @ 2010-07-27 17:28 UTC (permalink / raw) To: Grant Likely Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Tue, 27 Jul 2010 10:58:33 -0600 Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> wrote: ... > >> > + spi@11900 { > >> > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; > >> > + cell-index = <9>; > >> > >> Try to drop the cell-index properties. They are almost always misused. > > > > Removing cell-index would require changing the spi driver's probe. > > Currently cell-index is used to set spi bus number. What could be used > > for bus enumeration instead? Is it okay to use part of the spi node > > address? e.g. obtaining the offset 0x11900, masking out the unrelated > > bits and shifting by 8 would deliver unique index 9 for PSC9 in SPI > > mode. This would work for all 12 PSC SPI controllers of mpc5121. > > Does the spi bus number really matter? The device tree context gives > you a firm association between spi masters and devices which doesn't > require assigning a specific bus number. The core spi code can > dynamically assign a bus number for the bus by setting bus_num to -1. The bus number is used in the mpc5121 psc spi driver to obtain correct clock for PSC in question (0 to 11) and to enable the PSC clock at probe time. Therefore using dynamically assigned bus number would require another change to the spi driver. ... > > This requires fixing the mpc5121 psc spi driver to create spi child > > nodes of the spi master node. I have already send the appropriate > > patch to spi-devel list, but it is not the right approach to call > > of_register_spi_devices() in each driver. > > It's not wrong; but it isn't ideal either. > > > Do you plan to fix it in > > core spi code in v2.6.36? > > Apparently I no longer have to because you've gone ahead and done it > for me anyway. :-) I'll take a look at that patch and send you my > comments. Okay, thanks, Anatolij ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 2/2] powerpc/mpc5121: add initial support for PDM360NG board 2010-07-27 17:28 ` Anatolij Gustschin @ 2010-07-27 17:43 ` Grant Likely 0 siblings, 0 replies; 16+ messages in thread From: Grant Likely @ 2010-07-27 17:43 UTC (permalink / raw) To: Anatolij Gustschin Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A, Wolfgang Grandegger On Tue, Jul 27, 2010 at 11:28 AM, Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> wrote: > On Tue, 27 Jul 2010 10:58:33 -0600 > Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> wrote: > ... >> >> > + spi@11900 { >> >> > + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; >> >> > + cell-index = <9>; >> >> >> >> Try to drop the cell-index properties. They are almost always misused. >> > >> > Removing cell-index would require changing the spi driver's probe. >> > Currently cell-index is used to set spi bus number. What could be used >> > for bus enumeration instead? Is it okay to use part of the spi node >> > address? e.g. obtaining the offset 0x11900, masking out the unrelated >> > bits and shifting by 8 would deliver unique index 9 for PSC9 in SPI >> > mode. This would work for all 12 PSC SPI controllers of mpc5121. >> >> Does the spi bus number really matter? The device tree context gives >> you a firm association between spi masters and devices which doesn't >> require assigning a specific bus number. The core spi code can >> dynamically assign a bus number for the bus by setting bus_num to -1. > > The bus number is used in the mpc5121 psc spi driver to obtain correct > clock for PSC in question (0 to 11) and to enable the PSC clock at probe > time. Therefore using dynamically assigned bus number would require another > change to the spi driver. That's unrelated to the bus number. Use cell-index value directly for obtaining the clock if you need to; but limit its exposure. Once Jeremy gets his common clock architecture merged, then we could probably migrate to that for obtaining the correct clock without cell-index. g. ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v4 2/2] powerpc/mpc5121: add initial support for PDM360NG board [not found] ` <1279892973-24110-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org> 2010-07-25 7:42 ` Grant Likely @ 2010-07-27 21:26 ` Anatolij Gustschin 1 sibling, 0 replies; 16+ messages in thread From: Anatolij Gustschin @ 2010-07-27 21:26 UTC (permalink / raw) To: linuxppc-dev-mnsaURCQ41sdnm+yROfE0A Cc: Detlev Zundel, Markus Fischer, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, Michael Weiss, Wolfgang Grandegger Adds IFM PDM360NG device tree and platform code. Currently following is supported: - Spansion S29GL512P 256 MB NOR flash - ST Micro NAND 1 GiB flash - DIU, please use "fbcon=map:5 video=fslfb:800x480-32@60" at the kernel command line to enable PrimeView PM070WL3 Display support. - FEC - I2C - RTC, EEPROM - MSCAN - PSC UART, please pass "console=tty0 console=ttyPSC5,115200" on the kernel command line. - SPI, ADS7845 Touchscreen - USB0/1 Host - USB0 OTG Host/Device - VIU, Overlay/Capture support Signed-off-by: Markus Fischer <markus.fischer.ec-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> Signed-off-by: Michael Weiss <michael.weiss-Jj5Fu8i2Z9Q@public.gmane.org> Signed-off-by: Detlev Zundel <dzu-ynQEQJNshbs@public.gmane.org> Signed-off-by: Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> --- v4: - set interrupt-parent in root node only, less verbose DT - add compatible property to phy node - update touch screen controller node (add reg and spi-max-frequency properties) - drop unnecessary (u32 *) casts - change return statement in pdm360ng_get_pendown_state() as suggested - use correct error codes (-ENODEV) when returning on errors - use of_iomap() to eliminate some code - rework pdm360ng_touchscreen_init() to only use bus notifier for addition of platform data for touch screen driver - eliminate machine_device_initcall(), code moved to init callback v3: - uncomment and correct .irq_flags field of touchscreen platform data struct as proposed extension to this data struct has been accepted and merged via input tree a short while ago v1 -> v2: - fix interrupt-parent property in nfc node - drop #address-cells in ipic node - remove device_type from ethernet-phy sub-node - remove device_type from ethernet node - add aliases node for eth0, needed for MAC address update by U-Boot - removed spaces around &ipic arch/powerpc/boot/dts/pdm360ng.dts | 410 ++++++++++++++++++++++++++++++++ arch/powerpc/platforms/512x/Kconfig | 7 + arch/powerpc/platforms/512x/Makefile | 1 + arch/powerpc/platforms/512x/pdm360ng.c | 129 ++++++++++ 4 files changed, 547 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/boot/dts/pdm360ng.dts create mode 100644 arch/powerpc/platforms/512x/pdm360ng.c diff --git a/arch/powerpc/boot/dts/pdm360ng.dts b/arch/powerpc/boot/dts/pdm360ng.dts new file mode 100644 index 0000000..94dfa5c --- /dev/null +++ b/arch/powerpc/boot/dts/pdm360ng.dts @@ -0,0 +1,410 @@ +/* + * Device Tree Source for IFM PDM360NG. + * + * Copyright 2009 - 2010 DENX Software Engineering. + * Anatolij Gustschin <agust-ynQEQJNshbs@public.gmane.org> + * + * Based on MPC5121E ADS dts. + * Copyright 2008 Freescale Semiconductor Inc. + * + * 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. + */ + +/dts-v1/; + +/ { + model = "pdm360ng"; + compatible = "ifm,pdm360ng"; + #address-cells = <1>; + #size-cells = <1>; + interrupt-parent = <&ipic>; + + aliases { + ethernet0 = ð0; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + PowerPC,5121@0 { + device_type = "cpu"; + reg = <0>; + d-cache-line-size = <0x20>; // 32 bytes + i-cache-line-size = <0x20>; // 32 bytes + d-cache-size = <0x8000>; // L1, 32K + i-cache-size = <0x8000>; // L1, 32K + timebase-frequency = <49500000>;// 49.5 MHz (csb/4) + bus-frequency = <198000000>; // 198 MHz csb bus + clock-frequency = <396000000>; // 396 MHz ppc core + }; + }; + + memory { + device_type = "memory"; + reg = <0x00000000 0x20000000>; // 512MB at 0 + }; + + nfc@40000000 { + compatible = "fsl,mpc5121-nfc"; + reg = <0x40000000 0x100000>; + interrupts = <0x6 0x8>; + #address-cells = <0x1>; + #size-cells = <0x1>; + bank-width = <0x1>; + chips = <0x1>; + + partition@0 { + label = "nand0"; + reg = <0x0 0x40000000>; + }; + }; + + sram@50000000 { + compatible = "fsl,mpc5121-sram"; + reg = <0x50000000 0x20000>; // 128K at 0x50000000 + }; + + localbus@80000020 { + compatible = "fsl,mpc5121-localbus"; + #address-cells = <2>; + #size-cells = <1>; + reg = <0x80000020 0x40>; + + ranges = <0x0 0x0 0xf0000000 0x10000000 /* Flash */ + 0x2 0x0 0x50040000 0x00020000>; /* CS2: MRAM */ + + flash@0,0 { + compatible = "amd,s29gl01gp", "cfi-flash"; + reg = <0 0x00000000 0x08000000 + 0 0x08000000 0x08000000>; + #address-cells = <1>; + #size-cells = <1>; + bank-width = <4>; + device-width = <2>; + + partition@0 { + label = "u-boot"; + reg = <0x00000000 0x00080000>; + read-only; + }; + partition@80000 { + label = "environment"; + reg = <0x00080000 0x00080000>; + read-only; + }; + partition@100000 { + label = "splash-image"; + reg = <0x00100000 0x00080000>; + read-only; + }; + partition@180000 { + label = "device-tree"; + reg = <0x00180000 0x00040000>; + }; + partition@1c0000 { + label = "kernel"; + reg = <0x001c0000 0x00500000>; + }; + partition@6c0000 { + label = "filesystem"; + reg = <0x006c0000 0x07940000>; + }; + }; + + mram0@2,0 { + compatible = "mtd-ram"; + reg = <2 0x00000 0x10000>; + bank-width = <2>; + }; + + mram1@2,10000 { + compatible = "mtd-ram"; + reg = <2 0x010000 0x10000>; + bank-width = <2>; + }; + }; + + soc@80000000 { + compatible = "fsl,mpc5121-immr"; + #address-cells = <1>; + #size-cells = <1>; + #interrupt-cells = <2>; + ranges = <0x0 0x80000000 0x400000>; + reg = <0x80000000 0x400000>; + bus-frequency = <66000000>; // 66 MHz ips bus + + // IPIC + // interrupts cell = <intr #, sense> + // sense values match linux IORESOURCE_IRQ_* defines: + // sense == 8: Level, low assertion + // sense == 2: Edge, high-to-low change + // + ipic: interrupt-controller@c00 { + compatible = "fsl,mpc5121-ipic", "fsl,ipic"; + interrupt-controller; + #address-cells = <0>; + #interrupt-cells = <2>; + reg = <0xc00 0x100>; + }; + + rtc@a00 { // Real time clock + compatible = "fsl,mpc5121-rtc"; + reg = <0xa00 0x100>; + interrupts = <79 0x8 80 0x8>; + }; + + reset@e00 { // Reset module + compatible = "fsl,mpc5121-reset"; + reg = <0xe00 0x100>; + }; + + clock@f00 { // Clock control + compatible = "fsl,mpc5121-clock"; + reg = <0xf00 0x100>; + }; + + pmc@1000{ //Power Management Controller + compatible = "fsl,mpc5121-pmc"; + reg = <0x1000 0x100>; + interrupts = <83 0x2>; + }; + + gpio@1100 { + compatible = "fsl,mpc5121-gpio"; + reg = <0x1100 0x100>; + interrupts = <78 0x8>; + }; + + can@1300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <12 0x8>; + reg = <0x1300 0x80>; + }; + + can@1380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <13 0x8>; + reg = <0x1380 0x80>; + }; + + i2c@1700 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1700 0x20>; + interrupts = <0x9 0x8>; + fsl,preserve-clocking; + + eeprom@50 { + compatible = "at,24c01"; + reg = <0x50>; + }; + + rtc@68 { + compatible = "stm,m41t00"; + reg = <0x68>; + }; + }; + + i2c@1740 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5121-i2c"; + reg = <0x1740 0x20>; + interrupts = <0xb 0x8>; + fsl,preserve-clocking; + }; + + i2ccontrol@1760 { + compatible = "fsl,mpc5121-i2c-ctrl"; + reg = <0x1760 0x8>; + }; + + axe@2000 { + compatible = "fsl,mpc5121-axe"; + reg = <0x2000 0x100>; + interrupts = <42 0x8>; + }; + + display@2100 { + compatible = "fsl,mpc5121-diu"; + reg = <0x2100 0x100>; + interrupts = <64 0x8>; + }; + + can@2300 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <90 0x8>; + reg = <0x2300 0x80>; + }; + + can@2380 { + compatible = "fsl,mpc5121-mscan"; + interrupts = <91 0x8>; + reg = <0x2380 0x80>; + }; + + viu@2400 { + compatible = "fsl,mpc5121-viu"; + reg = <0x2400 0x400>; + interrupts = <67 0x8>; + }; + + mdio@2800 { + compatible = "fsl,mpc5121-fec-mdio"; + reg = <0x2800 0x200>; + #address-cells = <1>; + #size-cells = <0>; + phy: ethernet-phy@0 { + compatible = "smsc,lan8700"; + reg = <0x1f>; + }; + }; + + eth0: ethernet@2800 { + compatible = "fsl,mpc5121-fec"; + reg = <0x2800 0x200>; + local-mac-address = [ 00 00 00 00 00 00 ]; + interrupts = <4 0x8>; + phy-handle = < &phy >; + }; + + // USB1 using external ULPI PHY + usb@3000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x3000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupts = <43 0x8>; + dr_mode = "host"; + phy_type = "ulpi"; + }; + + // USB0 using internal UTMI PHY + usb@4000 { + compatible = "fsl,mpc5121-usb2-dr"; + reg = <0x4000 0x600>; + #address-cells = <1>; + #size-cells = <0>; + interrupts = <44 0x8>; + dr_mode = "otg"; + phy_type = "utmi_wide"; + fsl,invert-pwr-fault; + }; + + // IO control + ioctl@a000 { + compatible = "fsl,mpc5121-ioctl"; + reg = <0xA000 0x1000>; + }; + + // 512x PSCs are not 52xx PSCs compatible + serial@11000 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <0>; + reg = <0x11000 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11100 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <1>; + reg = <0x11100 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11200 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <2>; + reg = <0x11200 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11300 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <3>; + reg = <0x11300 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11400 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <4>; + reg = <0x11400 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11600 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <6>; + reg = <0x11600 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11800 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <8>; + reg = <0x11800 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + serial@11B00 { + compatible = "fsl,mpc5121-psc-uart", "fsl,mpc5121-psc"; + cell-index = <11>; + reg = <0x11B00 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + }; + + pscfifo@11f00 { + compatible = "fsl,mpc5121-psc-fifo"; + reg = <0x11f00 0x100>; + interrupts = <40 0x8>; + }; + + spi@11900 { + compatible = "fsl,mpc5121-psc-spi", "fsl,mpc5121-psc"; + cell-index = <9>; + #address-cells = <1>; + #size-cells = <0>; + reg = <0x11900 0x100>; + interrupts = <40 0x8>; + fsl,rx-fifo-size = <16>; + fsl,tx-fifo-size = <16>; + + // 7845 touch screen controller + ts@0 { + compatible = "ti,ads7846"; + reg = <0x0>; + spi-max-frequency = <3000000>; + // pen irq is GPIO25 + interrupts = <78 0x8>; + }; + }; + + dma@14000 { + compatible = "fsl,mpc5121-dma"; + reg = <0x14000 0x1800>; + interrupts = <65 0x8>; + }; + }; +}; diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig index e9dca28..27b0651 100644 --- a/arch/powerpc/platforms/512x/Kconfig +++ b/arch/powerpc/platforms/512x/Kconfig @@ -25,3 +25,10 @@ config MPC5121_GENERIC Compatible boards include: Protonic LVT base boards (ZANMCU and VICVT2). + +config PDM360NG + bool "ifm PDM360NG board" + depends on PPC_MPC512x + select DEFAULT_UIMAGE + help + This option enables support for the PDM360NG board. diff --git a/arch/powerpc/platforms/512x/Makefile b/arch/powerpc/platforms/512x/Makefile index 90be2f5..4efc1c4 100644 --- a/arch/powerpc/platforms/512x/Makefile +++ b/arch/powerpc/platforms/512x/Makefile @@ -4,3 +4,4 @@ obj-y += clock.o mpc512x_shared.o obj-$(CONFIG_MPC5121_ADS) += mpc5121_ads.o mpc5121_ads_cpld.o obj-$(CONFIG_MPC5121_GENERIC) += mpc5121_generic.o +obj-$(CONFIG_PDM360NG) += pdm360ng.o diff --git a/arch/powerpc/platforms/512x/pdm360ng.c b/arch/powerpc/platforms/512x/pdm360ng.c new file mode 100644 index 0000000..1eb8b0f --- /dev/null +++ b/arch/powerpc/platforms/512x/pdm360ng.c @@ -0,0 +1,129 @@ +/* + * Copyright (C) 2010 DENX Software Engineering + * + * Anatolij Gustschin, <agust-ynQEQJNshbs@public.gmane.org> + * + * PDM360NG board setup + * + * This 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. + * + */ + +#include <linux/kernel.h> +#include <linux/io.h> +#include <linux/of_platform.h> + +#include <asm/machdep.h> +#include <asm/ipic.h> + +#include "mpc512x.h" + +#if defined(CONFIG_TOUCHSCREEN_ADS7846) || \ + defined(CONFIG_TOUCHSCREEN_ADS7846_MODULE) +#include <linux/interrupt.h> +#include <linux/spi/ads7846.h> +#include <linux/spi/spi.h> +#include <linux/notifier.h> + +static void *pdm360ng_gpio_base; + +static int pdm360ng_get_pendown_state(void) +{ + u32 reg; + + reg = in_be32(pdm360ng_gpio_base + 0xc); + if (reg & 0x40) + setbits32(pdm360ng_gpio_base + 0xc, 0x40); + + reg = in_be32(pdm360ng_gpio_base + 0x8); + + /* return 1 if pen is down */ + return (reg & 0x40) == 0; +} + +static struct ads7846_platform_data pdm360ng_ads7846_pdata = { + .model = 7845, + .get_pendown_state = pdm360ng_get_pendown_state, + .irq_flags = IRQF_TRIGGER_LOW, +}; + +static int __init pdm360ng_penirq_init(void) +{ + struct device_node *np; + + np = of_find_compatible_node(NULL, NULL, "fsl,mpc5121-gpio"); + if (!np) { + pr_err("%s: Can't find 'mpc5121-gpio' node\n", __func__); + return -ENODEV; + } + + pdm360ng_gpio_base = of_iomap(np, 0); + of_node_put(np); + if (!pdm360ng_gpio_base) { + pr_err("%s: Can't map gpio regs.\n", __func__); + return -ENODEV; + } + out_be32(pdm360ng_gpio_base + 0xc, 0xffffffff); + setbits32(pdm360ng_gpio_base + 0x18, 0x2000); + setbits32(pdm360ng_gpio_base + 0x10, 0x40); + + return 0; +} + +static int pdm360ng_touchscreen_notifier_call(struct notifier_block *nb, + unsigned long event, void *__dev) +{ + struct device *dev = __dev; + + if ((event == BUS_NOTIFY_ADD_DEVICE) && + of_device_is_compatible(dev->of_node, "ti,ads7846")) { + dev->platform_data = &pdm360ng_ads7846_pdata; + return NOTIFY_OK; + } + return NOTIFY_DONE; +} + +static struct notifier_block pdm360ng_touchscreen_nb = { + .notifier_call = pdm360ng_touchscreen_notifier_call, +}; + +static void __init pdm360ng_touchscreen_init(void) +{ + if (pdm360ng_penirq_init()) + return; + + bus_register_notifier(&spi_bus_type, &pdm360ng_touchscreen_nb); +} +#else +static inline void __init pdm360ng_touchscreen_init(void) +{ +} +#endif /* CONFIG_TOUCHSCREEN_ADS7846 */ + +void __init pdm360ng_init(void) +{ + mpc512x_init(); + pdm360ng_touchscreen_init(); +} + +static int __init pdm360ng_probe(void) +{ + unsigned long root = of_get_flat_dt_root(); + + return of_flat_dt_is_compatible(root, "ifm,pdm360ng"); +} + +define_machine(pdm360ng) { + .name = "PDM360NG", + .probe = pdm360ng_probe, + .setup_arch = mpc512x_setup_diu, + .init = pdm360ng_init, + .init_early = mpc512x_init_diu, + .init_IRQ = mpc512x_init_IRQ, + .get_irq = ipic_get_irq, + .calibrate_decr = generic_calibrate_decr, + .restart = mpc512x_restart, +}; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
end of thread, other threads:[~2010-07-27 21:26 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1272659448-23302-1-git-send-email-agust@denx.de>
[not found] ` <1272659448-23302-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org>
2010-04-30 20:30 ` [PATCH 2/2] powerpc/mpc5121: add initial support for PDM360NG board Anatolij Gustschin
[not found] ` <1272659448-23302-2-git-send-email-agust-ynQEQJNshbs@public.gmane.org>
2010-05-02 14:54 ` Grant Likely
2010-05-03 9:22 ` Anatolij Gustschin
[not found] ` <g2tfa686aa41005020754t46934f13hbd5c75a6b7668a54-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-05-03 16:34 ` Scott Wood
[not found] ` <4BDEFB13.5060407-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2010-05-19 21:27 ` Grant Likely
2010-05-19 21:37 ` Scott Wood
[not found] ` <4BF45A1F.70100-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2010-05-19 21:47 ` Grant Likely
[not found] ` <AANLkTinxuFz3SSLZtF3yGYtRN8nsW8_6EDiqzmJilYE9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-06-22 21:39 ` Scott Wood
2010-05-03 10:23 ` [PATCH v2 " Anatolij Gustschin
[not found] ` <1272882222-12253-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org>
2010-07-23 13:49 ` [PATCH v3 " Anatolij Gustschin
[not found] ` <1279892973-24110-1-git-send-email-agust-ynQEQJNshbs@public.gmane.org>
2010-07-25 7:42 ` Grant Likely
[not found] ` <AANLkTi=Ee6HR+Ux0g4V1+81DXUmsO=UDhi6jWsDgGchv-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-07-27 10:36 ` Anatolij Gustschin
2010-07-27 16:58 ` Grant Likely
[not found] ` <AANLkTika=KeKcaY_FF8tN5Fy-H+cozjZsGObXoQhKiVh-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-07-27 17:28 ` Anatolij Gustschin
2010-07-27 17:43 ` Grant Likely
2010-07-27 21:26 ` [PATCH v4 " Anatolij Gustschin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox