Re: [PATCH] pinctrl: add a gpio and pinctrl driver for samsung io-pad controllers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 16, 2011 at 01:36:40AM +0530, Thomas Abraham wrote:

> +/* Note: The pin group list needs to be further populated */
> +static unsigned int exynos4210_spi0_pins[] = {
> +	EXYNOS4_GPB(0),	EXYNOS4_GPB(1),	EXYNOS4_GPB(2),	EXYNOS4_GPB(3),
> +};

Given the large number of SoCs in play here might it not be better to
arrange things so that the data part can go in the SoC directory, or at
least in a file separate to the main driver.  Otherwise the file is
going to get enormous but that's mostly going to be data tables.

Also, I'd expect these pin lists to include information on the function
number.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux