This patch series compose of 2 patches.
First patch, adding support for S5PV210 FIMD variant to Exynos driver.
Compatible for this soc was already existing in documentation.
Second patch, brings back possibility to use drivers depending on
DRM_EXYNOS, on Samsung S5PV210/S5PC110 series based systems.
Both patches has been tested on (not yet mainlined), an S5PV210 based
Samsung Galaxy S (i9000) phone.
Paweł Chmiel (1):
drm/exynos: Allow DRM_EXYNOS on s5pv210.
Tomasz Figa (1):
drm/exynos: fimd: Add support for S5PV210 FIMD variant
drivers/gpu/drm/exynos/Kconfig | 2 +-
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 8 ++++++++
2 files changed, 9 insertions(+), 1 deletion(-)
--
2.7.4
This patch brings back possibility to use drivers depending on
DRM_EXYNOS, on Samsung S5PV210/S5PC110 series based systems.
Fixes: dbbc925bb83a ("drm/exynos: depend on ARCH_EXYNOS for DRM_EXYNOS")
Signed-off-by: Paweł Chmiel <[email protected]>
---
drivers/gpu/drm/exynos/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/exynos/Kconfig b/drivers/gpu/drm/exynos/Kconfig
index 735ce47688f9..1548a784ef71 100644
--- a/drivers/gpu/drm/exynos/Kconfig
+++ b/drivers/gpu/drm/exynos/Kconfig
@@ -1,6 +1,6 @@
config DRM_EXYNOS
tristate "DRM Support for Samsung SoC EXYNOS Series"
- depends on OF && DRM && (ARCH_S3C64XX || ARCH_EXYNOS || ARCH_MULTIPLATFORM)
+ depends on OF && DRM && (ARCH_S3C64XX || ARCH_S5PV210 || ARCH_EXYNOS || ARCH_MULTIPLATFORM)
select DRM_KMS_HELPER
select VIDEOMODE_HELPERS
select SND_SOC_HDMI_CODEC if SND_SOC
--
2.7.4
From: Tomasz Figa <[email protected]>
This patch adds support for FIMD variant found on S5PV210 SoC.
Except CLKSEL bit availability, it is identical to Exynos4210.
Tested-by: Paweł Chmiel <[email protected]>
Signed-off-by: Tomasz Figa <[email protected]>
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index d42ae2bc3e56..01b1570d0c3a 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -121,6 +121,12 @@ static struct fimd_driver_data s3c64xx_fimd_driver_data = {
.has_limited_fmt = 1,
};
+static struct fimd_driver_data s5pv210_fimd_driver_data = {
+ .timing_base = 0x0,
+ .has_shadowcon = 1,
+ .has_clksel = 1,
+};
+
static struct fimd_driver_data exynos3_fimd_driver_data = {
.timing_base = 0x20000,
.lcdblk_offset = 0x210,
@@ -193,6 +199,8 @@ struct fimd_context {
static const struct of_device_id fimd_driver_dt_match[] = {
{ .compatible = "samsung,s3c6400-fimd",
.data = &s3c64xx_fimd_driver_data },
+ { .compatible = "samsung,s5pv210-fimd",
+ .data = &s5pv210_fimd_driver_data },
{ .compatible = "samsung,exynos3250-fimd",
.data = &exynos3_fimd_driver_data },
{ .compatible = "samsung,exynos4210-fimd",
--
2.7.4
On Fri, Apr 20, 2018 at 6:32 PM, Paweł Chmiel
<[email protected]> wrote:
> From: Tomasz Figa <[email protected]>
>
> This patch adds support for FIMD variant found on S5PV210 SoC.
> Except CLKSEL bit availability, it is identical to Exynos4210.
>
> Tested-by: Paweł Chmiel <[email protected]>
> Signed-off-by: Tomasz Figa <[email protected]>
You need to add your Signed-off-by here as well. It is necessary every
time you touch a patch (and sending it is also a "touch" action).
Best regards,
Krzysztof
> ---
> drivers/gpu/drm/exynos/exynos_drm_fimd.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> index d42ae2bc3e56..01b1570d0c3a 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> @@ -121,6 +121,12 @@ static struct fimd_driver_data s3c64xx_fimd_driver_data = {
> .has_limited_fmt = 1,
> };
>
> +static struct fimd_driver_data s5pv210_fimd_driver_data = {
> + .timing_base = 0x0,
> + .has_shadowcon = 1,
> + .has_clksel = 1,
> +};
> +
> static struct fimd_driver_data exynos3_fimd_driver_data = {
> .timing_base = 0x20000,
> .lcdblk_offset = 0x210,
> @@ -193,6 +199,8 @@ struct fimd_context {
> static const struct of_device_id fimd_driver_dt_match[] = {
> { .compatible = "samsung,s3c6400-fimd",
> .data = &s3c64xx_fimd_driver_data },
> + { .compatible = "samsung,s5pv210-fimd",
> + .data = &s5pv210_fimd_driver_data },
> { .compatible = "samsung,exynos3250-fimd",
> .data = &exynos3_fimd_driver_data },
> { .compatible = "samsung,exynos4210-fimd",
> --
> 2.7.4
>
On Saturday, April 21, 2018 6:55:17 PM CEST Krzysztof Kozlowski wrote:
> On Fri, Apr 20, 2018 at 6:32 PM, Paweł Chmiel
> <[email protected]> wrote:
> > From: Tomasz Figa <[email protected]>
> >
> > This patch adds support for FIMD variant found on S5PV210 SoC.
> > Except CLKSEL bit availability, it is identical to Exynos4210.
> >
> > Tested-by: Paweł Chmiel <[email protected]>
> > Signed-off-by: Tomasz Figa <[email protected]>
>
> You need to add your Signed-off-by here as well. It is necessary every
> time you touch a patch (and sending it is also a "touch" action).
Hi
Thanks for feedback, i've send a v2 version of this patchset with this issue fixed.
>
> Best regards,
> Krzysztof
>
> > ---
> > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> > index d42ae2bc3e56..01b1570d0c3a 100644
> > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> > @@ -121,6 +121,12 @@ static struct fimd_driver_data s3c64xx_fimd_driver_data = {
> > .has_limited_fmt = 1,
> > };
> >
> > +static struct fimd_driver_data s5pv210_fimd_driver_data = {
> > + .timing_base = 0x0,
> > + .has_shadowcon = 1,
> > + .has_clksel = 1,
> > +};
> > +
> > static struct fimd_driver_data exynos3_fimd_driver_data = {
> > .timing_base = 0x20000,
> > .lcdblk_offset = 0x210,
> > @@ -193,6 +199,8 @@ struct fimd_context {
> > static const struct of_device_id fimd_driver_dt_match[] = {
> > { .compatible = "samsung,s3c6400-fimd",
> > .data = &s3c64xx_fimd_driver_data },
> > + { .compatible = "samsung,s5pv210-fimd",
> > + .data = &s5pv210_fimd_driver_data },
> > { .compatible = "samsung,exynos3250-fimd",
> > .data = &exynos3_fimd_driver_data },
> > { .compatible = "samsung,exynos4210-fimd",
> > --
> > 2.7.4
> >
>