Hello, On Thursday, February 09, 2012 1:28 PM Kukjin Kim wrote: > > ARM: Exynos: JPEG board definition > > > > [...] > > > > > +#define EXYNOS4_PA_JPEG 0x11840000 > > + > > #define EXYNOS4_PA_G2D 0x12800000 > > > > #define EXYNOS4_PA_I2S0 0x03830000 > > @@ -164,6 +166,7 @@ > > #define S5P_PA_FIMC1 EXYNOS4_PA_FIMC1 > > #define S5P_PA_FIMC2 EXYNOS4_PA_FIMC2 > > #define S5P_PA_FIMC3 EXYNOS4_PA_FIMC3 > > +#define S5P_PA_JPEG EXYNOS4_PA_JPEG > > I'm not sure we _really_ need to re-define for JPEG. > Why not just use EXYNOS4_PA_JPEG? The s5p-jpeg driver can be also used on S5PV210 so this approach seems to be fine, the patch only lacks definitions for the S5PV210 series. Best regards -- Marek Szyprowski Samsung Poland R&D Center -- 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