Re: [PATCH] staging: android: binder: fix some sparse warnings and long lines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Resend in plain text, sorry...

> Em 23 de abril de 2012 14:23, Emil Goode <emilgoode@xxxxxxxxx> escreveu:
>
>> This patch silences some sparse warnings and adds some line breaks
>> to take the 80 char limit coding style rule in consideration.
>>
>> Signed-off-by: Emil Goode <emilgoode@xxxxxxxxx>
>
>
> Nice work Emil!
>
> What do you think to put the sparse warnings in the commit message?
>
> I think it is better to understand  what are you trying to fix.
>
>>
>> ---
>>  drivers/staging/android/binder.c |   14 ++++++++++----
>>  1 file changed, 10 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/staging/android/binder.c
>> b/drivers/staging/android/binder.c
>> index c283212..5d31417 100644
>> --- a/drivers/staging/android/binder.c
>> +++ b/drivers/staging/android/binder.c
>> @@ -1600,15 +1600,21 @@ static void binder_transaction(struct binder_proc
>> *proc,
>>                case BINDER_TYPE_BINDER:
>>                case BINDER_TYPE_WEAK_BINDER: {
>>                        struct binder_ref *ref;
>> -                       struct binder_node *node = binder_get_node(proc,
>> fp->binder);
>> +                       struct binder_node *node = binder_get_node(proc,
>> +                                                    (void __user
>> *)fp->binder);
>>                        if (node == NULL) {
>> -                               node = binder_new_node(proc, fp->binder,
>> fp->cookie);
>> +                               node = binder_new_node(proc,
>> +                                               (void __user
>> *)fp->binder,
>> +                                               (void __user
>> *)fp->cookie);
>>                                if (node == NULL) {
>>                                        return_error = BR_FAILED_REPLY;
>>                                        goto err_binder_new_node_failed;
>>                                }
>> -                               node->min_priority = fp->flags &
>> FLAT_BINDER_FLAG_PRIORITY_MASK;
>> -                               node->accept_fds = !!(fp->flags &
>> FLAT_BINDER_FLAG_ACCEPTS_FDS);
>> +                               node->min_priority = fp->flags &
>> +
>> FLAT_BINDER_FLAG_PRIORITY_MASK;
>> +
>> +                               node->accept_fds = !!(fp->flags &
>> +
>> FLAT_BINDER_FLAG_ACCEPTS_FDS);
>>                        }
>>                        if (fp->cookie != node->cookie) {
>>                                binder_user_error("binder: %d:%d sending
>> u%p "
>> --
>> 1.7.9.5
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe
>> kernel-janitors" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
>
>
>
> --
> Att,
>
> Marcos Paulo de Souza
> Acadêmico de Ciencia da Computação - FURB - SC
> "Uma vida sem desafios é uma vida sem razão"
> "A life without challenges, is a non reason life"




--
Att,

Marcos Paulo de Souza
Acadêmico de Ciencia da Computação - FURB - SC
"Uma vida sem desafios é uma vida sem razão"
"A life without challenges, is a non reason life"
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux