From: Ivan Djelic <ivan.djelic@parrot.com>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Andrew Morton <akpm@linux-foundation.org>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Artem Bityutskiy <dedekind1@gmail.com>
Subject: Re: [PATCH/RFC v2 1/3] Shared BCH ECC library
Date: Wed, 9 Mar 2011 19:02:09 +0100 [thread overview]
Message-ID: <20110309180209.GA29754@parrot.com> (raw)
In-Reply-To: <alpine.LFD.2.00.1103082206270.2787@localhost6.localdomain6>
On Tue, Mar 08, 2011 at 09:08:36PM +0000, Thomas Gleixner wrote:
> On Tue, 8 Mar 2011, Artem Bityutskiy wrote:
>
> > On Tue, 2011-03-08 at 12:15 +0100, Ivan Djelic wrote:
> > > On Mon, Feb 07, 2011 at 02:09:30PM +0000, Ivan Djelic wrote:
> > > > Hello,
> > > >
> > > > The first patch of this series contains a new generic BCH ECC library.
> > >
> > > Hi,
> > >
> > > It has been a month since this patch was posted, and I did not receive any
> > > comment/review yet.
> >
> > I can add it to my l2 tree and may be dwmw2 can merge it then. But could
> > you please re-send a version with better Kconfig help messages?
>
> I had a quick look. The code looks pretty clean and well documented,
> though I did not dig into the details. There are a few comments
> "lifted from ...", I hope the code which is copied does not cause any
> license issues.
The code you are referring to is in the public domain (from the famous "Bit
Twiddling Hacks" page). I added this information in version v3 of my patch.
Thanks for the feedback!
Ivan
WARNING: multiple messages have this Message-ID (diff)
From: Ivan Djelic <ivan.djelic@parrot.com>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Artem Bityutskiy <dedekind1@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH/RFC v2 1/3] Shared BCH ECC library
Date: Wed, 9 Mar 2011 19:02:09 +0100 [thread overview]
Message-ID: <20110309180209.GA29754@parrot.com> (raw)
In-Reply-To: <alpine.LFD.2.00.1103082206270.2787@localhost6.localdomain6>
On Tue, Mar 08, 2011 at 09:08:36PM +0000, Thomas Gleixner wrote:
> On Tue, 8 Mar 2011, Artem Bityutskiy wrote:
>
> > On Tue, 2011-03-08 at 12:15 +0100, Ivan Djelic wrote:
> > > On Mon, Feb 07, 2011 at 02:09:30PM +0000, Ivan Djelic wrote:
> > > > Hello,
> > > >
> > > > The first patch of this series contains a new generic BCH ECC library.
> > >
> > > Hi,
> > >
> > > It has been a month since this patch was posted, and I did not receive any
> > > comment/review yet.
> >
> > I can add it to my l2 tree and may be dwmw2 can merge it then. But could
> > you please re-send a version with better Kconfig help messages?
>
> I had a quick look. The code looks pretty clean and well documented,
> though I did not dig into the details. There are a few comments
> "lifted from ...", I hope the code which is copied does not cause any
> license issues.
The code you are referring to is in the public domain (from the famous "Bit
Twiddling Hacks" page). I added this information in version v3 of my patch.
Thanks for the feedback!
Ivan
next prev parent reply other threads:[~2011-03-09 18:02 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-07 14:09 [PATCH/RFC v2 1/3] Shared BCH ECC library Ivan Djelic
2011-02-07 14:09 ` Ivan Djelic
2011-02-07 14:09 ` [PATCH/RFC v2 2/3] mtd: nand: add software BCH ECC support Ivan Djelic
2011-02-07 14:09 ` Ivan Djelic
2011-02-07 14:09 ` [PATCH/RFC v2 3/3] mtd: nand: enable software BCH ECC in nand simulator Ivan Djelic
2011-02-07 14:09 ` Ivan Djelic
2011-03-08 11:15 ` [PATCH/RFC v2 1/3] Shared BCH ECC library Ivan Djelic
2011-03-08 11:15 ` Ivan Djelic
2011-03-08 12:39 ` Artem Bityutskiy
2011-03-08 12:39 ` Artem Bityutskiy
2011-03-08 21:08 ` Thomas Gleixner
2011-03-08 21:08 ` Thomas Gleixner
2011-03-09 18:02 ` Ivan Djelic [this message]
2011-03-09 18:02 ` Ivan Djelic
2011-03-08 12:37 ` Artem Bityutskiy
2011-03-08 12:37 ` Artem Bityutskiy
-- strict thread matches above, loose matches on Subject: below --
2011-02-02 10:50 Ivan Djelic
2011-02-06 15:03 ` Artem Bityutskiy
2011-02-06 15:03 ` Artem Bityutskiy
2011-02-07 14:16 ` Ivan Djelic
2011-02-07 14:16 ` Ivan Djelic
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=20110309180209.GA29754@parrot.com \
--to=ivan.djelic@parrot.com \
--cc=akpm@linux-foundation.org \
--cc=dedekind1@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=tglx@linutronix.de \
/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.