From: tanmay@codeaurora.org
To: Rob Herring <robh@kernel.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: Fri, 10 Jul 2020 10:07:56 -0700 [thread overview]
Message-ID: <2e867e903db9be91a988a37b7508abd0@codeaurora.org> (raw)
In-Reply-To: <20200709202110.GA814782@bogus>
Thanks for reviews Rob.
On 2020-07-09 13:21, Rob Herring wrote:
> 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.
Sure I will wait till previous patch is resolved.
>
>> https://patchwork.freedesktop.org/patch/369859/
>
> Probably the same goes for this too, but I care less as it's not the
> binding...
Above patch is not compile time dependency, but without it we see issues
during runtime.
So will be removed from dependency list.
Thanks.
>
>>
>> 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
>>
prev parent reply other threads:[~2020-07-10 17:08 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 ` [PATCH v8 0/6] Add support for DisplayPort driver on SnapDragon Rob Herring
2020-07-10 17:07 ` tanmay [this message]
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=2e867e903db9be91a988a37b7508abd0@codeaurora.org \
--to=tanmay@codeaurora.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=robh@kernel.org \
--cc=sam@ravnborg.org \
--cc=seanpaul@chromium.org \
--cc=swboyd@chromium.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).