Re: [PATCH 1/3] drm/exynos: Control DISP1BLK system register in FIMD

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ajay,

On 25 June 2014 19:45, Ajay Kumar <ajaykumar.rs@xxxxxxxxxxx> wrote:
> Exynos SOC have a DISP1BLK register where we can select
> the path for FIMD output. We can redirect the video data
> directly to DP/MIPI interface, or we can pass it via
> image enhancement chips.
>
> Since we don't use any image enhancement chips in exynos-drm,
> we need to set FIMD BYPASS in DISP1BLK.
>
> Signed-off-by: Ajay Kumar <ajaykumar.rs@xxxxxxxxxxx>
> ---
>  .../devicetree/bindings/video/samsung-fimd.txt     |    2 ++
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c           |   19 +++++++++++++++++++
>  include/video/samsung_fimd.h                       |    5 +++++
>  3 files changed, 26 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/video/samsung-fimd.txt b/Documentation/devicetree/bindings/video/samsung-fimd.txt
> index 2dad41b..034e3458 100644
> --- a/Documentation/devicetree/bindings/video/samsung-fimd.txt
> +++ b/Documentation/devicetree/bindings/video/samsung-fimd.txt
> @@ -44,6 +44,8 @@ Optional Properties:
>  - display-timings: timing settings for FIMD, as described in document [1].
>                 Can be used in case timings cannot be provided otherwise
>                 or to override timings provided by the panel.
> +- samsung,sysreg-phandle - handle to syscon node, used to control the system
> +                          registers for DISP1BLK.
>
>  The device node can contain 'port' child nodes according to the bindings defined
>  in [2]. The following are properties specific to those nodes:
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> index 33161ad..a3c855b 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
> @@ -20,6 +20,8 @@
>  #include <linux/of_device.h>
>  #include <linux/pm_runtime.h>
>  #include <linux/component.h>
> +#include <linux/mfd/syscon.h>
> +#include <linux/regmap.h>
>
>  #include <video/of_display_timing.h>
>  #include <video/of_videomode.h>
> @@ -68,6 +70,7 @@
>
>  struct fimd_driver_data {
>         unsigned int timing_base;
> +       unsigned int sysreg_disp1blk_offset;
>
>         unsigned int has_shadowcon:1;
>         unsigned int has_clksel:1;
> @@ -83,11 +86,13 @@ static struct fimd_driver_data s3c64xx_fimd_driver_data = {
>  static struct fimd_driver_data exynos4_fimd_driver_data = {
>         .timing_base = 0x0,
>         .has_shadowcon = 1,
> +       .sysreg_disp1blk_offset = 0x210,

offset should be defined as Macros in reg file.

>  };
>
>  static struct fimd_driver_data exynos5_fimd_driver_data = {
>         .timing_base = 0x20000,
>         .has_shadowcon = 1,
> +       .sysreg_disp1blk_offset = 0x214,
>  };

Same as above. Rest looks fine to me.

Regards,
Rahul Sharma.

>
>  struct fimd_win_data {
> @@ -112,6 +117,7 @@ struct fimd_context {
>         struct clk                      *bus_clk;
>         struct clk                      *lcd_clk;
>         void __iomem                    *regs;
> +       struct regmap                   *sysreg;
>         struct drm_display_mode         mode;
>         struct fimd_win_data            win_data[WINDOWS_NR];
>         unsigned int                    default_win;
> @@ -760,6 +766,12 @@ static int fimd_poweron(struct exynos_drm_manager *mgr)
>
>         pm_runtime_get_sync(ctx->dev);
>
> +       if (ctx->sysreg)
> +               regmap_update_bits(ctx->sysreg,
> +                                  ctx->driver_data->sysreg_disp1blk_offset,
> +                                  FIMDBYPASS_MASK,
> +                                  FIMDBYPASS << FIMDBYPASS_SHIFT);
> +
>         ret = clk_prepare_enable(ctx->bus_clk);
>         if (ret < 0) {
>                 DRM_ERROR("Failed to prepare_enable the bus clk [%d]\n", ret);
> @@ -998,6 +1010,13 @@ static int fimd_probe(struct platform_device *pdev)
>         if (IS_ERR(ctx->display))
>                 return PTR_ERR(ctx->display);
>
> +       ctx->sysreg = syscon_regmap_lookup_by_phandle(
> +                               pdev->dev.of_node, "samsung,sysreg-phandle");
> +       if (IS_ERR(ctx->sysreg)) {
> +               DRM_INFO("DISP1BLK system register is not configured\n");
> +               ctx->sysreg = NULL;
> +       }
> +
>         pm_runtime_enable(&pdev->dev);
>
>         ret = component_add(&pdev->dev, &fimd_component_ops);
> diff --git a/include/video/samsung_fimd.h b/include/video/samsung_fimd.h
> index b039320..b92267d 100644
> --- a/include/video/samsung_fimd.h
> +++ b/include/video/samsung_fimd.h
> @@ -462,3 +462,8 @@
>  #define FIMD_V8_VIDTCON2       0x20018
>  #define FIMD_V8_VIDTCON3       0x2001C
>  #define FIMD_V8_VIDCON1                0x20004
> +
> +/* SYSREG DISP1BLK definitions */
> +#define FIMDBYPASS_SHIFT                       15
> +#define FIMDBYPASS_MASK                        (1 << FIMDBYPASS_SHIFT)
> +#define FIMDBYPASS                             1
> --
> 1.7.9.5
>
> --
> 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
--
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




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux