Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-28 Thread Felix Brack
Hello Simon, On 27.11.2017 18:16, Simon Glass wrote: > However if you find that you really do need to blur the line between > the regulator and the pmic, and share the pdata structure, it is not > the end of the world. It seems wrong to me, so I'd like to avoid it if > possible. We should have a

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-27 Thread Simon Glass
Hi Felix, On 27 November 2017 at 06:51, Felix Brack wrote: > Hello Simon, > > On 26.11.2017 12:38, Simon Glass wrote: >> Hi Felix, >> >> On 23 November 2017 at 08:36, Felix Brack wrote: >>> >>> >>> On 22.11.2017 11:39, Felix Brack wrote: Hello Simon,

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-27 Thread Felix Brack
Hello Simon, On 26.11.2017 12:38, Simon Glass wrote: > Hi Felix, > > On 23 November 2017 at 08:36, Felix Brack wrote: >> >> >> On 22.11.2017 11:39, Felix Brack wrote: >>> Hello Simon, >>> >>> Many thanks for taking the time to review my patch. >>> >>> On 20.11.2017 16:38, Simon

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-27 Thread Felix Brack
Hello Simon On 26.11.2017 12:39, Simon Glass wrote: > Hi Felix, > > On 22 November 2017 at 03:39, Felix Brack wrote: >> Hello Simon, >> >> Many thanks for taking the time to review my patch. >> >> On 20.11.2017 16:38, Simon Glass wrote: >>> Hi Felix, >>> >>> On 8 November 2017 at

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-26 Thread Simon Glass
Hi Felix, On 22 November 2017 at 03:39, Felix Brack wrote: > Hello Simon, > > Many thanks for taking the time to review my patch. > > On 20.11.2017 16:38, Simon Glass wrote: >> Hi Felix, >> >> On 8 November 2017 at 04:04, Felix Brack wrote: >>> Texas Instrument's

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-26 Thread Simon Glass
Hi Felix, On 23 November 2017 at 08:36, Felix Brack wrote: > > > On 22.11.2017 11:39, Felix Brack wrote: >> Hello Simon, >> >> Many thanks for taking the time to review my patch. >> >> On 20.11.2017 16:38, Simon Glass wrote: >>> Hi Felix, >>> + +/* platform data */

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-23 Thread Felix Brack
On 22.11.2017 11:39, Felix Brack wrote: > Hello Simon, > > Many thanks for taking the time to review my patch. > > On 20.11.2017 16:38, Simon Glass wrote: >> Hi Felix, >> >>> + >>> +/* platform data */ >>> +struct tps65910_pdata { >>> + u32 supply[TPS65910_NUM_SUPPLIES]; /* regulator

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-22 Thread Felix Brack
Hello Simon, Many thanks for taking the time to review my patch. On 20.11.2017 16:38, Simon Glass wrote: > Hi Felix, > > On 8 November 2017 at 04:04, Felix Brack wrote: >> Texas Instrument's TPS65910 PMIC contains 3 buck DC-DC converts, one >> boost DC-DC converter and 8 LDOs.

Re: [U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-20 Thread Simon Glass
Hi Felix, On 8 November 2017 at 04:04, Felix Brack wrote: > Texas Instrument's TPS65910 PMIC contains 3 buck DC-DC converts, one > boost DC-DC converter and 8 LDOs. This patch implements driver model > support for the TPS65910 PMIC and its regulators making the get/set > API for

[U-Boot] [PATCH] power: pmic/regulator: Add basic support for TPS65910

2017-11-08 Thread Felix Brack
Texas Instrument's TPS65910 PMIC contains 3 buck DC-DC converts, one boost DC-DC converter and 8 LDOs. This patch implements driver model support for the TPS65910 PMIC and its regulators making the get/set API for regulator value/enable available. This patch depends on the patch "am33xx: Add a