linux-kernel - Re: [PATCH v7 0/3] drm/bridge_connector: implement OOB HPD handling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e16ebefd-2014-4089-baad-6c0d09813144@linaro.org>
Date:   Wed, 13 Sep 2023 15:55:08 +0300
From:   Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To:     Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Zimmermann <tzimmermann@...e.de>,
        David Airlie <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>,
        Jani Nikula <jani.nikula@...ux.intel.com>,
        Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
        Rodrigo Vivi <rodrigo.vivi@...el.com>,
        Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Heikki Krogerus <heikki.krogerus@...ux.intel.com>
Cc:     Andrzej Hajda <andrzej.hajda@...el.com>,
        Neil Armstrong <neil.armstrong@...aro.org>,
        Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
        Simon Ser <contact@...rsion.fr>, Janne Grunau <j@...nau.net>,
        dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
        intel-gfx@...ts.freedesktop.org, linux-arm-msm@...r.kernel.org,
        linux-usb@...r.kernel.org
Subject: Re: [PATCH v7 0/3] drm/bridge_connector: implement OOB HPD handling

On 25/08/2023 02:56, Dmitry Baryshkov wrote:
> Note, numbering for this series starts from v5, since there were several
> revisions for this patchset under a different series title ([1]).
> 
> USB altmodes code would send OOB notifications to the drm_connector
> specified in the device tree. However as the MSM DP driver uses
> drm_bridge_connector, there is no way to receive these event directly.
> Implement a bridge between oob_hotplug_event and drm_bridge's
> hpd_notify.
> 
> Merge strategy: since this series touches i915 code, it might make sense
> to merge all three patches through drm-intel.

Dear drm-misc and drm-intel maintainers. Since the merge window has 
ended and the trees are fully open for the patches, I'd like to massage 
this patch series. We have R-B on all three patches. Heikki has acked 
the first patch, so it seems to be fine from the i915 point of view.

Is it fine to be merged via drm-misc? Would you like to pick it up into 
drm-intel?


> 
> [1] https://patchwork.freedesktop.org/series/103449/
> 
> Changes since v6:
> - Rebased on top of linux-next. Fixed the freshly added
>    new drm_connector_oob_hotplug_event() call.
> 
> Changes since v5:
> - Fixed checkpatch warning in the first patch (noted by intel-gfx CI).
> 
> Changes since v4:
> - Picked up the patchset
> - Dropped msm-specific patches
> - Changed drm_bridge_connector_oob_hotplug_event to call connector's HPD
>    callback directly, rather than going through the last bridge's
>    hpd_notify
> - Added proper fwnode for the drm_bridge_connector
> 
> Bjorn Andersson (1):
>    drm: Add HPD state to drm_connector_oob_hotplug_event()
> 
> Dmitry Baryshkov (2):
>    drm/bridge_connector: stop filtering events in
>      drm_bridge_connector_hpd_cb()
>    drm/bridge_connector: implement oob_hotplug_event
> 
>   drivers/gpu/drm/drm_bridge_connector.c        | 34 ++++++++++++++-----
>   drivers/gpu/drm/drm_connector.c               |  6 ++--
>   .../gpu/drm/i915/display/intel_display_core.h |  3 ++
>   drivers/gpu/drm/i915/display/intel_dp.c       | 17 ++++++++--
>   drivers/usb/typec/altmodes/displayport.c      | 17 +++++-----
>   include/drm/drm_connector.h                   |  6 ++--
>   6 files changed, 60 insertions(+), 23 deletions(-)
> 

-- 
With best wishes
Dmitry