mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Andy Yan" <andyshrk@163.com>
To: "Heiko Stübner" <heiko@sntech.de>
Cc: krzk+dt@kernel.org, joro@8bytes.org, cl@rock-chips.com,
	robh@kernel.org,  hjc@rock-chips.com, vkoul@kernel.org,
	devicetree@vger.kernel.org,  detlev.casanova@collabora.com,
	cristian.ciocaltea@collabora.com,
	 dri-devel@lists.freedesktop.org, iommu@lists.linux.dev,
	 linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,  linux-phy@lists.infradead.org,
	linux-rockchip@lists.infradead.org,
	 "Andy Yan" <andy.yan@rock-chips.com>
Subject: Re:Re: [PATCH v2 0/8] Add display subsystem dt node on rk3576
Date: Sun, 29 Dec 2024 14:48:36 +0800 (CST)	[thread overview]
Message-ID: <6ff7470.1060.194112c804a.Coremail.andyshrk@163.com> (raw)
In-Reply-To: <4950097.GXAFRqVoOG@diego>



Hi Heiko,

At 2024-12-29 02:36:36, "Heiko Stübner" <heiko@sntech.de> wrote:
>Hi Andy,
>
>Am Samstag, 28. Dezember 2024, 13:21:43 CET schrieb Andy Yan:
>> From: Andy Yan <andy.yan@rock-chips.com>
>> 
>> 
>> As the VOP[0] and HDMI[1] driver have already been submitted for review.
>> This series send all display related device tree part together.
>> [0] https://lore.kernel.org/linux-rockchip/20241219073931.3997788-1-andyshrk@163.com/T/#t
>> [1] https://lore.kernel.org/linux-rockchip/20241225103741.364597-1-andyshrk@163.com/T/#t
>
>not to confuse you too much, I mainly meant that we were missing the
>"arm64: dts: rockchip:" patches from this series.

Sorry, I misunderstood it in V2 . Before go to the next step, I would like to confirm it again:
Does that mean I should:
1.  the dts patches(6/8, 7/8 8/8) should be sent as a separate series.
2. PATCH 5/8 should still go with the rk3576 vop driver[0]
3. PATCH 3/8 should still go with the rk3576 hdmi dirver[1]
4. PATCH 1/8, 2/8 4/8  should still be sent separate  ?

 
>
>The dt-binding patches are (already) part of the driver changes and
>should stay part of those series, to prevent confusion of what part goes
>where :-) .
>
>
>Heiko
>
>
>> Changes in v2:
>> - Wrap commit message according to Linux coding style
>> - Make "rockchip,rk3588-hdptx-phy" const for "rockchip,rk3576-hdptx-phy"
>> - Make declare phy/ropll/lcpll reset line are not exit on rk3576
>> - describe constraint SOC by SOC for vop2 binding, as interrupts of rk3576
>>   is very different from others
>> - Drop Krzysztof's Reviewed-by for vopp2 bindings, as this version changed a lot.
>> 
>> Changes in v1:
>> - ordered by soc name
>> - Add description for newly added interrupt
>> 
>> Andy Yan (8):
>>   dt-bindings: soc: rockchip: add rk3576 hdptxphy grf syscon
>>   dt-bindings: phy: Add rk3576 hdptx phy
>>   dt-bindings: display: rockchip: Add rk3576 hdmi controller
>>   dt-bindings: iommu: rockchip: Add Rockchip RK3576
>>   dt-bindings: display: vop2: Add rk3576 support
>>   arm64: dts: rockchip: Add vop for rk3576
>>   arm64: dts: rockchip: Add hdmi for rk3576
>>   arm64: dts: rockchip: Enable hdmi display on sige5
>> 
>>  .../rockchip/rockchip,rk3588-dw-hdmi-qp.yaml  |   1 +
>>  .../display/rockchip/rockchip-vop2.yaml       |  70 ++++++++--
>>  .../bindings/iommu/rockchip,iommu.yaml        |   1 +
>>  .../phy/rockchip,rk3588-hdptx-phy.yaml        |  62 ++++++---
>>  .../devicetree/bindings/soc/rockchip/grf.yaml |   1 +
>>  .../boot/dts/rockchip/rk3576-armsom-sige5.dts |  47 +++++++
>>  arch/arm64/boot/dts/rockchip/rk3576.dtsi      | 126 ++++++++++++++++++
>>  7 files changed, 276 insertions(+), 32 deletions(-)
>> 
>> 
>
>
>
>

  reply	other threads:[~2024-12-29  6:49 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-28 12:21 Andy Yan
2024-12-28 12:21 ` [PATCH v2 1/8] dt-bindings: soc: rockchip: add rk3576 hdptxphy grf syscon Andy Yan
2024-12-28 12:21 ` [PATCH v2 2/8] dt-bindings: phy: Add rk3576 hdptx phy Andy Yan
2024-12-28 12:21 ` [PATCH v2 3/8] dt-bindings: display: rockchip: Add rk3576 hdmi controller Andy Yan
2024-12-28 12:21 ` [PATCH v2 4/8] dt-bindings: iommu: rockchip: Add Rockchip RK3576 Andy Yan
2024-12-28 12:21 ` [PATCH v2 5/8] dt-bindings: display: vop2: Add rk3576 support Andy Yan
2024-12-28 14:09   ` Rob Herring (Arm)
2024-12-29 10:13   ` Krzysztof Kozlowski
2024-12-29 10:49     ` Andy Yan
2024-12-31  8:23       ` Krzysztof Kozlowski
2024-12-31  8:43         ` Andy Yan
2024-12-28 12:21 ` [PATCH v2 6/8] arm64: dts: rockchip: Add vop for rk3576 Andy Yan
2024-12-28 12:21 ` [PATCH v2 7/8] arm64: dts: rockchip: Add hdmi " Andy Yan
2024-12-28 12:21 ` [PATCH v2 8/8] arm64: dts: rockchip: Enable hdmi display on sige5 Andy Yan
2024-12-28 18:36 ` [PATCH v2 0/8] Add display subsystem dt node on rk3576 Heiko Stübner
2024-12-29  6:48   ` Andy Yan [this message]
2024-12-29 14:35     ` Heiko Stübner

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=6ff7470.1060.194112c804a.Coremail.andyshrk@163.com \
    --to=andyshrk@163.com \
    --cc=andy.yan@rock-chips.com \
    --cc=cl@rock-chips.com \
    --cc=cristian.ciocaltea@collabora.com \
    --cc=detlev.casanova@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=heiko@sntech.de \
    --cc=hjc@rock-chips.com \
    --cc=iommu@lists.linux.dev \
    --cc=joro@8bytes.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-phy@lists.infradead.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=robh@kernel.org \
    --cc=vkoul@kernel.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

all inboxes | Powered by JetHome®