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;
Message-ID: <a8d05899-edfa-8036-6672-c4e7e79d60d5@gmail.com>
Date:   Thu, 19 Jan 2023 19:26:48 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [PATCH] arm64: dts: mt8183: jacuzzi: Move panel under aux-bus
Content-Language: en-US
To:     Hsin-Yi Wang <hsinyi@chromium.org>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>
References: <20221228113204.1551180-1-hsinyi@chromium.org>
From:   Matthias Brugger <matthias.bgg@gmail.com>
In-Reply-To: <20221228113204.1551180-1-hsinyi@chromium.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 28/12/2022 12:32, Hsin-Yi Wang wrote:
> Read edp panel edid through aux bus, which is a more preferred way. Also
> use a more generic compatible since each jacuzzi models use different
> panels.
> 
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>

Applied, thanks.

> ---
>   .../dts/mediatek/mt8183-kukui-jacuzzi.dtsi    | 26 ++++++++++---------
>   1 file changed, 14 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi
> index 3ac83be53627..543286ce9cea 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi
> @@ -6,18 +6,6 @@
>   #include "mt8183-kukui.dtsi"
>   
>   / {
> -	panel: panel {
> -		compatible = "auo,b116xw03";
> -		power-supply = <&pp3300_panel>;
> -		backlight = <&backlight_lcd0>;
> -
> -		port {
> -			panel_in: endpoint {
> -				remote-endpoint = <&anx7625_out>;
> -			};
> -		};
> -	};
> -
>   	pp1200_mipibrdg: pp1200-mipibrdg {
>   		compatible = "regulator-fixed";
>   		regulator-name = "pp1200_mipibrdg";
> @@ -181,6 +169,20 @@ anx7625_out: endpoint {
>   				remote-endpoint = <&panel_in>;
>   			};
>   		};
> +
> +		aux-bus {
> +			panel: panel {
> +				compatible = "edp-panel";
> +				power-supply = <&pp3300_panel>;
> +				backlight = <&backlight_lcd0>;
> +
> +				port {
> +					panel_in: endpoint {
> +						remote-endpoint = <&anx7625_out>;
> +					};
> +				};
> +			};
> +		};
>   	};
>   };
>