All of lore.kernel.org
 help / color / mirror / Atom feed
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Joe Perches <joe@perches.com>
Cc: linux-mtd@lists.infradead.org,
	David Woodhouse <dwmw2@infradead.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4] jffs2: Convert most D1/D2 macros to jffs2_dbg
Date: Wed, 14 Mar 2012 12:54:45 +0200	[thread overview]
Message-ID: <1331722485.2140.13.camel@sauron.fi.intel.com> (raw)
In-Reply-To: <1331656495.11405.4.camel@joe2Laptop>

[-- Attachment #1: Type: text/plain, Size: 795 bytes --]

On Tue, 2012-03-13 at 09:34 -0700, Joe Perches wrote:
> These functions are conditionally compiled today.
> 
> Introducing a bitmask allows the same conditional
> compilation optimization.

Yeah, I know, but I think this is useless and can be just killed.

> David?  as nominal maintainer, do you have an opinion?

But you are right that I do not give guarantee David will accept this. I
can only get this to my l2-mtd.git tree.

> btw: Artem, you are not listed in the MAINTAINERS
> entry for jffs2.  Should you be?

No, I am not JFFS2 maintainer and would not want to be because nowadays
JFFS2 is on its slow death path and very few people take care of it.

P.S.: it looks like this is where whole MTD stuff is trending as
well :-(

-- 
Best Regards,
Artem Bityutskiy

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Joe Perches <joe@perches.com>
Cc: David Woodhouse <dwmw2@infradead.org>,
	linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4] jffs2: Convert most D1/D2 macros to jffs2_dbg
Date: Wed, 14 Mar 2012 12:54:45 +0200	[thread overview]
Message-ID: <1331722485.2140.13.camel@sauron.fi.intel.com> (raw)
In-Reply-To: <1331656495.11405.4.camel@joe2Laptop>

[-- Attachment #1: Type: text/plain, Size: 795 bytes --]

On Tue, 2012-03-13 at 09:34 -0700, Joe Perches wrote:
> These functions are conditionally compiled today.
> 
> Introducing a bitmask allows the same conditional
> compilation optimization.

Yeah, I know, but I think this is useless and can be just killed.

> David?  as nominal maintainer, do you have an opinion?

But you are right that I do not give guarantee David will accept this. I
can only get this to my l2-mtd.git tree.

> btw: Artem, you are not listed in the MAINTAINERS
> entry for jffs2.  Should you be?

No, I am not JFFS2 maintainer and would not want to be because nowadays
JFFS2 is on its slow death path and very few people take care of it.

P.S.: it looks like this is where whole MTD stuff is trending as
well :-(

-- 
Best Regards,
Artem Bityutskiy

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2012-03-14 10:52 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-15 23:56 [PATCH 0/4] jffs2: logging message neatening Joe Perches
2012-02-15 23:56 ` [PATCH 1/4] jffs2: Convert most D1/D2 macros to jffs2_dbg Joe Perches
2012-03-05 16:30   ` Artem Bityutskiy
2012-03-05 16:30     ` Artem Bityutskiy
2012-03-05 16:56     ` Joe Perches
2012-03-05 16:56       ` Joe Perches
2012-03-09 11:53       ` Artem Bityutskiy
2012-03-09 11:53         ` Artem Bityutskiy
2012-03-09 18:49         ` Joe Perches
2012-03-09 18:49           ` Joe Perches
2012-03-12 13:12           ` Artem Bityutskiy
2012-03-12 13:12             ` Artem Bityutskiy
2012-03-12 15:33             ` Joe Perches
2012-03-12 15:33               ` Joe Perches
2012-03-12 16:35               ` Artem Bityutskiy
2012-03-12 16:35                 ` Artem Bityutskiy
2012-03-12 17:11                 ` Joe Perches
2012-03-12 17:11                   ` Joe Perches
2012-03-13 12:39                   ` Artem Bityutskiy
2012-03-13 12:39                     ` Artem Bityutskiy
2012-03-13 16:34                     ` Joe Perches
2012-03-13 16:34                       ` Joe Perches
2012-03-14 10:54                       ` Artem Bityutskiy [this message]
2012-03-14 10:54                         ` Artem Bityutskiy
2012-03-09 21:59         ` [PATCH l2-mtd] jffs2: Fix build when CONFIG_JFFS2_FS_DEBUG > 0 Joe Perches
2012-03-09 21:59           ` Joe Perches
2012-03-12 13:50           ` Artem Bityutskiy
2012-03-12 13:50             ` Artem Bityutskiy
2012-02-15 23:56 ` [PATCH 2/4] jffs2: Convert printks to pr_<level> Joe Perches
2012-02-15 23:56 ` [PATCH 3/4] jffs2: Use pr_fmt and remove jffs: from formats Joe Perches
2012-02-15 23:56 ` [PATCH 4/4] jffs2: Standardize JFFS_<LEVEL> uses Joe Perches
2012-02-29 12:08 ` [PATCH 0/4] jffs2: logging message neatening Artem Bityutskiy
2012-02-29 12:08   ` Artem Bityutskiy
2012-02-29 12:12   ` Artem Bityutskiy
2012-02-29 12:12     ` Artem Bityutskiy
2012-02-29 19:02   ` Joe Perches
2012-02-29 19:02     ` Joe Perches
2012-03-01  7:36     ` Artem Bityutskiy
2012-03-01  7:36       ` Artem Bityutskiy

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=1331722485.2140.13.camel@sauron.fi.intel.com \
    --to=dedekind1@gmail.com \
    --cc=dwmw2@infradead.org \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.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.