[PATCH v2 00/38] Thirty eight patches big Exynos cleanup

Arnd Bergmann arnd at arndb.de
Tue Jun 18 14:29:09 EDT 2013


On Tuesday 18 June 2013, Tomasz Figa wrote:
> OK. Looking forward to it.
> 
> Btw. This leaves us with two things broken:
>  - sdhci-s3c checks for defined(CONFIG_SOC_EXYNOS4212), but does not check 
> for defined(CONFIG_SOC_EXYNOS4412),
>  - exynos-thermal checks for defined(CONFIG_SOC_EXYNOS4412), but does not 
> check for defined(CONFIG_SOC_EXYNOS4212).
> 
> I think this must be fixed, because selecting only Exynos4212 in kernel 
> config you end up without exynos-thermal, while selecting only Exynos4412 
> breaks sdhci-s3c.

I wonder what we actually gain by having all these fine-grained checks
in the kernel. Why not just reduce the compile-time checks to just
EXYNOS4 and EXYNOS5?

The drivers can easily implement code for all variants using
run-time checks, but I think it would be easier if building an EXYNOS4
kernel would just run on all EXYNOS4 machines.

	Arnd



More information about the linux-arm-kernel mailing list