On 10/11/16 5:58 PM, Michael Cyr wrote:
Signed-off-by: Michael Cyr <mike...@us.ibm.com>
  drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 1082 +++++++++++++-----------------
  drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.h |    5 +-
  2 files changed, 486 insertions(+), 601 deletions(-)

I would make the first patch reorganization and styling fixes, no actual code changes. Then for each of the commits make sure you amend them to give more description
as to what you are doing in each specific patch.

i.e. First patch:

ibmvscsis: Re-organization of commands/styling

Need to move functions around in order for the proceeding patches to work.

Patch 2/3 and 3/3 look fine to me after you change the title to include "ibmvscsis:" in it.


To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to