[microblaze-uclinux] Re: [microblaze-uclinux] RE: [PATCH v3] Device tree bindings for Xilinx devices

[Stephen Neuendorffer]

> -----Original Message-----
> From: owner-microblaze-uclinux at itee.uq.edu.au 
> [mailto:owner-microblaze-uclinux at itee.uq.edu.au] On Behalf Of 
> Michal Simek
> Sent: Friday, October 19, 2007 7:28 PM
> To: microblaze-uclinux at itee.uq.edu.au
> Cc: Stephen Neuendorffer; Grant Likely; Leonid; Arnd 
> Bergmann; linuxppc-dev at ozlabs.org; Wolfgang Reissnegger
> Subject: [microblaze-uclinux] Re: [microblaze-uclinux] RE: 
> [PATCH v3] Device tree bindings for Xilinx devices
> 
> Hi Steve and all,
> >Here's a full .dts generated using an updated version of
> >gen_mhs_devtree.py, following the proposal.
> >It happens to be a microblaze system, but you get the idea.
> 
> I think that is no good idea generate dts with all information.
> Especially information about PVR - number 2 means - Full PVR 
> and you can
> obtain information directly from PVR. It is waste of memory space.
> 			xilinx,pvr = <2>;

PowerPC does something with the powerpc equivalent of the PVR.
We should just do what they do...
 
> In my opinion will be better generate only parameters which 
> you want not all.
> That smells with unusable parameters.

In the long term, this may be true.  In the short term:
1) dtb size is not the key problem
2) making sure that everything works is a key problem.
3) The code that generates the dts should be as simple as possible,
so that we can easily document what it does.

In the long term, I'm all for optimizing the device tree that gets
built,
assuming that it appears to be a problem in real systems.

Steve