Re: [PATCH 5/7] ssb: Use bool function return values of true/false not 1/0

From: Kalle Valo
Date: Mon Mar 30 2015 - 14:39:28 EST


Michael BÃsch <m@xxxxxxx> writes:

> On Mon, 30 Mar 2015 10:43:21 -0700
> Joe Perches <joe@xxxxxxxxxxx> wrote:
>
>> Use the normal return values for bool functions
>>
>> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
>> ---
>> drivers/ssb/driver_gige.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/ssb/driver_gige.c b/drivers/ssb/driver_gige.c
>> index e973405..ebee6b0 100644
>> --- a/drivers/ssb/driver_gige.c
>> +++ b/drivers/ssb/driver_gige.c
>> @@ -242,7 +242,7 @@ static int ssb_gige_probe(struct ssb_device *sdev,
>> bool pdev_is_ssb_gige_core(struct pci_dev *pdev)
>> {
>> if (!pdev->resource[0].name)
>> - return 0;
>> + return false;
>> return (strcmp(pdev->resource[0].name, SSB_GIGE_MEM_RES_NAME) == 0);
>> }
>> EXPORT_SYMBOL(pdev_is_ssb_gige_core);
>
> Looks good.
>
> Signed-off-by: Michael Buesch <m@xxxxxxx>
>
> Kalle, can you take this one, please?

Sure, I'll apply it in the next few days.

--
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/