From patchwork Mon Jun 20 19:05:07 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Viliam Mateicka X-Patchwork-Id: 898072 Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by demeter1.kernel.org (8.14.4/8.14.4) with ESMTP id p5KJ5SKo011301 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Mon, 20 Jun 2011 19:05:50 GMT Received: from canuck.infradead.org ([2001:4978:20e::1]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1QYjmf-0006Sr-DS; Mon, 20 Jun 2011 19:05:17 +0000 Received: from localhost ([127.0.0.1] helo=canuck.infradead.org) by canuck.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1QYjmf-0008JK-1k; Mon, 20 Jun 2011 19:05:17 +0000 Received: from mail-vx0-f177.google.com ([209.85.220.177]) by canuck.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1QYjmZ-0008J2-6c for linux-arm-kernel@lists.infradead.org; Mon, 20 Jun 2011 19:05:15 +0000 Received: by vxd3 with SMTP id 3so3054998vxd.36 for ; Mon, 20 Jun 2011 12:05:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=osTrv+yj5rQmzXdlE69g9nWe77+yMj8+Vq+ZrGpJ7kM=; b=u9nWVn4TCb+qqZCC0AkTrKWHPj3DJPAm1eJs6vQdAzWfeznk/QdKh6i1sT/CVoBoJ+ oEr6niM8qAgQl5SIeU+ABREdotuNkVALwj/HR6+PY5XmJRytIAvYmbIwIFA+dmjy1RCS j+fzHKknSHnhjyJru2llOB6f8I61Vi0zzS798= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=WaeFVLPura2nAmMHRG4R/FE20ULUQwv5Gbx6TaIOEPB5vjKtvpXPLx0HGNhQT/gJq4 SVXiay5zNVKviI0ibTq13P3fs3gjHtpbMeuB0DTxFC6wYw2SrtRhU1GULe4Kxv3HFd7i YoHD8UXqeSD4rwKnPP9/SRiWtBt36o7U5k2Nw= MIME-Version: 1.0 Received: by 10.220.75.200 with SMTP id z8mr1853615vcj.32.1308596708039; Mon, 20 Jun 2011 12:05:08 -0700 (PDT) Received: by 10.220.178.198 with HTTP; Mon, 20 Jun 2011 12:05:07 -0700 (PDT) In-Reply-To: <20110620173444.GG26623@legolas.emea.dhcp.ti.com> References: <20110620151522.GN14262@legolas.emea.dhcp.ti.com> <20110620153645.GQ14262@legolas.emea.dhcp.ti.com> <20110620160650.GA26623@legolas.emea.dhcp.ti.com> <20110620170551.GE26623@legolas.emea.dhcp.ti.com> <20110620173444.GG26623@legolas.emea.dhcp.ti.com> Date: Mon, 20 Jun 2011 21:05:07 +0200 Message-ID: Subject: Re: [PATCH] s3c2410_udc: fix custom UDC command handling From: Viliam Mateicka To: balbi@ti.com X-CRM114-Version: 20090807-BlameThorstenAndJenny ( TRE 0.7.6 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20110620_150511_518009_08188BE6 X-CRM114-Status: GOOD ( 31.01 ) X-Spam-Score: -0.8 (/) X-Spam-Report: SpamAssassin version 3.3.1 on canuck.infradead.org summary: Content analysis details: (-0.8 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [209.85.220.177 listed in list.dnswl.org] 0.0 FREEMAIL_FROM Sender email is freemail (viliam.mateicka[at]gmail.com) -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature 0.0 RFC_ABUSE_POST Both abuse and postmaster missing on sender domain 0.0 T_TO_NO_BRKTS_FREEMAIL T_TO_NO_BRKTS_FREEMAIL Cc: Kukjin Kim , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Skacore Systems , linux-usb@vger.kernel.org, Alan Stern , Ben Dooks , linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Mon, 20 Jun 2011 19:05:50 +0000 (UTC) X-MIME-Autoconverted: from quoted-printable to 8bit by demeter1.kernel.org id p5KJ5SKo011301 Look. I didn't wanna spend more time with the issue because for me - it was solved. It was just the stupid byrocratic system you've enforced to minimize your work - which I would say its complete non-sence if it could ever have a problem with whitespace or line break. I understand that on the source level every character it's important to the look of the code, but at least when you try to force somebody to use some rules (which most of it at my copy were applied fine just fine by myself), at least use a tool which take a care about it (the look of the kernel code). Just look how much time it took to upstream a little change (valid on the first look), just because of my ignorance to some the formating rules (linebreaks and tabs namely - if you have a proper editor which takes care about it.) and a faulty mailer (I). Anyway, if the Documentation/SubmittingPatches would mention git format-patch and git send-email I would have use it and there would'nt be any trouble using faulty mailer. Instead as a first option there is solution which almost everytime produces such a problems. So many thanks, for doing your job and keeping the bad asses using wrong tools out of the formating heaven. I could imagine that this would be hard (to get one line into the kernel) but still haven't thought that hard. I would like to thank you for your patience and the exact amount of right doingness, while showing me what's wrong instead of telling me how to do it properly. Also liked the diff between our patches: player tmp # diff -ur wastetime wastetime2 + int value; Vilo On Mon, Jun 20, 2011 at 7:34 PM, Felipe Balbi wrote: > Hi, > > On Mon, Jun 20, 2011 at 07:27:18PM +0200, Viliam Mateicka wrote: >> I wonder then, who applies the patch first. One ass can't stop >> progress (was there some progress afterall ?) because of one >> line-break (which git produces by default afterall) and several >> non-conforming whitespace conversions. I think you should stop >> replying to me, because I'm kinda starting to like trolling afterall. > > Look, maybe you're just new to git/linux and that's fine. The only > problem we had from day one is that you have been a complete dick when > replying. > > If you go that road, that's what you'll get in return. > > Just so you see the properly formatted patch, I'm quoting below. See > that there's no line breaks introduced by git. > > commit c27c9b5312f4b2a7e66ee772943355d77fc2c550 > Author: Viliam Mateicka > Date:   Mon Jun 20 19:08:27 2011 +0300 > >    usb: gadget: s3c2410_udc: fix custom UDC command handling > >    There is a bug in Samsung's UDC driver, which is >    completely disabling the USB device when a custom >    UDC command is used. > >    Following patch seems to get the right behavior >    (e.g. enabling pull-up instead of disabling then >    Vcc is applied). > >    Signed-off-by: Viliam Mateicka > >    [ balbi@ti.com : manually re-created the patch because >        author was unable to send a proper one ] > >    Signed-off-by: Felipe Balbi > > 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) >                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; > > > (if you see any line breaks that's your mailer, it's better to check > from the archives [1]) > > [1] http://marc.info/?l=linux-usb > > -- > balbi > --- wastetime 2011-06-20 20:33:25.000000000 +0200 +++ wastetime2 2011-06-20 20:30:21.000000000 +0200 @@ -2,11 +2,12 @@ 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) - 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; +@@ -1567,7 +1567,7 @@ static void s3c2410_udc_command(enum +s3c2410_udc_cmd_e cmd) + 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)) {