[PATCH v2 0/2] can: etas_es58x: clean-up of new GCC W=1 and old checkpath warnings

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

 



The kernel recently added new warnings, one of which triggers a known
false positive on the etas_es58x module. In an effort to keep
es58x_etas free of any W=1 and W=2 (excluding those produced by
foreign headers), add a workaround to silence it.

While at it, this series also fix a checkpatch warning which I knew
existed for a long time but was too lazy to tackle.

* Changelog *

v1 -> v2:

  * v1 had two different check logics for the version numbers:

    - check that none of the sub-version number are zero to make sure
      the parsing succeeded  
    - check that all of the sub-version number fit the expected digit
      range to please GCC.

    v2 simplifies things by merging those two logics together.

Vincent Mailhol (2):
  can: etas_es58x: rework the version check logic to silence
    -Wformat-truncation
  can: etas_es58x: add missing a blank line after declaration

 drivers/net/can/usb/etas_es58x/es58x_core.c   |  1 +
 drivers/net/can/usb/etas_es58x/es58x_core.h   |  6 +-
 .../net/can/usb/etas_es58x/es58x_devlink.c    | 61 +++++++++++++------
 3 files changed, 46 insertions(+), 22 deletions(-)

-- 
2.25.1




[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux