[v2] Add DSI Port B input support for LT9611 HDMI bridge

[PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Hongyang Zhao 1 week, 2 days ago

The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
Port B. Update the devicetree to use port@1 which corresponds to
Port B input on the LT9611.

Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
Reviewed-by: Roger Shimizu <rosh@debian.org>
---
 arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts b/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
index 0b64a0b91202..f47efca42d48 100644
--- a/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
+++ b/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
@@ -755,10 +755,10 @@ ports {
 			#address-cells = <1>;
 			#size-cells = <0>;
 
-			port@0 {
-				reg = <0>;
+			port@1 {
+				reg = <1>;
 
-				lt9611_a: endpoint {
+				lt9611_b: endpoint {
 					remote-endpoint = <&mdss_dsi0_out>;
 				};
 			};
@@ -801,7 +801,7 @@ &mdss_dsi {
 };
 
 &mdss_dsi0_out {
-	remote-endpoint = <&lt9611_a>;
+	remote-endpoint = <&lt9611_b>;
 	data-lanes = <0 1 2 3>;
 };
 

-- 
2.43.0

Re: [PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Krzysztof Kozlowski 1 day, 16 hours ago

On Wed, Jan 28, 2026 at 07:15:47PM +0800, Hongyang Zhao wrote:
> The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
> Port B. Update the devicetree to use port@1 which corresponds to
> Port B input on the LT9611.
> 
> Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> Reviewed-by: Roger Shimizu <rosh@debian.org>

Where did this happen?

https://lore.kernel.org/all/?q=f%3Arosh%40debian.org

And again, v1 was completely different so how pre-v1-internal-review
could be applied to this v2?

Honestly, initial guidance is useful but continuous development behind
the closed doors is not.

Best regards,
Krzysztof

Re: [PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Roger Shimizu 1 day, 10 hours ago

On Thu, Feb 5, 2026 at 5:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Wed, Jan 28, 2026 at 07:15:47PM +0800, Hongyang Zhao wrote:
> > The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
> > Port B. Update the devicetree to use port@1 which corresponds to
> > Port B input on the LT9611.
> >
> > Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> > Reviewed-by: Roger Shimizu <rosh@debian.org>
>
> Where did this happen?
>
> https://lore.kernel.org/all/?q=f%3Arosh%40debian.org
>
> And again, v1 was completely different so how pre-v1-internal-review
> could be applied to this v2?
>
> Honestly, initial guidance is useful but continuous development behind
> the closed doors is not.

FYI. v2 was updated per review feedback, which is public:
https://lore.kernel.org/all/7d9041a3-9d2b-469a-9fa7-89d53bbd2a1f@linaro.org/

-Roger

Re: [PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Krzysztof Kozlowski 22 hours ago

On 05/02/2026 20:36, Roger Shimizu wrote:
> On Thu, Feb 5, 2026 at 5:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On Wed, Jan 28, 2026 at 07:15:47PM +0800, Hongyang Zhao wrote:
>>> The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
>>> Port B. Update the devicetree to use port@1 which corresponds to
>>> Port B input on the LT9611.
>>>
>>> Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
>>> Reviewed-by: Roger Shimizu <rosh@debian.org>
>>
>> Where did this happen?
>>
>> https://lore.kernel.org/all/?q=f%3Arosh%40debian.org
>>
>> And again, v1 was completely different so how pre-v1-internal-review
>> could be applied to this v2?
>>
>> Honestly, initial guidance is useful but continuous development behind
>> the closed doors is not.
> 
> FYI. v2 was updated per review feedback, which is public:
> https://lore.kernel.org/all/7d9041a3-9d2b-469a-9fa7-89d53bbd2a1f@linaro.org/

What? I asked about the tag. Do you understand how kernel review process
works?

Best regards,
Krzysztof

Re: [PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Roger Shimizu 24 minutes ago

On Thu, Feb 5, 2026 at 11:08 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 05/02/2026 20:36, Roger Shimizu wrote:
> > On Thu, Feb 5, 2026 at 5:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >>
> >> On Wed, Jan 28, 2026 at 07:15:47PM +0800, Hongyang Zhao wrote:
> >>> The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
> >>> Port B. Update the devicetree to use port@1 which corresponds to
> >>> Port B input on the LT9611.
> >>>
> >>> Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> >>> Reviewed-by: Roger Shimizu <rosh@debian.org>
> >>
> >> Where did this happen?
> >>
> >> https://lore.kernel.org/all/?q=f%3Arosh%40debian.org
> >>
> >> And again, v1 was completely different so how pre-v1-internal-review
> >> could be applied to this v2?
> >>
> >> Honestly, initial guidance is useful but continuous development behind
> >> the closed doors is not.
> >
> > FYI. v2 was updated per review feedback, which is public:
> > https://lore.kernel.org/all/7d9041a3-9d2b-469a-9fa7-89d53bbd2a1f@linaro.org/
>
> What? I asked about the tag. Do you understand how kernel review process
> works?

Yes, v2 was reviewed in private.
I'll inform Hongyang to remove the tag when he sends the next series.

-Roger

Re: [PATCH v2 3/3] arm64: dts: qcom: qcs6490-rubikpi3: Use lt9611 DSI Port B

Posted by Konrad Dybcio 1 week, 2 days ago

On 1/28/26 12:15 PM, Hongyang Zhao wrote:
> The LT9611 HDMI bridge on RubikPi3 has DSI physically connected to
> Port B. Update the devicetree to use port@1 which corresponds to
> Port B input on the LT9611.
> 
> Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> Reviewed-by: Roger Shimizu <rosh@debian.org>
> ---

Fixes: f055a39f6874 ("arm64: dts: qcom: Add qcs6490-rubikpi3 board dts")

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad