[PATCH 02/10] ux500: Adding new board files for snowball.
Arnd Bergmann
arnd at arndb.de
Sat Mar 12 04:50:00 EST 2011
On Saturday 12 March 2011 00:08:04 mathieu.poirier at linaro.org wrote:
> From: Mathieu J. Poirier <mathieu.poirier at linaro.org>
>
> Signed-off-by: Mathieu Poirier <mathieu.poirier at linaro.org>
This changelog would be a good place to explain what the board
is and point to http://www.igloocommunity.org/.
The patch looks good. One question though:
> +
> +static struct pl022_ssp_controller ssp0_platform_data = {
> + .bus_id = 0,
> +#ifdef CONFIG_STE_DMA40
> + .enable_dma = 1,
> + .dma_filter = stedma40_filter,
> + .dma_rx_param = &ssp0_dma_cfg_rx,
> + .dma_tx_param = &ssp0_dma_cfg_tx,
> +#else
> + .enable_dma = 0,
> +#endif
> + /* on this platform, gpio 31,142,144,214 &
> + * 224 are connected as chip selects
> + */
> + .num_chipselect = 5,
> +};
You have a lot of #ifdef CONFIG_STE_DMA40. Is that something
that realistically would be disabled for many users?
If not, you could simply "select STE_DMA40" from the board
Kconfig symbol and make all these unconditional.
Arnd
More information about the linux-arm-kernel
mailing list