From: Scott Wood <scottwood@freescale•com>
To: Zhang Haijun-B42677 <B42677@freescale•com>
Cc: Wood Scott-B07421 <B07421@freescale•com>,
Huang Changming-R66093 <r66093@freescale•com>,
"linuxppc-dev@lists•ozlabs.org" <linuxppc-dev@lists•ozlabs.org>,
Xie Xiaobo-R63061 <r63061@freescale•com>
Subject: Re: 答复: 答复: [PATCH 2/2] powerpc/85xx: add the P1020RDB-PD DTS support
Date: Tue, 9 Jul 2013 11:09:22 -0500 [thread overview]
Message-ID: <1373386162.8183.189@snotra> (raw)
In-Reply-To: <99E897753B6F7048BD8CCDB4661D02E1396F13@039-SN2MPN1-021.039d.mgd.msft.net> (from B42677@freescale.com on Tue Jul 9 00:34:34 2013)
On 07/09/2013 12:34:34 AM, Zhang Haijun-B42677 wrote:
> > cpld@2,0 {
> > compatible =3D "fsl, p1020rdb-cpld";
> > reg =3D <0x2 0x0 0x20000>;
> > read-only;
> > };
>=20
> Again, what is read-only supposed to mean here? I don't care that =20
> it's
> copied from the PC version. It doesn't make sense, and is not
> documented in a binding.
>=20
> Mostly CPLD only used to read configuration from. Supposed not been =20
> changed.
The device tree describes the hardware, not the intended use (the =20
partition stuff is not a good example). Plus, any property you use =20
must be documented in a binding.
And please use proper quoting in your e-mail replies. Don't just =20
insert your response as if it were part of what I wrote. You should be =20
able to make your e-mail client do this automatically (if you can't, =20
get a new e-mail client).
-Scott=
prev parent reply other threads:[~2013-07-09 16:09 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-01 4:12 [PATCH 1/2] [PATCH] powerpc/85xx: add P1020RDB-PD platform support Haijun Zhang
2013-07-01 4:12 ` [PATCH 2/2] powerpc/85xx: add the P1020RDB-PD DTS support Haijun Zhang
2013-07-03 19:09 ` Scott Wood
2013-07-04 2:04 ` 答复: " Zhang Haijun-B42677
2013-07-08 17:08 ` Scott Wood
[not found] ` <99E897753B6F7048BD8CCDB4661D02E1393DBA@039-SN2MPN1-021.039d.mgd.msft.net>
2013-07-09 5:34 ` 答复: " Zhang Haijun-B42677
2013-07-09 16:09 ` Scott Wood [this message]
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=1373386162.8183.189@snotra \
--to=scottwood@freescale$(echo .)com \
--cc=B07421@freescale$(echo .)com \
--cc=B42677@freescale$(echo .)com \
--cc=linuxppc-dev@lists$(echo .)ozlabs.org \
--cc=r63061@freescale$(echo .)com \
--cc=r66093@freescale$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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