On Fri, Jan 15, 2021 at 11:03:43PM +0900, Damien Le Moal wrote: > Fix the Sifive clint compatible string definition to make the value > "sifive,fu540-c000-clint" optional, allowing a DT to specify only > "sifive,clint0" for its generic implementation without any tweak. > With this change, a Sifive clint node can now be defined as explained > in the compatible string description. NACK. How do you know you'll never need some tweak? Adding it to the DT later defeats the whole point of not having to do DT updates to implement work-arounds and/or enhancements. > > Cc: Anup Patel <anup.patel@xxxxxxx> > Cc: Rob Herring <robh@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Signed-off-by: Damien Le Moal <damien.lemoal@xxxxxxx> > --- > .../devicetree/bindings/timer/sifive,clint.yaml | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/timer/sifive,clint.yaml b/Documentation/devicetree/bindings/timer/sifive,clint.yaml > index 2a0e9cd9fbcf..c6b1f37a2949 100644 > --- a/Documentation/devicetree/bindings/timer/sifive,clint.yaml > +++ b/Documentation/devicetree/bindings/timer/sifive,clint.yaml > @@ -22,12 +22,13 @@ description: > > properties: > compatible: > - items: > - - const: sifive,fu540-c000-clint > - - const: sifive,clint0 > + oneOf: > + - items: > + - enum: > + - sifive,fu540-c000-clint > + - sifive,clint0 > > description: > - Should be "sifive,<chip>-clint" and "sifive,clint<version>". > Supported compatible strings are - > "sifive,fu540-c000-clint" for the SiFive CLINT v0 as integrated > onto the SiFive FU540 chip, and "sifive,clint0" for the SiFive > -- > 2.29.2 >
