Re: [PATCH 3/4] arm64: dts: qcom: add 'chassis-type' property

[Stephan Gerhold]

On Sat, Oct 16, 2021 at 12:20:24PM +0200, Arnaud Ferraris wrote:
> A new 'chassis-type' root node property has recently been approved for
> the device-tree specification, in order to provide a simple way for
> userspace to detect the device form factor and adjust their behavior
> accordingly.
> 
> This patch fills in this property for end-user devices (such as laptops,
> smartphones and tablets) based on Qualcomm ARM64 processors.
> 
> Signed-off-by: Arnaud Ferraris <arnaud.ferraris@xxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dts            | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts                  | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts                  | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts            | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dts            | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts            | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dts            | 1 +
>  arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dts           | 1 +
> [...]
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dts b/arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dts
> index 670bd1bebd73..265e539e7e99 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dts
> @@ -9,6 +9,7 @@
>  / {
>  	model = "Alcatel OneTouch Idol 3 (4.7)";
>  	compatible = "alcatel,idol347", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts b/arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts
> index cee451e59385..6bc0a29d4b4f 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts
> @@ -8,6 +8,7 @@
>  / {
>  	model = "Asus Zenfone 2 Laser";
>  	compatible = "asus,z00l", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts b/arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts
> index e0075b574190..42d93d3fba36 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts
> @@ -25,6 +25,7 @@
>  / {
>  	model = "Huawei Ascend G7";
>  	compatible = "huawei,g7", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts b/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts
> index 30716eb8fb2d..9c83c96d2c8a 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts
> @@ -11,6 +11,7 @@
>  / {
>  	model = "Longcheer L8150";
>  	compatible = "longcheer,l8150", "qcom,msm8916-v1-qrd/9-v1", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dts b/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dts
> index 27845189ac2b..f9ce123471d4 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dts
> @@ -10,6 +10,7 @@
>  / {
>  	model = "BQ Aquaris X5 (Longcheer L8910)";
>  	compatible = "longcheer,l8910", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts
> index 6cc2eaeb1d33..4ba11b020f9b 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dts
> @@ -7,6 +7,7 @@
>  / {
>  	model = "Samsung Galaxy A3U (EUR)";
>  	compatible = "samsung,a3u-eur", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	reg_panel_vdd3: regulator-panel-vdd3 {
>  		compatible = "regulator-fixed";
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dts
> index c2eff5aebf85..d978c9ac179d 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dts
> @@ -7,6 +7,7 @@
>  / {
>  	model = "Samsung Galaxy A5U (EUR)";
>  	compatible = "samsung,a5u-eur", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	reg_touch_key: regulator-touch-key {
>  		compatible = "regulator-fixed";
> diff --git a/arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dts b/arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dts
> index 4e20cc0008f7..69a44c6f57fc 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dts
> @@ -11,6 +11,7 @@
>  / {
>  	model = "Xiaomi Redmi 2 (Wingtech WT88047)";
>  	compatible = "wingtech,wt88047", "qcom,msm8916";
> +	chassis-type = "handset";
>  
>  	aliases {
>  		serial0 = &blsp1_uart2;

FWIW:
Reviewed-by: Stephan Gerhold <stephan@xxxxxxxxxxx> # msm8916

Thanks for going through the work to add it to so many devices! :)
Stephan