On Wed, Oct 24, 2007 at 01:13:33AM +0200, Marian Balakowicz wrote:
> Add device tree source file for TQM5200 board.
>
> Signed-off-by: Marian Balakowicz <[EMAIL PROTECTED]>
> ---
>
> arch/powerpc/boot/dts/tqm5200.dts | 236
> +++++++++++++++++++++++++++++++++++++
> 1 files changed, 236 insertions(+), 0 deletions(-)
> create mode 100644 arch/powerpc/boot/dts/tqm5200.dts
>
>
> diff --git a/arch/powerpc/boot/dts/tqm5200.dts
> b/arch/powerpc/boot/dts/tqm5200.dts
> new file mode 100644
> index 0000000..01c7778
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/tqm5200.dts
> @@ -0,0 +1,236 @@
> +/*
> + * TQM5200 board Device Tree Source
[snip]
> + [EMAIL PROTECTED] {
I thought we were moving towards calling these just /[EMAIL PROTECTED]
> + model = "fsl,mpc5200";
> + compatible = "mpc5200";
That should have a vendor prefix.
[snip]
> + [EMAIL PROTECTED] { // PSC1
> + device_type = "serial";
> + compatible = "mpc5200-psc-uart";
> + port-number = <0>; // Logical port assignment
How are these port-number things used? The device tree shouldn't
generally contain information that isn't inherent to the hardware.
There can be reasons for hacks like this, but we should avoid them if
possible.
> + cell-index = <0>;
cell-index should only be used if the index number is used when
manipulating the hardware (e.g. if there's a global control register
which takes this number).
[snip]
> + [EMAIL PROTECTED] {
> + device_type = "ata";
No such thing as device_type = "ata", drop it. In general, never
include a device_type unless a binding explicitly says to do so.
[snip]
> + [EMAIL PROTECTED] {
> + model = "fsl,lpb";
> + compatible = "lpb";
Not nearly specific enough. Must include a vendor prefix at least,
and should have a lot more revision information. You should always be
able to pick the right driver with compatible alone, "model" should
generally be for human consumption, the driver shouldn't need it.
> + device_type = "lpb";
Drop this. Again, presence of a device_type property is the
exception, not the rule.
> + ranges = <0 fc000000 02000000>;
You need #address-cells and #size-cells properties, too.
[snip]
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
