Re: [PATCH v8 01/17] regulator: dt-bindings: describe the PMU module of the QCA6390 package

Bartosz Golaszewski
2024-06-05T09:13:17Z

On Tue, Jun 4, 2024 at 7:30 PM Bjorn Helgaas <helgaas@kernel.org> wrote:
>
> On Tue, May 28, 2024 at 09:03:09PM +0200, Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >
> > The QCA6390 package contains discreet modules for WLAN and Bluetooth. They
>
> s/discreet/discrete/
>
> > are powered by the Power Management Unit (PMU) that takes inputs from the
> > host and provides LDO outputs. This document describes this module.
>
> LDO? Again below, but maybe this is obvious to everybody.
>

Yes, this is an acceptable abbreviation to use, it's all over the
bindings and regulator drivers.

> "This document describes this module" seems possibly unnecessary.
>
> > +description:
> > + The QCA6390 package contains discreet modules for WLAN and Bluetooth. They
>
> s/discreet/discrete/
>
> > + are powered by the Power Management Unit (PMU) that takes inputs from the
> > + host and provides LDO outputs. This document describes this module.
>
> > + vddpcie1p3-supply:
> > + description: VDD_PCIE_1P3 supply regulator handle<S-Del>
>
> s/<S-Del>// ?

Eek, bad copy-paste.

Bart