Re: [PATCH V2] dt-bindings: mtd: partitions: Include TP-Link SafeLoader in allowed list

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

 



On Fri, Jun 02, 2023 at 11:56:29PM +0200, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@xxxxxxxxxx>
> 
> It's supposed to be there and it's needed for proper validation of DTS
> files.
> 
> This fixes following errors for Northstar based TP-Link routers:

>         bcm47081-tplink-archer-c5-v2.dtb: flash@0: partitions:compatible: 'oneOf' conditional failed, one must be fixed:
>                 ['tplink,safeloader-partitions'] is too short
>                 'fixed-partitions' was expected
>                 'sercomm,sc-partitions' was expected
>         'arm,arm-firmware-suite' was expected
>         'brcm,bcm4908-partitions' was expected
>         'brcm,bcm947xx-cfe-partitions' was expected
>         'linksys,ns-partitions' was expected
>         'qcom,smem-part' was expected
>         'redboot-fis' was expected

Something like this would have been sufficient, dumping the whole error
into the commit message bloats it for no real gain, but I was asking for
you to say what the issue was, not asking for a resubmission (yet anyway).

Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>

Thanks,
Conor.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux