| Message ID | 20240724-hdmi-tx-v7-0-e44a20553464@freebox.fr |
|---|---|
| Headers | show |
| Series | HDMI TX support in msm8998 | expand |
Hello Rob, Abhinav, Dmitry, This series might be ready this time? Are all the patches supposed to go through https://gitlab.freedesktop.org/drm/msm.git ? (Probably not... patches 5 & 6 are probably supposed to go through Bjorn's tree?) Regards On 24/07/2024 17:01, Marc Gonzalez wrote: > DT bits required for HDMI TX support in qcom APQ8098 (MSM8998 cousin) > > --- > Changes in v7: > - prefix the labels hdmi: and hdmi_phy: with mdss_ (Konrad) > - DID NOT MODIFY patch 3, based on conversation between Conor & Dmitry > - tested 40+40 boots with/without maxcpus=1 => no iommu panic witnessed > - Collect tags from Konrad & Dmitry (hopefully b4 did the right thing) > - Link to v6: https://lore.kernel.org/r/20240715-hdmi-tx-v6-0-d27f029627ad@freebox.fr > > Changes in v6: > - Fold HDMI PHY driver submission into this series > => [PATCH v2] drm/msm: add msm8998 hdmi phy/pll support > => Link to v2: https://lore.kernel.org/all/20240704-hdmi-phy-v2-1-a7f5af202cb5@freebox.fr/ > - Rebase onto v6.10 > - Move drivers/gpu/drm/msm/hdmi/hdmi.xml.h to drivers/gpu/drm/msm/registers/display/hdmi.xml > - Add copyright attribution > - Remove all dead/debug/temporary code > => Link to v1: https://lore.kernel.org/all/63337d63-67ef-4499-8a24-5f6e9285c36b@freebox.fr/ > - split HDMI PHY driver patch in 2 parts (PHY & TX) > - Use same regulator names as msm8996 (Dmitry) > - Remove printk statements > - Add Vinod's Ack on patch 1 > - Expand commit message on patch 4 = HDMI PHY driver > - Link to v5: https://lore.kernel.org/r/20240627-hdmi-tx-v5-0-355d5c1fbc3c@freebox.fr > > Changes in v5: > - Fix property & property-names for TX pinctrl in DTSI (Konrad) > - NOT CHANGED: clock trees for TX & PHY based on Dmitry & Jeffrey's remarks > - Link to v4: https://lore.kernel.org/r/20240613-hdmi-tx-v4-0-4af17e468699@freebox.fr > > Changes in v4: > - Collect tags since v3 > - Reword patch 1 subject (Vinod) > - Link to v3: https://lore.kernel.org/r/20240606-hdmi-tx-v3-0-9d7feb6d3647@freebox.fr > > Changes in v3 > - Address Rob's comments on patch 2: > - 'maxItems: 5' for clocks in the 8996 if/then schema > - match the order of 8996 for the clock-names in common > > --- > Arnaud Vrac (2): > drm/msm: add msm8998 hdmi phy/pll support > arm64: dts: qcom: add HDMI nodes for msm8998 > > Marc Gonzalez (4): > dt-bindings: phy: add qcom,hdmi-phy-8998 > dt-bindings: display/msm: hdmi: add qcom,hdmi-tx-8998 > drm/msm/hdmi: add "qcom,hdmi-tx-8998" compatible > arm64: dts: qcom: msm8998: add HDMI GPIOs > > .../devicetree/bindings/display/msm/hdmi.yaml | 28 +- > .../devicetree/bindings/phy/qcom,hdmi-phy-qmp.yaml | 1 + > arch/arm64/boot/dts/qcom/msm8998.dtsi | 128 +++- > drivers/gpu/drm/msm/Makefile | 1 + > drivers/gpu/drm/msm/hdmi/hdmi.c | 1 + > drivers/gpu/drm/msm/hdmi/hdmi.h | 8 + > drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 5 + > drivers/gpu/drm/msm/hdmi/hdmi_phy_8998.c | 779 +++++++++++++++++++++ > drivers/gpu/drm/msm/registers/display/hdmi.xml | 89 +++ > 9 files changed, 1037 insertions(+), 3 deletions(-) > --- > base-commit: 003a26f5074bfe024603cf76f8fd486a5344f307 > change-id: 20240606-hdmi-tx-00ee8e7ddbac
On Wed, 24 Jul 2024 17:01:33 +0200, Marc Gonzalez wrote: > DT bits required for HDMI TX support in qcom APQ8098 (MSM8998 cousin) > Applied, thanks! [1/6] dt-bindings: phy: add qcom,hdmi-phy-8998 https://gitlab.freedesktop.org/lumag/msm/-/commit/8886def25eef [2/6] dt-bindings: display/msm: hdmi: add qcom,hdmi-tx-8998 https://gitlab.freedesktop.org/lumag/msm/-/commit/656810411b1d [3/6] drm/msm/hdmi: add "qcom,hdmi-tx-8998" compatible https://gitlab.freedesktop.org/lumag/msm/-/commit/a61eb17f40e8 [4/6] drm/msm: add msm8998 hdmi phy/pll support https://gitlab.freedesktop.org/lumag/msm/-/commit/caedbf17c48d Best regards,
On 05/09/2024 05:33, Dmitry Baryshkov wrote: > On Wed, 24 Jul 2024 17:01:33 +0200, Marc Gonzalez wrote: > >> DT bits required for HDMI TX support in qcom APQ8098 (MSM8998 cousin) > > Applied, thanks! > > [1/6] dt-bindings: phy: add qcom,hdmi-phy-8998 > https://gitlab.freedesktop.org/lumag/msm/-/commit/8886def25eef > [2/6] dt-bindings: display/msm: hdmi: add qcom,hdmi-tx-8998 > https://gitlab.freedesktop.org/lumag/msm/-/commit/656810411b1d > [3/6] drm/msm/hdmi: add "qcom,hdmi-tx-8998" compatible > https://gitlab.freedesktop.org/lumag/msm/-/commit/a61eb17f40e8 > [4/6] drm/msm: add msm8998 hdmi phy/pll support > https://gitlab.freedesktop.org/lumag/msm/-/commit/caedbf17c48d Hello Bjorn, Now that the binding & driver parts have been merged, (Thanks, Dmitry! And thanks to all reviewers) am I right to assume that you are supposed to pick up patches 5 & 6 for the msm8998 DTSI? (Hopefully in time for 6.12-rc1) Regards
On Wed, 24 Jul 2024 17:01:33 +0200, Marc Gonzalez wrote: > DT bits required for HDMI TX support in qcom APQ8098 (MSM8998 cousin) > Applied, thanks! [5/6] arm64: dts: qcom: msm8998: add HDMI GPIOs commit: 1b97f00d1b11ed6e349816ba7e60fa76e477fcbb [6/6] arm64: dts: qcom: add HDMI nodes for msm8998 commit: 2150c87db80cf7eed3939f32cbb18393055c58ce Best regards,
DT bits required for HDMI TX support in qcom APQ8098 (MSM8998 cousin) --- Changes in v7: - prefix the labels hdmi: and hdmi_phy: with mdss_ (Konrad) - DID NOT MODIFY patch 3, based on conversation between Conor & Dmitry - tested 40+40 boots with/without maxcpus=1 => no iommu panic witnessed - Collect tags from Konrad & Dmitry (hopefully b4 did the right thing) - Link to v6: https://lore.kernel.org/r/20240715-hdmi-tx-v6-0-d27f029627ad@freebox.fr Changes in v6: - Fold HDMI PHY driver submission into this series => [PATCH v2] drm/msm: add msm8998 hdmi phy/pll support => Link to v2: https://lore.kernel.org/all/20240704-hdmi-phy-v2-1-a7f5af202cb5@freebox.fr/ - Rebase onto v6.10 - Move drivers/gpu/drm/msm/hdmi/hdmi.xml.h to drivers/gpu/drm/msm/registers/display/hdmi.xml - Add copyright attribution - Remove all dead/debug/temporary code => Link to v1: https://lore.kernel.org/all/63337d63-67ef-4499-8a24-5f6e9285c36b@freebox.fr/ - split HDMI PHY driver patch in 2 parts (PHY & TX) - Use same regulator names as msm8996 (Dmitry) - Remove printk statements - Add Vinod's Ack on patch 1 - Expand commit message on patch 4 = HDMI PHY driver - Link to v5: https://lore.kernel.org/r/20240627-hdmi-tx-v5-0-355d5c1fbc3c@freebox.fr Changes in v5: - Fix property & property-names for TX pinctrl in DTSI (Konrad) - NOT CHANGED: clock trees for TX & PHY based on Dmitry & Jeffrey's remarks - Link to v4: https://lore.kernel.org/r/20240613-hdmi-tx-v4-0-4af17e468699@freebox.fr Changes in v4: - Collect tags since v3 - Reword patch 1 subject (Vinod) - Link to v3: https://lore.kernel.org/r/20240606-hdmi-tx-v3-0-9d7feb6d3647@freebox.fr Changes in v3 - Address Rob's comments on patch 2: - 'maxItems: 5' for clocks in the 8996 if/then schema - match the order of 8996 for the clock-names in common --- Arnaud Vrac (2): drm/msm: add msm8998 hdmi phy/pll support arm64: dts: qcom: add HDMI nodes for msm8998 Marc Gonzalez (4): dt-bindings: phy: add qcom,hdmi-phy-8998 dt-bindings: display/msm: hdmi: add qcom,hdmi-tx-8998 drm/msm/hdmi: add "qcom,hdmi-tx-8998" compatible arm64: dts: qcom: msm8998: add HDMI GPIOs .../devicetree/bindings/display/msm/hdmi.yaml | 28 +- .../devicetree/bindings/phy/qcom,hdmi-phy-qmp.yaml | 1 + arch/arm64/boot/dts/qcom/msm8998.dtsi | 128 +++- drivers/gpu/drm/msm/Makefile | 1 + drivers/gpu/drm/msm/hdmi/hdmi.c | 1 + drivers/gpu/drm/msm/hdmi/hdmi.h | 8 + drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 5 + drivers/gpu/drm/msm/hdmi/hdmi_phy_8998.c | 779 +++++++++++++++++++++ drivers/gpu/drm/msm/registers/display/hdmi.xml | 89 +++ 9 files changed, 1037 insertions(+), 3 deletions(-) --- base-commit: 003a26f5074bfe024603cf76f8fd486a5344f307 change-id: 20240606-hdmi-tx-00ee8e7ddbac Best regards,