All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cruz Julian Bishop <cruzjbishop@gmail.com>
To: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 0/7] Fat checkstyle patch set (1)
Date: Sun, 12 Aug 2012 21:01:55 +1000	[thread overview]
Message-ID: <50278D23.3000008@gmail.com> (raw)
In-Reply-To: <877gt4behg.fsf@devron.myhome.or.jp>

On 12/08/12 20:18, OGAWA Hirofumi wrote:
> Cruz Julian Bishop <cruzjbishop@gmail.com> writes:
>
>> Hi (and to the mailing list, again).
>>
>> This is a patch series that fixes checkpatch issues in fs/fat.
>>
>> I didn't get to do all of the files, but I'll be sure to come back for
>> them later :)
>>
>> Please let me know if I did something wrong - I'll be sure to take it on
>> for my next patches.
> Is this for -mm tree? Please make sure there is no conflict. 
It's actually based on -next, but I think that's taken over what -mm
was doing in the past, anyway.
>
> Yeah, small patch is preferred, but it should be the logical separation,
> and one patch would be better in this case, IMHO.
>
> Personally, I'm not against this type cleanup only if there is no
> conflict though (i.e. doesn't increase maintain cost). So, if akpm is
> fine to take this, then I'm ok.
Thank you - I'll remember that in the future
>
>> Cruz Julian Bishop (7):
>>   fs/fat: Fix a checkpatch issue in namei_msdos.c
>>   fs/fat: Fix some checkpatch issues in fat.h
>>   fs/fat: Changed indentation of some comments in fat.h
>>   fs/fat: Fix two checkstyle issues in cache.c
>>   fs/fat: Fixes some small checkpatch issues is dir.c
>>   fs/fat: Fix all other checkstyle issues in dir.c
>>   fs/fat: Fix checkpatch issues in fatent.c
> There was one strange new line in a patch. Otherwise,
>
> Acked-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Again, thank you :)


      reply	other threads:[~2012-08-12 11:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-12  9:44 [PATCH 0/7] Fat checkstyle patch set (1) Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 1/7] fs/fat: Fix a checkpatch issue in namei_msdos.c Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 2/7] fs/fat: Fix some checkpatch issues in fat.h Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 3/7] fs/fat: Changed indentation of some comments " Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 4/7] fs/fat: Fix two checkpatch issues in cache.c Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 5/7] fs/fat: Fixes some small checkpatch issues in dir.c Cruz Julian Bishop
2012-08-12  9:44 ` [PATCH 6/7] fs/fat: Fix all other " Cruz Julian Bishop
2012-08-12 10:14   ` OGAWA Hirofumi
2012-08-12  9:44 ` [PATCH 7/7] fs/fat: Fix checkpatch issues in fatent.c Cruz Julian Bishop
2012-08-12 10:18 ` [PATCH 0/7] Fat checkstyle patch set (1) OGAWA Hirofumi
2012-08-12 11:01   ` Cruz Julian Bishop [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50278D23.3000008@gmail.com \
    --to=cruzjbishop@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.