On Thu, Jul 25, 2013 at 12:21:41PM -0500, Dan Murphy wrote: > On 07/25/2013 09:37 AM, Tom Rini wrote: > > On Tue, Jul 23, 2013 at 01:34:15PM -0500, Dan Murphy wrote: > >> On 07/19/2013 02:00 PM, Tom Rini wrote: > >>> From: Greg Guyotte <gguyo...@ti.com> > >>> > >>> Add a driver for the TPS65217 PMIC that is found in the Beaglebone > >>> family of boards. > > Can we add the public reference to the technical manual in the commit message? > > I found > <http://www.ti.com/lit/ug/slvu580b/slvu580b.pdf>http://www.ti.com/lit/ds/symlink/tps65217b.pdf
I've got the stable link in the header file, now (also for tps65910). [snip] > >>> +{ > >>> + if ((dc_cntrl_reg != DEFDCDC1) && (dc_cntrl_reg != DEFDCDC2) && > >>> + (dc_cntrl_reg != DEFDCDC3)) > >> What do these magic numbers mean? Are these HEX numbers or a string? > > OK, it took me a minute to understand your question here. These are > > defines to register names, matching the TRM for the part. The register > > names are however annoyingly and easily confused as hex values. > > Maybe we can rename the #defines so they are not so confusing. Maybe > something like > > #define TPS65217_<register name> <register offset> > > So these will become > #define TPS65217_DEFDCDC1 0xe > #define TPS65217_DEFDCDC2 0xf > #define TPS65217_DEFDCDC3 0x10 > > This will at least keep other defines like CHIPID and SEQ unique as well. I can do that, yeah. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot