Re: [PATCH v7 1/8] dt-bindings: net: bluetooth: Add device tree bindings for QTI chip wcn3990

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

On 2018-06-20 20:24, Rob Herring wrote:
On Wed, Jun 20, 2018 at 5:33 AM, Balakrishna Godavarthi
<bgodavar@xxxxxxxxxxxxxx> wrote:
Hi Rob,

please find my comments inline.

On 2018-06-18 18:59, Rob Herring wrote:

On Sat, Jun 16, 2018 at 12:27 AM, Balakrishna Godavarthi
<bgodavar@xxxxxxxxxxxxxx> wrote:

Signed-off-by: Balakrishna Godavarthi <bgodavar@xxxxxxxxxxxxxx>

[...]

-Example:
+ - vddpa-supply: Bluetooth VDD PA regulator handle
+ - vddio-supply: Bluetooth VDD IO regulator handle
+ - vddldo-supply: Bluetooth VDD LDO regulator handle. Kept under
optional
+ parameters as some of the chipsets doesn't require ldo
or
+                 it may use from same vddio.


You need to be explicit as to which properties apply or don't apply to
specific compatible strings.


[Bala]: will update.

Supplies should be based on inputs to the chip, so if 1 regulator
supplies 2 inputs you still need both.

[Bala]: sorry i did't get you, all these regulators are single output line
regulators.
         i.e. <supply-name>- supply, need single regulator input as a
handle.
         and these line regulators generate fixed voltage output.
         please let me know, if i have cleared your query

I was referring to "it may use from same vddio". If you mean vddldo is
left disconnected, then omitting the property is fine. If you mean
vddio and vddldo are connected to the same regulator, then you should
have both properties in the binding.

Rob

 may be pharse is confusing, will correct the text.
 we require both vregs to turn on wcn3990.

--
Regards
Balakrishna.
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux