Re: [PATCH 2/2] staging: media: davinci_vpfe: Remove spaces before semicolons

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

 



Hi Lisa,

On Tue, Dec 10, 2013 at 8:38 PM, Lisa Nguyen <lisa@xxxxxxxxxxxxxxx> wrote:
> Hi Prabhakar,
>
> On Tue, Dec 10, 2013 at 7:04 AM, Prabhakar Lad
> <prabhakar.csengg@xxxxxxxxx> wrote:
>> Hi Lisa,
>>
>> On Tue, Dec 10, 2013 at 8:27 PM, Lisa Nguyen <lisa@xxxxxxxxxxxxxxx> wrote:
>>> Hi everyone,
>>>
>>> On Tue, Dec 10, 2013 at 6:34 AM, Prabhakar Lad
>>> <prabhakar.csengg@xxxxxxxxx> wrote:
>>>> Hi Laurent,
>>>>
>>>> On Tue, Dec 10, 2013 at 7:34 PM, Laurent Pinchart
>>>> <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
>>>>> Hi Prabhakar,
>>>>>
>>>>> On Wednesday 30 October 2013 13:20:25 Prabhakar Lad wrote:
>>>>>> On Tue, Oct 29, 2013 at 2:53 AM, Lisa Nguyen <lisa@xxxxxxxxxxxxxxx> wrote:
>>>>>> > Remove unnecessary spaces before semicolons to meet kernel
>>>>>> > coding style.
>>>>>> >
>>>>>> > Signed-off-by: Lisa Nguyen <lisa@xxxxxxxxxxxxxxx>
>>>>>>
>>>>>> Acked-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx>
>>>>>
>>>>> Do you plan to send a pull request for these two patches ?
>>>>>
>>>> I had asked for a change in the first patch but Lisa never turned back :(
>>>> anyway I'll fix it and issue a pull request today to Mauro.
>>>
>>> My apologies. What happened was that I originally had sent these two
>>> patches to the staging mailing list. Greg KH advised me to send these
>>> to Mauro and the linux-media mailing list instead. As a result, there
>>> was a debate about the way the return statement was written in my
>>> first patch between Greg and a fellow developer, so I wasn't sure who
>>> to listen to. I was in the midst of changing jobs, so this didn't take
>>> top priority.
>>>
>> Ok, do you plan to post it now ?
>
> To be clear, I'd only have to update the first patch, correct?

Yes!

Regards,
--Prabhakar Lad
--
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