Messages in this thread Patch in this message | | | From | Shawn Sung (宋孝謙) <> | Subject | Re: [PATCH 1/1] drm/mediatek: Fix errors when reporting rotation capability | Date | Fri, 27 Oct 2023 01:30:52 +0000 |
| |
Hi CK,
On Thu, 2023-10-26 at 08:29 +0000, CK Hu (胡俊光) wrote: > Hi, Hsiao-chien: > > On Thu, 2023-10-26 at 08:27 +0000, Shawn Sung (宋孝謙) wrote: > > Hi CK, > > > > On Thu, 2023-10-26 at 01:53 +0000, CK Hu (胡俊光) wrote: > > > Hi, Hsiao-chien: > > > > > > On Mon, 2023-09-18 at 15:10 +0800, Hsiao Chien Sung wrote: > > > > For CRTCs that doesn't support rotation should still return > > > > DRM_MODE_ROTATE_0. Since both OVL and OVL adaptor on MTK chip > > > > doesn't support rotation, return the capability of the > > > > hardware accordingly. > > > > > > > > Fixes: df4444577118 ("drm/mediatek: Support 180 degree > > > > rotation") > > > > Fixes: 84d805753983 ("drm/mediatek: Support reflect-y plane > > > > rotation") > > > > > > > > Signed-off-by: Hsiao Chien Sung <shawn.sung@mediatek.com> > > > > --- > > > > drivers/gpu/drm/mediatek/mtk_disp_drv.h | 1 + > > > > drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 8 +------- > > > > drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 5 +++++ > > > > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 1 + > > > > drivers/gpu/drm/mediatek/mtk_drm_plane.c | 2 +- > > > > 5 files changed, 9 insertions(+), 8 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/mediatek/mtk_disp_drv.h > > > > b/drivers/gpu/drm/mediatek/mtk_disp_drv.h > > > > index 2254038519e1..f4c7dba8f73d 100644 > > > > --- a/drivers/gpu/drm/mediatek/mtk_disp_drv.h > > > > +++ b/drivers/gpu/drm/mediatek/mtk_disp_drv.h > > > > @@ -120,6 +120,7 @@ void > > > > mtk_ovl_adaptor_register_vblank_cb(struct > > > > device *dev, void (*vblank_cb)(vo > > > > void mtk_ovl_adaptor_unregister_vblank_cb(struct device *dev); > > > > void mtk_ovl_adaptor_enable_vblank(struct device *dev); > > > > void mtk_ovl_adaptor_disable_vblank(struct device *dev); > > > > +unsigned int mtk_ovl_adaptor_supported_rotations(struct device > > > > *dev); > > > > void mtk_ovl_adaptor_start(struct device *dev); > > > > void mtk_ovl_adaptor_stop(struct device *dev); > > > > unsigned int mtk_ovl_adaptor_layer_nr(struct device *dev); > > > > diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c > > > > b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c > > > > index 2bffe4245466..5fd1f6ae41f3 100644 > > > > --- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c > > > > +++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c > > > > @@ -288,8 +288,7 @@ unsigned int mtk_ovl_layer_nr(struct device > > > > *dev) > > > > > > > > unsigned int mtk_ovl_supported_rotations(struct device *dev) > > > > { > > > > - return DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180 | > > > > - DRM_MODE_REFLECT_X | DRM_MODE_REFLECT_Y; > > > > + return DRM_MODE_ROTATE_0 | DRM_MODE_REFLECT_X | > > > > DRM_MODE_REFLECT_Y; > > > > > > If ovl enable reflect_x and reflect_y at the same time, it is > > > rotate > > > 180, right? > > > > > > Regards, > > > CK > > > > Yes, if the user do such an operation, the result is equal to 180 > > degrees rotation. > > So ovl support rotate 180, why do you remove this capability? >
Thank you for checking. Will run IGT test to verify again.
Best, Shawn
| |