答复: 答复: [PATCH 2/2] powerpc/85xx: add the P1020RDB-PD DTS support

[Scott Wood]

On 07/09/2013 12:34:34 AM, Zhang Haijun-B42677 wrote:
> > cpld at 2,0 {
> >           compatible = "fsl, p1020rdb-cpld";
> >           reg = <0x2 0x0 0x20000>;
> >           read-only;
> > };
> 
> Again, what is read-only supposed to mean here?  I don't care that  
> it's
> copied from the PC version.  It doesn't make sense, and is not
> documented in a binding.
> 
> Mostly CPLD only used to read configuration from. Supposed not been  
> changed.

The device tree describes the hardware, not the intended use (the  
partition stuff is not a good example).  Plus, any property you use  
must be documented in a binding.

And please use proper quoting in your e-mail replies.  Don't just  
insert your response as if it were part of what I wrote.  You should be  
able to make your e-mail client do this automatically (if you can't,  
get a new e-mail client).

-Scott