From: Wolfram Sang <w.sang@pengutronix.de>
To: Lee Jones <lee.jones@linaro.org>
Cc: Linus Walleij <linus.walleij@linaro.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
STEricsson_nomadik_linux@list.st.com,
linus.walleij@stericsson.com, arnd@arndb.de,
linux-i2c@vger.kernel.org
Subject: Re: [PATCH 12/19] i2c-nomadik: Register sub-devices when passed via Device Tree
Date: Fri, 14 Sep 2012 13:32:12 +0200 [thread overview]
Message-ID: <20120914113212.GD2630@pengutronix.de> (raw)
In-Reply-To: <20120914101542.GT3374@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 693 bytes --]
> > Documentation: Device Tree binding information for i2c-nomadik driver
> > (Documentation should go along with the code)
>
> I'm still not too sure about how this benefits anyone. You can ensure
> the documentation is placed in an adjacent patch, so it's just as easy
> to make reference to.
It makes reviewing a lot easier if the documentation what is supposed to
be coded and the actual code are in one place. It also prevents what
happened in this case where the documentation never showed up on the
i2c-list.
--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2012-09-14 11:32 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1347016499-29354-1-git-send-email-lee.jones@linaro.org>
[not found] ` <1347016499-29354-1-git-send-email-lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-09-07 11:14 ` [PATCH 12/19] i2c-nomadik: Register sub-devices when passed via Device Tree Lee Jones
[not found] ` <1347016499-29354-13-git-send-email-lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-09-10 11:42 ` Linus Walleij
2012-09-12 10:52 ` Wolfram Sang
2012-09-14 8:27 ` Lee Jones
[not found] ` <20120914082754.GE3374-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-09-14 8:41 ` Wolfram Sang
[not found] ` <20120914084141.GA2630-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2012-09-14 9:02 ` Lee Jones
2012-09-14 9:39 ` Wolfram Sang
[not found] ` <20120914093945.GB2630-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2012-09-14 10:15 ` Lee Jones
2012-09-14 11:32 ` Wolfram Sang [this message]
[not found] ` <20120914113212.GD2630-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2012-09-19 20:12 ` Lee Jones
[not found] ` <20120919201220.GA5035-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-10-06 11:25 ` Wolfram Sang
[not found] ` <CACRpkdaCrus9SOpxyK+kV2gKr3sEZJsHCn6ivn2gA3ri0rcvHw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-09-14 8:22 ` Lee Jones
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=20120914113212.GD2630@pengutronix.de \
--to=w.sang@pengutronix.de \
--cc=STEricsson_nomadik_linux@list.st.com \
--cc=arnd@arndb.de \
--cc=lee.jones@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-i2c@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).