diff mbox series

[v1,07/16] test/qtest/aspeed_smc-test: Fix coding style

Message ID 20241018053112.1886173-8-jamin_lin@aspeedtech.com (mailing list archive)
State New
Headers show
Series Fix write incorrect data into flash in user mode | expand

Commit Message

Jamin Lin Oct. 18, 2024, 5:31 a.m. UTC
Fix coding style issues from checkpatch.pl

Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 tests/qtest/aspeed_smc-test.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Thomas Huth Oct. 18, 2024, 8:25 a.m. UTC | #1
On 18/10/2024 07.31, Jamin Lin wrote:
> Fix coding style issues from checkpatch.pl
> 
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
>   tests/qtest/aspeed_smc-test.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/qtest/aspeed_smc-test.c b/tests/qtest/aspeed_smc-test.c
> index c713a3700b..4673371d95 100644
> --- a/tests/qtest/aspeed_smc-test.c
> +++ b/tests/qtest/aspeed_smc-test.c
> @@ -353,7 +353,8 @@ static void test_read_page_mem(void)
>       uint32_t page[FLASH_PAGE_SIZE / 4];
>       int i;
>   
> -    /* Enable 4BYTE mode for controller. This is should be strapped by
> +    /*
> +     * Enable 4BYTE mode for controller. This is should be strapped by
>        * HW for CE0 anyhow.
>        */
>       spi_ce_ctrl(1 << CRTL_EXTENDED0);
> @@ -394,7 +395,8 @@ static void test_write_page_mem(void)
>       uint32_t page[FLASH_PAGE_SIZE / 4];
>       int i;
>   
> -    /* Enable 4BYTE mode for controller. This is should be strapped by
> +    /*
> +     * Enable 4BYTE mode for controller. This is should be strapped by
>        * HW for CE0 anyhow.
>        */
>       spi_ce_ctrl(1 << CRTL_EXTENDED0);

Reviewed-by: Thomas Huth <thuth@redhat.com>
Cédric Le Goater Oct. 21, 2024, 12:02 p.m. UTC | #2
On 10/18/24 07:31, Jamin Lin wrote:
> Fix coding style issues from checkpatch.pl
> 
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>


Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.


> ---
>   tests/qtest/aspeed_smc-test.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/qtest/aspeed_smc-test.c b/tests/qtest/aspeed_smc-test.c
> index c713a3700b..4673371d95 100644
> --- a/tests/qtest/aspeed_smc-test.c
> +++ b/tests/qtest/aspeed_smc-test.c
> @@ -353,7 +353,8 @@ static void test_read_page_mem(void)
>       uint32_t page[FLASH_PAGE_SIZE / 4];
>       int i;
>   
> -    /* Enable 4BYTE mode for controller. This is should be strapped by
> +    /*
> +     * Enable 4BYTE mode for controller. This is should be strapped by
>        * HW for CE0 anyhow.
>        */
>       spi_ce_ctrl(1 << CRTL_EXTENDED0);
> @@ -394,7 +395,8 @@ static void test_write_page_mem(void)
>       uint32_t page[FLASH_PAGE_SIZE / 4];
>       int i;
>   
> -    /* Enable 4BYTE mode for controller. This is should be strapped by
> +    /*
> +     * Enable 4BYTE mode for controller. This is should be strapped by
>        * HW for CE0 anyhow.
>        */
>       spi_ce_ctrl(1 << CRTL_EXTENDED0);
diff mbox series

Patch

diff --git a/tests/qtest/aspeed_smc-test.c b/tests/qtest/aspeed_smc-test.c
index c713a3700b..4673371d95 100644
--- a/tests/qtest/aspeed_smc-test.c
+++ b/tests/qtest/aspeed_smc-test.c
@@ -353,7 +353,8 @@  static void test_read_page_mem(void)
     uint32_t page[FLASH_PAGE_SIZE / 4];
     int i;
 
-    /* Enable 4BYTE mode for controller. This is should be strapped by
+    /*
+     * Enable 4BYTE mode for controller. This is should be strapped by
      * HW for CE0 anyhow.
      */
     spi_ce_ctrl(1 << CRTL_EXTENDED0);
@@ -394,7 +395,8 @@  static void test_write_page_mem(void)
     uint32_t page[FLASH_PAGE_SIZE / 4];
     int i;
 
-    /* Enable 4BYTE mode for controller. This is should be strapped by
+    /*
+     * Enable 4BYTE mode for controller. This is should be strapped by
      * HW for CE0 anyhow.
      */
     spi_ce_ctrl(1 << CRTL_EXTENDED0);