mbox series

[v2,0/3] Add additional YUV420 bus format check for dw-meson's bridge enable

Message ID 20230612112354.4034489-1-adrian.larumbe@collabora.com (mailing list archive)
Headers show
Series Add additional YUV420 bus format check for dw-meson's bridge enable | expand

Message

Adrián Larumbe June 12, 2023, 11:23 a.m. UTC 
This is v2 of:
https://lore.kernel.org/dri-devel/20230528140001.1057084-1-adrian.larumbe@collabora.com/

The only difference is having added an actual commit message to patch
number 3 in the series.

Adrián Larumbe (3):
  drm/meson: dw-hdmi: change YUV420 selection logic at clock setup
  dw-hdmi: truly enforce 420-only formats when drm mode demands it
  dw-hdmi: remove dead code and fix indentation

 drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 35 +++++++++--------------
 drivers/gpu/drm/meson/meson_dw_hdmi.c     |  4 +--
 include/drm/bridge/dw_hdmi.h              |  2 ++
 3 files changed, 18 insertions(+), 23 deletions(-)

Comments

Neil Armstrong June 12, 2023, 12:34 p.m. UTC | #1 
Hi,

On 12/06/2023 13:23, Adrián Larumbe wrote:
> This is v2 of:
> https://lore.kernel.org/dri-devel/20230528140001.1057084-1-adrian.larumbe@collabora.com/
> 
> The only difference is having added an actual commit message to patch
> number 3 in the series.

Seems you didn't keep my Acked-by/Reviewed-by and you used my old narmstrong@baylibre.com
email address.... and you only selected the Meson & Bridge maintainers and not the reviewers.

> 
> Adrián Larumbe (3):
>    drm/meson: dw-hdmi: change YUV420 selection logic at clock setup
>    dw-hdmi: truly enforce 420-only formats when drm mode demands it

The subject should be: "drm/bridge: dw-hdmi: ..."

>    dw-hdmi: remove dead code and fix indentation

Ditto

> 
>   drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 35 +++++++++--------------
>   drivers/gpu/drm/meson/meson_dw_hdmi.c     |  4 +--
>   include/drm/bridge/dw_hdmi.h              |  2 ++
>   3 files changed, 18 insertions(+), 23 deletions(-)
> 

Please resend a v3 with all those changes,
Thanks

Neil