[U-Boot] [PATCH V2 00/13] PMIC/REGULATOR cleanup and Sandbox tests

Simon Glass
2015-05-18T20:43:59Z

Hi Przemyslaw,

On 15 May 2015 at 10:21, Przemyslaw Marczak <p.marczak@samsung.com> wrote:
> Hello Simon,
>
>
> On 05/15/2015 03:55 PM, Simon Glass wrote:
>>
>> On 13 May 2015 at 05:38, Przemyslaw Marczak <p.marczak@samsung.com> wrote:
>>>
>>> Hello Simon,
>>>
>>> This patchset fixes the build issues. It changes the last three
>>> top commits of u-boot-dm/next. So it is rebased onto:
>>> 4e9a6eb dm: board:samsung: power_init_board: add requirement of
>>> CONFIG_DM_PMIC
>>>
>>> The rebased version, can be fetched from:
>>> https://github.com/bobenstein/u-boot.git
>>> branch: dm-pmic-v5-sandbox
>>>
>>> Best regards,
>>> Przemyslaw Marczak
>>>
>>> Przemyslaw Marczak (13):
>>> odroid: dts: add 'voltage-regulators' description to max77686 node
>>> odroid: enable driver model pmic/regulator API and MAX77686 drivers
>>> dm: pmic: code cleanup of PMIC uclass driver
>>> dm: regulator: uclass driver code cleanup
>>> common: cmd pmic: command cleanup
>>> common: cmd regulator: command cleanup
>>> doc: driver-model: pmic-framework.txt - cleanup
>>> sandbox: i2c: search child emul dev and check its uclass id
>>> sandbox: add: sandbox PMIC device drivers: I2C emul, pmic, regulator
>>> test: dm: add sandbox PMIC framework tests
>>> test: dm: test.dts - move to sandbox dts directory
>>> sandbox: dts: add sandbox_pmic.dtsi and include it to sandbox.dts and
>>> test.dts
>>> sandbox: defconfig: enable support of sandbox PMIC drivers
>>>
>>> arch/arm/dts/exynos4412-odroid.dts | 253 ++++++++++++++++++
>>> arch/sandbox/dts/Makefile | 1 +
>>> arch/sandbox/dts/sandbox.dts | 4 +
>>> arch/sandbox/dts/sandbox_pmic.dtsi | 78 ++++++
>>> arch/sandbox/dts/test.dts | 236 ++++++++++++++++
>>> board/samsung/common/misc.c | 1 +
>>> board/samsung/odroid/odroid.c | 76 +++---
>>> common/cmd_pmic.c | 131 ++++-----
>>> common/cmd_regulator.c | 239 +++++++++--------
>>> configs/odroid_defconfig | 8 +-
>>> configs/sandbox_defconfig | 7 +
>>> doc/device-tree-bindings/pmic/sandbox.txt | 35 +++
>>> doc/device-tree-bindings/regulator/sandbox.txt | 45 ++++
>>> doc/driver-model/pmic-framework.txt | 20 +-
>>> drivers/i2c/sandbox_i2c.c | 20 +-
>>> drivers/power/pmic/Kconfig | 27 +-
>>> drivers/power/pmic/Makefile | 3 +-
>>> drivers/power/pmic/i2c_pmic_emul.c | 142 ++++++++++
>>> drivers/power/pmic/max77686.c | 15 +-
>>> drivers/power/pmic/pmic-uclass.c | 31 +--
>>> drivers/power/pmic/sandbox.c | 79 ++++++
>>> drivers/power/regulator/Kconfig | 32 ++-
>>> drivers/power/regulator/Makefile | 1 +
>>> drivers/power/regulator/regulator-uclass.c | 104 +++++---
>>> drivers/power/regulator/sandbox.c | 355
>>> +++++++++++++++++++++++++
>>> include/configs/odroid.h | 5 -
>>> include/dt-bindings/pmic/sandbox_pmic.h | 35 +++
>>> include/power/pmic.h | 39 +--
>>> include/power/regulator.h | 116 ++++----
>>> include/power/sandbox_pmic.h | 138 ++++++++++
>>> test/dm/.gitignore | 1 -
>>> test/dm/Makefile | 2 +
>>> test/dm/pmic.c | 69 +++++
>>> test/dm/regulator.c | 325
>>> ++++++++++++++++++++++
>>> test/dm/test-dm.sh | 3 +-
>>> test/dm/test-main.c | 3 +-
>>> test/dm/test.dts | 230 ----------------
>>> 37 files changed, 2287 insertions(+), 622 deletions(-)
>>> create mode 100644 arch/sandbox/dts/sandbox_pmic.dtsi
>>> create mode 100644 arch/sandbox/dts/test.dts
>>> create mode 100644 doc/device-tree-bindings/pmic/sandbox.txt
>>> create mode 100644 doc/device-tree-bindings/regulator/sandbox.txt
>>> create mode 100644 drivers/power/pmic/i2c_pmic_emul.c
>>> create mode 100644 drivers/power/pmic/sandbox.c
>>> create mode 100644 drivers/power/regulator/sandbox.c
>>> create mode 100644 include/dt-bindings/pmic/sandbox_pmic.h
>>> create mode 100644 include/power/sandbox_pmic.h
>>> delete mode 100644 test/dm/.gitignore
>>> create mode 100644 test/dm/pmic.c
>>> create mode 100644 test/dm/regulator.c
>>> delete mode 100644 test/dm/test.dts
>>>
>>> --
>>> 1.9.1
>>>
>>
>> I've pulled this in from u-boot-dm/next to master.
>>
>> - Simon
>>
>
> Thank you, this is great information!
> And also thank you for the review of all the versions.

No problem - congratulations on getting this major piece of work into
mainline. I'll see if I can try it out for another SOC.

Regards,
Simon