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