[v1] can: Kconfig: add missing COMPILE_TEST

[PATCH 0/3] can: Kconfig: add missing COMPILE_TEST

Vincent Mailhol posted 3 patches 2 months, 3 weeks ago

Diff against v2
Download series mbox

There is a newer version of this series

drivers/net/can/Kconfig         | 2 +-
drivers/net/can/sja1000/Kconfig | 2 +-
drivers/net/can/ti_hecc.c       | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH 0/3] can: Kconfig: add missing COMPILE_TEST

Posted by Vincent Mailhol 2 months, 3 weeks ago

The ti_hecc and tscan1 CAN drivers can not be built on an x86_64
platform. Add the COMPILE_TEST dependency to allow build testing.

Doing that, a so far unnoticed W=0 warning showed up in ti_hecc. Fix
this warning. To prevent any potential noise in some future git
bisect, the warning is fixed before introducing COMPILE_TEST.

Note that the mscan and mpc5xxx drivers have the same issue but those
two use some helper functions, such as in_8() and out_8(), which are
only available on the powerpc platform. Those two drivers would
require some deeper code refactor to be built on x86_64 and are thus
left out of scope.

Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Vincent Mailhol (3):
      can: ti_hecc: fix -Woverflow compiler warning
      can: ti_hecc: Kconfig: add COMPILE_TEST
      can: tscan1: Kconfig: add COMPILE_TEST

 drivers/net/can/Kconfig         | 2 +-
 drivers/net/can/sja1000/Kconfig | 2 +-
 drivers/net/can/ti_hecc.c       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
---
base-commit: a52f9f0d77f20efc285908a28b5697603b6597c7
change-id: 20250713-can-compile-test-933d513473a1

Best regards,
-- 
Vincent Mailhol <mailhol.vincent@wanadoo.fr>