Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread David Miller
From: Paul Mackerras <[EMAIL PROTECTED]> Date: Fri, 29 Aug 2008 15:40:36 +1000 > The main remaining substantial technical issue is how we detect very > early on that we are a kdump kernel. I think the policy should be > that the kernel copies itself down to 0 if it's not a kdump kernel and > runs

Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread Paul Mackerras
Kumar Gala writes: > Also, can we get this on ppc32 (head_32.S)? Probably. I'll give it a go. Paul. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread Paul Mackerras
David Woodhouse writes: > Is this mature enough for us to consider putting it in Fedora? We'd > _love_ to stop building a separate kdump kernel for ppc64... Almost, but not quite. We'll need some modifications to yaboot (it refuses to accept an ET_DYN image, for instance), plus I'm currently see

libfdt: Fix bugs in fdt_get_path()

2008-08-28 Thread David Gibson
The current implementation of fdt_get_path() has a couple of bugs, fixed by this patch. First, contrary to its documentation, on success it returns the length of the node's path, rather than 0. The testcase is correspondingly wrong, and the patch fixes this as well. Second, in some circumstances

Re: [PATCH 1/2] leds: make the default trigger name const

2008-08-28 Thread Trent Piepho
The default_trigger fields of struct gpio_led and thus struct led_classdev are pretty much always assigned from a string literal, which means the string can't be modified. Which is fine, since there is no reason to modify the string and in fact it never is. But they should be marked const to prev

Re: [RFC][USB] powerpc: Workaround for the PPC440EPX USBH_23 errata

2008-08-28 Thread Benjamin Herrenschmidt
On Thu, 2008-08-28 at 17:33 -0400, Alan Stern wrote: > Is there some reason why it doesn't work already? All the patch does > is suspend the OHCI root hub when you plug in the memory stick -- but > the root hub should already be suspended. > > Unless the memory stick is already plugged in when th

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Benjamin Herrenschmidt
> I understand what you're saying, I've been here before :) However, I > was never able to convince myself that it's safe without the lwarx/ > stwcx. There's hashing code that wanks around with the HASHPTE bit > doing a RMW without holding any lock (other than lwarx/stwcx-ing the > PTE it

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Scott Wood
Alan Stern wrote: This was done deliberately. The relevant standards state that a USB device can have no more than one peripheral interface. Does building a kernel image that can run on different hardware without rebuilding also violate the "relevant standards"? And who's to say that there

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Becky Bruce
On Aug 28, 2008, at 11:07 AM, Scott Wood wrote: Becky Bruce wrote: I'm pretty sure I went through this in great detail at one point and concluded that I did in fact need the lwarx/stwcx. IIRC, it has to do with other non-set_pte_at writers not necessarily holding the page table lock. FYI

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Arnd Bergmann
On Thursday 28 August 2008, Alan Stern wrote: > On Thu, 28 Aug 2008, Scott Wood wrote: > > > Alan Stern wrote: > > > This was done deliberately.  The relevant standards state that a USB > > > device can have no more than one peripheral interface. > > > > Does building a kernel image that can run

Re: [RFC][USB] powerpc: Workaround for the PPC440EPX USBH_23 errata

2008-08-28 Thread Alan Stern
On Thu, 28 Aug 2008, Steven A. Falco wrote: > Alan Stern wrote: > > Your original post mentioned that the 440EPx has only one USB 2.0 host > > port. Then how can you use a keyboard and memory stick at the same > > time? You'd have to plug them into a hub -- in which case only one > > controller

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Becky Bruce
Great, so *you* got my email, and I did not. I love our mailserver! On Aug 28, 2008, at 3:28 PM, Scott Wood wrote: Becky Bruce wrote: On Aug 28, 2008, at 11:07 AM, Scott Wood wrote: Becky Bruce wrote: I'm pretty sure I went through this in great detail at one point and concluded that I did

Re: [RFC][USB] powerpc: Workaround for the PPC440EPX USBH_23 errata

2008-08-28 Thread Steven A. Falco
Alan Stern wrote: > Your original post mentioned that the 440EPx has only one USB 2.0 host > port. Then how can you use a keyboard and memory stick at the same > time? You'd have to plug them into a hub -- in which case only one > controller would be needed, the one driving the hub. The patch wo

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Scott Wood
Becky Bruce wrote: On Aug 28, 2008, at 11:07 AM, Scott Wood wrote: Becky Bruce wrote: I'm pretty sure I went through this in great detail at one point and concluded that I did in fact need the lwarx/stwcx. IIRC, it has to do with other non-set_pte_at writers not necessarily holding the page t

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Alan Stern
On Thu, 28 Aug 2008, Scott Wood wrote: > Alan Stern wrote: > > This was done deliberately. The relevant standards state that a USB > > device can have no more than one peripheral interface. > > Does building a kernel image that can run on different hardware without > rebuilding also violate the

Re: [RFC][USB] powerpc: Workaround for the PPC440EPX USBH_23 errata

2008-08-28 Thread Alan Stern
On Thu, 28 Aug 2008, Vitaly Bordug wrote: > > This doesn't explain why the fatal error occurs. > > > On certain 44x set of SoCs, only one controller is able to function, > e.g. technically they are mutually exclusive. > > There used to be recommendation to use only hi-speed or full-speed > devic

Re: [RFC][USB] powerpc: Workaround for the PPC440EPX USBH_23 errata

2008-08-28 Thread Vitaly Bordug
В Wed, 27 Aug 2008 10:36:20 -0400 (EDT) Alan Stern <[EMAIL PROTECTED]> пишет: > On Wed, 27 Aug 2008, Vitaly Bordug wrote: > > > A published errata for ppc440epx states, that when running Linux > > with both EHCI and OHCI modules loaded, the EHCI module experiences > > a fatal error when a high-sp

Re: RFC: Could cpm2_clk_setup and cpm2_set_pin be exported ?

2008-08-28 Thread Scott Wood
Laurent Pinchart wrote: On Thursday 28 August 2008, Scott Wood wrote: On Thu, Aug 28, 2008 at 05:57:13PM +0200, Laurent Pinchart wrote: I'm facing a situation where I need to call cpm2_clk_setup and cpm2_set_pin from a device driver compiled as a module. Before submitting a patch to export bot

Re: RFC: Could cpm2_clk_setup and cpm2_set_pin be exported ?

2008-08-28 Thread Scott Wood
On Thu, Aug 28, 2008 at 05:57:13PM +0200, Laurent Pinchart wrote: > I'm facing a situation where I need to call cpm2_clk_setup and > cpm2_set_pin from a device driver compiled as a module. Before > submitting a patch to export both functions, I'd like to make sure > there isn't a cleaner way to imp

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Alan Stern
On Thu, 28 Aug 2008, Arnd Bergmann wrote: > Not addressing this driver in particular, but the USB gadget layer in > general: This is a horrible interface, since every gadget driver exports > the same symbols, you can never build a kernel that includes more than > one gadget driver. Even if the dri

Re: [BUG] linux-next: Tree for August 26 - Badness at kernel/notifier.c:25

2008-08-28 Thread Milton Miller
David Woodhouse dwmw2 at infradead.org Fri Aug 29 00:55:07 EST 2008 > On Thu, 2008-08-28 at 15:23 +0100, David Woodhouse wrote: >> On Thu, 2008-08-28 at 00:38 +1000, Stephen Rothwell wrote: >>> Hi Arjan, >>> >>> On Thu, 28 Aug 2008 00:33:08 +1000 Stephen Rothwell >> canb.auug.org.au> wrote:

Re: RFC: Could cpm2_clk_setup and cpm2_set_pin be exported ?

2008-08-28 Thread Laurent Pinchart
On Thursday 28 August 2008, Scott Wood wrote: > On Thu, Aug 28, 2008 at 05:57:13PM +0200, Laurent Pinchart wrote: > > I'm facing a situation where I need to call cpm2_clk_setup and > > cpm2_set_pin from a device driver compiled as a module. Before > > submitting a patch to export both functions, I'

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Scott Wood
On Thu, Aug 28, 2008 at 05:43:33PM +0800, Li Yang wrote: > +config USB_GADGET_FSL_QE > + boolean "Freescale QE/CPM USB Device Controller" > + depends on FSL_SOC && (QUICC_ENGINE || CPM) > + help > +Some of Freescale PowerPC processors have a Full Speed > +QE/CPM2 USB con

Re: [PATCH 2/4] Add cpufreq driver for the IBM PowerPC 750GX

2008-08-28 Thread Kevin Diggs
Arnd Bergmann wrote: On Wednesday 27 August 2008, Kevin Diggs wrote: Arnd Bergmann wrote: I think the module_exit() function should leave the frequency in a well-defined state, so the easiest way to get there is probably to delete the timer, and then manually set the frequency. I really do

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Scott Wood
Becky Bruce wrote: I'm pretty sure I went through this in great detail at one point and concluded that I did in fact need the lwarx/stwcx. IIRC, it has to do with other non-set_pte_at writers not necessarily holding the page table lock. FYI, the existing 32-bit PTE code is doing atomic updates

RFC: Could cpm2_clk_setup and cpm2_set_pin be exported ?

2008-08-28 Thread Laurent Pinchart
Hi everybody, I'm facing a situation where I need to call cpm2_clk_setup and cpm2_set_pin from a device driver compiled as a module. Before submitting a patch to export both functions, I'd like to make sure there isn't a cleaner way to implement the desired functionality without calling functio

Re: [PATCH v2] POWERPC: Allow 32-bit pgtable code to support 36-bit physical

2008-08-28 Thread Becky Bruce
On Aug 27, 2008, at 6:43 PM, Scott Wood wrote: Becky Bruce wrote: #if _PAGE_HASHPTE != 0 +#ifndef CONFIG_PTE_64BIT pte_update(ptep, ~_PAGE_HASHPTE, pte_val(pte) & ~_PAGE_HASHPTE); #else + /* +* We have to do the write of the 64b pte as 2 stores. This +* code assu

Re: Why does one "stw" fail with address translation disabled in PPC405EP?

2008-08-28 Thread Zhou Rui
Hi, all: Well, as described before, the problem happens at "0xc434 : stw r12,64(r11)". At this moment, address translation is disabled and physical addresses are used, but r11 contains 0x03072da0 which is a physical address out of the range of 0x0 and 0x01ff. I check backward and s

Re: [PATCH] usb: add Freescale QE/CPM USB peripheral controller driver

2008-08-28 Thread Arnd Bergmann
On Thursday 28 August 2008, Li Yang wrote: > Some of Freescale SoC chips have a QE or CPM co-processor which > supports full speed USB. The driver adds device mode support > of both QE and CPM USB controller to Linux USB gadget. The > driver is tested with MPC8360 and MPC8272, and should work wit

Re: [BUG] linux-next: Tree for August 26 - Badness at kernel/notifier.c:25

2008-08-28 Thread David Woodhouse
On Thu, 2008-08-28 at 15:23 +0100, David Woodhouse wrote: > On Thu, 2008-08-28 at 00:38 +1000, Stephen Rothwell wrote: > > Hi Arjan, > > > > On Thu, 28 Aug 2008 00:33:08 +1000 Stephen Rothwell <[EMAIL PROTECTED]> > > wrote: > > > > > > The original reported trace was during setup_system which is

Re: [BUG] linux-next: Tree for August 26 - Badness at kernel/notifier.c:25

2008-08-28 Thread David Woodhouse
On Thu, 2008-08-28 at 00:38 +1000, Stephen Rothwell wrote: > Hi Arjan, > > On Thu, 28 Aug 2008 00:33:08 +1000 Stephen Rothwell <[EMAIL PROTECTED]> wrote: > > > > The original reported trace was during setup_system which is very early in > > the boot. > > But, of course, that version didn't have t

Re: [PATCH] ppc4xx_pci: necessary fixes for 4GB RAM size

2008-08-28 Thread Stefan Roese
On Thursday 28 August 2008, Josh Boyer wrote: > > 1. total_memory should be phys_addr_t not unsigned long > > 2. is_power_of_2() works with u32 so I just inlined (size & (size-1)) != > > 0 instead. > > Also this patch fixes default initialization: res->end should be > > 0x7fff not 0x8000. >

Re: [PATCH] ppc4xx_pci: necessary fixes for 4GB RAM size

2008-08-28 Thread Josh Boyer
On Fri, 22 Aug 2008 11:43:35 +0400 Ilya Yanok <[EMAIL PROTECTED]> wrote: > 1. total_memory should be phys_addr_t not unsigned long > 2. is_power_of_2() works with u32 so I just inlined (size & (size-1)) != 0 > instead. > Also this patch fixes default initialization: res->end should be 0x7fff >

Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread Geert Uytterhoeven
On Thu, 28 Aug 2008, David Woodhouse wrote: > On Wed, 2008-08-13 at 11:27 +1000, Paul Mackerras wrote: > > The following series of patches implement support for a relocatable > > kernel by building it as a position-independent executable (PIE). > > When the linker is given the -pie flag, it creates

Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread Kumar Gala
On Aug 28, 2008, at 7:12 AM, David Woodhouse wrote: On Wed, 2008-08-13 at 11:27 +1000, Paul Mackerras wrote: The following series of patches implement support for a relocatable kernel by building it as a position-independent executable (PIE). When the linker is given the -pie flag, it creates

Re: [PATCH 0/5] Relocatable 64-bit kernel using linker PIE support

2008-08-28 Thread David Woodhouse
On Wed, 2008-08-13 at 11:27 +1000, Paul Mackerras wrote: > The following series of patches implement support for a relocatable > kernel by building it as a position-independent executable (PIE). > When the linker is given the -pie flag, it creates an executable that > contains dynamic relocations w

Re: checkpatch nits ...

2008-08-28 Thread Andy Whitcroft
On Wed, Aug 27, 2008 at 09:49:44AM +0200, Wolfram Sang wrote: > On Sat, Aug 23, 2008 at 10:57:21AM +0200, Arnd Bergmann wrote: > > > On Saturday 23 August 2008, Kevin Diggs wrote: > > > WARNING: externs should be avoided in .c files > > > #1137: FILE: powerpc/kernel/cpu/pll_if.c:369: > > > +      

Re: [PATCH 2/4] Add cpufreq driver for the IBM PowerPC 750GX

2008-08-28 Thread Arnd Bergmann
On Wednesday 27 August 2008, Kevin Diggs wrote: > Arnd Bergmann wrote: > > > > Module parameter names should be short, so just "minmax" would > > be a good name, but better put the module_param() line right > > after that. If it's a bool type, I would just leave out the > > initialization. > > >