On Thu, Mar 13, 2014 at 10:12 AM, Pavel Machek <pavel@xxxxxx> wrote: > Hi! > >> > 30*HZ means 30 seconds in the kernel... what is hard to understand >> > about it? >> >> Well I might be picky, but since it is a charging algorithm dealing with >> ampères, volts, constant-current/constant-voltage, watchdogs and >> timeouts, all stated in SI units, it would be nice if all such constants >> were specified in simple units instead of kernel-specific terms. > > I agree HZ is badly named, but hopefully anyone working on kernel is > already familiar with it. > > But... what would actually help: I believe we should introduce > milivolt_t, miliamp_t, milisec_t etc... types. Storing milivolts in > int, then having comment saying "milivolts" is just wrong. Hm! I bet the regulator subsystem maintainers have opinions on that. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
