Daniel Vetter
>> ,Matthias Brugger ,DRI Development
>> ,Linux ARM
>> ,"moderated list:ARM/Mediatek SoC
>> support" ,linux-kernel
>> ,opensource.ker...@vivo.com
>> 主题:Re: [PATCH] drm/mediatek: cleanup coding style in mediatek a bit>Hi,
>> Bernard:
>&
Hi Bernard,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on pza/reset/next]
[also build test ERROR on drm-intel/for-linux-next drm-tip/drm-tip linus/master
v5.7-rc3 next-20200430]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve
st:ARM/Mediatek SoC
> support" ,linux-kernel
> ,opensource.ker...@vivo.com
> 主题:Re: [PATCH] drm/mediatek: cleanup coding style in mediatek a bit>Hi,
> Bernard:
> >
> >Bernard Zhao 於 2020年4月27日 週一 下午3:53寫道:
> >>
> >> This code change is to make cod
m/mediatek: cleanup coding style in mediatek a bit>Hi, Bernard:
>
>Bernard Zhao 於 2020年4月27日 週一 下午3:53寫道:
>>
>> This code change is to make code bit more readable.
>> Optimise array size align to HDMI macro define.
>> Add check if len is overange.
>
>One patch should
Hi, Bernard:
Bernard Zhao 於 2020年4月27日 週一 下午3:53寫道:
>
> This code change is to make code bit more readable.
> Optimise array size align to HDMI macro define.
> Add check if len is overange.
One patch should just do one thing, but this do three things.
So break this into three patches.
Regards,
> This code change is to make code bit more readable.
> Optimise array size align to HDMI macro define.
> Add check if len is overange.
I find it safer to handle also such source code adjustments
by a small patch series together with improved commit messages.
Regards,
Markus
_
This code change is to make code bit more readable.
Optimise array size align to HDMI macro define.
Add check if len is overange.
Signed-off-by: Bernard Zhao
---
drivers/gpu/drm/mediatek/mtk_hdmi.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drive