* [PATCH 0/2] Support exynos5420-fimd @ 2016-02-12 13:31 Chanho Park 2016-02-12 13:31 ` [PATCH 1/2] drm/exynos: add exynos5420 support for fimd Chanho Park 2016-02-12 13:31 ` [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible Chanho Park 0 siblings, 2 replies; 8+ messages in thread From: Chanho Park @ 2016-02-12 13:31 UTC (permalink / raw) To: Inki Dae, Kukjin Kim, Krzysztof Kozlowski, linux-samsung-soc, dri-devel Cc: Chanho Park This patchset introduces exynos5420-fimd compatible to support a mic-bypass option from display path. Since exynos5420, the option has been introduced and controlled by system register. The option is only available for exynos5420/exynos5422 and 5800, we need to add the new exynos5420-fimd compatible. Chanho Park (2): drm/exynos: add exynos5420 support for fimd ARM: dts: add exynos5420-fimd compatible .../bindings/display/exynos/samsung-fimd.txt | 3 ++- arch/arm/boot/dts/exynos5420.dtsi | 1 + drivers/gpu/drm/exynos/exynos_drm_fimd.c | 31 +++++++++++++++++++++- 3 files changed, 33 insertions(+), 2 deletions(-) -- 2.5.0 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/2] drm/exynos: add exynos5420 support for fimd 2016-02-12 13:31 [PATCH 0/2] Support exynos5420-fimd Chanho Park @ 2016-02-12 13:31 ` Chanho Park 2016-02-23 10:46 ` Inki Dae 2016-02-12 13:31 ` [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible Chanho Park 1 sibling, 1 reply; 8+ messages in thread From: Chanho Park @ 2016-02-12 13:31 UTC (permalink / raw) To: Inki Dae, Kukjin Kim, Krzysztof Kozlowski, linux-samsung-soc, dri-devel Cc: Chanho Park, Joonyoung Shim, Seung-Woo Kim This patch adds a exynos5420 driver data to support mic_bypass option to bypass the mic from display out path. The mic(Mobile image compressor) compresses RGB data from fimd and send the compressed data to the mipi dsi. The bypass option can be founded from system register and the bit is 11. The option bit has been introduced since exynos5420. The only difference between exynos5250 and exynos5420/exynos5422 is existence of the bit. Until the MIC is defined and enabled from device tree, the bypass mic will be default option. Cc: Inki Dae <inki.dae@samsung.com> Cc: Joonyoung Shim <jy0922.shim@samsung.com> Cc: Seung-Woo Kim <sw0312.kim@samsung.com> Signed-off-by: Chanho Park <chanho61.park@samsung.com> --- .../bindings/display/exynos/samsung-fimd.txt | 3 ++- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 31 +++++++++++++++++++++- 2 files changed, 32 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt b/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt index 27c3ce0..c7c6b9a 100644 --- a/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt +++ b/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt @@ -12,7 +12,8 @@ Required properties: "samsung,exynos3250-fimd"; /* for Exynos3250/3472 SoCs */ "samsung,exynos4210-fimd"; /* for Exynos4 SoCs */ "samsung,exynos4415-fimd"; /* for Exynos4415 SoC */ - "samsung,exynos5250-fimd"; /* for Exynos5 SoCs */ + "samsung,exynos5250-fimd"; /* for Exynos5250 SoCs */ + "samsung,exynos5420-fimd"; /* for Exynos5420/5422/5800 SoCs */ - reg: physical base address and length of the FIMD registers set. diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c index 70194d0..41c3bb2 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c @@ -94,12 +94,14 @@ struct fimd_driver_data { unsigned int lcdblk_offset; unsigned int lcdblk_vt_shift; unsigned int lcdblk_bypass_shift; + unsigned int lcdblk_mic_bypass_shift; unsigned int has_shadowcon:1; unsigned int has_clksel:1; unsigned int has_limited_fmt:1; unsigned int has_vidoutcon:1; unsigned int has_vtsel:1; + unsigned int has_mic_bypass:1; }; static struct fimd_driver_data s3c64xx_fimd_driver_data = { @@ -145,6 +147,18 @@ static struct fimd_driver_data exynos5_fimd_driver_data = { .has_vtsel = 1, }; +static struct fimd_driver_data exynos5420_fimd_driver_data = { + .timing_base = 0x20000, + .lcdblk_offset = 0x214, + .lcdblk_vt_shift = 24, + .lcdblk_bypass_shift = 15, + .lcdblk_mic_bypass_shift = 11, + .has_shadowcon = 1, + .has_vidoutcon = 1, + .has_vtsel = 1, + .has_mic_bypass = 1, +}; + struct fimd_context { struct device *dev; struct drm_device *drm_dev; @@ -184,6 +198,8 @@ static const struct of_device_id fimd_driver_dt_match[] = { .data = &exynos4415_fimd_driver_data }, { .compatible = "samsung,exynos5250-fimd", .data = &exynos5_fimd_driver_data }, + { .compatible = "samsung,exynos5420-fimd", + .data = &exynos5420_fimd_driver_data }, {}, }; MODULE_DEVICE_TABLE(of, fimd_driver_dt_match); @@ -461,6 +477,18 @@ static void fimd_commit(struct exynos_drm_crtc *crtc) return; } + /* TODO: When MIC is enabled for display path, the lcdblk_mic_bypass + * bit should be cleared. + */ + if (driver_data->has_mic_bypass && ctx->sysreg && + regmap_update_bits(ctx->sysreg, + driver_data->lcdblk_offset, + 0x1 << driver_data->lcdblk_mic_bypass_shift, + 0x1 << driver_data->lcdblk_mic_bypass_shift)) { + DRM_ERROR("Failed to update sysreg for bypass mic.\n"); + return; + } + /* setup horizontal and vertical display size. */ val = VIDTCON2_LINEVAL(mode->vdisplay - 1) | VIDTCON2_HOZVAL(mode->hdisplay - 1) | @@ -861,7 +889,8 @@ static void fimd_dp_clock_enable(struct exynos_drm_crtc *crtc, bool enable) * clock. On these SoCs the bootloader may enable it but any * power domain off/on will reset it to disable state. */ - if (ctx->driver_data != &exynos5_fimd_driver_data) + if (ctx->driver_data != &exynos5_fimd_driver_data || + ctx->driver_data != &exynos5420_fimd_driver_data) return; val = enable ? DP_MIE_CLK_DP_ENABLE : DP_MIE_CLK_DISABLE; -- 2.5.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] drm/exynos: add exynos5420 support for fimd 2016-02-12 13:31 ` [PATCH 1/2] drm/exynos: add exynos5420 support for fimd Chanho Park @ 2016-02-23 10:46 ` Inki Dae 0 siblings, 0 replies; 8+ messages in thread From: Inki Dae @ 2016-02-23 10:46 UTC (permalink / raw) To: Chanho Park, Kukjin Kim, Krzysztof Kozlowski, linux-samsung-soc, dri-devel Cc: Chanho Park, Joonyoung Shim, Seung-Woo Kim Picked it up. Thanks, Inki Dae 2016년 02월 12일 22:31에 Chanho Park 이(가) 쓴 글: > This patch adds a exynos5420 driver data to support mic_bypass > option to bypass the mic from display out path. > The mic(Mobile image compressor) compresses RGB data from fimd > and send the compressed data to the mipi dsi. > The bypass option can be founded from system register and the bit > is 11. The option bit has been introduced since exynos5420. The > only difference between exynos5250 and exynos5420/exynos5422 is > existence of the bit. Until the MIC is defined and enabled from > device tree, the bypass mic will be default option. > > Cc: Inki Dae <inki.dae@samsung.com> > Cc: Joonyoung Shim <jy0922.shim@samsung.com> > Cc: Seung-Woo Kim <sw0312.kim@samsung.com> > Signed-off-by: Chanho Park <chanho61.park@samsung.com> > --- > .../bindings/display/exynos/samsung-fimd.txt | 3 ++- > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 31 +++++++++++++++++++++- > 2 files changed, 32 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt b/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt > index 27c3ce0..c7c6b9a 100644 > --- a/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt > +++ b/Documentation/devicetree/bindings/display/exynos/samsung-fimd.txt > @@ -12,7 +12,8 @@ Required properties: > "samsung,exynos3250-fimd"; /* for Exynos3250/3472 SoCs */ > "samsung,exynos4210-fimd"; /* for Exynos4 SoCs */ > "samsung,exynos4415-fimd"; /* for Exynos4415 SoC */ > - "samsung,exynos5250-fimd"; /* for Exynos5 SoCs */ > + "samsung,exynos5250-fimd"; /* for Exynos5250 SoCs */ > + "samsung,exynos5420-fimd"; /* for Exynos5420/5422/5800 SoCs */ > > - reg: physical base address and length of the FIMD registers set. > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > index 70194d0..41c3bb2 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > @@ -94,12 +94,14 @@ struct fimd_driver_data { > unsigned int lcdblk_offset; > unsigned int lcdblk_vt_shift; > unsigned int lcdblk_bypass_shift; > + unsigned int lcdblk_mic_bypass_shift; > > unsigned int has_shadowcon:1; > unsigned int has_clksel:1; > unsigned int has_limited_fmt:1; > unsigned int has_vidoutcon:1; > unsigned int has_vtsel:1; > + unsigned int has_mic_bypass:1; > }; > > static struct fimd_driver_data s3c64xx_fimd_driver_data = { > @@ -145,6 +147,18 @@ static struct fimd_driver_data exynos5_fimd_driver_data = { > .has_vtsel = 1, > }; > > +static struct fimd_driver_data exynos5420_fimd_driver_data = { > + .timing_base = 0x20000, > + .lcdblk_offset = 0x214, > + .lcdblk_vt_shift = 24, > + .lcdblk_bypass_shift = 15, > + .lcdblk_mic_bypass_shift = 11, > + .has_shadowcon = 1, > + .has_vidoutcon = 1, > + .has_vtsel = 1, > + .has_mic_bypass = 1, > +}; > + > struct fimd_context { > struct device *dev; > struct drm_device *drm_dev; > @@ -184,6 +198,8 @@ static const struct of_device_id fimd_driver_dt_match[] = { > .data = &exynos4415_fimd_driver_data }, > { .compatible = "samsung,exynos5250-fimd", > .data = &exynos5_fimd_driver_data }, > + { .compatible = "samsung,exynos5420-fimd", > + .data = &exynos5420_fimd_driver_data }, > {}, > }; > MODULE_DEVICE_TABLE(of, fimd_driver_dt_match); > @@ -461,6 +477,18 @@ static void fimd_commit(struct exynos_drm_crtc *crtc) > return; > } > > + /* TODO: When MIC is enabled for display path, the lcdblk_mic_bypass > + * bit should be cleared. > + */ > + if (driver_data->has_mic_bypass && ctx->sysreg && > + regmap_update_bits(ctx->sysreg, > + driver_data->lcdblk_offset, > + 0x1 << driver_data->lcdblk_mic_bypass_shift, > + 0x1 << driver_data->lcdblk_mic_bypass_shift)) { > + DRM_ERROR("Failed to update sysreg for bypass mic.\n"); > + return; > + } > + > /* setup horizontal and vertical display size. */ > val = VIDTCON2_LINEVAL(mode->vdisplay - 1) | > VIDTCON2_HOZVAL(mode->hdisplay - 1) | > @@ -861,7 +889,8 @@ static void fimd_dp_clock_enable(struct exynos_drm_crtc *crtc, bool enable) > * clock. On these SoCs the bootloader may enable it but any > * power domain off/on will reset it to disable state. > */ > - if (ctx->driver_data != &exynos5_fimd_driver_data) > + if (ctx->driver_data != &exynos5_fimd_driver_data || > + ctx->driver_data != &exynos5420_fimd_driver_data) > return; > > val = enable ? DP_MIE_CLK_DP_ENABLE : DP_MIE_CLK_DISABLE; > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible 2016-02-12 13:31 [PATCH 0/2] Support exynos5420-fimd Chanho Park 2016-02-12 13:31 ` [PATCH 1/2] drm/exynos: add exynos5420 support for fimd Chanho Park @ 2016-02-12 13:31 ` Chanho Park 2016-02-15 0:57 ` Krzysztof Kozlowski 1 sibling, 1 reply; 8+ messages in thread From: Chanho Park @ 2016-02-12 13:31 UTC (permalink / raw) To: Inki Dae, Kukjin Kim, Krzysztof Kozlowski, linux-samsung-soc, dri-devel Cc: Chanho Park This patch changes the compatible of exynos5420 fimd to "exynos5420-fimd". To support mic bypass from display path, the new compatible is introduced for exynos5420. Cc: Inki Dae <inki.dae@samsung.com> Cc: Kukjin Kim <kgene@kernel.org> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com> Signed-off-by: Chanho Park <chanho61.park@samsung.com> --- arch/arm/boot/dts/exynos5420.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi index 48a0a55..252c7a3 100644 --- a/arch/arm/boot/dts/exynos5420.dtsi +++ b/arch/arm/boot/dts/exynos5420.dtsi @@ -1151,6 +1151,7 @@ }; &fimd { + compatible = "samsung,exynos5420-fimd"; clocks = <&clock CLK_SCLK_FIMD1>, <&clock CLK_FIMD1>; clock-names = "sclk_fimd", "fimd"; power-domains = <&disp_pd>; -- 2.5.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible 2016-02-12 13:31 ` [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible Chanho Park @ 2016-02-15 0:57 ` Krzysztof Kozlowski 2016-02-15 3:59 ` Krzysztof Kozlowski 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-02-15 0:57 UTC (permalink / raw) To: Chanho Park, Inki Dae, Kukjin Kim, linux-samsung-soc, dri-devel Cc: Chanho Park On 12.02.2016 22:31, Chanho Park wrote: > This patch changes the compatible of exynos5420 fimd > to "exynos5420-fimd". To support mic bypass from display > path, the new compatible is introduced for exynos5420. > > Cc: Inki Dae <inki.dae@samsung.com> > Cc: Kukjin Kim <kgene@kernel.org> > Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com> > Signed-off-by: Chanho Park <chanho61.park@samsung.com> > --- > arch/arm/boot/dts/exynos5420.dtsi | 1 + > 1 file changed, 1 insertion(+) > Looks okay to me and also looks independent from patch #1. I will apply it for late v4.6 if patch #1 got accepted by Inki. Anyway, for reference: Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible 2016-02-15 0:57 ` Krzysztof Kozlowski @ 2016-02-15 3:59 ` Krzysztof Kozlowski 2016-04-21 10:06 ` Krzysztof Kozlowski 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-02-15 3:59 UTC (permalink / raw) To: Chanho Park, Inki Dae, Kukjin Kim, linux-samsung-soc, dri-devel Cc: Chanho Park On 15.02.2016 09:57, Krzysztof Kozlowski wrote: > On 12.02.2016 22:31, Chanho Park wrote: >> This patch changes the compatible of exynos5420 fimd >> to "exynos5420-fimd". To support mic bypass from display >> path, the new compatible is introduced for exynos5420. >> >> Cc: Inki Dae <inki.dae@samsung.com> >> Cc: Kukjin Kim <kgene@kernel.org> >> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com> >> Signed-off-by: Chanho Park <chanho61.park@samsung.com> >> --- >> arch/arm/boot/dts/exynos5420.dtsi | 1 + >> 1 file changed, 1 insertion(+) >> > > Looks okay to me and also looks independent from patch #1. I will apply > it for late v4.6 if patch #1 got accepted by Inki. > > Anyway, for reference: > Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Stupid me, of course it cannot go independently through my tree. Please feel free to take it through drm-exynos with my reviewed-by tag. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible 2016-02-15 3:59 ` Krzysztof Kozlowski @ 2016-04-21 10:06 ` Krzysztof Kozlowski 2016-04-21 13:11 ` Inki Dae 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-04-21 10:06 UTC (permalink / raw) To: Chanho Park, Inki Dae, Kukjin Kim, linux-samsung-soc, dri-devel Cc: Chanho Park On 02/15/2016 04:59 AM, Krzysztof Kozlowski wrote: > On 15.02.2016 09:57, Krzysztof Kozlowski wrote: >> On 12.02.2016 22:31, Chanho Park wrote: >>> This patch changes the compatible of exynos5420 fimd >>> to "exynos5420-fimd". To support mic bypass from display >>> path, the new compatible is introduced for exynos5420. >>> >>> Cc: Inki Dae <inki.dae@samsung.com> >>> Cc: Kukjin Kim <kgene@kernel.org> >>> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com> >>> Signed-off-by: Chanho Park <chanho61.park@samsung.com> >>> --- >>> arch/arm/boot/dts/exynos5420.dtsi | 1 + >>> 1 file changed, 1 insertion(+) >>> >> >> Looks okay to me and also looks independent from patch #1. I will apply >> it for late v4.6 if patch #1 got accepted by Inki. >> >> Anyway, for reference: >> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > > Stupid me, of course it cannot go independently through my tree. Please > feel free to take it through drm-exynos with my reviewed-by tag. Inki did not pick it up, so applied now for late v4.7. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible 2016-04-21 10:06 ` Krzysztof Kozlowski @ 2016-04-21 13:11 ` Inki Dae 0 siblings, 0 replies; 8+ messages in thread From: Inki Dae @ 2016-04-21 13:11 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Chanho Park, Kukjin Kim, linux-samsung-soc@vger.kernel.org, DRI mailing list, Chanho Park 2016-04-21 19:06 GMT+09:00 Krzysztof Kozlowski <k.kozlowski@samsung.com>: > On 02/15/2016 04:59 AM, Krzysztof Kozlowski wrote: >> On 15.02.2016 09:57, Krzysztof Kozlowski wrote: >>> On 12.02.2016 22:31, Chanho Park wrote: >>>> This patch changes the compatible of exynos5420 fimd >>>> to "exynos5420-fimd". To support mic bypass from display >>>> path, the new compatible is introduced for exynos5420. >>>> >>>> Cc: Inki Dae <inki.dae@samsung.com> >>>> Cc: Kukjin Kim <kgene@kernel.org> >>>> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com> >>>> Signed-off-by: Chanho Park <chanho61.park@samsung.com> >>>> --- >>>> arch/arm/boot/dts/exynos5420.dtsi | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>> >>> Looks okay to me and also looks independent from patch #1. I will apply >>> it for late v4.6 if patch #1 got accepted by Inki. >>> >>> Anyway, for reference: >>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >> >> Stupid me, of course it cannot go independently through my tree. Please >> feel free to take it through drm-exynos with my reviewed-by tag. > > Inki did not pick it up, so applied now for late v4.7. > Forgot it. Sorry for this Chanho and Krzysztof. Thanks, Inki Dae > Best regards, > Krzysztof > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-04-21 13:11 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-02-12 13:31 [PATCH 0/2] Support exynos5420-fimd Chanho Park 2016-02-12 13:31 ` [PATCH 1/2] drm/exynos: add exynos5420 support for fimd Chanho Park 2016-02-23 10:46 ` Inki Dae 2016-02-12 13:31 ` [PATCH 2/2] ARM: dts: add exynos5420-fimd compatible Chanho Park 2016-02-15 0:57 ` Krzysztof Kozlowski 2016-02-15 3:59 ` Krzysztof Kozlowski 2016-04-21 10:06 ` Krzysztof Kozlowski 2016-04-21 13:11 ` Inki Dae
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.