Thanks for the comments! On 09/13/2014 02:27 AM, Hartmut Knaack wrote: > Stanimir Varbanov schrieb, Am 11.09.2014 17:13: >> The voltage ADC is peripheral of Qualcomm SPMI PMIC chips. It has >> 15bits resolution and register space inside PMIC accessible across >> SPMI bus. >> >> The vadc driver registers itself through IIO interface. >> > Looks already pretty good. Things you should consider in regard of common > coding style are to use the variable name ret instead of rc, since it is used > in almost all adc drivers and thus makes reviewing a bit easier. Besides > that, you seem to use unsigned as well as unsigned int, so to be consistent, > please stick to one of them. Other comments in line.
OK, I will take into account your comments. <snip> -- regards, Stan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
