Re: [PATCH] Macintosh: fix brace and trailing statement coding style issues in adb-iop.c This is a patch to the adb-iop.c file that cleans up brace and trailing statement warnings found by the check

2010-03-09 Thread Grant Likely
Hi Michael, Thanks for the patch. However, whitespace changes like this usually aren't worth bothering with. Yeah, sure, they are technically violations to the coding style, but they aren't egregious and aren't a serious impediment to readability. In general I don't want to see purely minor cod

Re: [RFC] powerpc: add support for new hcall H_BEST_ENERGY

2010-03-09 Thread Vaidyanathan Srinivasan
* Dipankar Sarma [2010-03-09 00:58:26]: > On Mon, Mar 08, 2010 at 12:20:06PM +0530, Vaidyanathan Srinivasan wrote: > > * Dipankar Sarma [2010-03-06 00:48:11]: > > > > > Shouldn't we create this only for supported platforms ? > > > > Hi Dipankar, > > > > Yes we will need a check like > > firmw

madness, was Re: [PATCH 2/2] Macintosh: fix coding style issues in adb.c This is a patch to the adb.c file that cleans up many warnings found by the checkpatch.pl tool

2010-03-09 Thread fthain
On Tue, 9 Mar 2010, Michael Beardsworth wrote: > - /* If a probe is in progress or we are sleeping, wait for it to > complete */ > + /* If a probe is in progress or we are sleeping, wait until complete */ If you must "fix" what aint broke, please try not to damage it. I'm sorry if your

[PATCH] Macintosh: fix brace and trailing statement coding style issues in adb-iop.c This is a patch to the adb-iop.c file that cleans up brace and trailing statement warnings found by the checkpatch.

2010-03-09 Thread Michael Beardsworth
From: Michael Beardsworth --- drivers/macintosh/adb-iop.c | 41 +++-- 1 files changed, 23 insertions(+), 18 deletions(-) diff --git a/drivers/macintosh/adb-iop.c b/drivers/macintosh/adb-iop.c index 4446966..e813589 100644 --- a/drivers/macintosh/adb-iop.c +

[PATCH 2/2] Macintosh: fix coding style issues in adb.c This is a patch to the adb.c file that cleans up many warnings found by the checkpatch.pl tool. Signed-off-by: Michael Beardsworth

2010-03-09 Thread Michael Beardsworth
--- drivers/macintosh/adb.c | 73 +++ 1 files changed, 36 insertions(+), 37 deletions(-) diff --git a/drivers/macintosh/adb.c b/drivers/macintosh/adb.c index 1c4ee6e..1f7a4f6 100644 --- a/drivers/macintosh/adb.c +++ b/drivers/macintosh/adb.c @@ -115,2

[PATCH] PPC: Skip over OF_DT_NOP when unflattening the device tree

2010-03-09 Thread Jason Gunthorpe
NOPs within the property section are skipped, but NOPs between OF_DT_END_NODE and OF_DT_BEGIN_NODE were not. My firmware NOPs out entire nodes depending on various environment parameters. of_scan_flat_dt already handles NOP more generally.. Signed-off-by: Jason Gunthorpe --- arch/powerpc/kernel

[PATCH] PPC32: Fixup pmd_page to work when ARCH_PFN_OFFSET is non-zero

2010-03-09 Thread Jason Gunthorpe
Instead of referencing mem_map directly, use pfn_to_page. Otherwise the kernel crashes when trying to start userspace if ARCH_PFN_OFFSET is non-zero and CONFIG_BOOKE is not defined Signed-off-by: Jason Gunthorpe --- arch/powerpc/include/asm/pgtable-ppc32.h |2 +- 1 files changed, 1 insertion

[PATCH] TPM: Provide a tpm_tis OF driver

2010-03-09 Thread Jason Gunthorpe
This provides an open firwmare driver binding for tpm_tis. OF is useful on arches where PNP is not used. Allow the tpm_tis driver to be selected if PNP or OF are compiled in. Tested on a PPC405 with a winbond LPC TPM. Signed-off-by: Jason Gunthorpe --- drivers/char/tpm/Kconfig |2 +- dri

[PATCH] Staging: macintosh: fix whitespace and style in adb.c This is a patch to the adb.c file, fixing the formatting of braces and tabs. Signed-off-by: Nick Savage

2010-03-09 Thread Nick Savage
--- drivers/macintosh/adb.c | 71 +++ 1 files changed, 35 insertions(+), 36 deletions(-) diff --git a/drivers/macintosh/adb.c b/drivers/macintosh/adb.c index 1c4ee6e..89891bb 100644 --- a/drivers/macintosh/adb.c +++ b/drivers/macintosh/adb.c @@ -115,2

Re: [LKML] [RFC PATCH v3 05/11] swiotlb: add swiotlb_set_default_size()

2010-03-09 Thread Konrad Rzeszutek Wilk
On Sun, Mar 07, 2010 at 01:11:46PM +0100, Albert Herranz wrote: > The current SWIOTLB code uses a default of 64MB for the IO TLB area. > This size can be influenced using a kernel command line parameter "swiotlb". > Unfortunately, the parsing of the kernel command line is done _after_ the > swiotlb

Re: [LKML] [RFC PATCH v3 04/11] swiotlb: support NOT_COHERENT_CACHE PowerPC platforms

2010-03-09 Thread Konrad Rzeszutek Wilk
On Sun, Mar 07, 2010 at 01:11:45PM +0100, Albert Herranz wrote: > The current SWIOTLB code does not support NOT_COHERENT_CACHE platforms. > This patch adds support for NOT_COHERENT_CACHE platforms to SWIOTLB by > adding two platform specific functions swiotlb_dma_sync_page() and > swiotlb_dma_sync(

Re: [LKML] [RFC PATCH v3 05/11] swiotlb: add swiotlb_set_default_size()

2010-03-09 Thread Albert Herranz
Konrad Rzeszutek Wilk wrote: > On Sun, Mar 07, 2010 at 01:11:46PM +0100, Albert Herranz wrote: >> The current SWIOTLB code uses a default of 64MB for the IO TLB area. >> This size can be influenced using a kernel command line parameter "swiotlb". >> Unfortunately, the parsing of the kernel command

Re: [LKML] [RFC PATCH v3 04/11] swiotlb: support NOT_COHERENT_CACHE PowerPC platforms

2010-03-09 Thread Albert Herranz
Konrad Rzeszutek Wilk wrote: > Hey Albert, > > I've been doing some posting in this area to split the physical / bus > address translation so that multiple platforms can utilize it. I was > wondering if it makes sense to utilize some of those concepts (ie, extend it > for DMA coherency) for your c

Re: cross-compiling Linux for PowerPC e200 core?

2010-03-09 Thread Detlev Zundel
Hi Segher, > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=33d9e9b > > There is no obvious way to get there via clicking afaics, the "commit" > search box doesn't work at least. Hrm. It works, but not like one would expect, i.e. check out the help: http://git.kerne

Re: BUG: Bad page map in process mm2

2010-03-09 Thread Steven A. Falco
Jake Magee wrote: > Steven, > > Try these patches... > http://patchwork.ozlabs.org/patch/34047/ > http://patchwork.ozlabs.org/patch/34113/ > Thanks for the pointer. I just tried 34047 and it fixed the problem. Steve > Both patches work for my situation, but I went with the second set