On Thu, Jun 15, 2023 at 5:19 PM Yann Gautier <yann.gautier@xxxxxxxxxxx> wrote: > > - description: Entry for STMicroelectronics variant of PL18x. > > This dedicated compatible is used by bootloaders. (...) > > - description: Entry for STMicroelectronics variant of PL18x for > > STM32MP25. This dedicated compatible is used by bootloaders. (...) > > Should I remove (or adapt) both descriptions? > > > > > > At the time the patch was done it was really just used by bootloaders. > But as it is now used in the driver for delay block, I should remove the > second sentence. Remove both. After "This dedicated compatible is used by bootloaders" there is an implicit "in the SDK provided by ST Microelectronics", and that is of no concern for DT bindings, which are (well, in theory) used by e.g. BSD or other operating systems and who knows what they will use and not, we don't put Linux specifics in there, neither Boot loader specifics nor ST SDK specifics. At least that is the little bureaucratic ambition we have. Yours, Linus Walleij