[PATCH v2] arch: configuration issue, random return value when disable 'CONFIG_BUG'

2013-05-25 Thread Chen Gang
If user disable 'CONFIG_BUG' in menuconfig, many areas not only will report compiling warnings, but also may return a random value from the function when BUG(). In fact, most of architectures need always let 'CONFIG_BUG' enabled, so '#ifdef CONFIG_BUG' is already not 'generic' enough in "include/

[PATCH] ppc: remove unused devm_ioremap_prot

2013-05-25 Thread Kyle McMartin
Added in 2008, but has never had any in-tree users, and no other architectures provide it. Also remove the header includes that the commit b41e5fff added to asm/io.h, and add an include of mutex.h for asm/eeh.h, which relied on io.h including device.h to provide it. Tested ppc64_defconfig in an xc

Re: [PATCH 2/2] net: mv643xx_eth: proper initialization for Kirkwood SoCs

2013-05-25 Thread Andrew Lunn
> > Why are you not keen on this? It seems like normal device driver > > practice, that is what the data field of of_device_id is typically > > used for.. > > I'm not keen on it because we don't have a document saying "All kirkwood > SoCs need PSC1 set to X after reset." We know it, but have we t

Re: [PATCH 1/2] ARM: kirkwood: proper retain MAC address workaround on DT ethernet

2013-05-25 Thread David Miller
From: Sebastian Hesselbarth Date: Wed, 22 May 2013 22:04:01 +0200 > + memcpy((void *)p->value, reg, 6); This cast is completely unnecessary, non-void to void pointer casts are automatic. If it is necessary, because p->value is const, then you are trying to change something b

Re: [PATCH] ppc: remove unused devm_ioremap_prot

2013-05-25 Thread Benjamin Herrenschmidt
On Sat, 2013-05-25 at 12:54 -0400, Kyle McMartin wrote: > Added in 2008, but has never had any in-tree users, and no other > architectures provide it. Also remove the header includes that > the commit b41e5fff added to asm/io.h, and add an include of mutex.h > for asm/eeh.h, which relied on io.h in

[PATCH v3, part1 03/10] PCI: Convert alloc_pci_dev(void) to pci_alloc_dev(bus) instead

2013-05-25 Thread Jiang Liu
From: Gu Zheng Use the new pci_alloc_dev(bus) to replace the existing using of alloc_pci_dev(void). v2: Follow Bjorn's correction to move pci_bus_put() to pci_release_dev() instead. v3: release reference to bus on error recovery path Signed-off-by: Gu Zheng Signed-off-by: Jiang Liu Cc: