public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH net v2] net: axienet: Fix register defines comment description
@ 2024-08-09  6:26 Radhey Shyam Pandey
  2024-08-11 16:23 ` Andrew Lunn
  2024-08-12  9:50 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Radhey Shyam Pandey @ 2024-08-09  6:26 UTC (permalink / raw)
  To: davem, edumazet, kuba, pabeni, michal.simek, ariane.keller,
	daniel, andrew
  Cc: netdev, linux-arm-kernel, linux-kernel, git, Radhey Shyam Pandey

In axiethernet header fix register defines comment description to be
inline with IP documentation. It updates MAC configuration register,
MDIO configuration register and frame filter control description.

Fixes: 8a3b7a252dca ("drivers/net/ethernet/xilinx: added Xilinx AXI Ethernet driver")
Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com>
---
- As suggested by Andrew renaming defines make it harder to
  backport fixes so drop define renaming.
- Drop removal of ununsed bit masks for PHYC register and
  MDIO interface MIS, MIP, MIE, MIC registers as there are
  other unused defines as well and may be used in future.
- Add fixes tag.
---
 drivers/net/ethernet/xilinx/xilinx_axienet.h | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet.h b/drivers/net/ethernet/xilinx/xilinx_axienet.h
index fa5500decc96..c7d9221fafdc 100644
--- a/drivers/net/ethernet/xilinx/xilinx_axienet.h
+++ b/drivers/net/ethernet/xilinx/xilinx_axienet.h
@@ -160,16 +160,16 @@
 #define XAE_RCW1_OFFSET		0x00000404 /* Rx Configuration Word 1 */
 #define XAE_TC_OFFSET		0x00000408 /* Tx Configuration */
 #define XAE_FCC_OFFSET		0x0000040C /* Flow Control Configuration */
-#define XAE_EMMC_OFFSET		0x00000410 /* EMAC mode configuration */
-#define XAE_PHYC_OFFSET		0x00000414 /* RGMII/SGMII configuration */
+#define XAE_EMMC_OFFSET		0x00000410 /* MAC speed configuration */
+#define XAE_PHYC_OFFSET		0x00000414 /* RX Max Frame Configuration */
 #define XAE_ID_OFFSET		0x000004F8 /* Identification register */
-#define XAE_MDIO_MC_OFFSET	0x00000500 /* MII Management Config */
-#define XAE_MDIO_MCR_OFFSET	0x00000504 /* MII Management Control */
-#define XAE_MDIO_MWD_OFFSET	0x00000508 /* MII Management Write Data */
-#define XAE_MDIO_MRD_OFFSET	0x0000050C /* MII Management Read Data */
+#define XAE_MDIO_MC_OFFSET	0x00000500 /* MDIO Setup */
+#define XAE_MDIO_MCR_OFFSET	0x00000504 /* MDIO Control */
+#define XAE_MDIO_MWD_OFFSET	0x00000508 /* MDIO Write Data */
+#define XAE_MDIO_MRD_OFFSET	0x0000050C /* MDIO Read Data */
 #define XAE_UAW0_OFFSET		0x00000700 /* Unicast address word 0 */
 #define XAE_UAW1_OFFSET		0x00000704 /* Unicast address word 1 */
-#define XAE_FMI_OFFSET		0x00000708 /* Filter Mask Index */
+#define XAE_FMI_OFFSET		0x00000708 /* Frame Filter Control */
 #define XAE_AF0_OFFSET		0x00000710 /* Address Filter 0 */
 #define XAE_AF1_OFFSET		0x00000714 /* Address Filter 1 */
 
@@ -308,7 +308,7 @@
  */
 #define XAE_UAW1_UNICASTADDR_MASK	0x0000FFFF
 
-/* Bit masks for Axi Ethernet FMI register */
+/* Bit masks for Axi Ethernet FMC register */
 #define XAE_FMI_PM_MASK			0x80000000 /* Promis. mode enable */
 #define XAE_FMI_IND_MASK		0x00000003 /* Index Mask */
 

base-commit: ee9a43b7cfe2d8a3520335fea7d8ce71b8cabd9d
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH net v2] net: axienet: Fix register defines comment description
  2024-08-09  6:26 [PATCH net v2] net: axienet: Fix register defines comment description Radhey Shyam Pandey
@ 2024-08-11 16:23 ` Andrew Lunn
  2024-08-12  9:50 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Andrew Lunn @ 2024-08-11 16:23 UTC (permalink / raw)
  To: Radhey Shyam Pandey
  Cc: davem, edumazet, kuba, pabeni, michal.simek, ariane.keller,
	daniel, netdev, linux-arm-kernel, linux-kernel, git

On Fri, Aug 09, 2024 at 11:56:09AM +0530, Radhey Shyam Pandey wrote:
> In axiethernet header fix register defines comment description to be
> inline with IP documentation. It updates MAC configuration register,
> MDIO configuration register and frame filter control description.
> 
> Fixes: 8a3b7a252dca ("drivers/net/ethernet/xilinx: added Xilinx AXI Ethernet driver")
> Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net v2] net: axienet: Fix register defines comment description
  2024-08-09  6:26 [PATCH net v2] net: axienet: Fix register defines comment description Radhey Shyam Pandey
  2024-08-11 16:23 ` Andrew Lunn
@ 2024-08-12  9:50 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-08-12  9:50 UTC (permalink / raw)
  To: Pandey, Radhey Shyam
  Cc: davem, edumazet, kuba, pabeni, michal.simek, ariane.keller,
	daniel, andrew, netdev, linux-arm-kernel, linux-kernel, git

Hello:

This patch was applied to netdev/net.git (main)
by David S. Miller <davem@davemloft.net>:

On Fri, 9 Aug 2024 11:56:09 +0530 you wrote:
> In axiethernet header fix register defines comment description to be
> inline with IP documentation. It updates MAC configuration register,
> MDIO configuration register and frame filter control description.
> 
> Fixes: 8a3b7a252dca ("drivers/net/ethernet/xilinx: added Xilinx AXI Ethernet driver")
> Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com>
> 
> [...]

Here is the summary with links:
  - [net,v2] net: axienet: Fix register defines comment description
    https://git.kernel.org/netdev/net/c/9ff2f816e2aa

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-08-12  9:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-09  6:26 [PATCH net v2] net: axienet: Fix register defines comment description Radhey Shyam Pandey
2024-08-11 16:23 ` Andrew Lunn
2024-08-12  9:50 ` patchwork-bot+netdevbpf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox