Re: [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064

[David Heidelberg <david@ixit.cz>]

Reviewed-by: David Heidelberg <david@ixit.cz>

On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Append silicon specific compatible qcom,mdss-dsi-ctrl-apq8064 to the
> mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
> for apq8064 against the yaml documentation.
>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> Cc: David Heidelberg <david@ixit.cz>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
>   arch/arm/boot/dts/qcom-apq8064.dtsi | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index d036dff4b14f7..f3624832e6841 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -1287,7 +1287,8 @@ mmss_sfpb: syscon@5700000 {
>   		};
>   
>   		dsi0: dsi@4700000 {
> -			compatible = "qcom,mdss-dsi-ctrl";
> +			compatible = "qcom,mdss-dsi-ctrl-apq8064",
> +				     "qcom,mdss-dsi-ctrl";
>   			label = "MDSS DSI CTRL->0";
>   			#address-cells = <1>;
>   			#size-cells = <0>;

-- 
David Heidelberg
Consultant Software Engineer