Re: [PATCH v2,2/2] drm: mediatek: Adjust the dpi output format to MT8186

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2022-08-19 at 14:43 -0400, Nícolas F. R. A. Prado wrote:
> Hi Xinlei,
> 
> On Fri, Aug 05, 2022 at 05:57:41PM +0800, xinlei.lee@xxxxxxxxxxxx
> wrote:
> > From: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx>
> > 
> > Dpi output needs to adjust the output format to dual edge for
> > MT8186.
> 
> Could you explain a bit more why this is needed? What does this
> configuration
> do? And why is MMSYS involved in a DPI configuration?
> 
> Also, is this a configuration specific to MT8186? I enabled the flag
> on MT8192
> just to test and didn't see any difference in behavior.
> 
> > 
> > Signed-off-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx>
> 
> If Jitao also helped in the writing of this code, then there should
> also be a
> co-developed-by tag for Jitao before the Signed-off-by [1].
> 
> [1] 
> https://urldefense.com/v3/__https://www.kernel.org/doc/html/latest/process/submitting-patches.html*when-to-use-acked-by-cc-and-co-developed-by__;Iw!!CTRNKA9wMg0ARbw!wlM0Ma8TeMR5EM0eUg1Z1JIDK_SF5rATf2UtVFJATWlzCSYABFbhD8yIo-_YxePWwQ$
>  
> 
> > Signed-off-by: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx>
> 
> [..]
> > --- a/drivers/gpu/drm/mediatek/mtk_dpi.c
> > +++ b/drivers/gpu/drm/mediatek/mtk_dpi.c
> 
> [..]
> > @@ -151,6 +154,7 @@ struct mtk_dpi_conf {
> >  	u32 yuv422_en_bit;
> >  	u32 csc_enable_bit;
> >  	u32 pixels_per_iter;
> > +	bool rgb888_dual_enable;
> 
> You're missing a description of this property in the struct's comment
> above:
> 
> drivers/gpu/drm/mediatek/mtk_dpi.c:160: warning: Function parameter
> or member 'rgb888_dual_enable' not described in 'mtk_dpi_conf'
> 
> Thanks,
> Nícolas

Hi Nícolas:

Thanks for your careful review.

First of all, MT8186 needs to set this bit of MMSYS because HW has been
modified at that time, SW needs to cooperate.
And the register (MMSYS) reserved for dpi will be used for output
format control (dual_edge/single_edge).
Since it is a modification for
MT8186, setting it on MT8192 will not have any effect.

If there are no other comments, I will revise it in the next edition:
1. Modify the label of jitao to Co-developed-by;
2. Add missing a description of this property in the mtk_dpi_conf.

Best Regards!
Xinlei






[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux