From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: Mason <slash.tmp@free.fr>
Cc: linux-mtd <linux-mtd@lists.infradead.org>,
Richard Weinberger <richard@nod.at>,
Sebastian Frias <sf84@laposte.net>,
Jean-Baptiste Lescher <jblescher@gmail.com>,
Thibaud Cornic <thibaud_cornic@sigmadesigns.com>
Subject: Re: [RFC preliminary v0.1] nfc driver for tango platform
Date: Wed, 31 Aug 2016 18:59:16 +0200 [thread overview]
Message-ID: <20160831185916.199c5433@bbrezillon> (raw)
In-Reply-To: <57C707F3.5080301@free.fr>
On Wed, 31 Aug 2016 18:38:11 +0200
Mason <slash.tmp@free.fr> wrote:
> On 30/08/2016 14:55, Mason wrote:
> > Hello linux-mtd,
> >
> > I've been writing a driver for the custom NAND Flash controller embedded
> > in tango chips, such as smp8758.
> >
> > I'd like to hear every and all feedback on what I did wrong so that I can
> > eventually submit the work upstream. Please bear in mind that this is a
> > preliminary version, so there are still a few rough edges.
>
> I didn't have time to finish, and make a proper patch, as Boris requested.
> But I did fix all the hard-coded values, and I wanted to post the result,
> as a reference.
I already told you on IRC that I would not review your code if it's
not submitted in the correct format. So, why do you even try to send a
new version?
On a more general note, I'm tired trying to convince you that the way
you do things is inappropriate (it usually happens on minor things, and
you keep arguing endlessly). So please try to listen to
other devs/maintainers comments instead constantly defying them.
>
> I will try to fix the two hard-coded timings. (Since these are chip-dependent
> parameters, I thought they should be passed via the device tree, but Boris
> thinks it might be possible to deduce at run-time?)
>
> I'm hoping to post a proper patch (with Kconfig and Makefile updated) some
> time tomorrow.
Why did you send this email then???
next prev parent reply other threads:[~2016-08-31 16:59 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-30 12:55 [RFC preliminary v0.1] nfc driver for tango platform Mason
2016-08-31 16:38 ` Mason
2016-08-31 16:59 ` Boris Brezillon [this message]
2016-08-31 17:10 ` Mason
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=20160831185916.199c5433@bbrezillon \
--to=boris.brezillon@free-electrons.com \
--cc=jblescher@gmail.com \
--cc=linux-mtd@lists.infradead.org \
--cc=richard@nod.at \
--cc=sf84@laposte.net \
--cc=slash.tmp@free.fr \
--cc=thibaud_cornic@sigmadesigns.com \
/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.