diff mbox series

firmware: ti_sci: rm: remove unneeded semicolon

Message ID 1612247013-17381-1-git-send-email-yang.lee@linux.alibaba.com (mailing list archive)
State New, archived
Headers show
Series firmware: ti_sci: rm: remove unneeded semicolon | expand

Commit Message

Yang Li Feb. 2, 2021, 6:23 a.m. UTC
Eliminate the following coccicheck warning:
./drivers/firmware/ti_sci.c:1762:2-3: Unneeded semicolon

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/firmware/ti_sci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Nishanth Menon Feb. 2, 2021, 5:17 p.m. UTC | #1
On 14:23-20210202, Yang Li wrote:
> Eliminate the following coccicheck warning:
> ./drivers/firmware/ti_sci.c:1762:2-3: Unneeded semicolon
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
> ---
>  drivers/firmware/ti_sci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
> index 235c7e7..5ae2040 100644
> --- a/drivers/firmware/ti_sci.c
> +++ b/drivers/firmware/ti_sci.c
> @@ -1759,7 +1759,7 @@ static int ti_sci_get_resource_range(const struct ti_sci_handle *handle,
>  		desc->num = resp->range_num;
>  		desc->start_sec = resp->range_start_sec;
>  		desc->num_sec = resp->range_num_sec;
> -	};
> +	}
>  


Uggh.. thanks..

Reviewed-by: Nishanth Menon <nm@ti.com>


Santosh: I know you have send your Soc fixes for merge, but will be
great if you could pick for a future version, this is minor enough not
to bother the stable, I think..
Santosh Shilimkar Feb. 2, 2021, 6:04 p.m. UTC | #2
On Feb 2, 2021, at 9:17 AM, Nishanth Menon <nm@ti.com> wrote:
> 
> On 14:23-20210202, Yang Li wrote:
>> Eliminate the following coccicheck warning:
>> ./drivers/firmware/ti_sci.c:1762:2-3: Unneeded semicolon
>> 
>> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
>> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
>> ---
>> drivers/firmware/ti_sci.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
>> index 235c7e7..5ae2040 100644
>> --- a/drivers/firmware/ti_sci.c
>> +++ b/drivers/firmware/ti_sci.c
>> @@ -1759,7 +1759,7 @@ static int ti_sci_get_resource_range(const struct ti_sci_handle *handle,
>> 		desc->num = resp->range_num;
>> 		desc->start_sec = resp->range_start_sec;
>> 		desc->num_sec = resp->range_num_sec;
>> -	};
>> +	}
>> 
> 
> 
> Uggh.. thanks..
> 
> Reviewed-by: Nishanth Menon <nm@ti.com>
> 
> 
> Santosh: I know you have send your Soc fixes for merge, but will be
> great if you could pick for a future version, this is minor enough not
> to bother the stable, I think..

Yes, this can wait for next one.

Regards,
Santosh
diff mbox series

Patch

diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
index 235c7e7..5ae2040 100644
--- a/drivers/firmware/ti_sci.c
+++ b/drivers/firmware/ti_sci.c
@@ -1759,7 +1759,7 @@  static int ti_sci_get_resource_range(const struct ti_sci_handle *handle,
 		desc->num = resp->range_num;
 		desc->start_sec = resp->range_start_sec;
 		desc->num_sec = resp->range_num_sec;
-	};
+	}
 
 fail:
 	ti_sci_put_one_xfer(&info->minfo, xfer);