Message ID | 20230624-sm6125-dpu-v1-0-1d5a638cebf2@somainline.org |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp6128897vqr; Fri, 23 Jun 2023 17:50:17 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4EjSebupBzOHqOShquFGggMag+WeXrI+DEovI2jtxbxCP1FOjQDVj7vy8oxOvPLSRdy6ti X-Received: by 2002:a05:6808:f07:b0:3a0:5b48:3db5 with SMTP id m7-20020a0568080f0700b003a05b483db5mr8092673oiw.22.1687567817129; Fri, 23 Jun 2023 17:50:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687567817; cv=none; d=google.com; s=arc-20160816; b=pFNXq/BtbtsWNDCnvg8bhHRc8Do+1KQNVzywM/ftKLjn24wEj/XBYoF1vrqOedQFIS mgeJnzqQoFAJ9GjFLVIrgkgiLGNSE5adsYqi7Lg1l4PJz32peWwKGCmNAtYh78pSdY3j I2Y6WujDNqoEEIUVz4NIoDYcZ2wnRGoEafPxBTSTPeWrr5pXBi9uz99BXijqlrcWlVCg Mw2ambZ86zgom0XvaHrP6zGCqg9QRBL347YTLaL0ScFryRZvDjajfpQTH6puGyqt5Qws DshcRpMadGGN/tqJwhXt2Q0KIXoFZu9EhdXWQpqQVG5rKodEUU1Tn24b1jXrhFQCvimC YL6Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:content-transfer-encoding:mime-version :message-id:date:subject:from; bh=w98Rrw4G4YUdYXPoEI9ppoMxnD4q4rOwfYmz/9UTVSQ=; fh=OfzUp+OcwbFUfZC2mGhS6GGXsXqM7NhjMWKyOt4HPQU=; b=FsCFS9Ly18OuVzG7NZNJeNBZABrynzU7MVVB9hbLWiuwPxNIN9PPIjKG9NnWd30jZA gU5hryc+Z2sttPGVNxWNrSI4oJbMhsHUsX42mzNyIwB275MW1QwcGoqWZ39FIJtIySl/ KHSd3egpOMbXGhAxpaiyRU9SWXZbPp3/mF5vHhTBg1YxksoAkYssnqHg+80wX+lV2hOG 8smYPEQ3LrHJ3ga3/4d9bMgAgPuFSw7eMWnmHo7RTHqJjNbrjJFutfakpSmtInrSUjN8 2qnvh33KPnEG5N6IV22fXzKECnLnt6DyIzA34rsIgCQoJ6FW54zO3ZaHxDIP0dxuvLTs 4jmA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y13-20020a17090322cd00b001a1deff6062si392730plg.281.2023.06.23.17.50.04; Fri, 23 Jun 2023 17:50:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230466AbjFXAlH (ORCPT <rfc822;maxin.john@gmail.com> + 99 others); Fri, 23 Jun 2023 20:41:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42412 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229496AbjFXAlG (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 23 Jun 2023 20:41:06 -0400 Received: from relay08.th.seeweb.it (relay08.th.seeweb.it [IPv6:2001:4b7a:2000:18::169]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 80CFA294B; Fri, 23 Jun 2023 17:41:03 -0700 (PDT) Received: from Marijn-Arch-PC.localdomain (94-211-6-86.cable.dynamic.v4.ziggo.nl [94.211.6.86]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by m-r2.th.seeweb.it (Postfix) with ESMTPSA id D454E3F78B; Sat, 24 Jun 2023 02:41:00 +0200 (CEST) From: Marijn Suijten <marijn.suijten@somainline.org> Subject: [PATCH 00/15] drm/msm: Add SM6125 MDSS/DPU hardware and enable Sony Xperia 10 II panel Date: Sat, 24 Jun 2023 02:40:58 +0200 Message-Id: <20230624-sm6125-dpu-v1-0-1d5a638cebf2@somainline.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAJs7lmQC/x2NSwqDQBAFryK9ToOOZiS5SnAxn5fYYCYyjSEg3 t0myyooaidFFSjdm50qvqLyKQbdpaE0h/ICSzYm17q+9W5gffvOXTmvGwfkdPP9CIyRLIhBwbG GkmZLyrYsJteKp/z+h8d0HCeH6zvdcQAAAA== To: Andy Gross <agross@kernel.org>, Bjorn Andersson <andersson@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Stephen Boyd <sboyd@kernel.org>, Rob Clark <robdclark@gmail.com>, Abhinav Kumar <quic_abhinavk@quicinc.com>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org>, Sean Paul <sean@poorly.run>, David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, Krishna Manikandan <quic_mkrishn@quicinc.com> Cc: ~postmarketos/upstreaming@lists.sr.ht, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, Konrad Dybcio <konrad.dybcio@linaro.org>, Martin Botka <martin.botka@somainline.org>, Jami Kettunen <jami.kettunen@somainline.org>, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski <krzk@kernel.org>, linux-clk@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, Lux Aliaga <they@mint.lgbt>, Marijn Suijten <marijn.suijten@somainline.org> X-Mailer: b4 0.12.2 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1769543111143964966?= X-GMAIL-MSGID: =?utf-8?q?1769543111143964966?= |
Series |
drm/msm: Add SM6125 MDSS/DPU hardware and enable Sony Xperia 10 II panel
|
|
Message
Marijn Suijten
June 24, 2023, 12:40 a.m. UTC
Bring up the SM6125 DPU now that all preliminary series (such as INTF TE) have been merged (for me to test the hardware properly), and most other conflicting work (barring ongoing catalog *improvements*) has made its way in as well or is still being discussed. The second part of the series complements that by immediately utilizing this hardware in DT, and even enabling the MDSS/DSI nodes complete with a 6.0" 1080x2520 panel for Sony's Seine PDX201 (Xperia 10 II). The last patch ("sm6125-seine: Configure MDSS, DSI and panel") depends on (an impending v2 of) my Sony panel collection series [1]. [1]: https://lore.kernel.org/linux-arm-msm/20230521-drm-panels-sony-v1-0-541c341d6bee@somainline.org/ --- Marijn Suijten (15): arm64: dts: qcom: sm6125: Sort spmi_bus node numerically by reg dt-bindings: clock: qcom,dispcc-sm6125: Remove unused GCC_DISP_AHB_CLK dt-bindings: clock: qcom,dispcc-sm6125: Require GCC PLL0 DIV clock dt-bindings: clock: qcom,dispcc-sm6125: Allow power-domains property dt-bindings: display/msm: dsi-controller-main: Document SM6125 dt-bindings: display/msm: sc7180-dpu: Describe SM6125 dt-bindings: display/msm: Add SM6125 MDSS drm/msm/dpu: Add SM6125 support drm/msm/mdss: Add SM6125 support dt-bindings: msm: dsi-phy-14nm: Document SM6125 variant drm/msm/dsi: Add 14nm phy configuration for SM6125 arm64: dts: qcom: sm6125: Switch fixed xo_board clock to RPM XO clock arm64: dts: qcom: sm6125: Add dispcc node arm64: dts: qcom: sm6125: Add display hardware nodes arm64: dts: qcom: sm6125-seine: Configure MDSS, DSI and panel .../bindings/clock/qcom,dispcc-sm6125.yaml | 17 +- .../bindings/display/msm/dsi-controller-main.yaml | 2 + .../bindings/display/msm/dsi-phy-14nm.yaml | 1 + .../bindings/display/msm/qcom,sc7180-dpu.yaml | 1 + .../bindings/display/msm/qcom,sm6125-mdss.yaml | 206 +++++++++++++++++ .../dts/qcom/sm6125-sony-xperia-seine-pdx201.dts | 59 +++++ arch/arm64/boot/dts/qcom/sm6125.dtsi | 244 +++++++++++++++++++-- .../gpu/drm/msm/disp/dpu1/catalog/dpu_5_4_sm6125.h | 173 +++++++++++++++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 6 + drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 + drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 + drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 1 + drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 15 ++ drivers/gpu/drm/msm/msm_mdss.c | 8 + 15 files changed, 712 insertions(+), 25 deletions(-) --- base-commit: 8d2be868b42c08290509c60515865f4de24ea704 change-id: 20230624-sm6125-dpu-aedc9637ee7b Best regards,
Comments
On 24.06.2023 02:40, Marijn Suijten wrote: > Bring up the SM6125 DPU now that all preliminary series (such as INTF > TE) have been merged (for me to test the hardware properly) We should not repeat the same mistake in the future.. Finding a balance between releasing early and releasing what we can declare working and tested code is hard, but we waaaaaaaay overstayed on this one.. Konrad , and most > other conflicting work (barring ongoing catalog *improvements*) has made > its way in as well or is still being discussed. > > The second part of the series complements that by immediately utilizing > this hardware in DT, and even enabling the MDSS/DSI nodes complete with > a 6.0" 1080x2520 panel for Sony's Seine PDX201 (Xperia 10 II). > > The last patch ("sm6125-seine: Configure MDSS, DSI and panel") depends > on (an impending v2 of) my Sony panel collection series [1]. > > [1]: https://lore.kernel.org/linux-arm-msm/20230521-drm-panels-sony-v1-0-541c341d6bee@somainline.org/ > > --- > Marijn Suijten (15): > arm64: dts: qcom: sm6125: Sort spmi_bus node numerically by reg > dt-bindings: clock: qcom,dispcc-sm6125: Remove unused GCC_DISP_AHB_CLK > dt-bindings: clock: qcom,dispcc-sm6125: Require GCC PLL0 DIV clock > dt-bindings: clock: qcom,dispcc-sm6125: Allow power-domains property > dt-bindings: display/msm: dsi-controller-main: Document SM6125 > dt-bindings: display/msm: sc7180-dpu: Describe SM6125 > dt-bindings: display/msm: Add SM6125 MDSS > drm/msm/dpu: Add SM6125 support > drm/msm/mdss: Add SM6125 support > dt-bindings: msm: dsi-phy-14nm: Document SM6125 variant > drm/msm/dsi: Add 14nm phy configuration for SM6125 > arm64: dts: qcom: sm6125: Switch fixed xo_board clock to RPM XO clock > arm64: dts: qcom: sm6125: Add dispcc node > arm64: dts: qcom: sm6125: Add display hardware nodes > arm64: dts: qcom: sm6125-seine: Configure MDSS, DSI and panel > > .../bindings/clock/qcom,dispcc-sm6125.yaml | 17 +- > .../bindings/display/msm/dsi-controller-main.yaml | 2 + > .../bindings/display/msm/dsi-phy-14nm.yaml | 1 + > .../bindings/display/msm/qcom,sc7180-dpu.yaml | 1 + > .../bindings/display/msm/qcom,sm6125-mdss.yaml | 206 +++++++++++++++++ > .../dts/qcom/sm6125-sony-xperia-seine-pdx201.dts | 59 +++++ > arch/arm64/boot/dts/qcom/sm6125.dtsi | 244 +++++++++++++++++++-- > .../gpu/drm/msm/disp/dpu1/catalog/dpu_5_4_sm6125.h | 173 +++++++++++++++ > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 6 + > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 1 + > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 + > drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 + > drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 1 + > drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 15 ++ > drivers/gpu/drm/msm/msm_mdss.c | 8 + > 15 files changed, 712 insertions(+), 25 deletions(-) > --- > base-commit: 8d2be868b42c08290509c60515865f4de24ea704 > change-id: 20230624-sm6125-dpu-aedc9637ee7b > > Best regards,
On 2023-06-24 03:42:46, Konrad Dybcio wrote: > On 24.06.2023 02:40, Marijn Suijten wrote: > > Bring up the SM6125 DPU now that all preliminary series (such as INTF > > TE) have been merged (for me to test the hardware properly) > We should not repeat the same mistake in the future.. Finding a > balance between releasing early and releasing what we can declare > working and tested code is hard, but we waaaaaaaay overstayed on > this one.. I don't understand what you mean by "mistake" at all. Yes the DPU catalog portion of this series sat in my local branch for a very long time. Yes it had to be rebased on top of conflicts many many times. However, that time has also been used to fix and extend DPU where necessary, instead of submitting a half-broken or half-incomplete catalog entry... Re "we overstayed": you could have asked to clean up and send my patch, so I don't take this as a mistake on my part as you are completely aware of my time schedule ;) > Konrad > , and most Also here, don't forget to re-quote my message if you break half-way in the line. > > other conflicting work (barring ongoing catalog *improvements*) has made > > its way in as well or is still being discussed. > > > > > The second part of the series complements that by immediately utilizing > > this hardware in DT, and even enabling the MDSS/DSI nodes complete with > > a 6.0" 1080x2520 panel for Sony's Seine PDX201 (Xperia 10 II). > > > > The last patch ("sm6125-seine: Configure MDSS, DSI and panel") depends > > on (an impending v2 of) my Sony panel collection series [1]. > > > > [1]: https://lore.kernel.org/linux-arm-msm/20230521-drm-panels-sony-v1-0-541c341d6bee@somainline.org/ > > > > --- > > Marijn Suijten (15): > > arm64: dts: qcom: sm6125: Sort spmi_bus node numerically by reg > > dt-bindings: clock: qcom,dispcc-sm6125: Remove unused GCC_DISP_AHB_CLK > > dt-bindings: clock: qcom,dispcc-sm6125: Require GCC PLL0 DIV clock > > dt-bindings: clock: qcom,dispcc-sm6125: Allow power-domains property > > dt-bindings: display/msm: dsi-controller-main: Document SM6125 > > dt-bindings: display/msm: sc7180-dpu: Describe SM6125 > > dt-bindings: display/msm: Add SM6125 MDSS > > drm/msm/dpu: Add SM6125 support > > drm/msm/mdss: Add SM6125 support > > dt-bindings: msm: dsi-phy-14nm: Document SM6125 variant > > drm/msm/dsi: Add 14nm phy configuration for SM6125 > > arm64: dts: qcom: sm6125: Switch fixed xo_board clock to RPM XO clock > > arm64: dts: qcom: sm6125: Add dispcc node > > arm64: dts: qcom: sm6125: Add display hardware nodes > > arm64: dts: qcom: sm6125-seine: Configure MDSS, DSI and panel > > > > .../bindings/clock/qcom,dispcc-sm6125.yaml | 17 +- > > .../bindings/display/msm/dsi-controller-main.yaml | 2 + > > .../bindings/display/msm/dsi-phy-14nm.yaml | 1 + > > .../bindings/display/msm/qcom,sc7180-dpu.yaml | 1 + > > .../bindings/display/msm/qcom,sm6125-mdss.yaml | 206 +++++++++++++++++ > > .../dts/qcom/sm6125-sony-xperia-seine-pdx201.dts | 59 +++++ > > arch/arm64/boot/dts/qcom/sm6125.dtsi | 244 +++++++++++++++++++-- > > .../gpu/drm/msm/disp/dpu1/catalog/dpu_5_4_sm6125.h | 173 +++++++++++++++ > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 6 + > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 1 + > > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 + > > drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 + > > drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 1 + > > drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 15 ++ > > drivers/gpu/drm/msm/msm_mdss.c | 8 + > > 15 files changed, 712 insertions(+), 25 deletions(-) > > --- > > base-commit: 8d2be868b42c08290509c60515865f4de24ea704 > > change-id: 20230624-sm6125-dpu-aedc9637ee7b > > > > Best regards,
On 25.06.2023 21:18, Marijn Suijten wrote: > On 2023-06-24 03:42:46, Konrad Dybcio wrote: >> On 24.06.2023 02:40, Marijn Suijten wrote: >>> Bring up the SM6125 DPU now that all preliminary series (such as INTF >>> TE) have been merged (for me to test the hardware properly) >> We should not repeat the same mistake in the future.. Finding a >> balance between releasing early and releasing what we can declare >> working and tested code is hard, but we waaaaaaaay overstayed on >> this one.. > > I don't understand what you mean by "mistake" at all. Yes the DPU > catalog portion of this series sat in my local branch for a very long > time. Yes it had to be rebased on top of conflicts many many times. > > However, that time has also been used to fix and extend DPU where > necessary, instead of submitting a half-broken or half-incomplete > catalog entry... > > Re "we overstayed": you could have asked to clean up and send my patch, > so I don't take this as a mistake on my part as you are completely aware > of my time schedule ;) I didn't mean to pick on you. I just wanted to emphasize that a more upstream-forward approach would have saved us quite some time on the rebasing and cleaning-up front. > >> Konrad >> , and most > > Also here, don't forget to re-quote my message if you break half-way in > the line. Ugh. All the time I've been doing this I thought thunderfox was smart enough to do it for me. Apparently not and you're the 1st one to point that out. Konrad > >>> other conflicting work (barring ongoing catalog *improvements*) has made >>> its way in as well or is still being discussed. >> >>> >>> The second part of the series complements that by immediately utilizing >>> this hardware in DT, and even enabling the MDSS/DSI nodes complete with >>> a 6.0" 1080x2520 panel for Sony's Seine PDX201 (Xperia 10 II). >>> >>> The last patch ("sm6125-seine: Configure MDSS, DSI and panel") depends >>> on (an impending v2 of) my Sony panel collection series [1]. >>> >>> [1]: https://lore.kernel.org/linux-arm-msm/20230521-drm-panels-sony-v1-0-541c341d6bee@somainline.org/ >>> >>> --- >>> Marijn Suijten (15): >>> arm64: dts: qcom: sm6125: Sort spmi_bus node numerically by reg >>> dt-bindings: clock: qcom,dispcc-sm6125: Remove unused GCC_DISP_AHB_CLK >>> dt-bindings: clock: qcom,dispcc-sm6125: Require GCC PLL0 DIV clock >>> dt-bindings: clock: qcom,dispcc-sm6125: Allow power-domains property >>> dt-bindings: display/msm: dsi-controller-main: Document SM6125 >>> dt-bindings: display/msm: sc7180-dpu: Describe SM6125 >>> dt-bindings: display/msm: Add SM6125 MDSS >>> drm/msm/dpu: Add SM6125 support >>> drm/msm/mdss: Add SM6125 support >>> dt-bindings: msm: dsi-phy-14nm: Document SM6125 variant >>> drm/msm/dsi: Add 14nm phy configuration for SM6125 >>> arm64: dts: qcom: sm6125: Switch fixed xo_board clock to RPM XO clock >>> arm64: dts: qcom: sm6125: Add dispcc node >>> arm64: dts: qcom: sm6125: Add display hardware nodes >>> arm64: dts: qcom: sm6125-seine: Configure MDSS, DSI and panel >>> >>> .../bindings/clock/qcom,dispcc-sm6125.yaml | 17 +- >>> .../bindings/display/msm/dsi-controller-main.yaml | 2 + >>> .../bindings/display/msm/dsi-phy-14nm.yaml | 1 + >>> .../bindings/display/msm/qcom,sc7180-dpu.yaml | 1 + >>> .../bindings/display/msm/qcom,sm6125-mdss.yaml | 206 +++++++++++++++++ >>> .../dts/qcom/sm6125-sony-xperia-seine-pdx201.dts | 59 +++++ >>> arch/arm64/boot/dts/qcom/sm6125.dtsi | 244 +++++++++++++++++++-- >>> .../gpu/drm/msm/disp/dpu1/catalog/dpu_5_4_sm6125.h | 173 +++++++++++++++ >>> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 6 + >>> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 1 + >>> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 + >>> drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 + >>> drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 1 + >>> drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 15 ++ >>> drivers/gpu/drm/msm/msm_mdss.c | 8 + >>> 15 files changed, 712 insertions(+), 25 deletions(-) >>> --- >>> base-commit: 8d2be868b42c08290509c60515865f4de24ea704 >>> change-id: 20230624-sm6125-dpu-aedc9637ee7b >>> >>> Best regards,
On 2023-06-26 11:41:39, Konrad Dybcio wrote: > On 25.06.2023 21:18, Marijn Suijten wrote: > > On 2023-06-24 03:42:46, Konrad Dybcio wrote: > >> On 24.06.2023 02:40, Marijn Suijten wrote: > >>> Bring up the SM6125 DPU now that all preliminary series (such as INTF > >>> TE) have been merged (for me to test the hardware properly) > >> We should not repeat the same mistake in the future.. Finding a > >> balance between releasing early and releasing what we can declare > >> working and tested code is hard, but we waaaaaaaay overstayed on > >> this one.. > > > > I don't understand what you mean by "mistake" at all. Yes the DPU > > catalog portion of this series sat in my local branch for a very long > > time. Yes it had to be rebased on top of conflicts many many times. > > > > However, that time has also been used to fix and extend DPU where > > necessary, instead of submitting a half-broken or half-incomplete > > catalog entry... > > > > Re "we overstayed": you could have asked to clean up and send my patch, > > so I don't take this as a mistake on my part as you are completely aware > > of my time schedule ;) > I didn't mean to pick on you. I just wanted to emphasize that a more > upstream-forward approach would have saved us quite some time on the > rebasing and cleaning-up front. That is how it comes across ;) - our dream is all about upstream-first but as you know this becomes a mess really quickly when things are blocked on dependencies and you're working on 5 different features and testing across ±8 different Sony platforms on ±14 different devices at once... all in a limited portion of free time. Fwiw cleaning-up would have had to happen either way, and would have taken the same amount of time regardless of whether this series is submitted now or two months ago. > >> Konrad > >> , and most > > > > Also here, don't forget to re-quote my message if you break half-way in > > the line. > Ugh. All the time I've been doing this I thought thunderfox was smart > enough to do it for me. Apparently not and you're the 1st one to point > that out. You're welcome! (Though I thought it should be visible in Thunderburd, unless you're not in plaintext mode? Does it still show the "this is quoted" line in front of the broken sentence?) - Marijn
On 26.06.2023 16:17, Marijn Suijten wrote: > On 2023-06-26 11:41:39, Konrad Dybcio wrote: >> On 25.06.2023 21:18, Marijn Suijten wrote: >>> On 2023-06-24 03:42:46, Konrad Dybcio wrote: >>>> On 24.06.2023 02:40, Marijn Suijten wrote: >>>>> Bring up the SM6125 DPU now that all preliminary series (such as INTF >>>>> TE) have been merged (for me to test the hardware properly) >>>> We should not repeat the same mistake in the future.. Finding a >>>> balance between releasing early and releasing what we can declare >>>> working and tested code is hard, but we waaaaaaaay overstayed on >>>> this one.. >>> >>> I don't understand what you mean by "mistake" at all. Yes the DPU >>> catalog portion of this series sat in my local branch for a very long >>> time. Yes it had to be rebased on top of conflicts many many times. >>> >>> However, that time has also been used to fix and extend DPU where >>> necessary, instead of submitting a half-broken or half-incomplete >>> catalog entry... >>> >>> Re "we overstayed": you could have asked to clean up and send my patch, >>> so I don't take this as a mistake on my part as you are completely aware >>> of my time schedule ;) >> I didn't mean to pick on you. I just wanted to emphasize that a more >> upstream-forward approach would have saved us quite some time on the >> rebasing and cleaning-up front. > > That is how it comes across ;) - our dream is all about upstream-first > but as you know this becomes a mess really quickly when things are > blocked on dependencies and you're working on 5 different features and > testing across ±8 different Sony platforms on ±14 different devices at > once... all in a limited portion of free time. > > Fwiw cleaning-up would have had to happen either way, and would have > taken the same amount of time regardless of whether this series is > submitted now or two months ago. > >>>> Konrad >>>> , and most >>> >>> Also here, don't forget to re-quote my message if you break half-way in >>> the line. >> Ugh. All the time I've been doing this I thought thunderfox was smart >> enough to do it for me. Apparently not and you're the 1st one to point >> that out. > > You're welcome! > (Though I thought it should be visible in Thunderburd, unless you're not > in plaintext mode? Does it still show the "this is quoted" line in > front of the broken sentence?) It doesn't, but the text stays blue (as if it was) Konrad > > - Marijn