[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20141223184407.GK9147@saruman>
Date: Tue, 23 Dec 2014 12:44:07 -0600
From: Felipe Balbi <balbi@...com>
To: Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
CC: Ricardo Ribalda Delgado <ricardo.ribalda@...il.com>,
Felipe Balbi <balbi@...com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Peter Chen <peter.chen@...escale.com>,
Andrzej Pietrasiewicz <andrzej.p@...sung.com>,
Linux USB Mailing List <linux-usb@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 06/17] net2280: Remove restart_dma inline function
definition
On Fri, Nov 28, 2014 at 11:16:16PM +0300, Sergei Shtylyov wrote:
> On 11/28/2014 06:21 PM, Ricardo Ribalda Delgado wrote:
>
> >Thanks for reviewing. I will fix it and resend it on the next version
> >of the patchset to avoid spamming the ML
>
> >I guess it is ok to add your Reviewed-by
>
> Yes, if you like.
>
> >Thanks!
>
> Not at all. :-)
>
> [...]
>
> >>On 11/28/2014 04:50 PM, Ricardo Ribalda Delgado wrote:
>
> >>>restart_dma is not used before it is declaration. Therefore we can
>
> >> s/it is/its/.
>
> Also s/declaration/definition/.
>
> >>>remove this definition.
>
> >> You're removing the declaration, not definition.
do I get a new version of this patch ?
--
balbi
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists