lkml.org 
[lkml]   [2015]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 5/6] power: bq27xxx_battery: Add support for additional bq27xxx family devices
Date
On Tuesday 28 July 2015 21:39:12 Andrew F. Davis wrote:
> Add support for additional devices and register equivalent family
> devices including the bq27010, bq27210, bq272500, bq27510, bq27520,
^^^^^^
maybe incorrect number?
> bq27530, bq27531, bq27541, bq27542, bq27546, bq27545, bq27441,
> bq27421, and the bq27641.
>
> To facilitate this process the register mapings have been moved to
> tables and other small cleanups have been made.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---

> static const struct i2c_device_id bq27xxx_id[] = {
> - { "bq27200", BQ27000 }, /* bq27200 is same as bq27000, but with i2c
> */ + { "bq27200", BQ27000 },
> + { "bq27210", BQ27010 },
> { "bq27500", BQ27500 },
> - { "bq27425", BQ27425 },
> - { "bq27742", BQ27742 },
> - { "bq27510", BQ27510 },
> + { "bq27510", BQ27500 },
> + { "bq27520", BQ27500 },
> + { "bq27530", BQ27530 },
> + { "bq27531", BQ27530 },
> + { "bq27541", BQ27541 },
> + { "bq27542", BQ27541 },
> + { "bq27546", BQ27541 },
> + { "bq27742", BQ27541 },
> + { "bq27545", BQ27545 },
> + { "bq27421", BQ27421 },
> + { "bq27425", BQ27421 },
> + { "bq27441", BQ27421 },
> + { "bq27621", BQ27421 },
> {},
> };

I'm trying to understand comparator which you used for sorting these
values... but I do not see any logic here. What about sorting list by
first value? Now it is long list of device names and for better ability
to read it, it would be better to have it in some order.

> -enum bq27xxx_chip { BQ27000, BQ27500, BQ27425, BQ27742, BQ27510 };
> +enum bq27xxx_chip {
> + BQ27000, /* bq27000, bq27200 */
> + BQ27010, /* bq27010, bq27210 */
> + BQ27500, /* bq27500, bq27510, bq27520 */
> + BQ27530, /* bq27530, bq27531 */
> + BQ27541, /* bq27541, bq27542, bq27546, bq27742 */
> + BQ27545, /* bq27545 */
> + BQ27421, /* bq27421, bq27425, bq27441, bq27621 */
> +};
>

Maybe same here? Putting BQ27421 before BQ27500?

--
Pali Rohár
pali.rohar@gmail.com
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2015-07-29 20:06    [W:0.129 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site