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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to