[Freedreno] [PATCH v6 01/10] drm/hdcp: Add drm_hdcp_atomic_check()

Sean Paul sean at poorly.run
Fri Jan 20 15:32:47 UTC 2023


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 at chromium.org>
> > 
> > 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 at intel.com>
> > Acked-by: Jani Nikula <jani.nikula at intel.com>
> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> > Reviewed-by: Abhinav Kumar <abhinavk at codeaurora.org>
> > Signed-off-by: Sean Paul <seanpaul at chromium.org>
> > Signed-off-by: Mark Yacoub <markyacoub at chromium.org>
> > Link: https://patchwork.freedesktop.org/patch/msgid/20210913175747.47456-2-sean@poorly.run #v1
> > Link: https://patchwork.freedesktop.org/patch/msgid/20210915203834.1439-2-sean@poorly.run #v2
> > Link: https://patchwork.freedesktop.org/patch/msgid/20211001151145.55916-2-sean@poorly.run #v3
> > Link: https://patchwork.freedesktop.org/patch/msgid/20211105030434.2828845-2-sean@poorly.run #v4
> > Link: https://patchwork.freedesktop.org/patch/msgid/20220411204741.1074308-2-sean@poorly.run #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 at lists.freedesktop.org, 
    jani.nikula at intel.com,
    intel-gfx at lists.freedesktop.org,
    freedreno at lists.freedesktop.org,
    rodrigo.vivi at intel.com
Cc: bjorn.andersson at linaro.org, 
    swboyd at chromium.org,
    abhinavk at codeaurora.org,
    markyacoub at chromium.org,
    Sean Paul <seanpaul at chromium.org>,
    Maarten Lankhorst <maarten.lankhorst at linux.intel.com>,
    Maxime Ripard <mripard at kernel.org>,
    Thomas Zimmermann <tzimmermann at suse.de>,
    David Airlie <airlied at linux.ie>,
    Daniel Vetter <daniel at ffwll.ch>,
    Jani Nikula <jani.nikula at linux.intel.com>,
    Joonas Lahtinen <joonas.lahtinen at linux.intel.com>,
    Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>

> 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?

Thanks again,

Sean

> 
> Best regards,
> Krzysztof
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS


More information about the Freedreno mailing list