Re: [PATCH] [POWERPC] Optimize counting distinct entries in the relocation sections

2007-11-11 Thread Paul Mackerras
Emil Medve writes: > (Not sure why the relocation tables could contain lots of duplicates and why > they are not trimmed at compile time by the linker. In some test cases, out of > 35K relocation entries only 1.5K were distinct/unique) Presumably you have lots of calls to the same function, or lo

Re: [PATCH] [POWERPC] Silence an annoying boot message

2007-11-11 Thread Olof Johansson
Hi Stephen, On Mon, Nov 12, 2007 at 01:53:39PM +1100, Stephen Rothwell wrote: > vmemmap_populate will printk (with KERN_WARNING) for a lot of pages > if CONFIG_SPARSEMEM_VMEMMAP is enabled (at least it does on iSeries). > Turn it into a DEBUG message. [...] > diff --git a/arch/powerpc/mm/init_64.c

of_iomap confusion

2007-11-11 Thread Benjamin Herrenschmidt
I just noticed this of_iomap() helper thing we have in prom_parse.c nowadays. It's a bit confusing as "iomap" is generally what we use to name the new iomap interface (ioport_map, pci_iomap, ...) and that requires -different- accessors than memory obtained from ioremap. This I think this needs to

[PATCH 3/4] Use embedded libfdt in the bootwrapper

2007-11-11 Thread David Gibson
This patch incorporates libfdt (from the source embedded in an earlier patch) into the wrapper.a library used by the bootwrapper. This includes adding a libfdt_env.h file, which the libfdt sources need in order to integrate into the bootwrapper environment, and a libfdt-wrapper.c which provides gl

[PATCH 4/4] Kill flatdevtree.c

2007-11-11 Thread David Gibson
Now that earlier patches have switched the bootwrapper to using libfdt for device tree manipulation, this patch removes the now unused flatdevtree.c and related files. Signed-off-by: David Gibson <[EMAIL PROTECTED]> --- arch/powerpc/boot/Makefile |2 arch/powerpc/boot/flatdevtree.

[PATCH 1/4] Merge dtc and libfdt upstream source

2007-11-11 Thread David Gibson
This very large patch incorporates a copy of dtc (including libfdt) into the kernel source, in arch/powerpc/boot/dtc-src. This patch only imports the upstream sources verbatim, later patches are needed to actually link it into the kernel Makefiles and use the embedded code during the kernel build.

[PATCH 2/4] Use embedded dtc in kernel builds

2007-11-11 Thread David Gibson
This patch alters the kernel makefiles to build dtc from the sources embedded in the previous patch. It also changes the arch/powerpc/boot/wrapper script to use the embedded dtc, rather than expecting a copy of dtc already installed on the system. Signed-off-by: David Gibson <[EMAIL PROTECTED]>

[0/4] Embed dtc and libfdt in the kernel (spin the third)

2007-11-11 Thread David Gibson
Another spin of my patches to to embed and use dtc in the kernel source, rather than relying on it as an external dependency, and to use libfdt in favour of flatdevtree.c in the bootwrapper. Only two changes from the last version: - trivial changes to make the patches apply to the current

RE: [PATCH 0/5 v3] Porting RapidIO driver from ppc to powerpc architecture and adding memory mapped RapidIO driver.

2007-11-11 Thread Zhang Wei-r63237
Hi, Randy, I'll post it soon and hope to get much feedback from you. Thanks! Wei. > -Original Message- > From: Randy Vinson [mailto:[EMAIL PROTECTED] > > Zhang Wei-r63237 wrote: > > Yes, I'm working on it. Do not worry about it. > How's this going? I've been working on this a bit myse

[PATCH] Demote clockevent printk to KERN_DEBUG.

2007-11-11 Thread Tony Breeds
These don't need to be seen by eveyone on every boot. Signed-off-by: Tony Breeds <[EMAIL PROTECTED]> --- arch/powerpc/kernel/time.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/kernel/time.c b/arch/powerpc/kernel/time.c index 4beb632..c0d7772 100644 --- a

Re: [PATCH] DTC: Polish up the DTS Version 1 implementation.

2007-11-11 Thread David Gibson
On Fri, Nov 09, 2007 at 08:32:57AM -0600, Jon Loeliger wrote: > So, like, the other day David Gibson mumbled: > > > > But you do take a hit w.r.t. *minimum* representation size - there's > > no form amongst all the possibilities here more compact than pure hex. > > Especially since spaces are opti

[PATCH] [POWERPC] Silence an annoying boot message

2007-11-11 Thread Stephen Rothwell
vmemmap_populate will printk (with KERN_WARNING) for a lot of pages if CONFIG_SPARSEMEM_VMEMMAP is enabled (at least it does on iSeries). Turn it into a DEBUG message. Signed-off-by: Stephen Rothwell <[EMAIL PROTECTED]> --- arch/powerpc/mm/init_64.c |2 +- 1 files changed, 1 insertions(+), 1

Re: Do not depend on MAX_ORDER when grouping pages by mobility

2007-11-11 Thread Stephen Rothwell
Hi Mel, I discovered recently that a kernel built with ppc64_defconfig no longer boots on legacy iSeries. It did in 2.6.23. I bisected down the commit d9c2340052278d8eb2ffb16b0484f8f794def4de ("Do not depend on MAX_ORDER when grouping pages by mobility") which fails while its parent is ok. Also,

Re: Gianfar ethernet device

2007-11-11 Thread Benjamin Herrenschmidt
On Sun, 2007-11-11 at 20:16 -0500, Jon Smirl wrote: > > Are there powerpc platforms without device trees? Not in arch/powerpc. Ben. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[RFC] Disabling platform bus and using only of_platform bus

2007-11-11 Thread Jon Smirl
Disabling platform bus and using only of_platform bus works for my mpc5200 system. Attached is a first pass at a patch implementing of_platform bus only. Suggestions on the correct ways to do the things needed? alsa soc in kernel uses platform bus, asoc v2 is under development and it removes asoc

Re: Gianfar ethernet device

2007-11-11 Thread Jon Smirl
On 11/11/07, Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > > The real solution is that gianfar support belongs in a device driver, > > not in a common file. That whole fsl_soc.c file is a catch-all of > > things that belong in device drivers. I haven't looked at every line > > in it, but 90

Re: Gianfar ethernet device

2007-11-11 Thread Benjamin Herrenschmidt
> The real solution is that gianfar support belongs in a device driver, > not in a common file. That whole fsl_soc.c file is a catch-all of > things that belong in device drivers. I haven't looked at every line > in it, but 90%+ of the code should be moved into device drivers. > > I'm preparing a

Re: pcspkr device, pnpPNP,100

2007-11-11 Thread Benjamin Herrenschmidt
On Sun, 2007-11-11 at 19:18 -0500, Jon Smirl wrote: > Using this scheme, which platforms should select the pcspkr hardware? Run a poll :-) I suppose at least chrp/prep/pegasos Ben. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlab

Re: Gianfar ethernet device

2007-11-11 Thread Jon Smirl
BTW, this exercise of disabling platform bus is turning up some real bugs. There are several tests in the kernel like this: if (!shost->shost_gendev.parent) shost->shost_gendev.parent = dev ? dev : &platform_bus; if (adap->dev.parent == NULL) { ada

Re: Gianfar ethernet device

2007-11-11 Thread Jon Smirl
On 11/11/07, Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > On Sun, 2007-11-11 at 18:30 -0500, Jon Smirl wrote: > > Does this patch add the right ifdefs in fsl_soc.c to make these > > drivers build on their proper platforms? As an experiment I have > > disabled platform bus on the mpc5200 an

Re: pcspkr device, pnpPNP,100

2007-11-11 Thread Jon Smirl
On 11/11/07, Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > On Sun, 2007-11-11 at 19:07 -0500, Jon Smirl wrote: > > > > This code should be in a device driver so that it can be selected with > > Kconfig. Can we #ifdef it now as a hint? I could make it a device > > driver but I don't have any

Re: pcspkr device, pnpPNP,100

2007-11-11 Thread Benjamin Herrenschmidt
On Sun, 2007-11-11 at 19:07 -0500, Jon Smirl wrote: > > This code should be in a device driver so that it can be selected with > Kconfig. Can we #ifdef it now as a hint? I could make it a device > driver but I don't have any hardware to test it on. > > I'm trying to get the mpc5200 kernel to bui

Re: pcspkr device, pnpPNP,100

2007-11-11 Thread Jon Smirl
On 11/11/07, Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > On Sun, 2007-11-11 at 18:35 -0500, Jon Smirl wrote: > > Which platform does pcspkr device, pnpPNP,100 belong to, amiga? I'd > > like to IFDEF add_pcspkr(void) in setup-common.c for the right > > platform. > > #ifdef is evil. What ab

Re: Gianfar ethernet device

2007-11-11 Thread Benjamin Herrenschmidt
On Sun, 2007-11-11 at 18:30 -0500, Jon Smirl wrote: > Does this patch add the right ifdefs in fsl_soc.c to make these > drivers build on their proper platforms? As an experiment I have > disabled platform bus on the mpc5200 and I'm only using of_platform > bus. Turning off platform bus is exposing

Re: pcspkr device, pnpPNP,100

2007-11-11 Thread Benjamin Herrenschmidt
On Sun, 2007-11-11 at 18:35 -0500, Jon Smirl wrote: > Which platform does pcspkr device, pnpPNP,100 belong to, amiga? I'd > like to IFDEF add_pcspkr(void) in setup-common.c for the right > platform. #ifdef is evil. What about multiplatform kernels ? Just test for the presence of the device in the

pcspkr device, pnpPNP,100

2007-11-11 Thread Jon Smirl
Which platform does pcspkr device, pnpPNP,100 belong to, amiga? I'd like to IFDEF add_pcspkr(void) in setup-common.c for the right platform. -- Jon Smirl [EMAIL PROTECTED] ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/

Re: Gianfar ethernet device

2007-11-11 Thread Jon Smirl
Does this patch add the right ifdefs in fsl_soc.c to make these drivers build on their proper platforms? As an experiment I have disabled platform bus on the mpc5200 and I'm only using of_platform bus. Turning off platform bus is exposing a lot of code that is getting built into my mpc5200 kernel t

Gianfar ethernet device

2007-11-11 Thread Jon Smirl
Which platforms have the Gianfar ethernet device? The gfar code in fsl_soc.c is getting built on the mpc5200 and it doesn't have the device. -- Jon Smirl [EMAIL PROTECTED] ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/

Re: [RFC] powermac: proper sleep management

2007-11-11 Thread Benjamin Herrenschmidt
> Just thought I'd send out the patch again, it has changed quite a bit > since last time because I cleaned up the code, made it depend on > CONFIG_SUSPEND, made the ioctl backward compatibility optional and some > other bits. > > The feared freezer vs. fuse sync deadlock is no longer present in

Re: [RFC] PMU: replace information ioctls and schedule for removal

2007-11-11 Thread Benjamin Herrenschmidt
On Thu, 2007-11-08 at 13:02 +0100, Johannes Berg wrote: > This patch adds sysfs attributes to the PMU to allow getting > the information on the PMU type and whether adb is present from > there instead of via the ioctl. The ioctl is made optional and > scheduled for removal. Acked-by: Benjamin He

Re: [RFC] PMU: don't lock_kernel()

2007-11-11 Thread Benjamin Herrenschmidt
On Thu, 2007-11-08 at 13:01 +0100, Johannes Berg wrote: > I see nothing that this lock_kernel() actually protects against > so remove it. > > Signed-off-by: Johannes Berg <[EMAIL PROTECTED]> Acked-by: Benjamin Herrenschmidt <[EMAIL PROTECTED]> > --- > drivers/macintosh/via-pmu.c |3 --- >

Re: [RFC] PMU: remove dead code

2007-11-11 Thread Benjamin Herrenschmidt
On Thu, 2007-11-08 at 13:00 +0100, Johannes Berg wrote: > Some code in via-pmu.c is never compiled because of "compile options" > within the file. Remove the code completely. > > Signed-off-by: Johannes Berg <[EMAIL PROTECTED]> Acked-by: Benjamin Herrenschmidt <[EMAIL PROTECTED]> > --- > drive

Re: IRQs in i2c-mpc.c

2007-11-11 Thread Benjamin Herrenschmidt
> Is the only error return from irq_of_parse_and_map() NO_IRQ? or can we > get standard negative returns? I started tracing out the returns from > irq_of_parse_and_map() but there are a lot of them. NP_IRQ is the only error return. Ben. ___ Linuxppc-

Re: [linux-pm] [RFC] powermac: proper sleep management

2007-11-11 Thread Pavel Machek
Hi! > This adds platform_suspend_ops for PMU based machines, directly in > the PMU driver. This finally allows suspending via /sys/power/state > on powerbooks. Thanks for doing this! -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/p

Re: IRQs in i2c-mpc.c

2007-11-11 Thread Jon Smirl
On 11/11/07, Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > > irq_of_parse_and_map() returns NO_IRQ when the irq parameter is missing. > > > > This API appears to be broken. In asm-powerpc/irq.h NO_IRQ is defined > > as (0). There is no way to tell an error in the attribute from a valid > >