On 20 January 2023 18:32:47 GMT+03:00, Sean Paul <sean@xxxxxxxxxx> wrote: >On Thu, Jan 19, 2023 at 11:37:52AM +0100, Krzysztof Kozlowski wrote: >> On 18/01/2023 20:30, Mark Yacoub wrote: >> > From: Sean Paul <seanpaul@xxxxxxxxxxxx> >> > >> > This patch moves the hdcp atomic check from i915 to drm_hdcp so other >> > drivers can use it. No functional changes, just cleaned up some of the >> > code when moving it over. >> > >> > Acked-by: Jani Nikula <jani.nikula@xxxxxxxxx> >> > Acked-by: Jani Nikula <jani.nikula@xxxxxxxxx> >> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> >> > Reviewed-by: Abhinav Kumar <abhinavk@xxxxxxxxxxxxxx> >> > Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx> >> > Signed-off-by: Mark Yacoub <markyacoub@xxxxxxxxxxxx> >> > Link: https://patchwork.freedesktop.org/patch/msgid/20210913175747.47456-2-sean@xxxxxxxxxx #v1 >> > Link: https://patchwork.freedesktop.org/patch/msgid/20210915203834.1439-2-sean@xxxxxxxxxx #v2 >> > Link: https://patchwork.freedesktop.org/patch/msgid/20211001151145.55916-2-sean@xxxxxxxxxx #v3 >> > Link: https://patchwork.freedesktop.org/patch/msgid/20211105030434.2828845-2-sean@xxxxxxxxxx #v4 >> > Link: https://patchwork.freedesktop.org/patch/msgid/20220411204741.1074308-2-sean@xxxxxxxxxx #v5 >> >> It seems all your previous versions were sent not to correct people and >> lists. Therefore we see it for the first time even though it is v6! > >Hi Krzysztof, >Thanks for your review comments. > >Here are the addresses the last version was sent to, who is missing? > >To: dri-devel@xxxxxxxxxxxxxxxxxxxxx, > jani.nikula@xxxxxxxxx, > intel-gfx@xxxxxxxxxxxxxxxxxxxxx, > freedreno@xxxxxxxxxxxxxxxxxxxxx, > rodrigo.vivi@xxxxxxxxx >Cc: bjorn.andersson@xxxxxxxxxx, > swboyd@xxxxxxxxxxxx, > abhinavk@xxxxxxxxxxxxxx, > markyacoub@xxxxxxxxxxxx, > Sean Paul <seanpaul@xxxxxxxxxxxx>, > Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>, > Maxime Ripard <mripard@xxxxxxxxxx>, > Thomas Zimmermann <tzimmermann@xxxxxxx>, > David Airlie <airlied@xxxxxxxx>, > Daniel Vetter <daniel@xxxxxxxx>, > Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>, > Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>, > Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> > >> It's >> not the first such weird CC list in chromium, so maybe your >> organisational process could be improved? Not only for you but for >> colleagues as well, so you all start using get_maintainers.pl on newest >> kernel (not something ancient)? > >I can't really speak for others, but I use MAINTAINERS from drm-tip. The >previous patch sets were sent before 24df12013853 ("MAINTAINERS: Add >Dmitry as MSM DRM driver co-maintainer"), which might explain why you think >there are absences? Current iteration of the patchset got at least three addresses wrong. They have been changed for various reasons. Thus I also can suppose that the list is incomplete and/or incorrect. > >Thanks again, > >Sean > >> >> Best regards, >> Krzysztof >> > -- With best wishes Dmitry
