diff mbox series

[v3,02/14] capp: fix endian conversion

Message ID 20191004040227.22186-3-npiggin@gmail.com
State Superseded
Headers show
Series little endian skiboot | expand

Checks

Context Check Description
snowpatch_ozlabs/apply_patch success Successfully applied on branch master (1785745d5a7eaefd7d0c135f2a3b0f5d86aefec5)
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot success Test snowpatch/job/snowpatch-skiboot on branch master
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot-dco success Signed-off-by present

Commit Message

Nicholas Piggin Oct. 4, 2019, 4:02 a.m. UTC 
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
---
 hw/capp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Donnellan Oct. 4, 2019, 4:29 a.m. UTC | #1 
On 4/10/19 6:02 am, Nicholas Piggin wrote:
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>

Sigh. Thanks for picking this up.

Reviewed-by: Andrew Donnellan <ajd@linux.ibm.com>

> ---
>   hw/capp.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/capp.c b/hw/capp.c
> index 709e6db14..269deb637 100644
> --- a/hw/capp.c
> +++ b/hw/capp.c
> @@ -168,7 +168,7 @@ int64_t capp_load_ucode(unsigned int chip_id, uint32_t opal_id,
>   
>   	/* 'CAPPULID' in ASCII */
>   	if ((be64_to_cpu(ucode->eyecatcher) != 0x43415050554C4944UL) ||
> -	    (be64_to_cpu(ucode->version != 1))) {
> +	    (be64_to_cpu(ucode->version) != 1)) {
>   		PHBERR(opal_id, chip_id, index,
>   		       "CAPP: ucode header invalid\n");
>   		return OPAL_HARDWARE;
>
diff mbox series

Patch

diff --git a/hw/capp.c b/hw/capp.c
index 709e6db14..269deb637 100644
--- a/hw/capp.c
+++ b/hw/capp.c
@@ -168,7 +168,7 @@  int64_t capp_load_ucode(unsigned int chip_id, uint32_t opal_id,
 
 	/* 'CAPPULID' in ASCII */
 	if ((be64_to_cpu(ucode->eyecatcher) != 0x43415050554C4944UL) ||
-	    (be64_to_cpu(ucode->version != 1))) {
+	    (be64_to_cpu(ucode->version) != 1)) {
 		PHBERR(opal_id, chip_id, index,
 		       "CAPP: ucode header invalid\n");
 		return OPAL_HARDWARE;