On Fri, Apr 1, 2022 at 4:36 PM Palmer Dabbelt <[email protected]> wrote:
>
> From: Palmer Dabbelt <[email protected]>
>
> As per 39bd2b6a3783 ("dt-bindings: Improve phandle-array schemas"), the
> phandle-array bindings have been disambiguated. This fixes the new
> RISC-V idle-states bindings to comply with the schema.
>
> Fixes: 1bd524f7e8d8 ("dt-bindings: Add common bindings for ARM and RISC-V idle states")
> Signed-off-by: Palmer Dabbelt <[email protected]>
> ---
> Changes since v2:
>
> * Add the missing schema requirement to riscv/cpus.yaml
>
> Changes since v1:
>
> * Only fix the RISC-V bindings, to avoid a merge conflict.
>
> ---
> .../devicetree/bindings/cpu/idle-states.yaml | 16 ++++++++--------
> .../devicetree/bindings/riscv/cpus.yaml | 2 ++
> 2 files changed, 10 insertions(+), 8 deletions(-)
Reviewed-by: Rob Herring <[email protected]>
Thanks for fixing quickly.
Rob
On Fri, 01 Apr 2022 14:39:24 PDT (-0700), Rob Herring wrote:
> On Fri, Apr 1, 2022 at 4:36 PM Palmer Dabbelt <[email protected]> wrote:
>>
>> From: Palmer Dabbelt <[email protected]>
>>
>> As per 39bd2b6a3783 ("dt-bindings: Improve phandle-array schemas"), the
>> phandle-array bindings have been disambiguated. This fixes the new
>> RISC-V idle-states bindings to comply with the schema.
>>
>> Fixes: 1bd524f7e8d8 ("dt-bindings: Add common bindings for ARM and RISC-V idle states")
>> Signed-off-by: Palmer Dabbelt <[email protected]>
>> ---
>> Changes since v2:
>>
>> * Add the missing schema requirement to riscv/cpus.yaml
>>
>> Changes since v1:
>>
>> * Only fix the RISC-V bindings, to avoid a merge conflict.
>>
>> ---
>> .../devicetree/bindings/cpu/idle-states.yaml | 16 ++++++++--------
>> .../devicetree/bindings/riscv/cpus.yaml | 2 ++
>> 2 files changed, 10 insertions(+), 8 deletions(-)
>
> Reviewed-by: Rob Herring <[email protected]>
>
> Thanks for fixing quickly.
Well, sorry for breaking it. I've got the DT checks running locally
now, we've got a handful of errors in RISC-V land. I'll clean those up
and then get something blocking my merges, so stuff like this is less
likely to happen.
Looks like Linus just merged my Part 2, I'll send along the Part 3 just
containing this as a fix.