lkml.org 
[lkml]   [2011]   [Jun]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
SubjectRe: [PATCH] s3c2410_udc: fix custom UDC command handling
From
Yeah I think that one line break and tabs are sabotaging your work.
So stop being an 8====O - hole. No wonder if you are paid by TI that
one line patch into the upstream takes you so long to comprehend.

Vilo

On Mon, Jun 20, 2011 at 6:06 PM, Felipe Balbi <balbi@ti.com> wrote:
> Hi,
>
> On Mon, Jun 20, 2011 at 05:57:31PM +0200, Skacore Systems wrote:
>> Oh and I've finally cloned the mainline git. So here it's the last
>> try. I hope the TI people will not sabotage the Samsung's drivers any
>> way longer !!!
>
> Now you're just making an ass of yourself.
>
> what do you think you will get making such kind of statements ? I just
> happen to be hired by TI, there's no such thing as TI trying to sabotage
> Samsung's drivers.
>
> I'll fix this one out myself just to make you stop bothering me, but
> don't expect me to put your patches in my priority list. If you really
> think TI is sabotaging samsung's drivers, look at the big re-factor we
> are doing on the entire Gadget framework, involving not only Samsung's
> drivers but all the others.
>
> Even though you cloned the tree you still make a mistake which will
> prevent me from applying the patch without manually editting it.
>
> On top of that, you decided not to follow our netiquette or our patch
> formatting rules just because you think it's "nonsense". If you really
> think that, why do you even bother sending the patch to this mailing
> list ?
>
>> diff --git a/drivers/usb/gadget/s3c2410_udc.c b/drivers/usb/gadget/s3c2410_udc.c
>> index 100f263..b6e1ca5 100644
>> --- a/drivers/usb/gadget/s3c2410_udc.c
>> +++ b/drivers/usb/gadget/s3c2410_udc.c
>> @@ -1567,7 +1567,7 @@ static void s3c2410_udc_command(enum
>> s3c2410_udc_cmd_e cmd)
>
> there shouldn't be this line break here.
>
>>                 return;
>>
>>         if (udc_info->udc_command) {
>> -               udc_info->udc_command(S3C2410_UDC_P_DISABLE);
>> +               udc_info->udc_command(cmd);
>>         } else if (gpio_is_valid(udc_info->pullup_pin)) {
>>                 int value;
>
> tabs where converted into spaces too.
>
> Next time, instead of been such an ass, spend some time learning the
> process we use to do development. You'll see things will go smoother.
>
> --
> balbi
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2011-06-20 18:45    [W:0.243 / U:0.452 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site