Re: [PATCH 12/73] powerpc: kvm e500/44x is not modular, so don't use module_init

2014-01-21 Thread Paul Gortmaker
On 14-01-21 04:22 PM, Paul Gortmaker wrote: > In powerpc, CONFIG_KVM is bool, and so are these three subarch > options, for the 44x and e500 variants. This means that any > module_exit() calls and functions used by them such as the > kvmppc_booke_exit() are dead code. Here we

[PATCH 10/73] powerpc: use device_initcall for registering rtc devices

2014-01-21 Thread Paul Gortmaker
e_initcall directly in this change means that the runtime impact is zero -- they will remain at level 6 in initcall ordering. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Geoff Levand Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/kernel/time.c| 2 +- ar

[PATCH 66/73] drivers/macintosh: delete non-required instances of include

2014-01-21 Thread Paul Gortmaker
s.org Signed-off-by: Paul Gortmaker --- drivers/macintosh/adb-iop.c | 1 - drivers/macintosh/ams/ams-input.c | 1 - drivers/macintosh/macio-adb.c | 1 - drivers/macintosh/via-macii.c | 1 - drivers/macintosh/via-pmu68k.c | 1 - dr

[PATCH 39/73] powerpc: delete another unrequired instance of

2014-01-21 Thread Paul Gortmaker
Most were already deleted in the 1st pass audit; this instance showed up more recently. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/sysdev/indirect_pci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch

[PATCH RFC 00/73] tree-wide: clean up some no longer required #include

2014-01-21 Thread Paul Gortmaker
ux-m...@lists.linux-m68k.org Cc: linux-m...@linux-mips.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-s...@vger.kernel.org Cc: sparcli...@vger.kernel.org Cc: x...@kernel.org Cc: net...@vger.kernel.org Cc: k...@vger.kernel.org Cc: s...@canb.auug.org.au Cc: ru...@rustcorp.com.au Cc: gre...@linuxfound

[PATCH 15/73] powerpc: don't use module_init in non-modular 83xx suspend code

2014-01-21 Thread Paul Gortmaker
irectly in this change means that the runtime impact is zero -- it will remain at level 6 in initcall ordering. Cc: Kumar Gala Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/platforms/83xx/suspend.c | 3 +-- 1 fi

[PATCH 14/73] powerpc: don't use module_init for non-modular core hugetlb code

2014-01-21 Thread Paul Gortmaker
ation from level 6-device to level 3-arch (i.e. slightly earlier). However no observable impact of that small difference has been observed during testing, or is expected. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/

[PATCH 13/73] powerpc: use subsys_initcall for Freescale Local Bus

2014-01-21 Thread Paul Gortmaker
r Gala Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/sysdev/fsl_lbc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/sysdev/fsl_lbc.c b/arch/powerpc/sysdev/fsl_lbc.c index d631022..38138cf 100644 --- a/arch/powerpc/sysdev/fsl_l

[PATCH 12/73] powerpc: kvm e500/44x is not modular, so don't use module_init

2014-01-21 Thread Paul Gortmaker
i Cc: Alexander Graf Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: k...@vger.kernel.org Cc: kvm-...@vger.kernel.org Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/include/asm/kvm_ppc.h | 1 - arch/powerpc/kvm/44x.c | 10 +- arch/powerpc/kvm/bo

[PATCH 11/73] powerpc: book3s KVM can be modular so it should use module.h

2014-01-21 Thread Paul Gortmaker
to module.h Cc: Gleb Natapov Cc: Paolo Bonzini Cc: Alexander Graf Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: k...@vger.kernel.org Cc: kvm-...@vger.kernel.org Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Paul Gortmaker --- arch/powerpc/kvm/book3s.c | 2 +- 1 file changed, 1

[PATCH v2] drivers/tty/hvc: don't use module_init in non-modular hyp. console code

2014-01-15 Thread Paul Gortmaker
k Wilk Cc: Boris Ostrovsky Cc: David Vrabel Cc: Greg Kroah-Hartman Signed-off-by: Paul Gortmaker --- [v2: unchanged; just added xen guys to Cc list, as hvc_xen isnt hooked into the MAINTAINERS file as of yet, so I forgot them.] diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_op

[PATCH] drivers/tty/hvc: don't use module_init in non-modular hyp. console code

2014-01-15 Thread Paul Gortmaker
Signed-off-by: Paul Gortmaker diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_opal.c index 6496872e2e47..b01659bd4f7c 100644 --- a/drivers/tty/hvc/hvc_opal.c +++ b/drivers/tty/hvc/hvc_opal.c @@ -255,13 +255,7 @@ static int __init hvc_opal_init(void) /* Register as a vio

Re: [PATCH V5 0/8] cpuidle/ppc: Enable deep idle states on PowerNV

2014-01-15 Thread Paul Gortmaker
On 14-01-15 03:07 AM, Preeti U Murthy wrote: [...] > > This patchset is based on mainline commit-id:8ae516aa8b8161254d3, and the I figured I'd give this a quick sanity build test for a few configs, but v3.13-rc1-141-g8ae516aa8b81 seems too old; Ben's ppc next branch is at v3.13-rc1-160-gfac515

[PATCH 2/4] powerpc: book3s kvm can be modular so it should use module.h

2014-01-13 Thread Paul Gortmaker
KVM support is tristate, so this file should be including module.h instead of export.h -- it only works currently because module_init is currently (mis)placed in init.h -- but we are intending to clean that up and relocate it to module.h Signed-off-by: Paul Gortmaker --- arch/powerpc/kvm

[PATCH 3/4] powerpc: use subsys_initcall for Freescale Local Bus

2014-01-13 Thread Paul Gortmaker
will thus change this registration from level 6-device to level 4-subsys (i.e. slightly earlier). However no observable impact of that small difference has been observed during testing, or is expected. Signed-off-by: Paul Gortmaker --- arch/powerpc/sysdev/fsl_lbc.c | 2 +- 1 file changed, 1

[PATCH 1/4] powerpc: use device_initcall for registering rtc devices

2014-01-13 Thread Paul Gortmaker
e_initcall directly in this change means that the runtime impact is zero -- they will remain at level 6 in initcall ordering. Signed-off-by: Paul Gortmaker --- arch/powerpc/kernel/time.c| 2 +- arch/powerpc/platforms/ps3/time.c | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --

[PATCH 4/4] powerpc: don't use module_init for non-modular core hugetlb code

2014-01-13 Thread Paul Gortmaker
ation from level 6-device to level 3-arch (i.e. slightly earlier). However no observable impact of that small difference has been observed during testing, or is expected. Signed-off-by: Paul Gortmaker --- arch/powerpc/mm/hugetlbpage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

[PATCH 0/4] remap non-modular uses of module_init properly

2014-01-13 Thread Paul Gortmaker
l where clearly arch_initcall or subsys_initcall are more appropriate. Boot tested on sbc8548 on powerpc next branch of today. Paul Gortmaker (4): powerpc: use device_initcall for registering rtc devices powerpc: book3s kvm can be modular so it should use module.h powerpc: use subsys_initcall for

[PATCH] powerpc: fix 8xx and 6xx final link failures

2014-01-09 Thread Paul Gortmaker
Only build board support files when required.") Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile index ca7f08c..4676e55 100644 --- a/arch/powerpc/boot/Makefile +++ b/arch/powerpc/boot/Makefile @@ -71,9 +71,9 @@ src-wlib-y := st

[PATCH] powerpc: delete non-required instances of include

2014-01-09 Thread Paul Gortmaker
off a case where that file was implicitly getting it from another header which itself didn't need it. Signed-off-by: Paul Gortmaker --- [ build tested for every arch/powerpc/config/* file, on next branch of benh/powerpc.git -- v3.13-rc1-92-gdece8ada993e ] arch/powerpc/include/

mpic build failure for 7447_hpc defconfig (bisected)

2014-01-08 Thread Paul Gortmaker
Commit 446f6d06fab0b49c61887ecbe8286d6aaa796637 ("powerpc/mpic: Properly set default triggers") breaks the mpc7447_hpc_defconfig as follows: CC arch/powerpc/sysdev/mpic.o arch/powerpc/sysdev/mpic.c: In function 'mpic_set_irq_type': arch/powerpc/sysdev/mpic.c:886:9: error: case label does no

[PATCH-next 00/32] Delete support for __cpuinit

2013-06-24 Thread Paul Gortmaker
ozlabs.org Cc: linux...@de.ibm.com Cc: linux-s...@vger.kernel.org Cc: linux...@vger.kernel.org Cc: sparcli...@vger.kernel.org Cc: x...@kernel.org Cc: linux-xte...@linux-xtensa.org Paul Gortmaker (32): init.h: remove __cpuinit sections from the kernel modpost: remove all traces of cpuinit/cpuexit sections alp

[PATCH-next 00/32] Delete support for __cpuinit

2013-06-24 Thread Paul Gortmaker
[Resending with only lists on Cc: -- previous mail header on the 00/32 was too long; failed to get passed vger's crap filters.] On 13-06-24 03:30 PM, Paul Gortmaker wrote: > This is the whole patch queue for removal of __cpuinit support > against the latest linux-next tree (Jun24th

[PATCH 04/32] powerpc: delete __cpuinit usage from all users

2013-06-24 Thread Paul Gortmaker
emoves all the powerpc uses of the __cpuinit macros. There are no __CPUINIT users in assembly files in powerpc. [1] https://lkml.org/lkml/2013/5/20/589 Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Josh Boyer Cc: Matt Porter Cc: Kumar Gala Cc: linuxppc-dev@lists.ozlabs.org Signed-of

Re: [PATCH] powerpc: delete __cpuinit usage from all users

2013-06-18 Thread Paul Gortmaker
On 13-06-17 04:10 PM, Paul Gortmaker wrote: > The __cpuinit type of throwaway sections might have made sense > some time ago when RAM was more constrained, but now the savings > do not offset the cost and complications. For example, the fix in > commit 5e427ec2d0 ("x86: Fix bit

[PATCH] powerpc: delete __cpuinit usage from all users

2013-06-17 Thread Paul Gortmaker
emoves all the powerpc uses of the __cpuinit macros. [1] https://lkml.org/lkml/2013/5/20/589 Signed-off-by: Paul Gortmaker --- [This was generated against today's linux-next tree ; I'm assuming all pending powerpc changes are in there currently.] arch/powerpc/include/asm/rtas.h

[PATCH net] gianfar: fix compile fail for NET_POLL=y due to struct packing

2013-02-24 Thread Paul Gortmaker
ere simply overlooked went undetected until now. Make the netpoll ones also use the gfar_irq() accessors. Reported-by: Benjamin Herrenschmidt Cc: Benjamin Herrenschmidt Cc: Claudiu Manoil Cc: Jianhua Xie Signed-off-by: Paul Gortmaker --- [compile tested for sbc8548 with NET_POLL=y] drivers/ne

Re: Gianfar breaks one of my test configs

2013-02-24 Thread Paul Gortmaker
On Sun, Feb 24, 2013 at 9:43 AM, Benjamin Herrenschmidt wrote: > Hi folks ! > > Current Linus tree as of this morning fails to build with one of my > (semi-random) test configs (attached): > > /home/benh/linux-powerpc-test/drivers/net/ethernet/freescale/gianfar.c: In > function 'gfar_netpoll': >

[PATCH 3/4] powerpc: add alternate dts file for sbc8548 boot via SODIMM

2013-01-23 Thread Paul Gortmaker
EFF0 E EFF2 E EFF4 E EFF6 E EFF8 E EFFAEFFCEFFE --- Signed-off-by: Paul Gortmaker --- arch/powerpc/boot/dts/sbc8548-altflash.dts | 115 + 1 file changed, 115 insertions(+) create mode 1

[PATCH 2/4] powerpc: update sbc8548 flash information to match recent u-boot

2013-01-23 Thread Paul Gortmaker
s based on. Now there is just the categorization of free space and bootloader images. Signed-off-by: Paul Gortmaker --- arch/powerpc/boot/dts/sbc8548.dts | 34 +++--- 1 file changed, 15 insertions(+), 19 deletions(-) diff --git a/arch/powerpc/boot/dts/sbc8548.dts b/arch/

[PATCH 1/4] powerpc: split sbc8548 dts file into pre and post chunks

2013-01-23 Thread Paul Gortmaker
e to using include files. Signed-off-by: Paul Gortmaker --- arch/powerpc/boot/dts/sbc8548-post.dtsi | 295 + arch/powerpc/boot/dts/sbc8548-pre.dtsi | 52 ++ arch/powerpc/boot/dts/sbc8548.dts | 322 +--- 3 files changed, 351

[PATCH 0/4] powerpc: update sbc8548 flash/mtd settings

2013-01-23 Thread Paul Gortmaker
powerpc: enable MTD options in sbc8548 defconfig (2013-01-23 14:49:10 -0500) ---- Paul Gortmaker (4): powerpc: split sbc8548 dts file into pre and post chunks powerpc: update sbc8548 flash information to match recent u-boot

[PATCH 4/4] powerpc: enable MTD options in sbc8548 defconfig

2013-01-23 Thread Paul Gortmaker
e of the u-boot image. The kernel configuration is the same in both cases; only the dtb needs to be changed in accordance with the JP12/SW2.8 settings. Signed-off-by: Paul Gortmaker --- arch/powerpc/configs/85xx/sbc8548_defconfig | 19 +++ 1 file changed, 19 insertions(+) diff --gi

Re: Anyone have a PrPmc 280/2800 handy?

2013-01-19 Thread Paul Gortmaker
On Thu, Jan 17, 2013 at 5:33 PM, Mark A. Greer wrote: > Hello. > > I want to make some fixups to the marvell hostbridge and > mpsc code and would like to test them on a PrPmc280/2800. > The problem is, I don't have one anymore. Hi Mark, Is there another platform that uses mpsc that can be used f

[PATCH] powerpc/mm: match variable types to API

2012-08-21 Thread Paul Gortmaker
njamin Herrenschmidt Signed-off-by: Joe MacDonald Signed-off-by: Paul Gortmaker --- arch/powerpc/mm/subpage-prot.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/mm/subpage-prot.c b/arch/powerpc/mm/subpage-prot.c index e4f8f1f..7c415dd 100644 --- a/arch/p

Re: [PATCH 2/3] mtd: delete SBC82xx/SBC8560 MTD mapping support

2012-06-29 Thread Paul Gortmaker
On 12-06-29 04:29 AM, Artem Bityutskiy wrote: > On Tue, 2012-06-26 at 15:54 -0400, Paul Gortmaker wrote: >> The SBC8260 support was dropped back when we moved from ppc to >> powerpc. We are now also dropping the support for the EOL SBC8560, >> so we can also delete this map

Re: [PATCH 3/3] serial/8250: delete WR SBC850 UART quirk handling

2012-06-27 Thread Paul Gortmaker
On 12-06-27 07:52 AM, Alan Cox wrote: > On Tue, 26 Jun 2012 15:54:29 -0400 > Paul Gortmaker wrote: > >> We've are dropping the support for the EOL SBC8560, so we can >> also delete this variant of the Alpha quirk support. >> >> Cc: Alan Cox >&

[PATCH 3/3] serial/8250: delete WR SBC850 UART quirk handling

2012-06-26 Thread Paul Gortmaker
We've are dropping the support for the EOL SBC8560, so we can also delete this variant of the Alpha quirk support. Cc: Alan Cox Cc: Greg Kroah-Hartman Cc: linux-ser...@vger.kernel.org Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250/8250.h |7 --- 1 files chang

[PATCH 2/3] mtd: delete SBC82xx/SBC8560 MTD mapping support

2012-06-26 Thread Paul Gortmaker
The SBC8260 support was dropped back when we moved from ppc to powerpc. We are now also dropping the support for the EOL SBC8560, so we can also delete this mapping support, as they were the only users of it. Cc: David Woodhouse Cc: linux-...@lists.infradead.org Signed-off-by: Paul Gortmaker

[PATCH 1/3] powerpc: remove Wind River SBC8560 support

2012-06-26 Thread Paul Gortmaker
us to remove some sbc8560 specific quirk handling in 8250 UART code, and some MTD mapping support. Cc: David Woodhouse Signed-off-by: Paul Gortmaker --- arch/powerpc/boot/Makefile |1 - arch/powerpc/boot/dts/sbc8560.dts | 406 --- arch/po

[PATCH 0/3] powerpc: delete WR sbc8560 board support.

2012-06-26 Thread Paul Gortmaker
via their respective trees if desired, and not risk introducing any build regressions by doing so. Paul --- Cc: David Woodhouse Cc: Alan Cox Cc: Greg Kroah-Hartman Paul Gortmaker (3): powerpc: remove Wind River SBC8560 support mtd: delete SBC82xx/SBC8560 MTD mapping support serial/8250: delete

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-25 Thread Paul Gortmaker
On 12-05-25 03:51 PM, Joe Perches wrote: > On Fri, 2012-05-25 at 11:58 -0400, Paul Gortmaker wrote: >> But you really shouldn't need the hardware to validate this kind of >> patch anyways -- aside from your code flow change in the irq routine of >> gianfar_ptp, you shoul

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-25 Thread Paul Gortmaker
[Re: [PATCH] gianfar:don't add FCB length to hard_header_len] On 24/05/2012 (Thu 09:16) Joe Perches wrote: > On Thu, 2012-05-24 at 17:04 +0200, Jan Ceuleers wrote: > > On 05/22/2012 09:18 PM, David Miller wrote: > > > From: Jiajun Wu > > > Date: Tue, 22 May 2012 17:00:48 +0800 > > > > > >> FCB(

Re: linux-next: Tree for May 10 (net/phy)

2012-05-10 Thread Paul Gortmaker
On Thu, May 10, 2012 at 5:40 PM, Randy Dunlap wrote: > On 05/10/2012 02:26 AM, Stephen Rothwell wrote: > >> Hi all, >> >> Changes since 20120508: > > > > ERROR: "of_mdio_find_bus" [drivers/net/phy/mdio-mux.ko] undefined! Not just randconfig, but also powerpc allmodconfig: http://kisskb.ellerman.

[PATCH] powerpc: fix compile fail in hugetlb cmdline parsing

2012-05-07 Thread Paul Gortmaker
need to printk out the "doing" value, so just add the arg as an "unused". Cc: Rusty Russell Cc: Jim Cromie Cc: Jason Baron Cc: Greg Kroah-Hartman Cc: Becky Bruce Cc: Benjamin Herrenschmidt Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/

[PATCH] powerpc: fix system.h fallout in sysdev/scom.c [chroma_defconfig]

2012-04-15 Thread Paul Gortmaker
/system.h" Add the debug header which contains powerpc_debugfs_root. Cc: David Howells Signed-off-by: Paul Gortmaker --- [Ben, let me know if you want to take this or have me keep it.] diff --git a/arch/powerpc/sysdev/scom.c b/arch/powerpc/sysdev/scom.c index 49a3ece..702256a 100644 --

[PATCH] powerpc: Option FB_FSL_DIU is not really optional for mpc512x

2012-04-03 Thread Paul Gortmaker
ef in this file, just remove the existing two, and make FB_FSL_DIU part of the dependency. The mpc512x_defconfig already has the line "CONFIG_FB_FSL_DIU=y" so this change should be zero impact on real world configs. Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/platforms/512x/Kc

bpf_slow_path: sibling call to `skb_copy_bits' not allowed multiple TOCs (ppc)

2012-03-28 Thread Paul Gortmaker
Hi all, Wondering if anyone has researched this, before I waste more time on it. The linux-next build of powerpc-allyesconfig has for quite some time shown this: LD .tmp_vmlinux1 powerpc-linux-ld: arch/powerpc/net/built-in.o: In function `bpf_slow_path_word': (.text+0x90): sibling call op

Re: [PATCH net-next 4/4] gianfar: use netif_tx_queue_stopped instead of __netif_subqueue_stopped

2012-03-18 Thread Paul Gortmaker
On Sun, Mar 18, 2012 at 5:55 PM, Eric Dumazet wrote: > On Sun, 2012-03-18 at 17:39 -0400, Paul Gortmaker wrote: >> The __netif_subqueue_stopped() just does the following: >> >>         struct netdev_queue *txq = netdev_get_tx_queue(dev, queue_index); >>         return

[PATCH net-next 4/4] gianfar: use netif_tx_queue_stopped instead of __netif_subqueue_stopped

2012-03-18 Thread Paul Gortmaker
Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ethernet/freescale/gianfar.c b/drivers/net/ethernet/freescale/gianfar.c index 6e66cc3..d9428f0 100644 --- a/drivers/net/ethernet/freescale

[PATCH net-next 3/4] gianfar: delete orphaned version strings and dead macros

2012-03-18 Thread Paul Gortmaker
There were two version strings, and neither one was being used. Also in the same proximity were some unused #define that were left over from the past. Delete them all. Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar.c |3 --- drivers/net/ethernet/freescale

[PATCH net-next 1/4] gianfar: Add support for byte queue limits.

2012-03-18 Thread Paul Gortmaker
introduced in gfar_clean_tx_ring, since it is now used often enough to warrant it, and it cleans up the readability somewhat as well. Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar.c | 19 --- 1 files changed, 16 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH net-next 2/4] gianfar: constify giant block of status descriptor strings

2012-03-18 Thread Paul Gortmaker
Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar_ethtool.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ethernet/freescale/gianfar_ethtool.c b/drivers/net/ethernet/freescale/gianfar_ethtool.c index 5a78d55..8d74efd 100644 --- a

[PATCH v2 net-next 0/4] Gianfar byte queue limits

2012-03-18 Thread Paul Gortmaker
x.git gianfar-bql for you to fetch changes up to 5407b14c6792d6ff122ecb1a2a6acffad60ef389: gianfar: use netif_tx_queue_stopped instead of __netif_subqueue_stopped (2012-03-18 17:11:22 -0400) Paul Gortmaker (4): gianfar: Add su

Re: [PATCH net-next 1/3] gianfar: Add support for byte queue limits.

2012-03-18 Thread Paul Gortmaker
On Sun, Mar 18, 2012 at 4:20 PM, Eric Dumazet wrote: > Le dimanche 18 mars 2012 à 12:56 -0400, Paul Gortmaker a écrit : > > ... > >>                * we add this skb back into the pool, if it's the right size >> @@ -2557,13 +2568,15 @@ static int gfar_clean_

Re: [PATCH net-next 0/3] Gianfar byte queue limits

2012-03-18 Thread Paul Gortmaker
On Sun, Mar 18, 2012 at 4:30 PM, Eric Dumazet wrote: > Le dimanche 18 mars 2012 à 12:56 -0400, Paul Gortmaker a écrit : >> The BQL support here is unchanged from what I posted earlier as an >> RFC[1] -- with the exception of the fact that I'm now happier with >> the runt

[PATCH net-next 3/3] gianfar: delete orphaned version strings and dead macros

2012-03-18 Thread Paul Gortmaker
There were two version strings, and neither one was being used. Also in the same proximity were some unused #define that were left over from the past. Delete them all. Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar.c |3 --- drivers/net/ethernet/freescale

[PATCH net-next 2/3] gianfar: constify giant block of status descriptor strings

2012-03-18 Thread Paul Gortmaker
Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar_ethtool.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ethernet/freescale/gianfar_ethtool.c b/drivers/net/ethernet/freescale/gianfar_ethtool.c index 5a78d55..8d74efd 100644 --- a

[PATCH net-next 1/3] gianfar: Add support for byte queue limits.

2012-03-18 Thread Paul Gortmaker
introduced in gfar_clean_tx_ring, since it is now used often enough to warrant it, and it cleans up the readability somewhat as well. Signed-off-by: Paul Gortmaker --- drivers/net/ethernet/freescale/gianfar.c | 19 --- 1 files changed, 16 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH net-next 0/3] Gianfar byte queue limits

2012-03-18 Thread Paul Gortmaker
mething that nobody would ever use for anything today. (aside from contrived tests like this) But it did give me an example of where I could see the effects of changing the BQL settings, and I'm reasonably confident they are working as expected. Paul. --- [1] http://lists.openwall.net/netd

Re: [PATCH] powerpc/srio: Fix the compile errors when building with 64bit

2012-03-02 Thread Paul Gortmaker
f different size > > Rewrote the corresponding code with the support of 64bit building. > > Signed-off-by: Liu Gang > Signed-off-by: Shaohui Xie > Signed-off-by: Paul Gortmaker Hi Liu, You can't just go adding a "Signed-off-by:" line for me to a p

Re: [PATCH net-next] ibm emac: delete module references; phy.c only supported as built-in

2012-02-27 Thread Paul Gortmaker
[Re: [PATCH net-next] ibm emac: delete module references; phy.c only supported as built-in] On 27/02/2012 (Mon 14:57) David Miller wrote: > From: Paul Gortmaker > Date: Mon, 27 Feb 2012 14:47:04 -0500 > > > The Makefile has it as "ibm_emac-y := mal.o core.o phy.o" so

[PATCH net-next] ibm emac: delete module references; phy.c only supported as built-in

2012-02-27 Thread Paul Gortmaker
The Makefile has it as "ibm_emac-y := mal.o core.o phy.o" so there is no way this can be built modular, so remove all references to module support. Signed-off-by: Paul Gortmaker diff --git a/drivers/net/ethernet/ibm/emac/phy.c b/drivers/net/ethernet/ibm/emac/phy.c index d3b9d10..fb96

[PATCH net-next] ucc_geth: separate out rx/tx ring alloc and free operations

2012-02-27 Thread Paul Gortmaker
represents a straight up relocation of the existing code into separate routines without changing any of the contained code itself. Local variables are relocated as necessary. Signed-off-by: Paul Gortmaker --- [boot tested net-next on mpc8360 once I fixed mpc836x_mds.dts] diff --git a/drivers/net

[PATCH] mpc836x: fix failed phy detection for ucc ethernet on MDS

2012-02-27 Thread Paul Gortmaker
e to first MDIO bus" but the older mpc836x didn't get one and hence was just failing with -EBUSY as follows: fsl-pq_mdio: probe of e0102120.mdio failed with error -16 ... net eth0: Could not attach to PHY eth0: Cannot initialize PHY, aborting. Add a TBI node and use the 1st free a

Re: [PATCH/RFC] rapidio: temporarily exclude FSL_RIO from 64 bit builds

2012-02-26 Thread Paul Gortmaker
Hi Ben, Given a week has passed with the absence of any feedback with respect to SRIO on 64 bit, are you OK with applying the below patch[1] pretty much as-is? Thanks, Paul. [1] http://patchwork.ozlabs.org/patch/142383/ --- On Tue, Feb 21, 2012 at 6:10 PM, Paul Gortmaker wrote: >

[PATCH] ppc-6xx: fix build failure in flipper-pic.c and hlwd-pic.c

2012-02-22 Thread Paul Gortmaker
sing build failures. Signed-off-by: Paul Gortmaker --- [seen in build tests on the linux-next tree from yesterday.] diff --git a/arch/powerpc/platforms/embedded6xx/flipper-pic.c b/arch/powerpc/platforms/embedded6xx/flipper-pic.c index 675335a..53d6eee 100644 --- a/arch/powerpc/platforms/e

[PATCH/RFC] rapidio: temporarily exclude FSL_RIO from 64 bit builds

2012-02-21 Thread Paul Gortmaker
/fsl_rmu.c:997: error: cast to pointer from integer of different size A quick inspection of the code leaves one with the impression that it was not explicitly written with 64 bit support in mind, so just block that as a possible config selection for now. Signed-off-by: Paul Gortmaker --- [This is

[RFC] Multi queue support in ethernet/freescale/ucc_geth.c

2012-02-02 Thread Paul Gortmaker
Hi Li, A while back DaveM mentioned that it would be good to break out the ring allocations[1] in this driver. I was looking at it, and in the process noticed this: $ grep 'numQueues.*=' drivers/net/ethernet/freescale/ucc_geth.c .numQueuesTx = 1, .numQueuesRx = 1, $ My interpr

Re: [PATCH] powerpc: dont include 8250 pre-setup if 8250 driver isn't enabled.

2012-02-02 Thread Paul Gortmaker
On Fri, Jan 20, 2012 at 10:50 PM, Paul Gortmaker wrote: > The legacy_serial code was setting things up based on the assumption > that the main 8250 driver would be loaded shortly after.  But some > randconfigs exposed an issue where early debug (UDB) UART support was > enabled, yet t

Re: [PATCH][v2] powerpc/85xx: Fix compiler error with THIS_MODULE and related

2012-02-01 Thread Paul Gortmaker
merged into the 3.2 baseline, since the export.h change was news in the last release, and not anything new for 3.3. Acked-by: Paul Gortmaker Paul. > arch/powerpc/sysdev/fsl_85xx_l2ctlr.c:229:20: error: expected declaration > specifiers or '...' before string constant > cc1: w

Re: [PATCH] powerpc: Define cpus_in_crash only for SMP kernels

2012-02-01 Thread Paul Gortmaker
On 12-02-01 01:55 PM, Paul E. McKenney wrote: > The variable cpus_in_crash gets a defined-but-not-used warning when > CONFIG_SMP=n, so fix by placing the variable under ifdef. I think Ben already merged a similar fix. http://patchwork.ozlabs.org/patch/136549/ P. > > Signed-off-by: Paul E. McKe

Re: [PATCH] powerpc/85xx: fix build failure from p1022 in SMP defconfig

2012-01-21 Thread Paul Gortmaker
On Sat, Jan 21, 2012 at 9:52 AM, Paul Gortmaker wrote: > Using the configs/mpc85xx_smp_defconfig shows this build error: > > arch/powerpc/platforms/85xx/p1022_ds.c:341: error: 'udbg_progress' undeclared > here (not in a function) > > Adding in the obvious missi

[PATCH] powerpc/85xx: fix build failure from p1022 in SMP defconfig

2012-01-21 Thread Paul Gortmaker
Using the configs/mpc85xx_smp_defconfig shows this build error: arch/powerpc/platforms/85xx/p1022_ds.c:341: error: 'udbg_progress' undeclared here (not in a function) Adding in the obvious missing header fixes it. Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/plat

[PATCH] powerpc: dont include 8250 pre-setup if 8250 driver isn't enabled.

2012-01-20 Thread Paul Gortmaker
. Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/kernel/legacy_serial.c b/arch/powerpc/kernel/legacy_serial.c index 3fea368..9a82a49 100644 --- a/arch/powerpc/kernel/legacy_serial.c +++ b/arch/powerpc/kernel/legacy_serial.c @@ -413,6 +413,14 @@ void __init find_legacy_serial_ports(void

[PATCH] powerpc/85xx: fix Kconfig warning about missing 8250 dependency

2012-01-19 Thread Paul Gortmaker
hich has unmet direct dependencies (HAS_IOMEM && SERIAL_8250_EXTENDED) Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/platforms/85xx/Kconfig b/arch/powerpc/platforms/85xx/Kconfig index d7946be..b221236 100644 --- a/arch/powerpc/platforms/85xx/Kconfig +++ b/arch/powerpc/platfo

[PATCH/RFC] gianfar: Add support for byte queue limits.

2012-01-06 Thread Paul Gortmaker
ant it, and it cleans up the readability somewhat as well. Signed-off-by: Paul Gortmaker --- [Too late for 3.3, but I figured I might as well get feedback from people in the interim. Passes basic boot test with NFS root on sbc8548 board.] diff --git a/drivers/net/ethernet/freescale/gianfar.c

[GIT PULL] tty-next: Fix Fsl 8250 BRK bug

2011-12-06 Thread Paul Gortmaker
aulg/linux fsl-uart-fix Paul Gortmaker (6): serial: move struct uart_8250_port from 8250.c to 8250.h serial: clean up parameter passing for 8250 Rx IRQ handling serial: export the key functions for an 8250 IRQ handler serial: make 8250 timeout use the specified IRQ handler

[PATCH] sbc834x: put full compat string in board match check

2011-12-05 Thread Paul Gortmaker
ble. Just add the E to the board detection string since all boards I've seen were manufactured with the E versions. Signed-off-by: Paul Gortmaker diff --git a/arch/powerpc/platforms/83xx/sbc834x.c b/arch/powerpc/platforms/83xx/sbc834x.c index af41d8c..f5a783a 100644 --- a/arch/powerpc

[PATCH 6/6] serial: add irq handler for Freescale 16550 errata.

2011-12-04 Thread Paul Gortmaker
s16550" is used in the compatible field of the serial device to mark UARTs known to have this issue. Thanks to Scott Wood for providing the errata data which led to a much cleaner fix. Signed-off-by: Paul Gortmaker --- arch/powerpc/kernel/legacy_serial.c |3 ++ drivers/tty/serial/8250_fsl

[PATCH 4/6] serial: make 8250 timeout use the specified IRQ handler

2011-12-04 Thread Paul Gortmaker
le_irq and call that, which for most will still be serial8250_default_handle_irq. Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/drivers/tty/serial/8250.c b/drivers/tty/serial/8250.c index 91afe7a..9e7780d

[PATCH 2/6] serial: clean up parameter passing for 8250 Rx IRQ handling

2011-12-04 Thread Paul Gortmaker
done. The value in this cleanup, is that it clarifies the API of the receive_chars prior to exporting it to other 8250-like drivers for shared usage. Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250.c | 17 +++-- 1 files changed, 11 insertions(+), 6 deletions(-) diff --

[PATCH 5/6] serial: manually inline serial8250_handle_port

2011-12-04 Thread Paul Gortmaker
handlers need to provide if not using serial8250_default_handle_irq. Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250.c | 23 --- 1 files changed, 8 insertions(+), 15 deletions(-) diff --git a/drivers/tty/serial/8250.c b/drivers/tty/serial/8250.c index 9e7780d

[PATCH 1/6] serial: move struct uart_8250_port from 8250.c to 8250.h

2011-12-04 Thread Paul Gortmaker
Since we want to promote sharing and move away from one single uart driver with a bunch of platform specific bugfixes all munged into one, relocate some header like material from the C file to the header. Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250.c | 26

[PATCH 3/6] serial: export the key functions for an 8250 IRQ handler

2011-12-04 Thread Paul Gortmaker
ound their own unique errata issues. The function names are given a serial8250 prefix, since they are now entering the global namespace. Signed-off-by: Paul Gortmaker --- drivers/tty/serial/8250.c | 29 +++-- include/linux/serial_8250.h |4 2 files c

[PATCH 0/6] RFCv2 Fix Fsl 8250 BRK bug

2011-12-04 Thread Paul Gortmaker
o all. (Again, the dts update from Kumar isn't shown here). Thanks to all who provided the feedback on v1. Paul. -- Paul Gortmaker (6): serial: move struct uart_8250_port from 8250.c to 8250.h serial: clean up parameter passing for 8250 Rx IRQ handling serial: export the key functi

Re: [PATCH 3/3] 8250: add workaround for MPC8[356]xx UART break IRQ storm

2011-12-02 Thread Paul Gortmaker
On 11-12-02 06:30 AM, Alan Cox wrote: >>> OK, I'll simply change the above to CONFIG_PPC then. >> >> It does, the bug is in the uart IP which I don't think we ever plan on >> fixing, so 32 or 64-bit parts will have it for ever and ever ;) > > It should be runtime selected, there should be no ifde

Re: [PATCH 3/3] 8250: add workaround for MPC8[356]xx UART break IRQ storm

2011-12-01 Thread Paul Gortmaker
On Thu, Dec 1, 2011 at 7:57 PM, Alan Cox wrote: > >> @@ -1553,7 +1554,15 @@ static void serial8250_handle_port(struct >> uart_8250_port *up) >>       spin_lock_irqsave(&up->port.lock, flags); >> >> -     status = serial_inp(up, UART_LSR); >> +     /* Workaround for IRQ storm errata on break with F

Re: [PATCH 1/3] serial: make bugs field not specific to 8250 type uarts.

2011-12-01 Thread Paul Gortmaker
On Thu, Dec 1, 2011 at 7:51 PM, Alan Cox wrote: >> Make the bugs field part of the globally visible struct >> uart_port and remove the 8250 specific one. > > Except all the bits in it are 8250 specific things or names that are > meaningless in generic form - no. I also don't want to encourage flag

Re: [RFC][PATCH] update FSL 16550 nodes to have...

2011-12-01 Thread Paul Gortmaker
On Mon, Nov 28, 2011 at 3:02 PM, Kumar Gala wrote: > Signed-off-by: Kumar Gala > --- > * Need to fixup the commit message I had this written when I was thinking of re-sending the dts with the other three, that is before I realized the dts patch would then overwhelm the other patches completely.

Re: [PATCH 3/3] 8250: add workaround for MPC8[356]xx UART break IRQ storm

2011-12-01 Thread Paul Gortmaker
On 11-12-01 06:51 PM, Scott Wood wrote: > On 12/01/2011 05:47 PM, Paul Gortmaker wrote: >> diff --git a/include/linux/serial_8250.h b/include/linux/serial_8250.h >> index 8c660af..b0f4042 100644 >> --- a/include/linux/serial_8250.h >> +++ b/include/linux/seria

[PATCH 3/3] 8250: add workaround for MPC8[356]xx UART break IRQ storm

2011-12-01 Thread Paul Gortmaker
s16550" is used in the compatible field of the serial device to mark UARTs known to have this issue. Thanks to Scott Wood for providing the errata data which led to a much cleaner fix. Signed-off-by: Paul Gortmaker --- arch/powerpc/kernel/legacy_serial.c | 11 +++ drivers/tt

[PATCH 1/3] serial: make bugs field not specific to 8250 type uarts.

2011-12-01 Thread Paul Gortmaker
so is that it helps pave the way for allowing arch or platform specific code to pass in information to the specific uart drivers about uart bugs known to impact certain platforms that would otherwise be hard to detect from within the context of the driver itself. Signed-off-by: Paul Gortmaker

[PATCH 0/3] RFC Fix Fsl 8250 BRK bug via letting plat code set bugs

2011-12-01 Thread Paul Gortmaker
1D, where sysrq was useless before; with this fix, it works as expected. Thanks, Paul. [1] http://patchwork.ozlabs.org/patch/46609/ [2] http://patchwork.ozlabs.org/patch/128070/ Paul Gortmaker (3): serial: make bugs field not specific to 8250 type uarts. serial: allow passing in ha

[PATCH 2/3] serial: allow passing in hardware bug info via platform device

2011-12-01 Thread Paul Gortmaker
can pass in this information. Also don't do a blanket overwrite of the bugs setting in the 8250.c driver. Finally, relocate the known bug #define list to a globally visible header so that the arch can assign any appropriate values from the list. Signed-off-by: Paul Gortmaker --- driver

Re: [PATCH] 8250: add workaround for MPC8[356]xx UART break IRQ storm

2011-11-24 Thread Paul Gortmaker
On 11-11-24 03:14 AM, Kumar Gala wrote: > > On Feb 26, 2010, at 1:25 PM, Paul Gortmaker wrote: > >> Sending a break on the SOC UARTs found in some MPC83xx/85xx/86xx >> chips seems to cause a short lived IRQ storm (/proc/interrupts >> typically shows somewhere

[GIT PULL] SRIO powerpc compile fix.

2011-11-14 Thread Paul Gortmaker
This fixes a compile regression in powerpc's serial rapidio, caused by the module.h split. It wasn't seen in defconfig/linux-next builds since it isn't enabled anywhere by default. Thanks, Paul. The following changes since commit 1ea6b8f48918282bdca0b32a34095504ee65bab5: Linux 3.2-rc1 (2011-

Re: [PATCH] fsl-rio: fix compile error

2011-11-11 Thread Paul Gortmaker
On 11-11-11 08:48 AM, Liu Gang wrote: > The "#include " was replaced by "#include " > in the patch "powerpc: various straight conversions from module.h --> > export.h". > This will cause the following compile problem: > arch/powerpc/sysdev/fsl_rio.c: In function 'fsl_rio_mcheck_exception': > arch/

Re: [PATCH v3] powerpc: 85xx: separate e500 from e500mc

2011-08-10 Thread Paul Gortmaker
On 11-08-10 12:01 PM, Scott Wood wrote: > On 08/10/2011 10:39 AM, Paul Gortmaker wrote: >> On Wed, Aug 10, 2011 at 1:21 AM, Baruch Siach wrote: >>> CONFIG_E500MC breaks e500/e500v2 systems. It defines L1_CACHE_SHIFT to 6, >>> thus >>> breaking clear_pages(

Re: [PATCH v3] powerpc: 85xx: separate e500 from e500mc

2011-08-10 Thread Paul Gortmaker
On Wed, Aug 10, 2011 at 1:21 AM, Baruch Siach wrote: > CONFIG_E500MC breaks e500/e500v2 systems. It defines L1_CACHE_SHIFT to 6, thus > breaking clear_pages(), probably others too. > > This patch adds a new "Processor Type" entry for e500mc, and makes e500 > systems > depend on PPC_E500_V1_V2. I

<    1   2   3   4   >