You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by li...@apache.org on 2026/10/09 10:30:42 UTC

(nuttx) branch master updated (324a0cd4379 -> b2e04cf122b)

This is an automated email from the ASF dual-hosted git repository.

GUIDINGLI pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


    from 324a0cd4379 boards/arm/stm32l4/nucleo-l432kc: Add support for external W25 SPI flash.
     new 6b4e6a03357 drivers/crypto/se05x: zero the size a failed read leaves
     new 7b58ee84fe8 tools/nxstyle: allow the NXP Plug&Trust names
     new b2e04cf122b drivers/crypto/se05x: nxstyle in pnt_se05x_api.c

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 drivers/crypto/pnt/pnt_se05x_api.c | 11 ++++++++++-
 tools/nxstyle.c                    |  8 ++++++++
 2 files changed, 18 insertions(+), 1 deletion(-)


(nuttx) 01/03: drivers/crypto/se05x: zero the size a failed read leaves

Posted by li...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

GUIDINGLI pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit 6b4e6a03357fa3df6d09a0afbc732897425029cf
Author: Royyan Zahir <ro...@gmail.com>
AuthorDate: Sun Oct 4 08:25:02 2026 +0400

    drivers/crypto/se05x: zero the size a failed read leaves
    
    pnt_se05x_get_data() compared the object size with the buffer even when ReadSize had failed and left it unset.
    
    Signed-off-by: Royyan Zahir <ro...@gmail.com>
---
 drivers/crypto/pnt/pnt_se05x_api.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/pnt/pnt_se05x_api.c b/drivers/crypto/pnt/pnt_se05x_api.c
index 46f2892576d..53f5fe319e9 100644
--- a/drivers/crypto/pnt/pnt_se05x_api.c
+++ b/drivers/crypto/pnt/pnt_se05x_api.c
@@ -247,7 +247,7 @@ int pnt_se05x_get_key(FAR struct se05x_dev_s *se05x,
 int pnt_se05x_get_data(FAR struct se05x_dev_s *se05x,
                        FAR struct se05x_key_transmission_s *get_key_args)
 {
-  uint16_t remainder;
+  uint16_t remainder = 0;
   smStatus_t status = Se05x_API_ReadSize(&(se05x->pnt->session),
                                          get_key_args->entry.id, &remainder);
 


(nuttx) 03/03: drivers/crypto/se05x: nxstyle in pnt_se05x_api.c

Posted by li...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

GUIDINGLI pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit b2e04cf122b42513262a68e6b8d170273f27c271
Author: Royyan Zahir <ro...@gmail.com>
AuthorDate: Fri Oct 9 06:07:48 2026 +0400

    drivers/crypto/se05x: nxstyle in pnt_se05x_api.c
    
    Add the blank line nxstyle wants after declarations.
    
    Signed-off-by: Royyan Zahir <ro...@gmail.com>
---
 drivers/crypto/pnt/pnt_se05x_api.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/crypto/pnt/pnt_se05x_api.c b/drivers/crypto/pnt/pnt_se05x_api.c
index 53f5fe319e9..44fbe79ee7c 100644
--- a/drivers/crypto/pnt/pnt_se05x_api.c
+++ b/drivers/crypto/pnt/pnt_se05x_api.c
@@ -94,6 +94,7 @@ static bool set_enable_pin(FAR struct se05x_dev_s *se05x, bool state)
 int pnt_se05x_open(FAR struct se05x_dev_s *se05x)
 {
   int ret;
+
   se05x->pnt = kmm_malloc(sizeof(struct pnt_handle));
 
   if (se05x->pnt == NULL)
@@ -150,6 +151,7 @@ int pnt_se05x_get_info(FAR struct se05x_dev_s *se05x,
 {
   bool result = select_card_manager(&(se05x->pnt->session));
   identify_rsp_t identify_response;
+
   if (result)
     {
       result = se05x_identify(&(se05x->pnt->session), &identify_response);
@@ -173,6 +175,7 @@ int pnt_se05x_get_uid(FAR struct se05x_dev_s *se05x,
   smStatus_t status = Se05x_API_CheckObjectExists(
       &(se05x->pnt->session), KSE05X_APPLETRESID_UNIQUE_ID, &dummy);
   int result = status == SM_OK ? 0 : -ENODATA;
+
   if (result == 0)
     {
       status = Se05x_API_ReadObject(&(se05x->pnt->session),
@@ -192,6 +195,7 @@ int pnt_se05x_generate_keypair(
       &(se05x->pnt->session), NULL, 0, generate_keypair_args->id,
       kSE05x_ECCurve_NIST_P256, NULL, 0, NULL, 0, kSE05x_INS_NA,
       kSE05x_KeyPart_Pair);
+
   return status == SM_OK ? 0 : -EIO;
 }
 
@@ -204,6 +208,7 @@ int pnt_se05x_set_public_key(
       kSE05x_ECCurve_NIST_P256, NULL, 0, set_publickey_args->content.buffer,
       set_publickey_args->content.buffer_size, kSE05x_INS_NA,
       kSE05x_KeyPart_Public);
+
   return status == SM_OK ? 0 : -EIO;
 }
 
@@ -220,6 +225,7 @@ int pnt_se05x_set_data(
     {
       size_t chunk_size =
           remainder > DATA_CHUNK_SIZE ? DATA_CHUNK_SIZE : remainder;
+
       status = Se05x_API_WriteBinary(
           &(se05x->pnt->session), NULL, set_publickey_args->entry.id, offset,
           first_cycle ? set_publickey_args->content.buffer_size : 0,
@@ -241,6 +247,7 @@ int pnt_se05x_get_key(FAR struct se05x_dev_s *se05x,
       Se05x_API_ReadObject(&(se05x->pnt->session), get_key_args->entry.id, 0,
                            0, get_key_args->content.buffer,
                            &get_key_args->content.buffer_content_size);
+
   return status == SM_OK ? 0 : -EIO;
 }
 
@@ -262,6 +269,7 @@ int pnt_se05x_get_data(FAR struct se05x_dev_s *se05x,
     {
       size_t chunk_size =
           remainder > DATA_CHUNK_SIZE ? DATA_CHUNK_SIZE : remainder;
+
       status = Se05x_API_ReadObject(
           &(se05x->pnt->session), get_key_args->entry.id, offset, chunk_size,
           get_key_args->content.buffer + offset, &chunk_size);
@@ -277,6 +285,7 @@ int pnt_se05x_delete_key(FAR struct se05x_dev_s *se05x, uint32_t key_id)
 {
   smStatus_t status =
       Se05x_API_DeleteSecureObject(&(se05x->pnt->session), key_id);
+
   return status == SM_OK ? 0 : -EIO;
 }
 


(nuttx) 02/03: tools/nxstyle: allow the NXP Plug&Trust names

Posted by li...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

GUIDINGLI pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit 7b58ee84fe801522020a4f00815daf1863d7926e
Author: Royyan Zahir <ro...@gmail.com>
AuthorDate: Fri Oct 9 06:07:48 2026 +0400

    tools/nxstyle: allow the NXP Plug&Trust names
    
    drivers/crypto/pnt calls the Plug&Trust middleware, whose API and types
    are mixed case (Se05x_API_*, SE05x_*, kSE05x_*, smStatus_t), so any
    change to those files failed the check.
    
    Signed-off-by: Royyan Zahir <ro...@gmail.com>
---
 tools/nxstyle.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tools/nxstyle.c b/tools/nxstyle.c
index 151c7bb38ec..1f3d43d943e 100644
--- a/tools/nxstyle.c
+++ b/tools/nxstyle.c
@@ -328,6 +328,14 @@ static const char *g_white_prefix[] =
 
   "SOPC_",
   "OpcUa_",
+
+  /* Ref:  drivers/crypto/pnt, NXP Plug&Trust middleware */
+
+  "Se05x",
+  "SE05x_",
+  "kSE05x_",
+  "smStatus_t",
+  "pScp03_",
   NULL
 };