drm/msm/dpu: remove hwversion field from data structures
The driver should not depend on hw revision for detecting features. Instead it should use features from the hw catalog. Drop the hwversion field from struct dpu_mdss_cfg and struct dpu_hw_blk_reg_map. Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Patchwork: https://patchwork.freedesktop.org/patch/488160/ Link: https://lore.kernel.org/r/20220602202447.1755115-4-dmitry.baryshkov@linaro.org Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
This commit is contained in:
parent
c87443150a
commit
129ae533fa
|
|
@ -2004,7 +2004,6 @@ struct dpu_mdss_cfg *dpu_hw_catalog_init(u32 hw_rev)
|
|||
for (i = 0; i < ARRAY_SIZE(cfg_handler); i++) {
|
||||
if (cfg_handler[i].hw_rev == hw_rev) {
|
||||
cfg_handler[i].cfg_init(dpu_cfg);
|
||||
dpu_cfg->hwversion = hw_rev;
|
||||
return dpu_cfg;
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -826,8 +826,6 @@ struct dpu_perf_cfg {
|
|||
* @mdss_irqs: Bitmap with the irqs supported by the target
|
||||
*/
|
||||
struct dpu_mdss_cfg {
|
||||
u32 hwversion;
|
||||
|
||||
const struct dpu_caps *caps;
|
||||
|
||||
u32 mdp_count;
|
||||
|
|
|
|||
|
|
@ -61,7 +61,6 @@ static const struct dpu_ctl_cfg *_ctl_offset(enum dpu_ctl ctl,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->ctl[i].base;
|
||||
b->length = m->ctl[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_CTL;
|
||||
return &m->ctl[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -169,7 +169,6 @@ static struct dpu_dsc_cfg *_dsc_offset(enum dpu_dsc dsc,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->dsc[i].base;
|
||||
b->length = m->dsc[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_DSC;
|
||||
return &m->dsc[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -83,7 +83,6 @@ static const struct dpu_dspp_cfg *_dspp_offset(enum dpu_dspp dspp,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->dspp[i].base;
|
||||
b->length = m->dspp[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_DSPP;
|
||||
return &m->dspp[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -403,7 +403,6 @@ static void __intr_offset(struct dpu_mdss_cfg *m,
|
|||
{
|
||||
hw->base_off = addr;
|
||||
hw->blk_off = m->mdp[0].base;
|
||||
hw->hwversion = m->hwversion;
|
||||
}
|
||||
|
||||
struct dpu_hw_intr *dpu_hw_intr_init(void __iomem *addr,
|
||||
|
|
|
|||
|
|
@ -80,7 +80,6 @@ static const struct dpu_intf_cfg *_intf_offset(enum dpu_intf intf,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->intf[i].base;
|
||||
b->length = m->intf[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_INTF;
|
||||
return &m->intf[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -46,7 +46,6 @@ static const struct dpu_lm_cfg *_lm_offset(enum dpu_lm mixer,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->mixer[i].base;
|
||||
b->length = m->mixer[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_LM;
|
||||
return &m->mixer[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -26,7 +26,6 @@ static const struct dpu_merge_3d_cfg *_merge_3d_offset(enum dpu_merge_3d idx,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->merge_3d[i].base;
|
||||
b->length = m->merge_3d[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_PINGPONG;
|
||||
return &m->merge_3d[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -54,7 +54,6 @@ static const struct dpu_pingpong_cfg *_pingpong_offset(enum dpu_pingpong pp,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->pingpong[i].base;
|
||||
b->length = m->pingpong[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_PINGPONG;
|
||||
return &m->pingpong[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -772,7 +772,6 @@ static const struct dpu_sspp_cfg *_sspp_offset(enum dpu_sspp sspp,
|
|||
b->base_off = addr;
|
||||
b->blk_off = catalog->sspp[i].base;
|
||||
b->length = catalog->sspp[i].len;
|
||||
b->hwversion = catalog->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_SSPP;
|
||||
return &catalog->sspp[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -288,7 +288,6 @@ static const struct dpu_mdp_cfg *_top_offset(enum dpu_mdp mdp,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->mdp[i].base;
|
||||
b->length = m->mdp[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_TOP;
|
||||
return &m->mdp[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -21,14 +21,12 @@
|
|||
* @blk_off: pipe offset relative to mdss offset
|
||||
* @length length of register block offset
|
||||
* @xin_id xin id
|
||||
* @hwversion mdss hw version number
|
||||
*/
|
||||
struct dpu_hw_blk_reg_map {
|
||||
void __iomem *base_off;
|
||||
u32 blk_off;
|
||||
u32 length;
|
||||
u32 xin_id;
|
||||
u32 hwversion;
|
||||
u32 log_mask;
|
||||
};
|
||||
|
||||
|
|
|
|||
|
|
@ -223,7 +223,6 @@ static const struct dpu_vbif_cfg *_top_offset(enum dpu_vbif vbif,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->vbif[i].base;
|
||||
b->length = m->vbif[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
b->log_mask = DPU_DBG_MASK_VBIF;
|
||||
return &m->vbif[i];
|
||||
}
|
||||
|
|
|
|||
|
|
@ -63,7 +63,6 @@ static const struct dpu_wb_cfg *_wb_offset(enum dpu_wb wb,
|
|||
b->base_off = addr;
|
||||
b->blk_off = m->wb[i].base;
|
||||
b->length = m->wb[i].len;
|
||||
b->hwversion = m->hwversion;
|
||||
return &m->wb[i];
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue