Re: [PATCH 01/16] dt-bindings: arm: qcom: add SM8550 AIM300

From: Krzysztof Kozlowski
Date: Tue Nov 21 2023 - 02:18:17 EST


On 21/11/2023 01:30, Tengfei Fan wrote:
>
>
> 在 11/20/2023 4:53 PM, Krzysztof Kozlowski 写道:
>> On 17/11/2023 11:22, Krzysztof Kozlowski wrote:
>>> On 17/11/2023 11:18, Tengfei Fan wrote:
>>>> Add board compatible for SM8550 AIM300.
>>>
>>> Subject, commit msg and compatible tell basically the same... and none
>>> of them tell me what is AIM300.
>>
>> Due to lack of explanation it is difficult to judge what's this. However
>> based on pieces of information I got, it looks like it is not a board,
>> so it is not suitable to be DTS, but DTSI. You still need a board...
>>
>> Best regards,
>> Krzysztof
>>
>
> Hi Krzysztof,
> AIM (Artificial Intelligence Module). This hardware platform can be used
> to develop AI related software based on Qualcomm chipset. I also will
> update this message to patch commit message.

Does "Module" means it is physical module?

>
> I also will sync with internal team about if we should create DTSI file
> as your suggestion.

Do they know how DTS should be organized upstream (not downstream)?

Best regards,
Krzysztof