[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