All of lore.kernel.org
 help / color / mirror / Atom feed
From: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
To: Cruz Julian Bishop <cruzjbishop@gmail.com>
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 19:18:19 +0900	[thread overview]
Message-ID: <877gt4behg.fsf@devron.myhome.or.jp> (raw)
In-Reply-To: <1344764687-4155-1-git-send-email-cruzjbishop@gmail.com> (Cruz Julian Bishop's message of "Sun, 12 Aug 2012 19:44:40 +1000")

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.

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.

> 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>

-- 
OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>

  parent reply	other threads:[~2012-08-12 10:18 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 ` OGAWA Hirofumi [this message]
2012-08-12 11:01   ` [PATCH 0/7] Fat checkstyle patch set (1) Cruz Julian Bishop

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=877gt4behg.fsf@devron.myhome.or.jp \
    --to=hirofumi@mail.parknet.co.jp \
    --cc=akpm@linux-foundation.org \
    --cc=cruzjbishop@gmail.com \
    --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.