diff mbox series

[net-next,v3] net: ethernet: ti: davinci_mdio: Update K3 SoCs list for errata i2329

Message ID 20231201132033.29576-1-r-gunasekaran@ti.com (mailing list archive)
State Accepted
Commit be5fc78a0084472dcc392cbea75f86202467437c
Delegated to: Netdev Maintainers
Headers show
Series [net-next,v3] net: ethernet: ti: davinci_mdio: Update K3 SoCs list for errata i2329 | expand

Checks

Context Check Description
netdev/series_format success Single patches do not need cover letters
netdev/tree_selection success Clearly marked for net-next
netdev/ynl success Generated files up to date; no warnings/errors;
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 1115 this patch: 1115
netdev/cc_maintainers warning 3 maintainers not CCed: u.kleine-koenig@pengutronix.de bhupesh.sharma@linaro.org robh@kernel.org
netdev/build_clang success Errors and warnings before: 1142 this patch: 1142
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 1142 this patch: 1142
netdev/checkpatch warning WARNING: Unknown link reference 'AM62x:', use 'Link:' or 'Closes:' instead WARNING: Unknown link reference 'AM64X:', use 'Link:' or 'Closes:' instead WARNING: Unknown link reference 'AM65X:', use 'Link:' or 'Closes:' instead WARNING: Unknown link reference 'J7200:', use 'Link:' or 'Closes:' instead WARNING: Unknown link reference 'J721E:', use 'Link:' or 'Closes:' instead WARNING: Unknown link reference 'J721S2:', use 'Link:' or 'Closes:' instead
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Ravi Gunasekaran Dec. 1, 2023, 1:20 p.m. UTC
The errata i2329 affects all the currently available silicon revisions of
AM62x, AM64x, AM65x, J7200, J721E and J721S2. So remove the revision
string from the SoC list.

The silicon revisions affected by the errata i2329 can be found under
the MDIO module in the "Advisories by Modules" section of each
SoC errata document listed below

AM62x: https://www.ti.com/lit/er/sprz487c/sprz487c.pdf
AM64X: https://www.ti.com/lit/er/sprz457g/sprz457g.pdf
AM65X: https://www.ti.com/lit/er/sprz452i/sprz452i.pdf
J7200: https://www.ti.com/lit/er/sprz491d/sprz491d.pdf
J721E: https://www.ti.com/lit/er/sprz455d/sprz455d.pdf
J721S2: https://www.ti.com/lit/er/sprz530b/sprz530b.pdf

Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
---
Changes since v2:
* Removed revision string for all the affected SoCs

Changes since v1:
* For J721E, retained the incorrect SR ID and added the correct one
* Add AM65x SR2.1 to the workaround list

v2: https://lore.kernel.org/all/20231020111738.14671-1-r-gunasekaran@ti.com/
v1: https://lore.kernel.org/all/20231018140009.1725-1-r-gunasekaran@ti.com/

 drivers/net/ethernet/ti/davinci_mdio.c | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

Comments

Nishanth Menon Dec. 3, 2023, 1:47 p.m. UTC | #1
On 18:50-20231201, Ravi Gunasekaran wrote:
> The errata i2329 affects all the currently available silicon revisions of
> AM62x, AM64x, AM65x, J7200, J721E and J721S2. So remove the revision
> string from the SoC list.
> 
> The silicon revisions affected by the errata i2329 can be found under
> the MDIO module in the "Advisories by Modules" section of each
> SoC errata document listed below
> 
> AM62x: https://www.ti.com/lit/er/sprz487c/sprz487c.pdf
> AM64X: https://www.ti.com/lit/er/sprz457g/sprz457g.pdf
> AM65X: https://www.ti.com/lit/er/sprz452i/sprz452i.pdf
> J7200: https://www.ti.com/lit/er/sprz491d/sprz491d.pdf
> J721E: https://www.ti.com/lit/er/sprz455d/sprz455d.pdf
> J721S2: https://www.ti.com/lit/er/sprz530b/sprz530b.pdf
> 
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
> ---
> Changes since v2:
> * Removed revision string for all the affected SoCs
> 
> Changes since v1:
> * For J721E, retained the incorrect SR ID and added the correct one
> * Add AM65x SR2.1 to the workaround list
> 
> v2: https://lore.kernel.org/all/20231020111738.14671-1-r-gunasekaran@ti.com/
> v1: https://lore.kernel.org/all/20231018140009.1725-1-r-gunasekaran@ti.com/
> 
>  drivers/net/ethernet/ti/davinci_mdio.c | 16 ++++++----------
>  1 file changed, 6 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/net/ethernet/ti/davinci_mdio.c b/drivers/net/ethernet/ti/davinci_mdio.c
> index 628c87dc1d28..8e07d4a1b6ba 100644
> --- a/drivers/net/ethernet/ti/davinci_mdio.c
> +++ b/drivers/net/ethernet/ti/davinci_mdio.c
> @@ -511,16 +511,12 @@ static const struct k3_mdio_soc_data am65_mdio_soc_data = {
>  };
>  
>  static const struct soc_device_attribute k3_mdio_socinfo[] = {
> -	{ .family = "AM62X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
> -	{ .family = "AM64X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
> -	{ .family = "AM64X", .revision = "SR2.0", .data = &am65_mdio_soc_data },
> -	{ .family = "AM65X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
> -	{ .family = "AM65X", .revision = "SR2.0", .data = &am65_mdio_soc_data },
> -	{ .family = "J7200", .revision = "SR1.0", .data = &am65_mdio_soc_data },
> -	{ .family = "J7200", .revision = "SR2.0", .data = &am65_mdio_soc_data },
> -	{ .family = "J721E", .revision = "SR1.0", .data = &am65_mdio_soc_data },
> -	{ .family = "J721E", .revision = "SR2.0", .data = &am65_mdio_soc_data },
> -	{ .family = "J721S2", .revision = "SR1.0", .data = &am65_mdio_soc_data},
> +	{ .family = "AM62X", .data = &am65_mdio_soc_data },
> +	{ .family = "AM64X", .data = &am65_mdio_soc_data },
> +	{ .family = "AM65X", .data = &am65_mdio_soc_data },
> +	{ .family = "J7200", .data = &am65_mdio_soc_data },
> +	{ .family = "J721E", .data = &am65_mdio_soc_data },
> +	{ .family = "J721S2", .data = &am65_mdio_soc_data },
>  	{ /* sentinel */ },


Much better. Thank you. and will avoid the conflicts we have with fixups
pending..

Reviewed-by: Nishanth Menon <nm@ti.com>
patchwork-bot+netdevbpf@kernel.org Dec. 5, 2023, 2:30 a.m. UTC | #2
Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 1 Dec 2023 18:50:33 +0530 you wrote:
> The errata i2329 affects all the currently available silicon revisions of
> AM62x, AM64x, AM65x, J7200, J721E and J721S2. So remove the revision
> string from the SoC list.
> 
> The silicon revisions affected by the errata i2329 can be found under
> the MDIO module in the "Advisories by Modules" section of each
> SoC errata document listed below
> 
> [...]

Here is the summary with links:
  - [net-next,v3] net: ethernet: ti: davinci_mdio: Update K3 SoCs list for errata i2329
    https://git.kernel.org/netdev/net-next/c/be5fc78a0084

You are awesome, thank you!
diff mbox series

Patch

diff --git a/drivers/net/ethernet/ti/davinci_mdio.c b/drivers/net/ethernet/ti/davinci_mdio.c
index 628c87dc1d28..8e07d4a1b6ba 100644
--- a/drivers/net/ethernet/ti/davinci_mdio.c
+++ b/drivers/net/ethernet/ti/davinci_mdio.c
@@ -511,16 +511,12 @@  static const struct k3_mdio_soc_data am65_mdio_soc_data = {
 };
 
 static const struct soc_device_attribute k3_mdio_socinfo[] = {
-	{ .family = "AM62X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
-	{ .family = "AM64X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
-	{ .family = "AM64X", .revision = "SR2.0", .data = &am65_mdio_soc_data },
-	{ .family = "AM65X", .revision = "SR1.0", .data = &am65_mdio_soc_data },
-	{ .family = "AM65X", .revision = "SR2.0", .data = &am65_mdio_soc_data },
-	{ .family = "J7200", .revision = "SR1.0", .data = &am65_mdio_soc_data },
-	{ .family = "J7200", .revision = "SR2.0", .data = &am65_mdio_soc_data },
-	{ .family = "J721E", .revision = "SR1.0", .data = &am65_mdio_soc_data },
-	{ .family = "J721E", .revision = "SR2.0", .data = &am65_mdio_soc_data },
-	{ .family = "J721S2", .revision = "SR1.0", .data = &am65_mdio_soc_data},
+	{ .family = "AM62X", .data = &am65_mdio_soc_data },
+	{ .family = "AM64X", .data = &am65_mdio_soc_data },
+	{ .family = "AM65X", .data = &am65_mdio_soc_data },
+	{ .family = "J7200", .data = &am65_mdio_soc_data },
+	{ .family = "J721E", .data = &am65_mdio_soc_data },
+	{ .family = "J721S2", .data = &am65_mdio_soc_data },
 	{ /* sentinel */ },
 };