Re: [PATCH 2/2] bsps/nand: Update Xilinx NAND driver

2023-01-30 Thread Gedare Bloom
ok to these two

On Thu, Jan 26, 2023 at 11:31 AM Kinsey Moore  wrote:
>
> This resovles gcc warnings by updating to the latest Xilinx NAND
> controller driver.
> ---
>  bsps/include/dev/nand/xnandpsu.h  |  4 +++-
>  bsps/include/dev/nand/xnandpsu_bbm.h  |  2 +-
>  bsps/include/dev/nand/xnandpsu_hw.h   |  2 +-
>  bsps/include/dev/nand/xnandpsu_onfi.h |  2 +-
>  bsps/shared/dev/nand/VERSION  | 22 +-
>  bsps/shared/dev/nand/xnandpsu.c   | 14 --
>  bsps/shared/dev/nand/xnandpsu_bbm.c   |  2 +-
>  bsps/shared/dev/nand/xnandpsu_onfi.c  |  2 +-
>  8 files changed, 29 insertions(+), 21 deletions(-)
>
> diff --git a/bsps/include/dev/nand/xnandpsu.h 
> b/bsps/include/dev/nand/xnandpsu.h
> index 3625299fb2..a93a74eb85 100644
> --- a/bsps/include/dev/nand/xnandpsu.h
> +++ b/bsps/include/dev/nand/xnandpsu.h
> @@ -7,7 +7,7 @@
>  /**
>  *
>  * @file xnandpsu.h
> -* @addtogroup nandpsu_v1_10
> +* @addtogroup Overview
>  * @{
>  * @details
>  *
> @@ -152,6 +152,8 @@
>  * 1.10  akm10/20/21Fix gcc warnings.
>  * 1.10  akm12/21/21Validate input parameters before use.
>  * 1.10  akm01/05/22Remove assert checks form static and internal 
> APIs.
> +* 1.11  akm03/31/22Fix unused parameter warning.
> +* 1.11  akm03/31/22Fix misleading-indentation warning.
>  *
>  * 
>  *
> diff --git a/bsps/include/dev/nand/xnandpsu_bbm.h 
> b/bsps/include/dev/nand/xnandpsu_bbm.h
> index 4d4fafffa3..b6b39dc990 100644
> --- a/bsps/include/dev/nand/xnandpsu_bbm.h
> +++ b/bsps/include/dev/nand/xnandpsu_bbm.h
> @@ -7,7 +7,7 @@
>  /**
>  *
>  * @file xnandpsu_bbm.h
> -* @addtogroup nandpsu_v1_10
> +* @addtogroup Overview
>  * @{
>  *
>  * This file implements the Bad Block Management(BBM) functionality. This is
> diff --git a/bsps/include/dev/nand/xnandpsu_hw.h 
> b/bsps/include/dev/nand/xnandpsu_hw.h
> index 47e6c8fddc..e3a648b136 100644
> --- a/bsps/include/dev/nand/xnandpsu_hw.h
> +++ b/bsps/include/dev/nand/xnandpsu_hw.h
> @@ -7,7 +7,7 @@
>  /**
>  *
>  * @file xnandpsu_hw.h
> -* @addtogroup nandpsu_v1_10
> +* @addtogroup Overview
>  * @{
>  *
>  * This file contains identifiers and low-level macros/functions for the 
> Arasan
> diff --git a/bsps/include/dev/nand/xnandpsu_onfi.h 
> b/bsps/include/dev/nand/xnandpsu_onfi.h
> index 7523ff48ea..97ea3c404e 100644
> --- a/bsps/include/dev/nand/xnandpsu_onfi.h
> +++ b/bsps/include/dev/nand/xnandpsu_onfi.h
> @@ -7,7 +7,7 @@
>  /**
>  *
>  * @file xnandpsu_onfi.h
> -* @addtogroup nandpsu_v1_10
> +* @addtogroup Overview
>  * @{
>  *
>  * This file defines all the ONFI 3.1 specific commands and values.
> diff --git a/bsps/shared/dev/nand/VERSION b/bsps/shared/dev/nand/VERSION
> index c0afe6e031..5e4eb00fe9 100644
> --- a/bsps/shared/dev/nand/VERSION
> +++ b/bsps/shared/dev/nand/VERSION
> @@ -5,16 +5,20 @@ Import from:
>
>  https://github.com/Xilinx/embeddedsw.git
>
> -commit 8a89579489c88ea5acd23d7d439ac928659c26cf
> -Author: msreeram 
> -AuthorDate: Wed Apr 6 23:24:38 2022 -0600
> +commit 5330a64c8efd14f0eef09befdbb8d3d738c33ec2
> +Refs: 
> +Author: Nicole Baze 
> +AuthorDate: Mon Oct 3 13:27:19 2022 -0700
>  Commit: Siva Addepalli 
> -CommitDate: Fri Apr 8 16:47:15 2022 +0530
> +CommitDate: Fri Oct 7 10:26:16 2022 +0530
>
> -update license file for EmbeddedSW 2022.1 release
> +xilpm: versal: server: Fix bug in AIE2 zeroization
>
> -Update license file for EmbeddedSW 2022.1 release
> +There is a bug in AIE2 zeriozation function when polling for memory
> +zeroization complete. Currently the entire memory register is being
> +checked against zero but instead we need to check the bits specific
> +to the memory tiles. This patch updates the zeroization check by
> +adding a mask so that only the desired bits are checked for zero.
>
> -Signed-off-by: Manikanta Sreeram 
> -
> -Acked-by : Meena Paleti 
> +Signed-off-by: Nicole Baze 
> +Acked-by: Jesus De Haro 
> diff --git a/bsps/shared/dev/nand/xnandpsu.c b/bsps/shared/dev/nand/xnandpsu.c
> index a2e2235906..89451d19f8 100644
> --- a/bsps/shared/dev/nand/xnandpsu.c
> +++ b/bsps/shared/dev/nand/xnandpsu.c
> @@ -7,7 +7,7 @@
>  /**
>  *
>  * @file xnandpsu.c
> -* @addtogroup nandpsu_v1_10
> +* @addtogroup Overview
>  * @{
>  *
>  * This file contains the implementation of the interface functions for
> @@ -88,6 +88,8 @@
>  * 1.10  akm10/20/21Fix gcc warnings.
>  * 1.10  akm12/21/21Validate input parameters before use.
>  * 1.10  akm01/05/22Remove assert checks form static and internal 
> APIs.
> +* 1.11  akm03/31/22Fix unused parameter warning.
> +* 1.11  akm03/31/22Fix misleading-indentation warning.
>  *
>  * 
>  *
> @@ -175,7 +177,7 @@ static s32 XNandPsu_ProgramPage(XNandPsu *InstancePtr, 
> u32 Target, u32 Page,
>  static s32 XNandPsu_ReadPage(XNandPsu *InstancePtr, u32 Target, u32 Page,
> u32 Col, u8 *Buf);
>
> -static 

[PATCH 2/2] bsps/nand: Update Xilinx NAND driver

2023-01-26 Thread Kinsey Moore
This resovles gcc warnings by updating to the latest Xilinx NAND
controller driver.
---
 bsps/include/dev/nand/xnandpsu.h  |  4 +++-
 bsps/include/dev/nand/xnandpsu_bbm.h  |  2 +-
 bsps/include/dev/nand/xnandpsu_hw.h   |  2 +-
 bsps/include/dev/nand/xnandpsu_onfi.h |  2 +-
 bsps/shared/dev/nand/VERSION  | 22 +-
 bsps/shared/dev/nand/xnandpsu.c   | 14 --
 bsps/shared/dev/nand/xnandpsu_bbm.c   |  2 +-
 bsps/shared/dev/nand/xnandpsu_onfi.c  |  2 +-
 8 files changed, 29 insertions(+), 21 deletions(-)

diff --git a/bsps/include/dev/nand/xnandpsu.h b/bsps/include/dev/nand/xnandpsu.h
index 3625299fb2..a93a74eb85 100644
--- a/bsps/include/dev/nand/xnandpsu.h
+++ b/bsps/include/dev/nand/xnandpsu.h
@@ -7,7 +7,7 @@
 /**
 *
 * @file xnandpsu.h
-* @addtogroup nandpsu_v1_10
+* @addtogroup Overview
 * @{
 * @details
 *
@@ -152,6 +152,8 @@
 * 1.10  akm10/20/21Fix gcc warnings.
 * 1.10  akm12/21/21Validate input parameters before use.
 * 1.10  akm01/05/22Remove assert checks form static and internal APIs.
+* 1.11  akm03/31/22Fix unused parameter warning.
+* 1.11  akm03/31/22Fix misleading-indentation warning.
 *
 * 
 *
diff --git a/bsps/include/dev/nand/xnandpsu_bbm.h 
b/bsps/include/dev/nand/xnandpsu_bbm.h
index 4d4fafffa3..b6b39dc990 100644
--- a/bsps/include/dev/nand/xnandpsu_bbm.h
+++ b/bsps/include/dev/nand/xnandpsu_bbm.h
@@ -7,7 +7,7 @@
 /**
 *
 * @file xnandpsu_bbm.h
-* @addtogroup nandpsu_v1_10
+* @addtogroup Overview
 * @{
 *
 * This file implements the Bad Block Management(BBM) functionality. This is
diff --git a/bsps/include/dev/nand/xnandpsu_hw.h 
b/bsps/include/dev/nand/xnandpsu_hw.h
index 47e6c8fddc..e3a648b136 100644
--- a/bsps/include/dev/nand/xnandpsu_hw.h
+++ b/bsps/include/dev/nand/xnandpsu_hw.h
@@ -7,7 +7,7 @@
 /**
 *
 * @file xnandpsu_hw.h
-* @addtogroup nandpsu_v1_10
+* @addtogroup Overview
 * @{
 *
 * This file contains identifiers and low-level macros/functions for the Arasan
diff --git a/bsps/include/dev/nand/xnandpsu_onfi.h 
b/bsps/include/dev/nand/xnandpsu_onfi.h
index 7523ff48ea..97ea3c404e 100644
--- a/bsps/include/dev/nand/xnandpsu_onfi.h
+++ b/bsps/include/dev/nand/xnandpsu_onfi.h
@@ -7,7 +7,7 @@
 /**
 *
 * @file xnandpsu_onfi.h
-* @addtogroup nandpsu_v1_10
+* @addtogroup Overview
 * @{
 *
 * This file defines all the ONFI 3.1 specific commands and values.
diff --git a/bsps/shared/dev/nand/VERSION b/bsps/shared/dev/nand/VERSION
index c0afe6e031..5e4eb00fe9 100644
--- a/bsps/shared/dev/nand/VERSION
+++ b/bsps/shared/dev/nand/VERSION
@@ -5,16 +5,20 @@ Import from:
 
 https://github.com/Xilinx/embeddedsw.git
 
-commit 8a89579489c88ea5acd23d7d439ac928659c26cf
-Author: msreeram 
-AuthorDate: Wed Apr 6 23:24:38 2022 -0600
+commit 5330a64c8efd14f0eef09befdbb8d3d738c33ec2
+Refs: 
+Author: Nicole Baze 
+AuthorDate: Mon Oct 3 13:27:19 2022 -0700
 Commit: Siva Addepalli 
-CommitDate: Fri Apr 8 16:47:15 2022 +0530
+CommitDate: Fri Oct 7 10:26:16 2022 +0530
 
-update license file for EmbeddedSW 2022.1 release
+xilpm: versal: server: Fix bug in AIE2 zeroization
 
-Update license file for EmbeddedSW 2022.1 release
+There is a bug in AIE2 zeriozation function when polling for memory
+zeroization complete. Currently the entire memory register is being
+checked against zero but instead we need to check the bits specific
+to the memory tiles. This patch updates the zeroization check by
+adding a mask so that only the desired bits are checked for zero.
 
-Signed-off-by: Manikanta Sreeram 
-
-Acked-by : Meena Paleti 
+Signed-off-by: Nicole Baze 
+Acked-by: Jesus De Haro 
diff --git a/bsps/shared/dev/nand/xnandpsu.c b/bsps/shared/dev/nand/xnandpsu.c
index a2e2235906..89451d19f8 100644
--- a/bsps/shared/dev/nand/xnandpsu.c
+++ b/bsps/shared/dev/nand/xnandpsu.c
@@ -7,7 +7,7 @@
 /**
 *
 * @file xnandpsu.c
-* @addtogroup nandpsu_v1_10
+* @addtogroup Overview
 * @{
 *
 * This file contains the implementation of the interface functions for
@@ -88,6 +88,8 @@
 * 1.10  akm10/20/21Fix gcc warnings.
 * 1.10  akm12/21/21Validate input parameters before use.
 * 1.10  akm01/05/22Remove assert checks form static and internal APIs.
+* 1.11  akm03/31/22Fix unused parameter warning.
+* 1.11  akm03/31/22Fix misleading-indentation warning.
 *
 * 
 *
@@ -175,7 +177,7 @@ static s32 XNandPsu_ProgramPage(XNandPsu *InstancePtr, u32 
Target, u32 Page,
 static s32 XNandPsu_ReadPage(XNandPsu *InstancePtr, u32 Target, u32 Page,
u32 Col, u8 *Buf);
 
-static s32 XNandPsu_CheckOnDie(XNandPsu *InstancePtr, OnfiParamPage *Param);
+static s32 XNandPsu_CheckOnDie(XNandPsu *InstancePtr);
 
 static void XNandPsu_SetEccAddrSize(XNandPsu *InstancePtr);
 
@@ -380,7 +382,7 @@ static s32 XNandPsu_FlashInit(XNandPsu *InstancePtr)
XNandPsu_InitTimingMode(InstancePtr, [Index]);