From: ryadav-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: hoegsberg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org
Subject: Re: [DPU PATCH v3 05/12] drm/msm/dpu: update dpu sub-block offsets wrt dpu base address
Date: Tue, 19 Jun 2018 14:29:29 +0530 [thread overview]
Message-ID: <2f600935fc7cc0eb0b78a4c2a8029434@codeaurora.org> (raw)
In-Reply-To: <1526311619-3309-6-git-send-email-ryadav-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Hi Sean,
Looks like this patch is missing on for-next branch.
Can you please help pull it in?
Thanks,
Rajesh
On 2018-05-14 20:56, Rajesh Yadav wrote:
> The dpu sub-block offsets were defined wrt mdss base address
> instead of dpu base address.
> Since, dpu is now defined as a separate device, update hw catalog
> offsets for all dpu sub blocks wrt dpu base address.
>
> Changes in v3:
> - none
>
> Changes in v2:
> - none
>
> Signed-off-by: Rajesh Yadav <ryadav@codeaurora.org>
> Reviewed-by: Sean Paul <seanpaul@chromium.org>
> ---
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 68
> +++++++++++------------
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c | 18 +++---
> 2 files changed, 43 insertions(+), 43 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> index c5b370f..2fd3254 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c
> @@ -80,7 +80,7 @@
> static struct dpu_mdp_cfg sdm845_mdp[] = {
> {
> .name = "top_0", .id = MDP_TOP,
> - .base = 0x1000, .len = 0x45C,
> + .base = 0x0, .len = 0x45C,
> .features = 0,
> .highest_bank_bit = 0x2,
> .has_dest_scaler = true,
> @@ -111,27 +111,27 @@
> static struct dpu_ctl_cfg sdm845_ctl[] = {
> {
> .name = "ctl_0", .id = CTL_0,
> - .base = 0x2000, .len = 0xE4,
> + .base = 0x1000, .len = 0xE4,
> .features = BIT(DPU_CTL_SPLIT_DISPLAY)
> },
> {
> .name = "ctl_1", .id = CTL_1,
> - .base = 0x2200, .len = 0xE4,
> + .base = 0x1200, .len = 0xE4,
> .features = BIT(DPU_CTL_SPLIT_DISPLAY)
> },
> {
> .name = "ctl_2", .id = CTL_2,
> - .base = 0x2400, .len = 0xE4,
> + .base = 0x1400, .len = 0xE4,
> .features = 0
> },
> {
> .name = "ctl_3", .id = CTL_3,
> - .base = 0x2600, .len = 0xE4,
> + .base = 0x1600, .len = 0xE4,
> .features = 0
> },
> {
> .name = "ctl_4", .id = CTL_4,
> - .base = 0x2800, .len = 0xE4,
> + .base = 0x1800, .len = 0xE4,
> .features = 0
> },
> };
> @@ -211,21 +211,21 @@
> }
>
> static struct dpu_sspp_cfg sdm845_sspp[] = {
> - SSPP_VIG_BLK("sspp_0", SSPP_VIG0, 0x5000,
> + SSPP_VIG_BLK("sspp_0", SSPP_VIG0, 0x4000,
> sdm845_vig_sblk_0, 0, DPU_CLK_CTRL_VIG0),
> - SSPP_VIG_BLK("sspp_1", SSPP_VIG1, 0x7000,
> + SSPP_VIG_BLK("sspp_1", SSPP_VIG1, 0x6000,
> sdm845_vig_sblk_1, 4, DPU_CLK_CTRL_VIG1),
> - SSPP_VIG_BLK("sspp_2", SSPP_VIG2, 0x9000,
> + SSPP_VIG_BLK("sspp_2", SSPP_VIG2, 0x8000,
> sdm845_vig_sblk_2, 8, DPU_CLK_CTRL_VIG2),
> - SSPP_VIG_BLK("sspp_3", SSPP_VIG3, 0xb000,
> + SSPP_VIG_BLK("sspp_3", SSPP_VIG3, 0xa000,
> sdm845_vig_sblk_3, 12, DPU_CLK_CTRL_VIG3),
> - SSPP_DMA_BLK("sspp_8", SSPP_DMA0, 0x25000,
> + SSPP_DMA_BLK("sspp_8", SSPP_DMA0, 0x24000,
> sdm845_dma_sblk_0, 1, DPU_CLK_CTRL_DMA0),
> - SSPP_DMA_BLK("sspp_9", SSPP_DMA1, 0x27000,
> + SSPP_DMA_BLK("sspp_9", SSPP_DMA1, 0x26000,
> sdm845_dma_sblk_1, 5, DPU_CLK_CTRL_DMA1),
> - SSPP_DMA_BLK("sspp_10", SSPP_DMA2, 0x29000,
> + SSPP_DMA_BLK("sspp_10", SSPP_DMA2, 0x28000,
> sdm845_dma_sblk_2, 9, DPU_CLK_CTRL_CURSOR0),
> - SSPP_DMA_BLK("sspp_11", SSPP_DMA3, 0x2b000,
> + SSPP_DMA_BLK("sspp_11", SSPP_DMA3, 0x2a000,
> sdm845_dma_sblk_3, 13, DPU_CLK_CTRL_CURSOR1),
> };
>
> @@ -252,17 +252,17 @@
> .lm_pair_mask = (1 << _lmpair) \
> }
> static struct dpu_lm_cfg sdm845_lm[] = {
> - LM_BLK("lm_0", LM_0, 0x45000, DSPP_0,
> + LM_BLK("lm_0", LM_0, 0x44000, DSPP_0,
> DS_0, PINGPONG_0, LM_1),
> - LM_BLK("lm_1", LM_1, 0x46000, DSPP_1,
> + LM_BLK("lm_1", LM_1, 0x45000, DSPP_1,
> DS_1, PINGPONG_1, LM_0),
> - LM_BLK("lm_2", LM_2, 0x47000, DSPP_2,
> + LM_BLK("lm_2", LM_2, 0x46000, DSPP_2,
> DS_MAX, PINGPONG_2, LM_5),
> LM_BLK("lm_3", LM_3, 0x0, DSPP_MAX,
> DS_MAX, PINGPONG_MAX, 0),
> LM_BLK("lm_4", LM_4, 0x0, DSPP_MAX,
> DS_MAX, PINGPONG_MAX, 0),
> - LM_BLK("lm_5", LM_5, 0x4a000, DSPP_3,
> + LM_BLK("lm_5", LM_5, 0x49000, DSPP_3,
> DS_MAX, PINGPONG_3, LM_2),
> };
>
> @@ -270,7 +270,7 @@
> * DSPP sub blocks config
> *************************************************************/
> static struct dpu_dspp_top_cfg sdm845_dspp_top = {
> - .name = "dspp_top", .base = 0x1300, .len = 0xc
> + .name = "dspp_top", .base = 0x300, .len = 0xc
> };
>
> static const struct dpu_dspp_sub_blks sdm845_dspp_sblk = {
> @@ -304,10 +304,10 @@
> }
>
> static struct dpu_dspp_cfg sdm845_dspp[] = {
> - DSPP_BLK("dspp_0", DSPP_0, 0x55000),
> - DSPP_BLK("dspp_1", DSPP_1, 0x57000),
> - DSPP_BLK("dspp_2", DSPP_2, 0x59000),
> - DSPP_BLK("dspp_3", DSPP_3, 0x5b000),
> + DSPP_BLK("dspp_0", DSPP_0, 0x54000),
> + DSPP_BLK("dspp_1", DSPP_1, 0x56000),
> + DSPP_BLK("dspp_2", DSPP_2, 0x58000),
> + DSPP_BLK("dspp_3", DSPP_3, 0x5a000),
> };
>
> /*************************************************************
> @@ -315,7 +315,7 @@
> *************************************************************/
> static const struct dpu_ds_top_cfg sdm845_ds_top = {
> .name = "ds_top_0", .id = DS_TOP,
> - .base = 0x61000, .len = 0xc,
> + .base = 0x60000, .len = 0xc,
> .maxinputwidth = DEFAULT_DPU_LINE_WIDTH,
> .maxoutputwidth = DEFAULT_DPU_OUTPUT_LINE_WIDTH,
> .maxupscale = MAX_UPSCALE_RATIO,
> @@ -365,10 +365,10 @@
> }
>
> static struct dpu_pingpong_cfg sdm845_pp[] = {
> - PP_BLK_TE("pingpong_0", PINGPONG_0, 0x71000),
> - PP_BLK_TE("pingpong_1", PINGPONG_1, 0x71800),
> - PP_BLK("pingpong_2", PINGPONG_2, 0x72000),
> - PP_BLK("pingpong_3", PINGPONG_3, 0x72800),
> + PP_BLK_TE("pingpong_0", PINGPONG_0, 0x70000),
> + PP_BLK_TE("pingpong_1", PINGPONG_1, 0x70800),
> + PP_BLK("pingpong_2", PINGPONG_2, 0x71000),
> + PP_BLK("pingpong_3", PINGPONG_3, 0x71800),
> };
>
> /*************************************************************
> @@ -384,10 +384,10 @@
> }
>
> static struct dpu_intf_cfg sdm845_intf[] = {
> - INTF_BLK("intf_0", INTF_0, 0x6B000, INTF_DP, 0),
> - INTF_BLK("intf_1", INTF_1, 0x6B800, INTF_DSI, 0),
> - INTF_BLK("intf_2", INTF_2, 0x6C000, INTF_DSI, 1),
> - INTF_BLK("intf_3", INTF_3, 0x6C800, INTF_DP, 1),
> + INTF_BLK("intf_0", INTF_0, 0x6A000, INTF_DP, 0),
> + INTF_BLK("intf_1", INTF_1, 0x6A800, INTF_DSI, 0),
> + INTF_BLK("intf_2", INTF_2, 0x6B000, INTF_DSI, 1),
> + INTF_BLK("intf_3", INTF_3, 0x6B800, INTF_DP, 1),
> };
>
> /*************************************************************
> @@ -401,7 +401,7 @@
> static struct dpu_wb_cfg sdm845_wb[] = {
> {
> .name = "wb_2", .id = WB_2,
> - .base = 0x66000, .len = 0x2c8,
> + .base = 0x65000, .len = 0x2c8,
> .features = WB2_SDM845_MASK,
> .sblk = &sdm845_wb2_sblk,
> .format_list = wb2_formats,
> @@ -414,7 +414,7 @@
> static struct dpu_cdm_cfg sdm845_cdm[] = {
> {
> .name = "cdm_0", .id = CDM_0,
> - .base = 0x7A200, .len = 0x224,
> + .base = 0x79200, .len = 0x224,
> .features = 0,
> .intf_connect = BIT(INTF_3),
> .wb_connect = BIT(WB_2)
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c
> index 73f084c..a7bced2 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c
> @@ -20,16 +20,16 @@
>
> /**
> * Register offsets in MDSS register file for the interrupt registers
> - * w.r.t. to the MDSS base
> + * w.r.t. to the MDP base
> */
> -#define MDP_SSPP_TOP0_OFF 0x1000
> -#define MDP_INTF_0_OFF 0x6B000
> -#define MDP_INTF_1_OFF 0x6B800
> -#define MDP_INTF_2_OFF 0x6C000
> -#define MDP_INTF_3_OFF 0x6C800
> -#define MDP_INTF_4_OFF 0x6D000
> -#define MDP_AD4_0_OFF 0x7D000
> -#define MDP_AD4_1_OFF 0x7E000
> +#define MDP_SSPP_TOP0_OFF 0x0
> +#define MDP_INTF_0_OFF 0x6A000
> +#define MDP_INTF_1_OFF 0x6A800
> +#define MDP_INTF_2_OFF 0x6B000
> +#define MDP_INTF_3_OFF 0x6B800
> +#define MDP_INTF_4_OFF 0x6C000
> +#define MDP_AD4_0_OFF 0x7C000
> +#define MDP_AD4_1_OFF 0x7D000
> #define MDP_AD4_INTR_EN_OFF 0x41c
> #define MDP_AD4_INTR_CLEAR_OFF 0x424
> #define MDP_AD4_INTR_STATUS_OFF 0x420
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno
next prev parent reply other threads:[~2018-06-19 8:59 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-14 15:26 [DPU PATCH v3 00/12] Refactor DPU device/driver hierarchy and add runtime_pm support Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 01/12] drm/msm: remove redundant pm_runtime_enable call from msm_drv Rajesh Yadav
[not found] ` <1526311619-3309-1-git-send-email-ryadav-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2018-05-14 15:26 ` [DPU PATCH v3 02/12] drm/msm/mdp5: subclass msm_mdss for mdp5 Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 03/12] drm/msm/dpu: add MDSS top level driver for dpu Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 04/12] drm/msm/dpu: create new platform driver for dpu device Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 05/12] drm/msm/dpu: update dpu sub-block offsets wrt dpu base address Rajesh Yadav
[not found] ` <1526311619-3309-6-git-send-email-ryadav-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2018-06-19 8:59 ` ryadav-sgV2jX0FEOL9JmXXK+q4OQ [this message]
2018-05-14 15:26 ` [DPU PATCH v3 06/12] drm/msm/dpu: use runtime_pm calls on dpu device Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 07/12] drm/msm/dpu: remove clock management code from dpu_power_handle Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 08/12] drm/msm/dpu: remove power " Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 09/12] drm/msm/dp: remove dpu_power_handle calls from dp driver Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 10/12] drm/msm/dpu: use runtime_pm calls in dpu_dbg Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 11/12] drm/msm/dpu: move dpu_power_handle to dpu folder Rajesh Yadav
2018-05-14 15:26 ` [DPU PATCH v3 12/12] drm/msm/dpu: add error handling in dpu_core_perf_crtc_update Rajesh Yadav
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2f600935fc7cc0eb0b78a4c2a8029434@codeaurora.org \
--to=ryadav-sgv2jx0feol9jmxxk+q4oq@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=hoegsberg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).