[PATCH 7/9] lpc2k: Add EmbeddedArtists LPC2478 Developer's Kit board

Kevin Wells kevin.wells at nxp.com
Thu Mar 17 18:07:46 EDT 2011


> +
> +#define EXTPOLAR	0x14c
> +
> +static unsigned long mfp_cfgs[] = {

Should this be initdata?

> +
> +MACHINE_START(LPC2478OEM, "Embedded Artists LPC2478 OEM Board")

Just a nit, but the machine ID should probably be unique to this
board and not the 2478, ie EA2478OEM. If someone else releases a
LPC2478 based board, it can't use this same id. This would apply
to the filename too (lpc2478oem.c) - the name is somewhat generic
and might be better named something like plat-ea2478oem.c.

> +	.init_irq	= lpc2478oem_init_irq,
> +	.timer		= &lpc2k_timer,
> +	.init_machine	= lpc2478oem_init_machine,
> +MACHINE_END
> --
> 1.7.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



More information about the linux-arm-kernel mailing list