On 07/29/2014 07:36 PM, Mark Brown wrote: > On Tue, Jul 29, 2014 at 06:28:58PM +0200, Javier Martinez Canillas wrote: > >> +#define tps65090_REG_VAR(_id, _sname, en_reg, _en_bits, _ops) \ >> + tps65090_REG_DESC(_id, _sname, en_reg, _en_bits, 0, 0, _ops) > > I'd expect this to describe a variable regulator when in fact it > seems it describes a switch with no regulation. Better name please. >
Yes, when writing the patch I thought of a switch as a non-fixed regulator whose output voltage is variable and depend on its input supply voltage. That's why I used VAR but I agree that is a bad name and I'll change it. Thanks a lot and best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
