Re: [PATCH] power_supply: bq25890: make chip_id int

From: Sebastian Reichel
Date: Tue Jun 02 2015 - 15:28:55 EST


Hi Laurentiu,

On Tue, Jun 02, 2015 at 01:36:51PM +0300, Laurentiu Palcu wrote:
> Smatch static checker correctly detected an impossible condition because
> chip_id was declared as u8, instead of int:
>
> drivers/power/bq25890_charger.c:843 bq25890_probe()
> warn: impossible condition '(bq->chip_id < 0) => (0-255 < 0)'
>
> Also, while at it, fix the return value too.

Thanks, queued for 4.2.

-- Sebastian

Attachment: signature.asc
Description: Digital signature