Re: [PATCH] arm64: dts: qcom: Remove unused and undocumented properties

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/16/2024 10:00 PM, Rob Herring wrote:
> On Mon, Dec 16, 2024 at 6:42 AM Konrad Dybcio
> <konrad.dybcio@xxxxxxxxxxxxxxxx> wrote:
>>
>> On 16.12.2024 4:42 AM, Aiqun Yu (Maria) wrote:
>>> On 11/16/2024 3:36 AM, Konrad Dybcio wrote:
>>>> On 15.11.2024 8:34 PM, Rob Herring (Arm) wrote:
>>>>> Remove properties which are both unused in the kernel and undocumented.
>>>>> Most likely they are leftovers from downstream.
>>>>>
>>>>> Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
>>>>> ---
>>>>
>>>> Funny, the sideinteraction compatible is undocumented and the touchscreen
>>>> driver is not upstream
>>>
>>> Does the compatible also need to be fixed to "focaltech,ft8719"? Refer
>>> to the info from [1].
>>
>> +Alexander, Caleb - would you like to look into this on axolotl?
>>
>>> Confirmed, the compatible "focaltech,fts8719" is not even in any merged
>>> bindings or drivers. Instead, "focaltech,ft8719" is in the current
>>> driver and bindings.
>>>
>>> By the way, I am curious about how Rob finds the unused properties.
>>> Could you share some steps?
>>
>> make ARCH=arm64 LLVM=1 -j24 CHECK_DTBS=1 dtbs
>>
>> returns all kinds of errors, including "undocumented property"
> 
> This case however was a dtschema change I have not yet committed.
> Plenty of other things to fix still with the above command though. :(

Thanks, Rob and Konrad, for the clarification. That answers my question
about the intention behind the current change.

-- 
Thx and BRs,
Aiqun(Maria) Yu




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux