Re: [PATCH 1/1] drivers/char/mem.c

2012-09-23 Thread Arnd Bergmann
On Saturday 22 September 2012, Teodori Serge wrote: > replace 'arch_has_dev_port()' to 'CONFIG_DEVPORT' > > modified: arch/powerpc/include/asm/io.h > modified: drivers/char/mem.c > modified: include/linux/io.h > > why do we need 'arch_has_dev_port()' if we have a 'CONFIG_DEVPORT'? > > Sign

Re: [PATCH] powerpc: Remove tlb batching hack for nighthawk

2012-09-23 Thread Anton Blanchard
On Fri, 21 Sep 2012 18:08:28 +1000 Michael Ellerman wrote: > In hpte_init_native() we call tlb_batching_enabled() to decide if we > should setup ppc_md.flush_hash_range. > > tlb_batching_enabled() checks the _unflattened_ device tree, to see > if we are running on a nighthawk. > > Since commit

Re: [PATCH 1/1] drivers/char/tpm: remove tasklet and cleanup

2012-09-23 Thread James Morris
On Wed, 12 Sep 2012, Ashley Lai wrote: > This patch removed the tasklet and moved the wait queue into the > private structure. It also cleaned up the response CRQ path. > > Signed-off-by: Ashley Lai Kent: any comment on this? You should probably push this to me via your tree. > --- > Jame

Re: [PATCH v3 1/2] mmc: Move mmc_delay() to include/linux/mmc/core.h

2012-09-23 Thread Chunhe Lan
On 09/21/2012 08:33 AM, Arnd Bergmann wrote: On Friday 21 September 2012, Chunhe Lan wrote: On 08/10/2012 09:27 AM, Arnd Bergmann wrote: On Friday 10 August 2012, Chunhe Lan wrote: cond_resched(); mdelay(ms); sets off alarm bells, and I would always replace that with msleep().

[RFC PATCH powerpc] Fix a lazy irq related WARING in arch_local_irq_restore()

2012-09-23 Thread Li Zhong
This patch tries to fix a WARNING in irq.c(below), when doing cpu offline/online. The reason is that if the preferred offline state is CPU_STATE_INACTIVE, when cpu offline, pseries_mach_cpu_die() calls extended_cede_processor() to cede the processor. After the hv call returns, the MSR_EE is enab

Re: [RFC PATCH powerpc] Fix a lazy irq related WARING in arch_local_irq_restore()

2012-09-23 Thread Benjamin Herrenschmidt
On Mon, 2012-09-24 at 11:56 +0800, Li Zhong wrote: > This patch tries to fix a WARNING in irq.c(below), when doing cpu > offline/online. > > The reason is that if the preferred offline state is CPU_STATE_INACTIVE, when > cpu offline, pseries_mach_cpu_die() calls extended_cede_processor() to cede

[PATCH][v2] powerpc/fsl-pci: use 'Header Type' to identify PCIE mode

2012-09-23 Thread Minghuan Lian
The original code uses 'Programming Interface' field to judge if PCIE is EP or RC mode, however, some latest silicons do not support this functionality. According to PCIE specification, 'Header Type' offset 0x0e is used to indicate header type, so change code to use 'Header Type' field to judge PCI

[PATCH v2] Powerpc 8xx CPM_UART desynchronisation

2012-09-23 Thread Christophe Leroy
This patch fixes a desynchronisation problem with CPM UART driver on Powerpc MPC8xx. The problem happens if data is received before the device is open by the user application. Signed-off-by: Christophe Leroy --- linux-3.5-vanilla/drivers/tty/serial/cpm_uart/cpm_uart_core.c 2012-07-21 22:5

[PATCH] Powerpc 8xx CPM_UART too many interrupts

2012-09-23 Thread Christophe Leroy
Setting the fifo to only 1 byte generates one interrupt every 1ms at 9600 bauds. This is too much. This patch reduces the threshold to speeds below 2400 bauds like in the 8250 UART driver. Signed-off-by: Christophe Leroy --- linux-3.5-vanilla/drivers/tty/serial/cpm_uart/cpm_uart_core.c 2

[PATCH] Powerpc 8xx CPM_UART maxidl should not depend on fifo size

2012-09-23 Thread Christophe Leroy
maxidl register was set to fifo size. There is no reason to set this register to same value as fifo size. Setting it now to 0x10 by default as in the UCC UART driver. Signed-off-by: Christophe Leroy --- linux-3.5-vanilla/drivers/tty/serial/cpm_uart/cpm_uart_core.c 2012-07-21 22:58:29.

[PATCH] Powerpc 8xx CPM_UART setting MAXIDL register proportionaly to baud rate

2012-09-23 Thread Christophe Leroy
MAXIDL is the timeout after which a receive buffer is closed when not full if no more characters are received. We calculate it from the baudrate so that the duration is always the same at standard rates: about 4ms. At 9600 bauds it gives a timeout of 4 characters, which is the timeout on the 8250 U

Re: [RFC PATCH powerpc] Fix a lazy irq related WARING in arch_local_irq_restore()

2012-09-23 Thread Li Zhong
On Mon, 2012-09-24 at 15:30 +1000, Benjamin Herrenschmidt wrote: > On Mon, 2012-09-24 at 11:56 +0800, Li Zhong wrote: > > This patch tries to fix a WARNING in irq.c(below), when doing cpu > > offline/online. > > > > The reason is that if the preferred offline state is CPU_STATE_INACTIVE, > > wh