| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 14 Sep 2023 03:55:03 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Parthiban Veerasooran <Parthiban.Veerasooran@...rochip.com>
Cc: davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
pabeni@...hat.com, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
corbet@....net, steen.hegelund@...rochip.com, rdunlap@...radead.org,
horms@...nel.org, casper.casan@...il.com, netdev@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, horatiu.vultur@...rochip.com,
Woojung.Huh@...rochip.com, Nicolas.Ferre@...rochip.com,
UNGLinuxDriver@...rochip.com, Thorsten.Kummermehr@...rochip.com
Subject: Re: [RFC PATCH net-next 5/6] microchip: lan865x: add driver support
for Microchip's LAN865X MACPHY
> +#define REG_STDR_RESET 0x00000003
This appears to be a standard register, so you should not need to
define it here.
> +#define REG_MAC_ADDR_BO 0x00010022
> +#define REG_MAC_ADDR_L 0x00010024
> +#define REG_MAC_ADDR_H 0x00010025
> +#define REG_MAC_NW_CTRL 0x00010000
> +#define REG_MAC_NW_CONFIG 0x00010001
> +#define REG_MAC_HASHL 0x00010020
> +#define REG_MAC_HASHH 0x00010021
> +#define REG_MAC_ADDR_BO 0x00010022
> +#define REG_MAC_ADDR_L 0x00010024
> +#define REG_MAC_ADDR_H 0x00010025
> +
> +#define CCS_Q0_TX_CFG 0x000A0081
> +#define CCS_Q0_RX_CFG 0x000A0082
These are proprietary vendor registers, so please add a prefix to make
this clear.
Andrew
Powered by blists - more mailing lists