Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Grant Likely
On Tue, Mar 23, 2010 at 3:42 PM, Timur Tabi ti...@freescale.com wrote: Define a binding for embedding a QE firmware blob into the device tree.  Also define a new property for the QE node that links to a firmware node. Signed-off-by: Timur Tabi ti...@freescale.com ---  

[PATCH 1/2] mm/vmalloc: Export purge_vmap_area_lazy()

2010-03-24 Thread Benjamin Herrenschmidt
Some powerpc code needs to ensure that all previous iounmap/vunmap has really been flushed out of the MMU hash table. Without that, various hotplug operations may fail when trying to return those pieces to the hypervisor due to existing active mappings. This exports purge_vmap_area_lazy() to

[PATCH 2/2] powerpc/pseries: Flush lazy kernel mappings after unplug operations

2010-03-24 Thread Benjamin Herrenschmidt
This ensures that the translations for unmapped IO mappings or unmapped memory are properly removed from the MMU hash table before such an unplug. Without this, the hypervisor refuses the unplug operations due to those resources still being mapped by the partition. Signed-off-by: Benjamin

Re: Linux patches for XIP on MPC8xx?

2010-03-24 Thread Benjamin Herrenschmidt
On Tue, 2010-03-23 at 20:44 +0100, Wolfgang Denk wrote: There were very few tests done with later kernel versions, and in all practical system configurations we tested we found that XIP did not give real benefits. Usually execution from flash was slower than when running from RAM, and even

Re: Linux patches for XIP on MPC8xx?

2010-03-24 Thread Vitaly Wool
Hi, There were very few tests done with later kernel versions, and in all practical system configurations we tested we found that XIP did not give real benefits. Usually execution from flash was slower than when running from RAM, and even booting a (inevitably uncompressed) kernel from flash

Booting headless PowerMac

2010-03-24 Thread Romain Goyet
Most people have had ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Booting headless PowerMac G5

2010-03-24 Thread Romain Goyet
Hi there, Here's a summary about how to boot a PowerMac G5 without a screen attached. As many people have noticed, default yaboot install won't boot unless a screen is attached. Actually, the workaround is really simple. Thing is, that's the ofboot.b script that messes things up. This

Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove

2010-03-24 Thread Benjamin Herrenschmidt
On Sun, 2010-03-21 at 08:04 +1100, Benjamin Herrenschmidt wrote: On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: From: Amit Shah amit.s...@redhat.com Alan pointed out a race in the code where hvc_remove is invoked. The recent virtio_console work is the first user of

Re: Booting headless PowerMac G5

2010-03-24 Thread Benjamin Herrenschmidt
On Wed, 2010-03-24 at 11:32 +0100, Romain Goyet wrote: Hi there, Here's a summary about how to boot a PowerMac G5 without a screen attached. As many people have noticed, default yaboot install won't boot unless a screen is attached. CC'ing Tony who is the latest victim of yaboot

Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove

2010-03-24 Thread Amit Shah
On (Wed) Mar 24 2010 [21:45:44], Benjamin Herrenschmidt wrote: On Sun, 2010-03-21 at 08:04 +1100, Benjamin Herrenschmidt wrote: On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: From: Amit Shah amit.s...@redhat.com Alan pointed out a race in the code where hvc_remove is

Re: [PATCH 2/4] panic: Allow taint flag for warnings to be changed from TAINT_WARN

2010-03-24 Thread Michael Ellerman
On Sat, 2010-03-20 at 23:05 +, Ben Hutchings wrote: WARN() is used in some places to report firmware or hardware bugs that are then worked-around. These bugs do not affect the stability of the kernel and should not set the usual TAINT_WARN flag. To allow for this, add WARN_TAINT() and

Re: Linuxppc-dev Digest, Vol 67, Issue 131

2010-03-24 Thread Heiko Schocher
Hello Daniel, Date: Wed, 24 Mar 2010 13:33:48 +1100 From: Daniel Ng daniel.ng1...@gmail.com To: linuxppc-...@ozlabs.org Subject: SPI Driver for CPM2 8260 (8272 family) Message-ID: 547eba1b1003231933x738b2660u364b12d43bf9...@mail.gmail.com Content-Type: text/plain; charset=ISO-8859-1

Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove

2010-03-24 Thread Alan Cox
On Sun, 21 Mar 2010 08:04:39 +1100 Benjamin Herrenschmidt b...@kernel.crashing.org wrote: On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: From: Amit Shah amit.s...@redhat.com Alan pointed out a race in the code where hvc_remove is invoked. The recent virtio_console work is

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Timur Tabi
On Wed, Mar 24, 2010 at 1:07 AM, Grant Likely grant.lik...@secretlab.ca wrote: +- fsl,firmware-phandle: +    Usage: required +    Value type: phandle +    Definition: Points to a firmware node (see QE Firmware Node below) +        that contains the firmware that should be uploaded for this

Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove

2010-03-24 Thread Amit Shah
On (Sun) Mar 21 2010 [10:07:25], Amit Shah wrote: On (Sun) Mar 21 2010 [08:04:39], Benjamin Herrenschmidt wrote: On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: From: Amit Shah amit.s...@redhat.com Alan pointed out a race in the code where hvc_remove is invoked. The

Re: [PATCH 01/10] irq: move some interrupt arch_* functions into struct irq_chip.

2010-03-24 Thread Ian Campbell
On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote: On Sun, 21 Mar 2010, Yinghai Lu wrote: From: Ian Campbell ian.campb...@citrix.com Move arch_init_copy_chip_data and arch_free_chip_data into function pointers in struct irq_chip since they operate on irq_desc-chip_data. Not

Re: [PATCH 01/10] irq: move some interrupt arch_* functions into struct irq_chip.

2010-03-24 Thread Ian Campbell
On Tue, 2010-03-23 at 07:10 +, Paul Mundt wrote: The function pointer thing itself is also a bit unorthodox to say the least. You're introducing and passing around an opaque type just so you can get to a 'return 0' in the xen case as far as I can tell, The ultimate aim is to have Xen use

Re: [PATCH 4/7] hvc_console: Fix race between hvc_close and hvc_remove

2010-03-24 Thread Amit Shah
On (Wed) Mar 24 2010 [11:37:21], Alan Cox wrote: On Sun, 21 Mar 2010 08:04:39 +1100 Benjamin Herrenschmidt b...@kernel.crashing.org wrote: On Fri, 2010-03-19 at 08:18 -0700, Greg Kroah-Hartman wrote: From: Amit Shah amit.s...@redhat.com Alan pointed out a race in the code where

Problem with multiple hvc consoles via virtio-console

2010-03-24 Thread Amit Shah
Hello, When multiple hvc console ports are initialised and used via the virtio_console driver, I can interact with only the first console, the 2nd one doesn't respond. If I call hvc_kick() even if hvc_poll() returns 0, the 2nd console becomes responsive. I've checked that hvc_poll() receives

Re: Booting headless PowerMac G5

2010-03-24 Thread Segher Boessenkool
The root of the problem is that Apple OF will not open an stdout for you on some machines if you don't enter OF manually, so yaboot text never shows up. This was somewhat worked around by the ofboot.b script but indeed, it seems to fail to check that there's a display to open before actually

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Segher Boessenkool
Why the phandle redirection? Why not just put the firmware blob into a property in the QE node, or as a subnode? Because there might be multiple QE devices on a single chip, and each will need to upload the same firmware. So instead of embedding the firmware multiple times, just embed it

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Grant Likely
On Wed, Mar 24, 2010 at 11:00 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: Why the phandle redirection?  Why not just put the firmware blob into a property in the QE node, or as a subnode? Because there might be multiple QE devices on a single chip, and each will need to upload

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Timur Tabi
Grant Likely wrote: On Wed, Mar 24, 2010 at 11:00 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: Why the phandle redirection? Why not just put the firmware blob into a property in the QE node, or as a subnode? Because there might be multiple QE devices on a single chip, and each

Re: [PATCH 01/10] irq: move some interrupt arch_* functions into struct irq_chip.

2010-03-24 Thread Thomas Gleixner
On Wed, 24 Mar 2010, Ian Campbell wrote: On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote: On Sun, 21 Mar 2010, Yinghai Lu wrote: From: Ian Campbell ian.campb...@citrix.com Move arch_init_copy_chip_data and arch_free_chip_data into function pointers in struct irq_chip

[PATCH 00/09] net: remove trailing spaces in messages

2010-03-24 Thread Frans Pop
This is the second patchset to remove trailing spaces in kernel messages. Patches in this set cover networking code, excepting wireless which will be submitted in a separate set. Patches have been rebased against current net-next. Benefits are: - general cleanup and consistency - minor

[PATCH 4/9] net/ps3_gelic: remove trailing space in messages

2010-03-24 Thread Frans Pop
ps3_gelic_wireless.c: also remove a stray p after a newline. Signed-off-by: Frans Pop elen...@planet.nl Cc: Benjamin Herrenschmidt b...@kernel.crashing.org Cc: linuxppc-...@ozlabs.org --- drivers/net/ps3_gelic_net.c |4 ++-- drivers/net/ps3_gelic_wireless.c | 34

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Grant Likely
On Wed, Mar 24, 2010 at 11:31 AM, Timur Tabi ti...@freescale.com wrote: Grant Likely wrote: On Wed, Mar 24, 2010 at 11:00 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: Why the phandle redirection?  Why not just put the firmware blob into a property in the QE node, or as a subnode?

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Mitch Bradley
Most firmware is 8-12KB, so this will make for one ugly DTS. Plus, there's the issue of distributing non-GPL firmware data inside a DTS, which is GPL. You've got the distribution problem that needs to be solved regardless because it cannot be part of U-Boot either. How do you plan to

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Timur Tabi
Grant Likely wrote: Thanks. This is important information when talking about his. You can also put it into an initrd and use the Linux firmware loader. We may also need to support non-Linux operating systems that don't use an initrd. For now, an initrd might work. I don't know how I'll

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Scott Wood
On Wed, Mar 24, 2010 at 11:07:42AM -0600, Grant Likely wrote: On Wed, Mar 24, 2010 at 11:00 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: Why the phandle redirection?  Why not just put the firmware blob into a property in the QE node, or as a subnode? Because there might be

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Timur Tabi
M. Warner Losh wrote: Is there some reason that the firmware couldn't be loaded by uboot as part of its initialization? Unfortunately, our hardware is frequently designed such that the same firmware needs to be loaded by U-Boot and then again by the kernel. -- Timur Tabi Linux kernel

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread M. Warner Losh
In message: 4baa4c8a.70...@freescale.com Timur Tabi ti...@freescale.com writes: : Most firmware is 8-12KB, so this will make for one ugly DTS. Plus, : there's the issue of distributing non-GPL firmware data inside a : DTS, which is GPL. Is there some reason that the firmware

Re: [PATCH 01/10] irq: move some interrupt arch_* functions into struct irq_chip.

2010-03-24 Thread Ian Campbell
On Wed, 2010-03-24 at 17:44 +, Thomas Gleixner wrote: On Wed, 24 Mar 2010, Ian Campbell wrote: On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote: On Sun, 21 Mar 2010, Yinghai Lu wrote: From: Ian Campbell ian.campb...@citrix.com Move arch_init_copy_chip_data and

Re: [PATCH 00/09] net: remove trailing spaces in messages

2010-03-24 Thread David Miller
From: Frans Pop elen...@planet.nl Date: Wed, 24 Mar 2010 18:40:15 +0100 This is the second patchset to remove trailing spaces in kernel messages. Patches in this set cover networking code, excepting wireless which will be submitted in a separate set. Patches have been rebased against

Re: [PATCH 01/10] irq: move some interrupt arch_* functions into struct irq_chip.

2010-03-24 Thread Thomas Gleixner
On Wed, 24 Mar 2010, Ian Campbell wrote: On Wed, 2010-03-24 at 17:44 +, Thomas Gleixner wrote: On Wed, 24 Mar 2010, Ian Campbell wrote: On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote: On Sun, 21 Mar 2010, Yinghai Lu wrote: From: Ian Campbell

Re: [PATCH 00/09] net: remove trailing spaces in messages

2010-03-24 Thread Frans Pop
On Wednesday 24 March 2010, David Miller wrote: All applied, but I have to make some fixups. For example, GIT warns about trailing whitespace on any lines that you touched so I had to fix those up before putting them into the tree. I did wonder about fixing that or not. But as the surrounding

Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware

2010-03-24 Thread Segher Boessenkool
Why the phandle redirection? Why not just put the firmware blob into a property in the QE node, or as a subnode? Because there might be multiple QE devices on a single chip, and each will need to upload the same firmware. So instead of embedding the firmware multiple times, just embed it

Re: [tpmdd-devel] [PATCH] TPM: Provide a tpm_tis OF driver

2010-03-24 Thread Jason Gunthorpe
On Thu, Mar 25, 2010 at 01:20:41AM -0300, Rajiv Andrade wrote: Why not +#ifdef CONFIG_OF + return of_register_platform_driver(tis_of_driver); +#elif CONFIG_PNP + return pnp_register_driver(tis_pnp_driver); +#endif ? AFAIK they are not exclusive options? I can't imagine anyone

Re: [PATCH 1/2] mm/vmalloc: Export purge_vmap_area_lazy()

2010-03-24 Thread Nick Piggin
On Wed, Mar 24, 2010 at 06:56:31PM +1100, Benjamin Herrenschmidt wrote: Some powerpc code needs to ensure that all previous iounmap/vunmap has really been flushed out of the MMU hash table. Without that, various hotplug operations may fail when trying to return those pieces to the hypervisor