Hello Krzysztof, Thank you for the review! On Tue, May 02, 2023 at 09:39:12AM +0200, Krzysztof Kozlowski wrote: > On 02/05/2023 03:38, Christian Hewitt wrote: > >> On 1 May 2023, at 7:51 pm, Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> wrote: > >> > >> Hi Dmitry, > >> > >> On Wed, Apr 26, 2023 at 11:58 AM Dmitry Rokosov > >> <ddrokosov@xxxxxxxxxxxxxx> wrote: > >>> > >>> Add the documentation for Amlogic A1 Peripherals clock driver, > >>> and A1 Peripherals clock controller bindings. > >> Maybe a native English speaker can comment on whether it's > >> "peripheral" or "peripherals". > > > > I’m not a grammar specialist, but I would write: > > > > “Add documentation and bindings for the Amlogic A1 SoC peripherals > > clock driver” > > > > Peripherals is the correct plural but reads better when you add > > context on the type of peripherals. > > Drop the "driver" references - from the binding itself and from commit > msg. The bindings are for hardware, not for the driver, so: "for the > Amlogic A1 SoC peripherals clock controller.". Okay, thank you for the suggestion! I will remove it in the next version. -- Thank you, Dmitry
