Hi Przemyslaw,
On 10 July 2015 at 05:53, Przemyslaw Marczak <p.marc...@samsung.com> wrote:
> Hello Simon,
>
> On 07/03/2015 02:16 AM, Simon Glass wrote:
>>
>> The driver name should not have a space in it. Also the regulator names
>> should match the case of the device tree. Fix these problems.
>>
>> Signed-off-by: Simon Glass <s...@chromium.org>
>> ---
>>
>> drivers/power/pmic/max77686.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/power/pmic/max77686.c b/drivers/power/pmic/max77686.c
>> index 3523b4a..dc5a54a 100644
>> --- a/drivers/power/pmic/max77686.c
>> +++ b/drivers/power/pmic/max77686.c
>> @@ -17,8 +17,8 @@
>> DECLARE_GLOBAL_DATA_PTR;
>>
>> static const struct pmic_child_info pmic_children_info[] = {
>> - { .prefix = "ldo", .driver = MAX77686_LDO_DRIVER },
>> - { .prefix = "buck", .driver = MAX77686_BUCK_DRIVER },
>> + { .prefix = "LDO", .driver = MAX77686_LDO_DRIVER },
>> + { .prefix = "BUCK", .driver = MAX77686_BUCK_DRIVER },
>> { },
>> };
>>
>> @@ -84,7 +84,7 @@ static const struct udevice_id max77686_ids[] = {
>> };
>>
>> U_BOOT_DRIVER(pmic_max77686) = {
>> - .name = "max77686 pmic",
>> + .name = "max77686_pmic",
>> .id = UCLASS_PMIC,
>> .of_match = max77686_ids,
>> .bind = max77686_bind,
>>
>
> Acked-by: Przemyslaw Marczak <p.marc...@samsung.com>
>
> This change is ok, but could you please consider Odroid U3
> (exynos4412-odroid.dts) within this patch set?
Ah, does that mean that these need to change to upper case to match the driver?
Applied to u-boot-dm.
>
> Regards
> --
> Przemyslaw Marczak
> Samsung R&D Institute Poland
> Samsung Electronics
> p.marc...@samsung.com
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
