Re: [PATCH 2/2] of: remove /proc/device-tree

2013-11-16 Thread Geert Uytterhoeven
On Thu, Mar 21, 2013 at 1:36 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-03-21 at 08:16 +, Grant Likely wrote: >> On Thu, Mar 21, 2013 at 7:43 AM, Benjamin Herrenschmidt >> wrote: >> > On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: >> >> > Shouldn't we have the symlink just be a

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-21 Thread Grant Likely
On Thu, Mar 21, 2013 at 12:36 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-03-21 at 08:16 +, Grant Likely wrote: >> On Thu, Mar 21, 2013 at 7:43 AM, Benjamin Herrenschmidt >> wrote: >> > On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: >> >> > Shouldn't we have the symlink just be a

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-21 Thread Benjamin Herrenschmidt
On Thu, 2013-03-21 at 08:16 +, Grant Likely wrote: > On Thu, Mar 21, 2013 at 7:43 AM, Benjamin Herrenschmidt > wrote: > > On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: > >> > Shouldn't we have the symlink just be a config option itself ? > >> > Eventually distros might want get rid of

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-21 Thread Grant Likely
On Thu, Mar 21, 2013 at 7:43 AM, Benjamin Herrenschmidt wrote: > On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: >> > Shouldn't we have the symlink just be a config option itself ? >> > Eventually distros might want get rid of it completely .. >> >> Why? It is the cheapest thing in the worl

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-21 Thread Benjamin Herrenschmidt
On Thu, 2013-03-21 at 07:35 +, Grant Likely wrote: > > Shouldn't we have the symlink just be a config option itself ? > > Eventually distros might want get rid of it completely .. > > Why? It is the cheapest thing in the world and it means the ABI > doesn't change at all. It's also gross and

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-21 Thread Grant Likely
On Thu, Mar 21, 2013 at 4:19 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-03-20 at 21:38 +, Grant Likely wrote: >> > NAK. It should at the very least be a CONFIG option for a while >> before >> > completely switching over. >> >> I'll modify patch 1 to create the symlink if CONFIG_PROC_DEVI

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Benjamin Herrenschmidt
On Wed, 2013-03-20 at 21:38 +, Grant Likely wrote: > > NAK. It should at the very least be a CONFIG option for a while > before > > completely switching over. > > I'll modify patch 1 to create the symlink if CONFIG_PROC_DEVICETREE is > not set. After the first patch can be applied we can leave

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Rob Landley
On 03/20/2013 11:24:54 AM, Daniel Mack wrote: On Wed, Mar 20, 2013 at 4:19 PM, Rob Herring wrote: > On 03/20/2013 09:51 AM, Grant Likely wrote: >> The same data is now available in sysfs, so we can remove the code >> that exports it in /proc and replace it with a symlink to the sysfs >> versio

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Grant Likely
On Wed, Mar 20, 2013 at 2:57 PM, Benjamin Herrenschmidt wrote: > On Wed, 2013-03-20 at 14:51 +, Grant Likely wrote: >> The same data is now available in sysfs, so we can remove the code >> that exports it in /proc and replace it with a symlink to the sysfs >> version. >> >> Tested on versatile

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Grant Likely
On Wed, Mar 20, 2013 at 4:24 PM, Daniel Mack wrote: > On Wed, Mar 20, 2013 at 4:19 PM, Rob Herring wrote: >> On 03/20/2013 09:51 AM, Grant Likely wrote: >>> The same data is now available in sysfs, so we can remove the code >>> that exports it in /proc and replace it with a symlink to the sysfs >

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Daniel Mack
On Wed, Mar 20, 2013 at 4:19 PM, Rob Herring wrote: > On 03/20/2013 09:51 AM, Grant Likely wrote: >> The same data is now available in sysfs, so we can remove the code >> that exports it in /proc and replace it with a symlink to the sysfs >> version. >> >> Tested on versatile qemu model and mpc520

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Benjamin Herrenschmidt
On Wed, 2013-03-20 at 14:51 +, Grant Likely wrote: > The same data is now available in sysfs, so we can remove the code > that exports it in /proc and replace it with a symlink to the sysfs > version. > > Tested on versatile qemu model and mpc5200 eval board. More testing > would be appreciate

Re: [PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Rob Herring
On 03/20/2013 09:51 AM, Grant Likely wrote: > The same data is now available in sysfs, so we can remove the code > that exports it in /proc and replace it with a symlink to the sysfs > version. > > Tested on versatile qemu model and mpc5200 eval board. More testing > would be appreciated. I would

[PATCH 2/2] of: remove /proc/device-tree

2013-03-20 Thread Grant Likely
The same data is now available in sysfs, so we can remove the code that exports it in /proc and replace it with a symlink to the sysfs version. Tested on versatile qemu model and mpc5200 eval board. More testing would be appreciated. Signed-off-by: Grant Likely Cc: Rob Herring Cc: Greg Kroah-Ha