[PATCH] staging: netlogic: removed 1 multiple assignment checkpatch warning

Roger H Newell newell.roger at gmail.com
Sat Mar 19 17:03:16 UTC 2016


On Sat, Mar 19, 2016 at 2:04 PM, Parth Sane <laerdevstudios at gmail.com> wrote:
> Hi,
> I haven’t received any feedback as to whether my patch has been accepted. Would request someone to kindly help me out. Maybe you could weigh in here Greg?
> Regards,
> Parth Sane
>> On 15-Mar-2016, at 10:16 PM, laerdevstudios at gmail.com wrote:
>>
>> From: Parth Sane <laerdevstudios at gmail.com>
>>
>> Fixed multiple assignment checkpatch warning by rearranging code.
>>
>> Signed-off-by: Parth Sane <laerdevstudios at gmail.com>
>>
>> ---
>> drivers/staging/netlogic/platform_net.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/staging/netlogic/platform_net.c b/drivers/staging/netlogic/platform_net.c
>> index 7806c2b..daee1b2 100644
>> --- a/drivers/staging/netlogic/platform_net.c
>> +++ b/drivers/staging/netlogic/platform_net.c
>> @@ -86,7 +86,8 @@ static void xlr_resource_init(struct resource *res, int offset, int irq)
>>
>>       res++;
>>       res->name = "gmac";
>> -     res->start = res->end = irq;
>> +     res->end = irq;
>> +     res->start = res->end;
>>       res->flags = IORESOURCE_IRQ;
>> }
>>
>> --
>> 2.1.4
>>
>
>
> _______________________________________________
> devel mailing list
> devel at linuxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
>

This patch is only 4 days old. It's likely in Gregs queue and you just
have to wait a bit longer.  I don't think you need the From: line in
the body of the patch. The Signed-off-by: should suffice.


More information about the devel mailing list