On 11/13/23 02:13, Nuno Sá wrote:
On Fri, 2023-11-10 at 18:50 +0200, Andy Shevchenko wrote:
On Fri, Nov 10, 2023 at 04:18:46PM +0100, Nuno Sa wrote:
The LTC4282 hot swap controller allows a board to be safely inserted and
removed from a live backplane. Using one or more external N-channel pass
transistors, board supply voltage and inrush current are ramped up at an
adjustable rate. An I2C interface and onboard ADC allows for monitoring
of board current, voltage, power, energy and fault status.
There are plenty of ltc42xx in the same folder and a lot of other chips.
Have you checked that none of the existing driver is not close enough and
brand new driver is indeed required? If so, add a respective paragraph
somewhere (cover letter? TBH I haven't read it, it's fine if it's already
explained there).
I can double check but not the last time I looked into those drivers
They are all different.
+ /*
+ * AVG = E / (Tconv * counter)
+ * We get energy in microJoule, hence dividing it by microSeconds
gives Watts. Therefore,
+ * multiplying by MICRO gives us microWatts.
+ */
Here you used long lines, somewhere else, much shorter. Please, choose one
limit and be consistent with it.
Yeah, maybe not too consistent. I'm still not sure what's the convention in
hwmon (I mention it in the cover).
Patches should pass "checkpatch --strict". Other than that, I'd expect consistency
and common sense, but that isn't exactly spelled out anywhere and would be
difficult to define. Personally, I would suggest something like "use limit of 80
columns for comments and otherwise try to stick with 80 columns unless it results
in odd line breaks. In that case, feel free to go up to 100 columns". Do I need to
spell that out in Documentation/hwmon/submitting-patches.rst ?
Not that anyone seems to read that document :-(. Sorry, frustrated maintainer
speaking.
Guenter
