Messages in this thread | | | Subject | Re: [PATCH 02/29] memstick: core: reorder functions This patch just reorders functions in memstick.c So that host specific and card driver specific functions are now grouped together. This makes it easier to understand the code. | From | Maxim Levitsky <> | Date | Tue, 26 Oct 2010 03:14:23 +0200 |
| |
On Mon, 2010-10-25 at 07:50 -0700, Alex Dubov wrote: > > --- On Fri, 22/10/10, Maxim Levitsky <maximlevitsky@gmail.com> wrote: > > > From: Maxim Levitsky <maximlevitsky@gmail.com> > > Subject: [PATCH 02/29] memstick: core: reorder functions This patch just reorders functions in memstick.c So that host specific and card driver specific functions are now grouped together. This makes it easier to understand the code. > > > I'm not sure this purely subjective patch is of any real utility. > And at any rate, it belongs to the end of the patchset (where the clean-up > normally go), not to the beginning.
Could somebody else comment on this? I mean memstick.c/h contains two types of functions. One is useful only for low level drivers, and other only for high level drivers. Whats wrong with at least grouping that together? It probably even better to put them to separate files...
As for doing that late, I exaplained that once in reply to Morton. It was much easier to split patches this way, and really doesn't matter otherwise.
Best regards, Maxim Levitsky
| |