Re: [PATCH v5 1/2] dt-bindings: arm: qcom: add sc7180-lazor board bindings

From: Doug Anderson
Date: Tue Aug 22 2023 - 09:41:25 EST


Hi,

On Tue, Aug 22, 2023 at 3:04 AM Sheng-Liang Pan
<sheng-liang.pan@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
>
> Introduce more sc7180-lazor sku and board version configuration,
> add no-eSIM SKU 10 for Lazor, no-eSIM SKU 15 and 18 for Limozeen,
> add new board version 10 for audio codec ALC5682i-VS.
>
> Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
> Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
> ---
>
> Changes in v3:
> - correct corresponding of new board and new sku
>
> Changes in v2:
> - add new entry rev9 with Parade bridge chip
> - correct newly create dts files
> .../devicetree/bindings/arm/qcom.yaml | 31 +++++++++++++++++++
> 1 file changed, 31 insertions(+)

In my response to v3 [1] I mentioned that you should make sure to add
Krzysztof's tag that he added on v2 [2], since v2 and v3 were nearly
identical. IMO, you don't need to send a v6 just for this but please
remember to do this in the future.

[1] https://lore.kernel.org/r/CAD=FV=WKTdaQgH4Cf2OCV9Uy6APjxfP8Pz+1-uWAcLC2PphYnw@xxxxxxxxxxxxxx
[2] https://lore.kernel.org/r/55d4090f-d2f3-ffb2-cc6f-a13222f14e47@xxxxxxxxxx