Re: [PATCH] staging: media: cxd2099: remove memcpy of similar structure variables

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

 



Hi Ezequiel,

On Mon, Aug 6, 2012 at 3:36 AM, Ezequiel Garcia <elezegarcia@xxxxxxxxx> wrote:
> Hi Devendra,
>
> Thanks for the patch,
>
> On Sun, Aug 5, 2012 at 5:40 PM, Devendra Naga
> <develkernel412222@xxxxxxxxx> wrote:
>> structure variables can be assigned, no memcpy needed,
>> remove the memcpy and use assignment for the cfg and en variables.
>>
>> Tested by Compilation Only
>>
>> Suggested-by: Ezequiel Garcia <elezegarcia@xxxxxxxxx>
>
> I'm not sure this is completely valid or useful.
>
> If you read Documentation/SubmittingPatches (which you should)
> you will find references to Acked-by, Reported-by, Tested-by,
> but not this one.
>
> You don't need to give me credit for the patch:
> it's *your* patch, all I did was a very simple suggestion :-)
>
Ok. I will remove the Suggested-by line and send out the patch again
with PATCH RESEND subject line.
Is it ok?

> Plus, there was some discussion called "Kernel Komedians" [1] where
> some developer expressed their concern on the number of weird signatures
> that have recently appeared.
>
> Regards,
> Ezequiel.
>
> [1] http://lwn.net/Articles/503829/

I will read out the Documentation and will try to send patches accordingly,

Thanks a lot for your help and your time,
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux