On Mon, 19 Jun 2023 at 09:29, Yann Gautier <yann.gautier@xxxxxxxxxxx> wrote: > > On 6/15/23 20:51, Linus Walleij wrote: > > 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 > > Hi, > > Thanks for all the reviews. > I'll update this patch in the v2, removing bootloader line and using enum. > > Ulf, should I send the new series now, or do you prefer to review the > whole series before? Actually I have already looked through the series and it looks good to me! Please submit a new version so we can get this queued up for v6.5. Kind regards Uffe
