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: <076badf1-520e-ed6f-c459-cf77c99bf0b9@collabora.com>
Date:   Tue, 30 May 2023 08:34:17 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
Subject: Re: [PATCH] arm64: mediatek: Propagate chassis-type where possible
To:     Matthias Brugger <matthias.bgg@gmail.com>
Cc:     krzysztof.kozlowski+dt@linaro.org, robh+dt@kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org, kernel@collabora.com,
        Kevin Hilman <khilman@baylibre.com>
References: <20230517101108.205654-1-angelogioacchino.delregno@collabora.com>
 <4f75238e-dd98-3e33-7187-4131d524919c@gmail.com>
Content-Language: en-US
From:   AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>
In-Reply-To: <4f75238e-dd98-3e33-7187-4131d524919c@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

Il 29/05/23 17:53, Matthias Brugger ha scritto:
> 
> 
> On 17/05/2023 12:11, AngeloGioacchino Del Regno wrote:
>> The chassis-type string identifies the form-factor of the system:
>> add this property to all device trees of devices for which the form
>> factor is known.
>>
>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> 
> Applied thanks!
> 
> I'm I correct that you are missing the chassis-type for some boards from pumpkin 
> series? My bet is they are embedded, but I'd prefer that someone with access to the 
> boards can clarify.
> 

You're correct. I am 95% sure that they are embedded... not 100% sure though.

If anyone with access to those boards can either send a patch, or clarify that
to us, that would be genuinely appreciated.

Cheers!
Angelo

> Regards,
> Matthias
> 
>> ---
>>
>> I had to ignore some devicetrees as I wasn't sure about the correct
>> chassis-type of some devices; this raises an issue that we shall solve
>> when reviewing new DTs, as we shall tell developers to add this
>> property so that we don't lose track of "what was what".
>>
>> Even though this property is technically "Optional, Recommended", I
>> think it's useful to have it as this may (or may not) be actively used
>> in the future (think about "handset" form factor and thermal envelope).
>>
>> Cheers!
>>
>>   arch/arm64/boot/dts/mediatek/mt2712-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt6755-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt6779-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt6795-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt6797-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts                  | 1 +
>>   arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts         | 1 +
>>   arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts                     | 1 +
>>   arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3.dts         | 1 +
>>   arch/arm64/boot/dts/mediatek/mt7986a-rfb.dts                     | 1 +
>>   arch/arm64/boot/dts/mediatek/mt7986b-rfb.dts                     | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dts                  | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts            | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts                 | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8173-elm.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8173-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-evb.dts                      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dts     | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts       | 1 +
>>   .../boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dts     | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dts       | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dts             | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dts       | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dts      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dts      | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dts         | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dts       | 1 +
>>   arch/arm64/boot/dts/mediatek/mt8186-evb.dts                      | 1 +
>>   28 files changed, 28 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
>> index d31a194124c9..fffdb7bbf889 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
>> @@ -11,6 +11,7 @@
>>   / {
>>       model = "MediaTek MT2712 evaluation board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt2712-evb", "mediatek,mt2712";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6755-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt6755-evb.dts
>> index e079b7932ba3..00b14f85c6a1 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6755-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt6755-evb.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek MT6755 EVB";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt6755-evb", "mediatek,mt6755";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6779-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt6779-evb.dts
>> index 164f5cbb3821..56b1bf06e26b 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6779-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt6779-evb.dts
>> @@ -10,6 +10,7 @@
>>   / {
>>       model = "MediaTek MT6779 EVB";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt6779-evb", "mediatek,mt6779";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6795-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt6795-evb.dts
>> index 1ed2f81edeff..e0d4d7a63139 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6795-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt6795-evb.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek MT6795 Evaluation Board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt6795-evb", "mediatek,mt6795";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt6797-evb.dts
>> index 2327e752d164..c927932afa0d 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6797-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-evb.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek MT6797 Evaluation Board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt6797-evb", "mediatek,mt6797";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts 
>> b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts
>> index eff9e8dbd076..9534cf3a09d0 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts
>> @@ -12,6 +12,7 @@
>>   / {
>>       model = "Mediatek X20 Development Board";
>> +    chassis-type = "embedded";
>>       compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts 
>> b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> index af3fe61e4093..4848164e8e59 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> @@ -15,6 +15,7 @@
>>   / {
>>       model = "Bananapi BPI-R64";
>> +    chassis-type = "embedded";
>>       compatible = "bananapi,bpi-r64", "mediatek,mt7622";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts 
>> b/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts
>> index b74e774c6eba..dad8e683aac5 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts
>> @@ -15,6 +15,7 @@
>>   / {
>>       model = "MediaTek MT7622 RFB1 board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3.dts 
>> b/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3.dts
>> index 33bd6febc160..7128f779c165 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3.dts
>> @@ -16,6 +16,7 @@
>>   / {
>>       model = "Bananapi BPI-R3";
>> +    chassis-type = "embedded";
>>       compatible = "bananapi,bpi-r3", "mediatek,mt7986a";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7986a-rfb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt7986a-rfb.dts
>> index 4f18b4a9a8c8..3ef371ca254e 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7986a-rfb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7986a-rfb.dts
>> @@ -11,6 +11,7 @@
>>   / {
>>       model = "MediaTek MT7986a RFB";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt7986a-rfb", "mediatek,mt7986a";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7986b-rfb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt7986b-rfb.dts
>> index 188ce82ae56c..dde190442e38 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7986b-rfb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7986b-rfb.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek MT7986b RFB";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt7986b-rfb", "mediatek,mt7986b";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dts
>> index 774a2f3fb4b2..ebf1a358f42a 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8167-pumpkin.dts
>> @@ -11,6 +11,7 @@
>>   / {
>>       model = "Pumpkin MT8167";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt8167-pumpkin", "mediatek,mt8167";
>>       memory@40000000 {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts
>> index 28433b94f7c7..256f245ac01d 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts
>> @@ -8,6 +8,7 @@
>>   / {
>>       model = "Google Hanawl";
>> +    chassis-type = "laptop";
>>       compatible = "google,hana-rev7", "mediatek,mt8173";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts
>> index c234296755e1..fcf0cb76a87c 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts
>> @@ -8,6 +8,7 @@
>>   / {
>>       model = "Google Hana";
>> +    chassis-type = "laptop";
>>       compatible = "google,hana-rev6", "google,hana-rev5",
>>                "google,hana-rev4", "google,hana-rev3",
>>                "google,hana", "mediatek,mt8173";
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-elm.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8173-elm.dts
>> index e9e4ac0b74b2..2390d04204e8 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8173-elm.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8173-elm.dts
>> @@ -8,6 +8,7 @@
>>   / {
>>       model = "Google Elm";
>> +    chassis-type = "laptop";
>>       compatible = "google,elm-rev8", "google,elm-rev7", "google,elm-rev6",
>>                "google,elm-rev5", "google,elm-rev4", "google,elm-rev3",
>>                "google,elm", "mediatek,mt8173";
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
>> index 755df5694234..5122963d8743 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
>> @@ -10,6 +10,7 @@
>>   / {
>>       model = "MediaTek MT8173 evaluation board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt8173-evb", "mediatek,mt8173";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
>> index 3e3f4b1b00f0..d8bd51807683 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
>> @@ -11,6 +11,7 @@
>>   / {
>>       model = "MediaTek MT8183 evaluation board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt8183-evb", "mediatek,mt8183";
>>       aliases {
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dts
>> index 1a2ec0787d3c..19c1e2bee494 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "Google burnet board";
>> +    chassis-type = "convertible";
>>       compatible = "google,burnet", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
>> index 0eca3ff8672a..552bfc726999 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "Google damu board";
>> +    chassis-type = "convertible";
>>       compatible = "google,damu", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dts
>> index bc2c57f0a827..8ac6bf5b17f9 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "Google juniper sku16 board";
>> +    chassis-type = "convertible";
>>       compatible = "google,juniper-sku16", "google,juniper", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dts
>> index 3a724e6f915c..fcce8ea1232e 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu-sku22.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek kakadu board sku22";
>> +    chassis-type = "tablet";
>>       compatible = "google,kakadu-rev3-sku22", "google,kakadu-rev2-sku22",
>>                "google,kakadu", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dts
>> index 89a139a0ee44..ebfabba72507 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dts
>> @@ -9,6 +9,7 @@
>>   / {
>>       model = "MediaTek kakadu board";
>> +    chassis-type = "tablet";
>>       compatible = "google,kakadu-rev3", "google,kakadu-rev2",
>>               "google,kakadu", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dts
>> index e3dd75bdaea4..7213cdcca612 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dts
>> @@ -12,6 +12,7 @@
>>   / {
>>       model = "MediaTek kodama sku16 board";
>> +    chassis-type = "tablet";
>>       compatible = "google,kodama-sku16", "google,kodama", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dts
>> index d81935ae07bc..bbf0cd1aa66d 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dts
>> @@ -12,6 +12,7 @@
>>   / {
>>       model = "MediaTek kodama sku272 board";
>> +    chassis-type = "tablet";
>>       compatible = "google,kodama-sku272", "google,kodama", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dts
>> index f4082fbe0517..a429ffeac3bd 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dts
>> @@ -12,6 +12,7 @@
>>   / {
>>       model = "MediaTek kodama sku288 board";
>> +    chassis-type = "tablet";
>>       compatible = "google,kodama-sku288", "google,kodama", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dts
>> index fb5ee91b6fe0..4ac75806fa94 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dts
>> @@ -14,6 +14,7 @@
>>   / {
>>       model = "MediaTek krane sku0 board";
>> +    chassis-type = "tablet";
>>       compatible = "google,krane-sku0", "google,krane", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dts
>> index 721d16f9c3b4..095279e55d50 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dts
>> @@ -14,6 +14,7 @@
>>   / {
>>       model = "MediaTek krane sku176 board";
>> +    chassis-type = "tablet";
>>       compatible = "google,krane-sku176", "google,krane", "mediatek,mt8183";
>>   };
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8186-evb.dts 
>> b/arch/arm64/boot/dts/mediatek/mt8186-evb.dts
>> index ed74a3617c13..2667a7424200 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8186-evb.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8186-evb.dts
>> @@ -7,6 +7,7 @@
>>   / {
>>       model = "MediaTek MT8186 evaluation board";
>> +    chassis-type = "embedded";
>>       compatible = "mediatek,mt8186-evb", "mediatek,mt8186";
>>       aliases {
> 

-- 
AngeloGioacchino Del Regno
Software Engineer

Collabora Ltd.
Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, UK
Registered in England & Wales, no. 5513718