<html><body><p>
<pre>
Hi CK,
On Fri, 2024-03-01 at 07:21 +0000, CK Hu (胡俊光) wrote:
> Hi, Hsiao-chien:
>
> On Thu, 2024-02-15 at 18:11 +0800, Hsiao Chien Sung wrote:
> > Set DRM mode configs limitation according to the hardware
> > capabilities
> > and pass the IGT checks as below:
> >
> > - The test "graphics.IgtKms.kms_plane" requires a frame buffer with
> > width of 4512 pixels (> 4096).
> > - The test "graphics.IgtKms.kms_cursor_crc" checks if the cursor
> > size
> > is
> > defined, and run the test with cursor size from 1x1 to 512x512.
> >
> > Please notice that the test conditions may change as IGT is
> > updated.
> >
> > Signed-off-by: Hsiao Chien Sung <shawn.sung@mediatek.com>
> > ---
> > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 25
> > +++++++++++++++++++++++++
> > drivers/gpu/drm/mediatek/mtk_drm_drv.h | 3 +++
> > 2 files changed, 28 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> > b/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> > index 890e1e93a2227..8cf157ec66ba6 100644
> > --- a/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> > +++ b/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> > @@ -296,6 +296,9 @@ static const struct mtk_mmsys_driver_data
> > mt8188_vdosys0_driver_data = {
> > .conn_routes = mt8188_mtk_ddp_main_routes,
> > .num_conn_routes = ARRAY_SIZE(mt8188_mtk_ddp_main_routes),
> > .mmsys_dev_num = 2,
> > +.max_pitch = GENMASK(15, 0),
> > +.min_width = 1,
> > +.min_height = 1,
> > };
> >
> > static const struct mtk_mmsys_driver_data mt8192_mmsys_driver_data
> > =
> > {
> > @@ -310,6 +313,9 @@ static const struct mtk_mmsys_driver_data
> > mt8195_vdosys0_driver_data = {
> > .main_path = mt8195_mtk_ddp_main,
> > .main_len = ARRAY_SIZE(mt8195_mtk_ddp_main),
> > .mmsys_dev_num = 2,
> > +.max_pitch = GENMASK(15, 0),
> > +.min_width = 1,
> > +.min_height = 1,
> > };
> >
> > static const struct mtk_mmsys_driver_data
> > mt8195_vdosys1_driver_data
> > = {
> > @@ -317,6 +323,9 @@ static const struct mtk_mmsys_driver_data
> > mt8195_vdosys1_driver_data = {
> > .ext_len = ARRAY_SIZE(mt8195_mtk_ddp_ext),
> > .mmsys_id = 1,
> > .mmsys_dev_num = 2,
> > +.max_pitch = GENMASK(15, 0),
> > +.min_width = 2, /* 2-pixel align when ethdr is bypassed */
> > +.min_height = 1,
> > };
> >
> > static const struct of_device_id mtk_drm_of_ids[] = {
> > @@ -495,6 +504,18 @@ static int mtk_drm_kms_init(struct drm_device
> > *drm)
> > for (j = 0; j < private->data->mmsys_dev_num; j++) {
> > priv_n = private->all_drm_private[j];
> >
> > +if (priv_n->data->max_pitch) {
> > +/* Save 4 bytes for the color depth
> > (pitch = width * bpp) */
> > +drm->mode_config.max_width = priv_n-
> > > data->max_pitch >> 2;
> >
> > +drm->mode_config.max_height = priv_n-
> > > data->max_pitch >> 2;
>
> I think the term 'pitch' is for width not for height. And I think you
> should not divide height by 4. So I would like to have priv_n->data-
> > max_height.
>
Got it. Will change another way to implement this part.
I'll remain the orginal logic that set the maximum width/height to 4096
and overwrite it only if the new value is defined in the driver data.
Regards,
Shawn
</pre>
</p></body></html><!--type:text--><!--{--><pre>************* MEDIATEK Confidentiality Notice ********************
The information contained in this e-mail message (including any
attachments) may be confidential, proprietary, privileged, or otherwise
exempt from disclosure under applicable laws. It is intended to be
conveyed only to the designated recipient(s). Any use, dissemination,
distribution, printing, retaining or copying of this e-mail (including its
attachments) by unintended recipient(s) is strictly prohibited and may
be unlawful. If you are not an intended recipient of this e-mail, or believe
that you have received this e-mail in error, please notify the sender
immediately (by replying to this e-mail), delete any and all copies of
this e-mail (including any attachments) from your system, and do not
disclose the content of this e-mail to any other person. Thank you!
</pre><!--}-->