Messages in this thread | | | Date | Wed, 31 Jul 2013 18:45:12 -0700 | From | Greg Kroah-Hartman <> | Subject | Re: [PATCH 1/5] Intel MIC Host Driver for X100 family. |
| |
On Wed, Jul 24, 2013 at 08:31:32PM -0700, Sudeep Dutt wrote: > This patch enables the following: > a) Initializes the Intel MIC X100 PCIe devices. > b) Boots and shuts down the card via sysfs entries. > c) Allocates and maps a device page for communication with the > card driver and updates the device page address via scratchpad > registers. > d) Provides sysfs entries for family, stepping, state, shutdown > status, kernel command line, IP address, ramdisk and log buffer > information.
That's a lot to do in one patch, almost 4 thousand lines. Can't you break this up into some more smaller, logical, and reviewable, pieces?
For example, I have no idea what b) is, and how to separate it from the things you do for a) and c).
thanks,
greg k-h
| |