From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail50-dub-R.bigfish.com (mail-dub.bigfish.com [213.199.154.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "*.bigfish.com", Issuer "*.bigfish.com" (not verified)) by ozlabs.org (Postfix) with ESMTP id 1C840DDE0E for ; Tue, 18 Dec 2007 05:25:10 +1100 (EST) MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----_=_NextPart_001_01C840DA.208E366C" Subject: RE: [PATCH 7/7] [POWERPC] Xilinx: Update booting-without-of. Date: Mon, 17 Dec 2007 10:24:57 -0800 References: <1197589413-5965-1-git-send-email-stephen.neuendorffer@xilinx.com><1197589413-5965-2-git-send-email-stephen.neuendorffer@xilinx.com><1197589413-5965-3-git-send-email-stephen.neuendorffer@xilinx.com><1197589413-5965-4-git-send-email-stephen.neuendorffer@xilinx.com><1197589413-5965-5-git-send-email-stephen.neuendorffer@xilinx.com><1197589413-5965-6-git-send-email-stephen.neuendorffer@xilinx.com><20071213234151.D0B3DE80061@mail20-sin.bigfish.com> From: "Stephen Neuendorffer" To: "Grant Likely" Message-Id: <20071217182459.7972AAE804F@mail50-dub.bigfish.com> Cc: linuxppc-dev@ozlabs.org, David Gibson , simekm2@fel.cvut.cz List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , This is a multi-part message in MIME format. ------_=_NextPart_001_01C840DA.208E366C Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable When the driver no longer requires the port number, it's easy to drop. = Until then, I'll keep it in. =20 Also, I'm not so sure that moving to completely generic names is really = worth the effort... All the 'semantically' interesting' information is = already in the device tree somewhere else. In the limit, the node name = could just be a randomly generated string. So now we have a matter of = taste: what is the right amount of detail to put in so that someone who = looks at the tree can easily understand what's going on, but not be = overwhelmed? The xilinx ip name seems to usually do that almost as well = as a 'generic name'. Anyway, you proved me wrong last time after a = bunch of mulling it over, so maybe I'll just take your word for it and = do it that way. :) =20 In other news, my computer seems to have died this morning, so = productivity may be low. :) =20 Steve ________________________________ From: glikely@secretlab.ca on behalf of Grant Likely Sent: Mon 12/17/2007 7:19 AM To: Stephen Neuendorffer Cc: simekm2@fel.cvut.cz; jwilliams@itee.uq.edu.au; = linuxppc-dev@ozlabs.org; David Gibson Subject: Re: [PATCH 7/7] [POWERPC] Xilinx: Update booting-without-of. > reg =3D ; > }; > }; > @@ -2513,6 +2521,9 @@ platforms are moved over to use the = flattened-device-tree model. > > Requred properties: > - current-speed : Baud rate of uartlite > + Optional properties: > + - port-number : unique ordinal index of the device. This > + property is required for a console on uartlite. And has already been discussed, drop the port-number property. I'll rework the uartlite driver to use aliases instead. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. grant.likely@secretlab.ca (403) 399-0195 ------_=_NextPart_001_01C840DA.208E366C Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Re: [PATCH 7/7] [POWERPC] Xilinx: Update = booting-without-of.=0A= =0A= =0A= =0A=
=0A=
When the = driver no longer requires the port number, it's easy to drop.  = Until then, I'll keep it in.
=0A=
 
=0A=
Also, I'm not so sure that = moving to completely generic names is really worth the effort...  = All the 'semantically' interesting' information is already in the device = tree somewhere else.  In the limit, the node name could = just be a randomly generated string.  So now we have a matter = of taste: what is the right amount of detail to put in so that someone = who looks at the tree can easily understand what's going on, but not be = overwhelmed?  The xilinx ip name seems to usually do that almost as = well as a 'generic name'.  Anyway, you proved me wrong last time = after a bunch of mulling it over, so maybe I'll just take your word for = it and do it that way. :)
=0A=
 
=0A=
In other news, my computer = seems to have died this morning, so productivity may be low. = :)
=0A=
 
=0A=
Steve
=0A=

=0A=
=0A=
=0A=
=0A=
From: = glikely@secretlab.ca on behalf of Grant Likely
Sent: Mon = 12/17/2007 7:19 AM
To: Stephen Neuendorffer
Cc: = simekm2@fel.cvut.cz; jwilliams@itee.uq.edu.au; linuxppc-dev@ozlabs.org; = David Gibson
Subject: Re: [PATCH 7/7] [POWERPC] Xilinx: Update = booting-without-of.
=0A=

>         &n= bsp;           &nb= sp;           reg =3D = <d1000fc0 = 20>;
>         &nbs= p;            = ;   = };
>          &nb= sp;      };
> @@ -2513,6 +2521,9 @@ = platforms are moved over to use the flattened-device-tree = model.
>
>        Requred = properties:
>         - = current-speed : Baud rate of uartlite
> = +            =    Optional properties:
> = +       - port-number : unique ordinal = index of the device. This
> = +         property is required = for a console on uartlite.

And has already been discussed, drop = the port-number property.  I'll
rework the uartlite driver to = use aliases instead.

Cheers,
g.

--
Grant Likely, = B.Sc., P.Eng.
Secret Lab Technologies = Ltd.
grant.likely@secretlab.ca
(403) = 399-0195

------_=_NextPart_001_01C840DA.208E366C--