lkml.org 
[lkml]   [2018]   [Jul]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 01/15] Introduce Cadence USBSSP DRD Driver - added gadget.c file.
Date
> On Tue, Jul 03, 2018 at 08:57:45PM +0100, Pawel Laszczak wrote:
> > From: Laszczak Pawel <pawell.cadence.com>
> >
> > Signed-off-by: Laszczak Pawel <pawell@cadence.com>
> > ---
>
> I can't take patches without any changelog text at all, sorry.
>
> Also, your subject lines need work, look at how all other usb patches are
> written, your long prefix is not really helping out here.
>
> > drivers/usb/usbssp/gadget.c | 2082
> > +++++++++++++++++++++++++++++++++++
> > 1 file changed, 2082 insertions(+)
> > create mode 100644 drivers/usb/usbssp/gadget.c
>
>
> Just splitting the driver up by file is hard to review, as you are just dumping
> individual files on us, not functionality, and we can not review anything until
> all of them are added. Can you break this up into logical chunks? Make
> these in a way that is easy to review, would you want to review this patch
> series in this way?

It's quite large driver and I thought that this form will be better and faster to review.
I didn't want to prepare too much patches.

I will prepare new patch set and I will tray to split it into logical chunk.

Thanks,
Pawel Laszczak

\
 
 \ /
  Last update: 2018-07-05 12:16    [W:0.069 / U:1.816 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site