> -----Original Message-----
> From: Haojian Zhuang [mailto:haojian.zhu...@gmail.com]
> Sent: 2013年4月13日 21:04
> To: Neil Zhang
> Cc: Grant Likely; linux-arm-ker...@lists.infradead.org;
> linux-kernel@vger.kernel.org; Chao Xie
> Subject: Re: [PATCH 3/4] ARM: mmp: bring up pxa988 with device tree
> support
>
> On Thu, Apr 11, 2013 at 11:38 AM, Neil Zhang <zhan...@marvell.com>
> wrote:
> > bring up pxa988 with device tree support.
> >
> > Signed-off-by: Neil Zhang <zhan...@marvell.com>
> > Signed-off-by: Chao Xie <chao....@marvell.com>
> > ---
> > arch/arm/boot/dts/pxa988-dkb.dts | 36 ++++++
> > arch/arm/boot/dts/pxa988.dtsi | 196
> +++++++++++++++++++++++++++++
> > arch/arm/mach-mmp/Kconfig | 25 ++++
> > arch/arm/mach-mmp/Makefile | 1 +
> > arch/arm/mach-mmp/common.c | 11 ++-
> > arch/arm/mach-mmp/include/mach/addr-map.h | 6 +
> > arch/arm/mach-mmp/mmpx-dt.c | 79
> ++++++++++++
> > drivers/clk/mmp/Makefile | 1 +
> > 8 files changed, 354 insertions(+), 1 deletions(-) create mode
> > 100644 arch/arm/boot/dts/pxa988-dkb.dts create mode 100644
> > arch/arm/boot/dts/pxa988.dtsi create mode 100644
> > arch/arm/mach-mmp/mmpx-dt.c
> >
> > diff --git a/arch/arm/boot/dts/pxa988-dkb.dts
> > b/arch/arm/boot/dts/pxa988-dkb.dts
> > new file mode 100644
> > index 0000000..2cee3ed
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/pxa988-dkb.dts
> > @@ -0,0 +1,36 @@
> > +/*
> > + * Copyright (C) 2012 Marvell Technology Group Ltd.
> > + * Author: Haojian Zhuang <haojian.zhu...@marvell.com>
> > + *
> > + * This program is free software; you can redistribute it and/or
> > +modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * publishhed by the Free Software Foundation.
> > + */
> > +
> > +/dts-v1/;
> > +/include/ "pxa988.dtsi"
> > +
> > +
> > + gpio@d4019000 {
> > + compatible = "mrvl,mmp-gpio";
> Since gpio-pxa driver is updated & merged just now, you need to update this
> as "marvell,mmp-gpio".
>
Ok, I'll update.
> > +
> > +/* PXA988 */
> > +static const struct of_dev_auxdata pxa988_auxdata_lookup[] __initconst
> = {
> > + OF_DEV_AUXDATA("mrvl,mmp-uart", 0xd4017000,
> "pxa2xx-uart.0", NULL),
> > + OF_DEV_AUXDATA("mrvl,mmp-uart", 0xd4018000,
> "pxa2xx-uart.1", NULL),
> > + OF_DEV_AUXDATA("mrvl,mmp-uart", 0xd4036000,
> "pxa2xx-uart.2", NULL),
> > + OF_DEV_AUXDATA("mrvl,mmp-twsi", 0xd4011000,
> "pxa2xx-i2c.0", NULL),
> > + OF_DEV_AUXDATA("mrvl,mmp-twsi", 0xd4037000,
> "pxa2xx-i2c.1", NULL),
> > + OF_DEV_AUXDATA("mrvl,mmp-gpio", 0xd4019000, "pxa-gpio",
> NULL),
> "marvell,mmp-gpio"
Best Regards,
Neil Zhang
