Re: [PATCH v4 3/3] hwmon: Add driver for Astera Labs PT5161L retimer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 15, 2024 at 06:05:18PM +0800, Cosmo Chou wrote:
> This driver implements support for temperature monitoring of Astera Labs
> PT5161L series PCIe retimer chips.
> 
> This driver implementation originates from the CSDK available at
> Link: https://github.com/facebook/openbmc/tree/helium/common/recipes-lib/retimer-v2.14
> The communication protocol utilized is based on the I2C/SMBus standard.
> 
> Signed-off-by: Cosmo Chou <chou.cosmo@xxxxxxxxx>

That gives me:

WARNING: Improper SPDX comment style for 'drivers/hwmon/pt5161l.c', please use '//' instead
#237: FILE: drivers/hwmon/pt5161l.c:1:
+/* SPDX-License-Identifier: GPL-2.0-or-later */

WARNING: Missing or malformed SPDX-License-Identifier tag in line 1
#237: FILE: drivers/hwmon/pt5161l.c:1:
+/* SPDX-License-Identifier: GPL-2.0-or-later */

CHECK: struct mutex definition without comment
#304: FILE: drivers/hwmon/pt5161l.c:68:
+	struct mutex lock;

CHECK: Lines should not end with a '('
#419: FILE: drivers/hwmon/pt5161l.c:183:
+		ret = pt5161l_write_block_data(

and many more of the latter. I'd fix it up if it was just one or two,
and I don't care about the mutex definition comment, but

	total: 0 errors, 2 warnings, 9 checks, 755 lines checked

is just too much.

>From Documentation/hwmon/submitting-patches.rst:

* Please run your patch through 'checkpatch --strict'. There should be no
  errors, no warnings, and few if any check messages. If there are any
  messages, please be prepared to explain.

Please fix and resubmit. No need to resubmit the other patches in the
series, since I already applied those.

Thanks,
Guenter




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux