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 New, archived
Headers show
Series [net-next,v3] net: ethernet: ti: davinci_mdio: Update K3 SoCs list for errata i2329 | expand

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 */ },
 };