[U-Boot] [PATCH 1/2] power: add PFUZE100 PMIC driver

Stefano Babic sbabic at denx.de
Tue Feb 4 10:04:42 CET 2014


Hi Tim,

On 03/02/2014 20:30, Tim Harvey wrote:
>> This is always printed, not only on your board - even from other board,
>> if any, that will use this pmic. If you want that your board always
>> prints the pmic revision, move this output to your board file - maybe in
>> checkboard().
> 
> I noticed that all the pmic drivers in drivers/power/pmic print
> something very generic in their pmic_init such as 'Board PMIC init'
> (even though no initialization or communication with the pmic has
> occurred).  I thought I would be a bit more specific and display
> exactly which PMIC init has been called and provide some details about
> the PMIC version.
> 
> I would agree that this can go into checkboard()
> 
> Is there value in adding the 'puts("Board PMIC init\n")? 

I do not think so

> I didn't
> want to display that if the pmic isn't present (as this PMIC isn't
> present on some Ventana boards).

Agree.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list