Those four that you just sent checkpatch clean. Merged into cifs-2.6.git Any additional Reviewed-by would be appreciated for those 4. On Mon, Jan 20, 2014 at 4:24 AM, Pavel Shilovsky <piastry@xxxxxxxxxxx> wrote: > 2014/1/20 Steve French <smfrench@xxxxxxxxx>: >> looks reasonable but am getting various minor whitespace errors >> scanning the patch series >> >> Can you fixup and make sure it is checkpatch clean? >> >> e.g. >> >> ERROR: trailing whitespace >> #123: FILE: fs/cifs/dir.c:650: >> +^I/*^M$ >> >> ERROR: trailing whitespace >> #124: FILE: fs/cifs/dir.c:651: >> +^I * BB Do not bother to decode buf since no local inode yet to put^M$ >> >> ERROR: trailing whitespace >> #125: FILE: fs/cifs/dir.c:652: >> +^I * timestamps in, but we can reuse it safely.^M$ >> > > Steve, are you sure you got right variants? I tested my patches with > checkpatch.pl and everything was ok. I applied them on top of your > current for-next branch and attached. > > -- > Best regards, > Pavel Shilovsky. -- Thanks, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
