From: Rob Herring <robh@kernel.org>
To: Tanmay Shah <tanmay@codeaurora.org>
Cc: swboyd@chromium.org, sam@ravnborg.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
freedreno@lists.freedesktop.org, seanpaul@chromium.org,
robdclark@gmail.com, daniel@ffwll.ch, airlied@linux.ie,
aravindh@codeaurora.org, abhinavk@codeaurora.org,
chandanu@codeaurora.org, varar@codeaurora.org
Subject: Re: [PATCH v8 0/6] Add support for DisplayPort driver on SnapDragon
Date: Thu, 9 Jul 2020 14:21:10 -0600 [thread overview]
Message-ID: <20200709202110.GA814782@bogus> (raw)
In-Reply-To: <20200630184507.15589-1-tanmay@codeaurora.org>
On Tue, Jun 30, 2020 at 11:45:01AM -0700, Tanmay Shah wrote:
> These patches add Display-Port driver on SnapDragon/msm hardware.
> This series also contains device-tree bindings for msm DP driver.
> It also contains Makefile and Kconfig changes to compile msm DP driver.
>
> The block diagram of DP driver is shown below:
>
>
> +-------------+
> |DRM FRAMEWORK|
> +------+------+
> |
> +----v----+
> | DP DRM |
> +----+----+
> |
> +----v----+
> +------------+| DP +----------++------+
> + +---+| DISPLAY |+---+ | |
> | + +-+-----+-+ | | |
> | | | | | | |
> | | | | | | |
> | | | | | | |
> v v v v v v v
> +------+ +------+ +---+ +----+ +----+ +---+ +-----+
> | DP | | DP | |DP | | DP | | DP | |DP | | DP |
> |PARSER| | HPD | |AUX| |LINK| |CTRL| |PHY| |POWER|
> +--+---+ +---+--+ +---+ +----+ +--+-+ +-+-+ +-----+
> | | |
> +--v---+ +v-----v+
> |DEVICE| | DP |
> | TREE | |CATALOG|
> +------+ +---+---+
> |
> +---v----+
> |CTRL/PHY|
> | HW |
> +--------+
>
> Changes in v7:
>
> - Modify cover letter description and fix title.
> - Introduce dp-controller.yaml for common bindings across SOC
> - Rename dp-sc7180.yaml to dp-controller-sc7180.yaml for SC7180 bindings
> - Rename compatible string to qcom,sc7180-dp
> - Add assigned-clocks and assigned-clock-parents properties in bindings
> - Remove redundant code from driver
> - Extend series to include HPD detection logic
>
> Changes in v8:
>
> - Add MDSS AHB clock in bindings
> - Replace mode->vrefresh use with drm_mode_vrefresh API
> - Remove redundant aux config code from parser and aux module
> - Assign default max lanes if data-lanes property is not available
> - Fix use-after-free during DP driver remove
> - Unregister hardware clocks during driver cleanup
>
> This series depends-on:
> https://patchwork.freedesktop.org/patch/366159/
If a single patch is a dependency, please coordinate your work and send
as 1 series.
To put it another way, I'm just going to ignore this series until the
dependency is sorted out.
> https://patchwork.freedesktop.org/patch/369859/
Probably the same goes for this too, but I care less as it's not the
binding...
>
> Chandan Uddaraju (4):
> dt-bindings: msm/dp: add bindings of DP/DP-PLL driver for Snapdragon
> drm: add constant N value in helper file
> drm/msm/dp: add displayPort driver support
> drm/msm/dp: add support for DP PLL driver
>
> Jeykumar Sankaran (1):
> drm/msm/dpu: add display port support in DPU
>
> Tanmay Shah (1):
> drm/msm/dp: Add Display Port HPD feature
>
> .../display/msm/dp-controller-sc7180.yaml | 144 ++
> .../bindings/display/msm/dp-controller.yaml | 61 +
> .../bindings/display/msm/dpu-sc7180.yaml | 11 +
> drivers/gpu/drm/i915/display/intel_display.c | 2 +-
> drivers/gpu/drm/msm/Kconfig | 16 +
> drivers/gpu/drm/msm/Makefile | 14 +
> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 29 +-
> .../drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 8 +
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 83 +-
> drivers/gpu/drm/msm/dp/dp_aux.c | 510 +++++
> drivers/gpu/drm/msm/dp/dp_aux.h | 29 +
> drivers/gpu/drm/msm/dp/dp_catalog.c | 1060 ++++++++++
> drivers/gpu/drm/msm/dp/dp_catalog.h | 104 +
> drivers/gpu/drm/msm/dp/dp_ctrl.c | 1707 +++++++++++++++++
> drivers/gpu/drm/msm/dp/dp_ctrl.h | 35 +
> drivers/gpu/drm/msm/dp/dp_display.c | 1017 ++++++++++
> drivers/gpu/drm/msm/dp/dp_display.h | 31 +
> drivers/gpu/drm/msm/dp/dp_drm.c | 168 ++
> drivers/gpu/drm/msm/dp/dp_drm.h | 18 +
> drivers/gpu/drm/msm/dp/dp_hpd.c | 69 +
> drivers/gpu/drm/msm/dp/dp_hpd.h | 79 +
> drivers/gpu/drm/msm/dp/dp_link.c | 1216 ++++++++++++
> drivers/gpu/drm/msm/dp/dp_link.h | 132 ++
> drivers/gpu/drm/msm/dp/dp_panel.c | 490 +++++
> drivers/gpu/drm/msm/dp/dp_panel.h | 95 +
> drivers/gpu/drm/msm/dp/dp_parser.c | 267 +++
> drivers/gpu/drm/msm/dp/dp_parser.h | 138 ++
> drivers/gpu/drm/msm/dp/dp_pll.c | 99 +
> drivers/gpu/drm/msm/dp/dp_pll.h | 61 +
> drivers/gpu/drm/msm/dp/dp_pll_10nm.c | 917 +++++++++
> drivers/gpu/drm/msm/dp/dp_pll_private.h | 111 ++
> drivers/gpu/drm/msm/dp/dp_power.c | 392 ++++
> drivers/gpu/drm/msm/dp/dp_power.h | 103 +
> drivers/gpu/drm/msm/dp/dp_reg.h | 517 +++++
> drivers/gpu/drm/msm/msm_drv.c | 2 +
> drivers/gpu/drm/msm/msm_drv.h | 59 +-
> include/drm/drm_dp_helper.h | 1 +
> 37 files changed, 9776 insertions(+), 19 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/display/msm/dp-controller-sc7180.yaml
> create mode 100644 Documentation/devicetree/bindings/display/msm/dp-controller.yaml
> create mode 100644 drivers/gpu/drm/msm/dp/dp_aux.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_aux.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_catalog.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_catalog.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_ctrl.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_ctrl.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_display.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_display.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_drm.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_drm.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_hpd.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_hpd.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_link.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_link.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_panel.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_panel.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_parser.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_parser.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_pll.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_pll.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_pll_10nm.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_pll_private.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_power.c
> create mode 100644 drivers/gpu/drm/msm/dp/dp_power.h
> create mode 100644 drivers/gpu/drm/msm/dp/dp_reg.h
>
>
> base-commit: 0a19b068acc47d05212f03e494381926dc0381e2
> prerequisite-patch-id: 8058026a54241aa728a91dd1685419afb249959e
> prerequisite-patch-id: ed730eb83f84501579332a0f0ab98f7ef649e868
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
next prev parent reply other threads:[~2020-07-09 20:21 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-30 18:45 [PATCH v8 0/6] Add support for DisplayPort driver on SnapDragon Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 1/6] dt-bindings: msm/dp: add bindings of DP/DP-PLL driver for Snapdragon Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 2/6] drm: add constant N value in helper file Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 3/6] drm/msm/dp: add displayPort driver support Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 4/6] drm/msm/dp: add support for DP PLL driver Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 5/6] drm/msm/dpu: add display port support in DPU Tanmay Shah
2020-06-30 18:45 ` [PATCH v8 6/6] drm/msm/dp: Add Display Port HPD feature Tanmay Shah
2020-07-09 20:21 ` Rob Herring [this message]
2020-07-10 17:07 ` [PATCH v8 0/6] Add support for DisplayPort driver on SnapDragon tanmay
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200709202110.GA814782@bogus \
--to=robh@kernel.org \
--cc=abhinavk@codeaurora.org \
--cc=airlied@linux.ie \
--cc=aravindh@codeaurora.org \
--cc=chandanu@codeaurora.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robdclark@gmail.com \
--cc=sam@ravnborg.org \
--cc=seanpaul@chromium.org \
--cc=swboyd@chromium.org \
--cc=tanmay@codeaurora.org \
--cc=varar@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).