Re: [PATCH v3 1/4] x86/of: Convert Intel's APIC bindings to YAML schema
From: Rahul Tanwar
Date: Thu Nov 24 2022 - 03:34:52 EST
On 24/11/2022 12:08 am, Krzysztof Kozlowski wrote:
> This email was sent from outside of MaxLinear.
>
>
> On 22/11/2022 08:39, Rahul Tanwar wrote:
>> Intel's APIC family of interrupt controllers support local APIC
>> (lapic) & I/O APIC (ioapic). Convert existing bindings for lapic
>> & ioapic from text to YAML schema. Separate lapic & ioapic schemas.
>> Addditionally, add description which was missing in text file and
>> add few more required standard properties which were also missing
>> in text file.
>>
>> Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>> Signed-off-by: Rahul Tanwar <rtanwar@xxxxxxxxxxxxx>
>> ---
>> .../intel,ce4100-ioapic.txt | 26 --------
>> .../intel,ce4100-ioapic.yaml | 62 +++++++++++++++++++
>> .../intel,ce4100-lapic.yaml | 49 +++++++++++++++
>> 3 files changed, 111 insertions(+), 26 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/intel,ce4100-ioapic.txt
>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/intel,ce4100-ioapic.yaml
>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/intel,ce4100-lapic.yaml
>>
>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC. It might happen, that command when run on an older
> kernel, gives you outdated entries. Therefore please be sure you base
> your patches on recent Linux kernel.
>
> You miss not only people but also lists, meaning this will not be
> automatically tested.
>
> So: No.
>
Agree that i made mistakes in email list earlier. But i fixed that
problem from v4 onwards thanks to Andy. From v4 onwards, To & Cc should
be correct. Thanks.
Regards,
Rahul
> Best regards,
> Krzysztof
>
>