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?
Yann
