Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
Message-ID: <eaf9759c-946b-a2fd-042d-235709201908@quicinc.com>
Date:   Tue, 3 Oct 2023 10:39:02 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.0
Subject: Re: [PATCH] dt-bindings: arm: qcom: drop the IPQ board types
Content-Language: en-US
To:     Bryan O'Donoghue <bryan.odonoghue@linaro.org>, <agross@kernel.org>,
        <andersson@kernel.org>, <konrad.dybcio@linaro.org>,
        <robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
        <conor+dt@kernel.org>, <linux-arm-msm@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20230927093130.4098385-1-quic_sridsn@quicinc.com>
 <3b7a6266-e2fc-4b56-8b41-a26042248c64@linaro.org>
From:   Sridharan S N <quic_sridsn@quicinc.com>
In-Reply-To: <3b7a6266-e2fc-4b56-8b41-a26042248c64@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 9/27/2023 5:01 PM, Bryan O'Donoghue wrote:
> On 27/09/2023 10:31, Sridharan S N wrote:
>> IPQ bootloaders do not need these information to select the
>> the DTB blob. So dropping the board names from board section.
>>
>> Signed-off-by: Sridharan S N <quic_sridsn@quicinc.com>
>> ---
>>   Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
>>   1 file changed, 14 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml 
>> b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index 7f80f48a0954..5b56f3d90d4d 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -91,24 +91,10 @@ description: |
>>     The 'board' element must be one of the following strings:
>>             adp
>> -        ap-al02-c2
>> -        ap-al02-c6
>> -        ap-al02-c7
>> -        ap-al02-c8
>> -        ap-al02-c9
>> -        ap-mi01.2
>> -        ap-mi01.3
>> -        ap-mi01.6
>> -        ap-mi01.9
>>           cdp
>> -        cp01-c1
>>           dragonboard
>> -        hk01
>> -        hk10-c1
>> -        hk10-c2
>>           idp
>>           liquid
>> -        rdp432-c2
>>           mtp
>>           qrd
>>           rb2
>
> All bootloaders ? You are 100% sure that all versions of downstream 
> u-boot don't rely on these strings ? Upstream u-boot too ?
>
> ---
> bod

Yes , we don't rely on these strings , instead downstream u-boot uses 
machid to identify the DTB.

Thanks,

Sridharan S N