Commit f287c66a authored by Justin Green's avatar Justin Green Committed by Chun-Kuang Hu

drm/mediatek: Refactor pixel format logic

Add an DDP component interface for querying pixel format support
and move list of supported pixel formats into DDP components
instead of mtk_drm_plane.c

Tested by running Chrome on an MT8195.
Signed-off-by: default avatarJustin Green <greenjustin@chromium.org>
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://patchwork.kernel.org/project/linux-mediatek/patch/20230309210416.1167020-2-greenjustin@chromium.org/Signed-off-by: default avatarChun-Kuang Hu <chunkuang.hu@kernel.org>
parent 813420eb
...@@ -96,6 +96,8 @@ void mtk_ovl_register_vblank_cb(struct device *dev, ...@@ -96,6 +96,8 @@ void mtk_ovl_register_vblank_cb(struct device *dev,
void mtk_ovl_unregister_vblank_cb(struct device *dev); void mtk_ovl_unregister_vblank_cb(struct device *dev);
void mtk_ovl_enable_vblank(struct device *dev); void mtk_ovl_enable_vblank(struct device *dev);
void mtk_ovl_disable_vblank(struct device *dev); void mtk_ovl_disable_vblank(struct device *dev);
const u32 *mtk_ovl_get_formats(struct device *dev);
size_t mtk_ovl_get_num_formats(struct device *dev);
void mtk_rdma_bypass_shadow(struct device *dev); void mtk_rdma_bypass_shadow(struct device *dev);
int mtk_rdma_clk_enable(struct device *dev); int mtk_rdma_clk_enable(struct device *dev);
...@@ -115,6 +117,8 @@ void mtk_rdma_register_vblank_cb(struct device *dev, ...@@ -115,6 +117,8 @@ void mtk_rdma_register_vblank_cb(struct device *dev,
void mtk_rdma_unregister_vblank_cb(struct device *dev); void mtk_rdma_unregister_vblank_cb(struct device *dev);
void mtk_rdma_enable_vblank(struct device *dev); void mtk_rdma_enable_vblank(struct device *dev);
void mtk_rdma_disable_vblank(struct device *dev); void mtk_rdma_disable_vblank(struct device *dev);
const u32 *mtk_rdma_get_formats(struct device *dev);
size_t mtk_rdma_get_num_formats(struct device *dev);
int mtk_mdp_rdma_clk_enable(struct device *dev); int mtk_mdp_rdma_clk_enable(struct device *dev);
void mtk_mdp_rdma_clk_disable(struct device *dev); void mtk_mdp_rdma_clk_disable(struct device *dev);
......
...@@ -67,6 +67,20 @@ ...@@ -67,6 +67,20 @@
#define OVL_CON_VIRT_FLIP BIT(9) #define OVL_CON_VIRT_FLIP BIT(9)
#define OVL_CON_HORZ_FLIP BIT(10) #define OVL_CON_HORZ_FLIP BIT(10)
static const u32 mt8173_formats[] = {
DRM_FORMAT_XRGB8888,
DRM_FORMAT_ARGB8888,
DRM_FORMAT_BGRX8888,
DRM_FORMAT_BGRA8888,
DRM_FORMAT_ABGR8888,
DRM_FORMAT_XBGR8888,
DRM_FORMAT_RGB888,
DRM_FORMAT_BGR888,
DRM_FORMAT_RGB565,
DRM_FORMAT_UYVY,
DRM_FORMAT_YUYV,
};
struct mtk_disp_ovl_data { struct mtk_disp_ovl_data {
unsigned int addr; unsigned int addr;
unsigned int gmc_bits; unsigned int gmc_bits;
...@@ -74,6 +88,8 @@ struct mtk_disp_ovl_data { ...@@ -74,6 +88,8 @@ struct mtk_disp_ovl_data {
bool fmt_rgb565_is_0; bool fmt_rgb565_is_0;
bool smi_id_en; bool smi_id_en;
bool supports_afbc; bool supports_afbc;
const u32 *formats;
size_t num_formats;
}; };
/* /*
...@@ -139,6 +155,20 @@ void mtk_ovl_disable_vblank(struct device *dev) ...@@ -139,6 +155,20 @@ void mtk_ovl_disable_vblank(struct device *dev)
writel_relaxed(0x0, ovl->regs + DISP_REG_OVL_INTEN); writel_relaxed(0x0, ovl->regs + DISP_REG_OVL_INTEN);
} }
const u32 *mtk_ovl_get_formats(struct device *dev)
{
struct mtk_disp_ovl *ovl = dev_get_drvdata(dev);
return ovl->data->formats;
}
size_t mtk_ovl_get_num_formats(struct device *dev)
{
struct mtk_disp_ovl *ovl = dev_get_drvdata(dev);
return ovl->data->num_formats;
}
int mtk_ovl_clk_enable(struct device *dev) int mtk_ovl_clk_enable(struct device *dev)
{ {
struct mtk_disp_ovl *ovl = dev_get_drvdata(dev); struct mtk_disp_ovl *ovl = dev_get_drvdata(dev);
...@@ -496,6 +526,8 @@ static const struct mtk_disp_ovl_data mt2701_ovl_driver_data = { ...@@ -496,6 +526,8 @@ static const struct mtk_disp_ovl_data mt2701_ovl_driver_data = {
.gmc_bits = 8, .gmc_bits = 8,
.layer_nr = 4, .layer_nr = 4,
.fmt_rgb565_is_0 = false, .fmt_rgb565_is_0 = false,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8173_ovl_driver_data = { static const struct mtk_disp_ovl_data mt8173_ovl_driver_data = {
...@@ -503,6 +535,8 @@ static const struct mtk_disp_ovl_data mt8173_ovl_driver_data = { ...@@ -503,6 +535,8 @@ static const struct mtk_disp_ovl_data mt8173_ovl_driver_data = {
.gmc_bits = 8, .gmc_bits = 8,
.layer_nr = 4, .layer_nr = 4,
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8183_ovl_driver_data = { static const struct mtk_disp_ovl_data mt8183_ovl_driver_data = {
...@@ -510,6 +544,8 @@ static const struct mtk_disp_ovl_data mt8183_ovl_driver_data = { ...@@ -510,6 +544,8 @@ static const struct mtk_disp_ovl_data mt8183_ovl_driver_data = {
.gmc_bits = 10, .gmc_bits = 10,
.layer_nr = 4, .layer_nr = 4,
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8183_ovl_2l_driver_data = { static const struct mtk_disp_ovl_data mt8183_ovl_2l_driver_data = {
...@@ -517,6 +553,8 @@ static const struct mtk_disp_ovl_data mt8183_ovl_2l_driver_data = { ...@@ -517,6 +553,8 @@ static const struct mtk_disp_ovl_data mt8183_ovl_2l_driver_data = {
.gmc_bits = 10, .gmc_bits = 10,
.layer_nr = 2, .layer_nr = 2,
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8192_ovl_driver_data = { static const struct mtk_disp_ovl_data mt8192_ovl_driver_data = {
...@@ -525,6 +563,8 @@ static const struct mtk_disp_ovl_data mt8192_ovl_driver_data = { ...@@ -525,6 +563,8 @@ static const struct mtk_disp_ovl_data mt8192_ovl_driver_data = {
.layer_nr = 4, .layer_nr = 4,
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.smi_id_en = true, .smi_id_en = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8192_ovl_2l_driver_data = { static const struct mtk_disp_ovl_data mt8192_ovl_2l_driver_data = {
...@@ -533,6 +573,8 @@ static const struct mtk_disp_ovl_data mt8192_ovl_2l_driver_data = { ...@@ -533,6 +573,8 @@ static const struct mtk_disp_ovl_data mt8192_ovl_2l_driver_data = {
.layer_nr = 2, .layer_nr = 2,
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.smi_id_en = true, .smi_id_en = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_ovl_data mt8195_ovl_driver_data = { static const struct mtk_disp_ovl_data mt8195_ovl_driver_data = {
...@@ -542,6 +584,8 @@ static const struct mtk_disp_ovl_data mt8195_ovl_driver_data = { ...@@ -542,6 +584,8 @@ static const struct mtk_disp_ovl_data mt8195_ovl_driver_data = {
.fmt_rgb565_is_0 = true, .fmt_rgb565_is_0 = true,
.smi_id_en = true, .smi_id_en = true,
.supports_afbc = true, .supports_afbc = true,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct of_device_id mtk_disp_ovl_driver_dt_match[] = { static const struct of_device_id mtk_disp_ovl_driver_dt_match[] = {
......
...@@ -55,8 +55,24 @@ ...@@ -55,8 +55,24 @@
#define RDMA_MEM_GMC 0x40402020 #define RDMA_MEM_GMC 0x40402020
static const u32 mt8173_formats[] = {
DRM_FORMAT_XRGB8888,
DRM_FORMAT_ARGB8888,
DRM_FORMAT_BGRX8888,
DRM_FORMAT_BGRA8888,
DRM_FORMAT_ABGR8888,
DRM_FORMAT_XBGR8888,
DRM_FORMAT_RGB888,
DRM_FORMAT_BGR888,
DRM_FORMAT_RGB565,
DRM_FORMAT_UYVY,
DRM_FORMAT_YUYV,
};
struct mtk_disp_rdma_data { struct mtk_disp_rdma_data {
unsigned int fifo_size; unsigned int fifo_size;
const u32 *formats;
size_t num_formats;
}; };
/* /*
...@@ -127,6 +143,20 @@ void mtk_rdma_disable_vblank(struct device *dev) ...@@ -127,6 +143,20 @@ void mtk_rdma_disable_vblank(struct device *dev)
rdma_update_bits(dev, DISP_REG_RDMA_INT_ENABLE, RDMA_FRAME_END_INT, 0); rdma_update_bits(dev, DISP_REG_RDMA_INT_ENABLE, RDMA_FRAME_END_INT, 0);
} }
const u32 *mtk_rdma_get_formats(struct device *dev)
{
struct mtk_disp_rdma *rdma = dev_get_drvdata(dev);
return rdma->data->formats;
}
size_t mtk_rdma_get_num_formats(struct device *dev)
{
struct mtk_disp_rdma *rdma = dev_get_drvdata(dev);
return rdma->data->num_formats;
}
int mtk_rdma_clk_enable(struct device *dev) int mtk_rdma_clk_enable(struct device *dev)
{ {
struct mtk_disp_rdma *rdma = dev_get_drvdata(dev); struct mtk_disp_rdma *rdma = dev_get_drvdata(dev);
...@@ -361,18 +391,26 @@ static int mtk_disp_rdma_remove(struct platform_device *pdev) ...@@ -361,18 +391,26 @@ static int mtk_disp_rdma_remove(struct platform_device *pdev)
static const struct mtk_disp_rdma_data mt2701_rdma_driver_data = { static const struct mtk_disp_rdma_data mt2701_rdma_driver_data = {
.fifo_size = SZ_4K, .fifo_size = SZ_4K,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_rdma_data mt8173_rdma_driver_data = { static const struct mtk_disp_rdma_data mt8173_rdma_driver_data = {
.fifo_size = SZ_8K, .fifo_size = SZ_8K,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_rdma_data mt8183_rdma_driver_data = { static const struct mtk_disp_rdma_data mt8183_rdma_driver_data = {
.fifo_size = 5 * SZ_1K, .fifo_size = 5 * SZ_1K,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct mtk_disp_rdma_data mt8195_rdma_driver_data = { static const struct mtk_disp_rdma_data mt8195_rdma_driver_data = {
.fifo_size = 1920, .fifo_size = 1920,
.formats = mt8173_formats,
.num_formats = ARRAY_SIZE(mt8173_formats),
}; };
static const struct of_device_id mtk_disp_rdma_driver_dt_match[] = { static const struct of_device_id mtk_disp_rdma_driver_dt_match[] = {
......
...@@ -856,7 +856,9 @@ static int mtk_drm_crtc_init_comp_planes(struct drm_device *drm_dev, ...@@ -856,7 +856,9 @@ static int mtk_drm_crtc_init_comp_planes(struct drm_device *drm_dev,
BIT(pipe), BIT(pipe),
mtk_drm_crtc_plane_type(mtk_crtc->layer_nr, mtk_drm_crtc_plane_type(mtk_crtc->layer_nr,
num_planes), num_planes),
mtk_ddp_comp_supported_rotations(comp)); mtk_ddp_comp_supported_rotations(comp),
mtk_ddp_comp_get_formats(comp),
mtk_ddp_comp_get_num_formats(comp));
if (ret) if (ret)
return ret; return ret;
......
...@@ -359,6 +359,8 @@ static const struct mtk_ddp_comp_funcs ddp_ovl = { ...@@ -359,6 +359,8 @@ static const struct mtk_ddp_comp_funcs ddp_ovl = {
.layer_config = mtk_ovl_layer_config, .layer_config = mtk_ovl_layer_config,
.bgclr_in_on = mtk_ovl_bgclr_in_on, .bgclr_in_on = mtk_ovl_bgclr_in_on,
.bgclr_in_off = mtk_ovl_bgclr_in_off, .bgclr_in_off = mtk_ovl_bgclr_in_off,
.get_formats = mtk_ovl_get_formats,
.get_num_formats = mtk_ovl_get_num_formats,
}; };
static const struct mtk_ddp_comp_funcs ddp_postmask = { static const struct mtk_ddp_comp_funcs ddp_postmask = {
...@@ -381,6 +383,8 @@ static const struct mtk_ddp_comp_funcs ddp_rdma = { ...@@ -381,6 +383,8 @@ static const struct mtk_ddp_comp_funcs ddp_rdma = {
.disable_vblank = mtk_rdma_disable_vblank, .disable_vblank = mtk_rdma_disable_vblank,
.layer_nr = mtk_rdma_layer_nr, .layer_nr = mtk_rdma_layer_nr,
.layer_config = mtk_rdma_layer_config, .layer_config = mtk_rdma_layer_config,
.get_formats = mtk_rdma_get_formats,
.get_num_formats = mtk_rdma_get_num_formats,
}; };
static const struct mtk_ddp_comp_funcs ddp_ufoe = { static const struct mtk_ddp_comp_funcs ddp_ufoe = {
......
...@@ -71,6 +71,8 @@ struct mtk_ddp_comp_funcs { ...@@ -71,6 +71,8 @@ struct mtk_ddp_comp_funcs {
void (*bgclr_in_off)(struct device *dev); void (*bgclr_in_off)(struct device *dev);
void (*ctm_set)(struct device *dev, void (*ctm_set)(struct device *dev,
struct drm_crtc_state *state); struct drm_crtc_state *state);
const u32 *(*get_formats)(struct device *dev);
size_t (*get_num_formats)(struct device *dev);
}; };
struct mtk_ddp_comp { struct mtk_ddp_comp {
...@@ -203,6 +205,24 @@ static inline void mtk_ddp_ctm_set(struct mtk_ddp_comp *comp, ...@@ -203,6 +205,24 @@ static inline void mtk_ddp_ctm_set(struct mtk_ddp_comp *comp,
comp->funcs->ctm_set(comp->dev, state); comp->funcs->ctm_set(comp->dev, state);
} }
static inline
const u32 *mtk_ddp_comp_get_formats(struct mtk_ddp_comp *comp)
{
if (comp->funcs && comp->funcs->get_formats)
return comp->funcs->get_formats(comp->dev);
return NULL;
}
static inline
size_t mtk_ddp_comp_get_num_formats(struct mtk_ddp_comp *comp)
{
if (comp->funcs && comp->funcs->get_num_formats)
return comp->funcs->get_num_formats(comp->dev);
return 0;
}
int mtk_ddp_comp_get_id(struct device_node *node, int mtk_ddp_comp_get_id(struct device_node *node,
enum mtk_ddp_comp_type comp_type); enum mtk_ddp_comp_type comp_type);
unsigned int mtk_drm_find_possible_crtc_by_comp(struct drm_device *drm, unsigned int mtk_drm_find_possible_crtc_by_comp(struct drm_device *drm,
......
...@@ -19,20 +19,6 @@ ...@@ -19,20 +19,6 @@
#include "mtk_drm_gem.h" #include "mtk_drm_gem.h"
#include "mtk_drm_plane.h" #include "mtk_drm_plane.h"
static const u32 formats[] = {
DRM_FORMAT_XRGB8888,
DRM_FORMAT_ARGB8888,
DRM_FORMAT_BGRX8888,
DRM_FORMAT_BGRA8888,
DRM_FORMAT_ABGR8888,
DRM_FORMAT_XBGR8888,
DRM_FORMAT_RGB888,
DRM_FORMAT_BGR888,
DRM_FORMAT_RGB565,
DRM_FORMAT_UYVY,
DRM_FORMAT_YUYV,
};
static const u64 modifiers[] = { static const u64 modifiers[] = {
DRM_FORMAT_MOD_LINEAR, DRM_FORMAT_MOD_LINEAR,
DRM_FORMAT_MOD_ARM_AFBC(AFBC_FORMAT_MOD_BLOCK_SIZE_32x8 | DRM_FORMAT_MOD_ARM_AFBC(AFBC_FORMAT_MOD_BLOCK_SIZE_32x8 |
...@@ -315,13 +301,19 @@ static const struct drm_plane_helper_funcs mtk_plane_helper_funcs = { ...@@ -315,13 +301,19 @@ static const struct drm_plane_helper_funcs mtk_plane_helper_funcs = {
int mtk_plane_init(struct drm_device *dev, struct drm_plane *plane, int mtk_plane_init(struct drm_device *dev, struct drm_plane *plane,
unsigned long possible_crtcs, enum drm_plane_type type, unsigned long possible_crtcs, enum drm_plane_type type,
unsigned int supported_rotations) unsigned int supported_rotations, const u32 *formats,
size_t num_formats)
{ {
int err; int err;
if (!formats || !num_formats) {
DRM_ERROR("no formats for plane\n");
return -EINVAL;
}
err = drm_universal_plane_init(dev, plane, possible_crtcs, err = drm_universal_plane_init(dev, plane, possible_crtcs,
&mtk_plane_funcs, formats, &mtk_plane_funcs, formats,
ARRAY_SIZE(formats), modifiers, type, NULL); num_formats, modifiers, type, NULL);
if (err) { if (err) {
DRM_ERROR("failed to initialize plane\n"); DRM_ERROR("failed to initialize plane\n");
return err; return err;
......
...@@ -48,6 +48,7 @@ to_mtk_plane_state(struct drm_plane_state *state) ...@@ -48,6 +48,7 @@ to_mtk_plane_state(struct drm_plane_state *state)
int mtk_plane_init(struct drm_device *dev, struct drm_plane *plane, int mtk_plane_init(struct drm_device *dev, struct drm_plane *plane,
unsigned long possible_crtcs, enum drm_plane_type type, unsigned long possible_crtcs, enum drm_plane_type type,
unsigned int supported_rotations); unsigned int supported_rotations, const u32 *formats,
size_t num_formats);
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment