* [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes
@ 2020-11-10 2:40 Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 1/3] drivers: net: xilinx_emaclite: Add missing parameter kerneldoc Andrew Lunn
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Andrew Lunn @ 2020-11-10 2:40 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: netdev, Michal Simek, Radhey Shyam Pandey, Andrew Lunn
kerneldoc, pointer issues, and add COMPILE_TEST support to easy
finding future issues via build testing.
v2:
Use uintptr_t instead of long
Added Acked-by's.
Andrew Lunn (3):
drivers: net: xilinx_emaclite: Add missing parameter kerneldoc
drivers: net: xilinx_emaclite: Fix -Wpointer-to-int-cast warnings with
W=1
drivers: net: xilinx_emaclite: Add COMPILE_TEST support
drivers/net/ethernet/xilinx/Kconfig | 2 +-
drivers/net/ethernet/xilinx/xilinx_emaclite.c | 14 ++++++++------
2 files changed, 9 insertions(+), 7 deletions(-)
--
2.29.2
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH net-next v2 1/3] drivers: net: xilinx_emaclite: Add missing parameter kerneldoc
2020-11-10 2:40 [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Andrew Lunn
@ 2020-11-10 2:40 ` Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 2/3] drivers: net: xilinx_emaclite: Fix -Wpointer-to-int-cast warnings with W=1 Andrew Lunn
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Andrew Lunn @ 2020-11-10 2:40 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: netdev, Michal Simek, Radhey Shyam Pandey, Andrew Lunn
The txqueue parameter to the watchdog callback is unused in this
driver. But it still needs to be documented.
Reviewed-by: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
drivers/net/ethernet/xilinx/xilinx_emaclite.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/xilinx/xilinx_emaclite.c b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
index 0c26f5bcc523..2c98e4cc07a5 100644
--- a/drivers/net/ethernet/xilinx/xilinx_emaclite.c
+++ b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
@@ -518,6 +518,7 @@ static int xemaclite_set_mac_address(struct net_device *dev, void *address)
/**
* xemaclite_tx_timeout - Callback for Tx Timeout
* @dev: Pointer to the network device
+ * @txqueue: Unused
*
* This function is called when Tx time out occurs for Emaclite device.
*/
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH net-next v2 2/3] drivers: net: xilinx_emaclite: Fix -Wpointer-to-int-cast warnings with W=1
2020-11-10 2:40 [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 1/3] drivers: net: xilinx_emaclite: Add missing parameter kerneldoc Andrew Lunn
@ 2020-11-10 2:40 ` Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 3/3] drivers: net: xilinx_emaclite: Add COMPILE_TEST support Andrew Lunn
2020-11-12 22:34 ` [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: Andrew Lunn @ 2020-11-10 2:40 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: netdev, Michal Simek, Radhey Shyam Pandey, Andrew Lunn
drivers/net/ethernet//xilinx/xilinx_emaclite.c:341:35: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
341 | addr = (void __iomem __force *)((u32 __force)addr ^
Use uintptr_t instead of u32 to avoid problems on 64 bit systems.
Also, cast the address to an unsigned long for printing.
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
drivers/net/ethernet/xilinx/xilinx_emaclite.c | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/net/ethernet/xilinx/xilinx_emaclite.c b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
index 2c98e4cc07a5..008b9a40faad 100644
--- a/drivers/net/ethernet/xilinx/xilinx_emaclite.c
+++ b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
@@ -97,7 +97,7 @@
#define ALIGNMENT 4
/* BUFFER_ALIGN(adr) calculates the number of bytes to the next alignment. */
-#define BUFFER_ALIGN(adr) ((ALIGNMENT - ((u32)adr)) % ALIGNMENT)
+#define BUFFER_ALIGN(adr) ((ALIGNMENT - ((uintptr_t)adr)) % ALIGNMENT)
#ifdef __BIG_ENDIAN
#define xemaclite_readl ioread32be
@@ -338,7 +338,7 @@ static int xemaclite_send_data(struct net_local *drvdata, u8 *data,
* if it is configured in HW
*/
- addr = (void __iomem __force *)((u32 __force)addr ^
+ addr = (void __iomem __force *)((uintptr_t __force)addr ^
XEL_BUFFER_OFFSET);
reg_data = xemaclite_readl(addr + XEL_TSR_OFFSET);
@@ -399,8 +399,9 @@ static u16 xemaclite_recv_data(struct net_local *drvdata, u8 *data, int maxlen)
* will correct on subsequent calls
*/
if (drvdata->rx_ping_pong != 0)
- addr = (void __iomem __force *)((u32 __force)addr ^
- XEL_BUFFER_OFFSET);
+ addr = (void __iomem __force *)
+ ((uintptr_t __force)addr ^
+ XEL_BUFFER_OFFSET);
else
return 0; /* No data was available */
@@ -1192,9 +1193,9 @@ static int xemaclite_of_probe(struct platform_device *ofdev)
}
dev_info(dev,
- "Xilinx EmacLite at 0x%08X mapped to 0x%08X, irq=%d\n",
+ "Xilinx EmacLite at 0x%08X mapped to 0x%08lX, irq=%d\n",
(unsigned int __force)ndev->mem_start,
- (unsigned int __force)lp->base_addr, ndev->irq);
+ (unsigned long __force)lp->base_addr, ndev->irq);
return 0;
error:
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH net-next v2 3/3] drivers: net: xilinx_emaclite: Add COMPILE_TEST support
2020-11-10 2:40 [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 1/3] drivers: net: xilinx_emaclite: Add missing parameter kerneldoc Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 2/3] drivers: net: xilinx_emaclite: Fix -Wpointer-to-int-cast warnings with W=1 Andrew Lunn
@ 2020-11-10 2:40 ` Andrew Lunn
2020-11-12 22:34 ` [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: Andrew Lunn @ 2020-11-10 2:40 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: netdev, Michal Simek, Radhey Shyam Pandey, Andrew Lunn
To improve build testing of this driver, add COMPILE_TEST support.
Acked-by: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
drivers/net/ethernet/xilinx/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/xilinx/Kconfig b/drivers/net/ethernet/xilinx/Kconfig
index d0d0d4fe9d40..3b2137d1f4c6 100644
--- a/drivers/net/ethernet/xilinx/Kconfig
+++ b/drivers/net/ethernet/xilinx/Kconfig
@@ -18,7 +18,7 @@ if NET_VENDOR_XILINX
config XILINX_EMACLITE
tristate "Xilinx 10/100 Ethernet Lite support"
- depends on PPC32 || MICROBLAZE || ARCH_ZYNQ || MIPS
+ depends on PPC32 || MICROBLAZE || ARCH_ZYNQ || MIPS || COMPILE_TEST
select PHYLIB
help
This driver supports the 10/100 Ethernet Lite from Xilinx.
--
2.29.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes
2020-11-10 2:40 [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Andrew Lunn
` (2 preceding siblings ...)
2020-11-10 2:40 ` [PATCH net-next v2 3/3] drivers: net: xilinx_emaclite: Add COMPILE_TEST support Andrew Lunn
@ 2020-11-12 22:34 ` Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: Jakub Kicinski @ 2020-11-12 22:34 UTC (permalink / raw)
To: Andrew Lunn; +Cc: netdev, Michal Simek, Radhey Shyam Pandey
On Tue, 10 Nov 2020 03:40:21 +0100 Andrew Lunn wrote:
> kerneldoc, pointer issues, and add COMPILE_TEST support to easy
> finding future issues via build testing.
>
> v2:
> Use uintptr_t instead of long
> Added Acked-by's.
Applied, thank you!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-11-12 22:35 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-10 2:40 [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 1/3] drivers: net: xilinx_emaclite: Add missing parameter kerneldoc Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 2/3] drivers: net: xilinx_emaclite: Fix -Wpointer-to-int-cast warnings with W=1 Andrew Lunn
2020-11-10 2:40 ` [PATCH net-next v2 3/3] drivers: net: xilinx_emaclite: Add COMPILE_TEST support Andrew Lunn
2020-11-12 22:34 ` [PATCH net-next v2 0/3] xilinx_emaclite W=1 fixes Jakub Kicinski
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).