[PATCH - 3rd attempt] ARM: Support for Embedian MXM-8x10 eval kit
daniel at caiaq.de
Mon Jan 18 16:51:24 EST 2010
On Mon, Jan 18, 2010 at 09:34:14PM +0200, Edwin Peer wrote:
> I believe we have finally managed to address all your review points - third time's a charm. :)
> Here's the diffstat against 2.6.33-rc4:
One more thing that caught my eye.
> +void __init mxm_8x10_mmc_init(void)
> + pxa_set_mci_info(&mxm_8x10_mci_platform_data);
> +inline void mxm_8x10_mmc_init(void)
You should make that inline function static.
> +void __init mxm_8x10_nand_init(void)
> + pxa3xx_set_nand_info(&mxm_8x10_nand_info);
> +inline mxm_8x10_nand_init(void)
Same here, and it should return void. Without any return type, it will
default to int, which is not what you want. And as you're on it, you can
also put the whole thing in one line (but that's really nit-picking).
More information about the linux-arm-kernel