[PATCH v7 13/15] ARM: mxs: Add initial mx23evk support

Shawn Guo shawn.guo at freescale.com
Mon Dec 20 05:17:10 EST 2010


Hi Uwe,

On Mon, Dec 20, 2010 at 10:48:01AM +0100, Uwe Kleine-König wrote:
> Hello Shawn,
> 
> On Sat, Dec 18, 2010 at 09:39:34PM +0800, Shawn Guo wrote:
> > Add initial mx23evk support with duart.
> > 
> > Signed-off-by: Shawn Guo <shawn.guo at freescale.com>
> > ---
> > Changes for v7:
> >  - Define mx23evk pads using naked pad "OR" mA/vol/pull configurations
> > 
> > Changes for v3:
> >  - Remove inclusion of hardware.h
> >  - Add __initconst for mx23evk_pads[]
> > 
> > Changes for v2:
> >  - Remove boot_params
> > 
> >  arch/arm/mach-mxs/mach-mx23evk.c |   59 ++++++++++++++++++++++++++++++++++++++
> >  1 files changed, 59 insertions(+), 0 deletions(-)
> >  create mode 100644 arch/arm/mach-mxs/mach-mx23evk.c
> > 
> > diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c
> > new file mode 100644
> > index 0000000..2aa77cc
> > --- /dev/null
> > +++ b/arch/arm/mach-mxs/mach-mx23evk.c
> > @@ -0,0 +1,59 @@
> > [...]
> > +static const iomux_cfg_t mx23evk_pads[] __initconst = {
> > +	/* duart */
> > +	MX23_PAD_PWM0__DUART_RX |
> > +		MXS_PAD_4MA,
> > +	MX23_PAD_PWM1__DUART_TX |
> > +		MXS_PAD_4MA,
> > +};
> I would keep these on a single line.  YMMV, so for me no need for a v8.
> 
> I will test this series later on hardware, and if no show stoppers
> appear I'd say this is ready to be merged.  The (maybe) remaining issues
> can then be resolved in follow up patches.
> 
My intension was to keep the pad configuration on different line from
the naked pad.  I will keep this comment and that extra space one
in mind, and address them later.

Thanks for the review and test effort. 

-- 
Regards,
Shawn




More information about the linux-arm-kernel mailing list