[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <742e4af7-ad70-6607-62b0-81d11cd5b04e@displaylink.com>
Date: Wed, 12 Feb 2020 09:41:12 +0000
From: Vladimir Stankovic <vladimir.stankovic@...playlink.com>
To: "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
Petar Kovacevic <petar.kovacevic@...playlink.com>,
Stefan Lugonjic <stefan.lugonjic@...playlink.com>,
Nikola Simic <nikola.simic@...playlink.com>,
Marko Miljkovic <marko.miljkovic@...playlink.com>
Subject: Re: [External] Re: [PATCH 1/1] usb: Add MA USB Host driver
On 11.2.20. 17:39, gregkh@...uxfoundation.org wrote:
> On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
>> 39 files changed, 8668 insertions(+)
>
> This is a bit hard, if not impossible, to review all in one huge patch.
>
> Can you resend this as a patch series, breaking it down into logical
> chunks, like all other kernel patches have?
>
> Also, why so many individual files? For only 8k lines, 39 files seems
> like a huge number.
>
> thanks,
>
> greg k-h
>
Will break it down into patch series and resend.
In regards of the file count, our intention was to ease the
troubleshooting efforts during development and have a clear separation
between logical parts of MA-USB implementation (i.e data in/out, isoch
in/out, etc.; each source file representing logical chunk).
Regards,
Vladimir.
Powered by blists - more mailing lists