* [PATCH net-next] can: peak_canfd: Remove unused macros
@ 2020-09-04 13:12 Wang Hai
2020-09-15 6:28 ` Marc Kleine-Budde
0 siblings, 1 reply; 2+ messages in thread
From: Wang Hai @ 2020-09-04 13:12 UTC (permalink / raw)
To: wg, mkl, davem, kuba, gustavoars; +Cc: linux-can, netdev, linux-kernel
CANFD_CLK_SEL_DIV_MASK and CANFD_OPTIONS_SET are
never used after they were introduced. Remove them.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wang Hai <wanghai38@huawei.com>
---
drivers/net/can/peak_canfd/peak_pciefd_main.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/net/can/peak_canfd/peak_pciefd_main.c b/drivers/net/can/peak_canfd/peak_pciefd_main.c
index 9469d4421afe..5f0f39d2fa28 100644
--- a/drivers/net/can/peak_canfd/peak_pciefd_main.c
+++ b/drivers/net/can/peak_canfd/peak_pciefd_main.c
@@ -83,7 +83,6 @@ MODULE_LICENSE("GPL v2");
#define CANFD_MISC_TS_RST 0x00000001 /* timestamp cnt rst */
/* CAN-FD channel Clock SELector Source & DIVider */
-#define CANFD_CLK_SEL_DIV_MASK 0x00000007
#define CANFD_CLK_SEL_DIV_60MHZ 0x00000000 /* SRC=240MHz only */
#define CANFD_CLK_SEL_DIV_40MHZ 0x00000001 /* SRC=240MHz only */
#define CANFD_CLK_SEL_DIV_30MHZ 0x00000002 /* SRC=240MHz only */
@@ -116,8 +115,6 @@ MODULE_LICENSE("GPL v2");
#define CANFD_CTL_IRQ_CL_DEF 16 /* Rx msg max nb per IRQ in Rx DMA */
#define CANFD_CTL_IRQ_TL_DEF 10 /* Time before IRQ if < CL (x100 µs) */
-#define CANFD_OPTIONS_SET (CANFD_OPTION_ERROR | CANFD_OPTION_BUSLOAD)
-
/* Tx anticipation window (link logical address should be aligned on 2K
* boundary)
*/
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] can: peak_canfd: Remove unused macros
2020-09-04 13:12 [PATCH net-next] can: peak_canfd: Remove unused macros Wang Hai
@ 2020-09-15 6:28 ` Marc Kleine-Budde
0 siblings, 0 replies; 2+ messages in thread
From: Marc Kleine-Budde @ 2020-09-15 6:28 UTC (permalink / raw)
To: Wang Hai, wg, davem, kuba, gustavoars; +Cc: linux-can, netdev, linux-kernel
[-- Attachment #1.1: Type: text/plain, Size: 1801 bytes --]
On 9/4/20 3:12 PM, Wang Hai wrote:
> CANFD_CLK_SEL_DIV_MASK and CANFD_OPTIONS_SET are
> never used after they were introduced. Remove them.
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wang Hai <wanghai38@huawei.com>
> ---
> drivers/net/can/peak_canfd/peak_pciefd_main.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/net/can/peak_canfd/peak_pciefd_main.c b/drivers/net/can/peak_canfd/peak_pciefd_main.c
> index 9469d4421afe..5f0f39d2fa28 100644
> --- a/drivers/net/can/peak_canfd/peak_pciefd_main.c
> +++ b/drivers/net/can/peak_canfd/peak_pciefd_main.c
> @@ -83,7 +83,6 @@ MODULE_LICENSE("GPL v2");
> #define CANFD_MISC_TS_RST 0x00000001 /* timestamp cnt rst */
>
> /* CAN-FD channel Clock SELector Source & DIVider */
> -#define CANFD_CLK_SEL_DIV_MASK 0x00000007
I would like to keep this for documentation purpose.
> #define CANFD_CLK_SEL_DIV_60MHZ 0x00000000 /* SRC=240MHz only */
> #define CANFD_CLK_SEL_DIV_40MHZ 0x00000001 /* SRC=240MHz only */
> #define CANFD_CLK_SEL_DIV_30MHZ 0x00000002 /* SRC=240MHz only */
> @@ -116,8 +115,6 @@ MODULE_LICENSE("GPL v2");
> #define CANFD_CTL_IRQ_CL_DEF 16 /* Rx msg max nb per IRQ in Rx DMA */
> #define CANFD_CTL_IRQ_TL_DEF 10 /* Time before IRQ if < CL (x100 µs) */
>
> -#define CANFD_OPTIONS_SET (CANFD_OPTION_ERROR | CANFD_OPTION_BUSLOAD)
> -
Okay, let's remove this.
> /* Tx anticipation window (link logical address should be aligned on 2K
> * boundary)
> */
>
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-15 6:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-04 13:12 [PATCH net-next] can: peak_canfd: Remove unused macros Wang Hai
2020-09-15 6:28 ` Marc Kleine-Budde
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).