Re: Merge dtc

2007-10-16 Thread Benjamin Herrenschmidt
You must have missed the thread where various people where complaining about how powerpc is the only architecture where they can't build kernels without some external tool that they don't have, etc., etc. We thought about shipping compiled DTBs for various platforms, but the problem there

Re: Merge dtc

2007-10-16 Thread Benjamin Herrenschmidt
On Tue, 2007-10-16 at 00:50 -0500, Kumar Gala wrote: Just out of interest who's complaining? We don't include mkimage for u-boot related builds and I haven't seen any gripes related to that. It's a pain in the neck since those are built even for things that don't need them (such as 4xx

Re: boards in arch/ppc - arch/powerpc for 85xx

2007-10-16 Thread Wolfgang Denk
Dear Kumar, in message [EMAIL PROTECTED] you wrote: 1. what processors does this actually support/run on. I'm guessing MPC8541, MPC8555? But not really sure IIRC there are 4 different versions of the board: - 8540, 8541, 8555 8560 do they all boot/run with the code that exists

Re: boards in arch/ppc - arch/powerpc for 85xx

2007-10-16 Thread Stefan Roese
On Tuesday 16 October 2007, Kumar Gala wrote: On Oct 16, 2007, at 12:28 AM, Stefan Roese wrote: On Tuesday 16 October 2007, Kumar Gala wrote: I'm willing to look into doing the port over, but would need some help testing. Yes, it would be greatly appreciated if you could start this

Re: Merge dtc

2007-10-16 Thread Kumar Gala
On Oct 16, 2007, at 1:01 AM, Benjamin Herrenschmidt wrote: On Tue, 2007-10-16 at 00:50 -0500, Kumar Gala wrote: Just out of interest who's complaining? We don't include mkimage for u-boot related builds and I haven't seen any gripes related to that. It's a pain in the neck since those

Re: Merge dtc

2007-10-16 Thread Kumar Gala
On Oct 16, 2007, at 1:00 AM, Benjamin Herrenschmidt wrote: You must have missed the thread where various people where complaining about how powerpc is the only architecture where they can't build kernels without some external tool that they don't have, etc., etc. We thought about

dtc: Improve support for string escapes

2007-10-16 Thread David Gibson
dtc supports the use of C-style escapes (\n, \t and so forth) in string property definitions via the data_copy_escape_string() function. However, while it supports the most common escape characters, it doesn't support the full set that C does, which is a potential gotcha. Worse, a bug in the

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Herbert Xu
Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: Note: I use msleep_interruptible(1); just like napi_disable(). However I'm not too happy that the hot loop that results of a pending signal here will spin without even a cpu_relax ... what do you guys think would be the best way to handle this ?

Re: [patch 12/13] fb: Move and rename extern declaration for global_mode_option

2007-10-16 Thread Geert Uytterhoeven
On Mon, 15 Oct 2007, Andrew Morton wrote: On Fri, 12 Oct 2007 16:51:04 +0200 Geert Uytterhoeven [EMAIL PROTECTED] wrote: -if (mode_option || (mode_option = global_mode_option)) { +if (mode_option || (mode_option = fb_mode_option)) { I guess that

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread David Miller
From: Benjamin Herrenschmidt [EMAIL PROTECTED] Date: Tue, 16 Oct 2007 17:37:03 +1000 On Tue, 2007-10-16 at 14:06 +0800, Herbert Xu wrote: Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: Note: I use msleep_interruptible(1); just like napi_disable(). However I'm not too happy that the

Re: [PATCH] powerpc64 vDSO: linker script indentation

2007-10-16 Thread Benjamin Herrenschmidt
On Mon, 2007-10-15 at 20:43 -0700, Roland McGrath wrote: This cleans up the formatting in the vDSO linker script, mostly just the use of whitespace. It's intended to approximate the kernel standard conventions for indenting C, treating elements of the linker script about like initialized

Re: [PATCH] powerpc32 vDSO: linker script indentation

2007-10-16 Thread Benjamin Herrenschmidt
On Mon, 2007-10-15 at 20:44 -0700, Roland McGrath wrote: This cleans up the formatting in the vDSO linker script, mostly just the use of whitespace. It's intended to approximate the kernel standard conventions for indenting C, treating elements of the linker script about like initialized

Re: [PATCH v2 2/2] [POWERPC] MPC8568E-MDS: add support for flash

2007-10-16 Thread Anton Vorontsov
On Mon, Oct 15, 2007 at 09:33:40PM +0400, Sergei Shtylyov wrote: Anton Vorontsov wrote: MPC8568E-MDS have 1 32MB Spansion x16 CFI flash chip. Let's use it. Signed-off-by: Anton Vorontsov [EMAIL PROTECTED] diff --git a/arch/powerpc/boot/dts/mpc8568mds.dts

Re: Merge dtc

2007-10-16 Thread Josh Boyer
On Tue, 2007-10-16 at 16:01 +1000, Benjamin Herrenschmidt wrote: On Tue, 2007-10-16 at 00:50 -0500, Kumar Gala wrote: Just out of interest who's complaining? We don't include mkimage for u-boot related builds and I haven't seen any gripes related to that. It's a pain in the neck since

Re: libfdt: libfdt_env.h must be included first

2007-10-16 Thread Jon Loeliger
So, like, the other day David Gibson mumbled: libfdt.h currently includes fdt.h, then libfdt_env.h. This is incorrect, because depending on the environment into which libfdt is embedded, libfdt_env.h may be needed to define datatypes used in fdt.h. This patch corrects the problem.

Re: dtc: Don't delete *.test.dtb between testgroups

2007-10-16 Thread Jon Loeliger
So, like, the other day David Gibson mumbled: The dtc/libfdt testsuite creates a number of .dtb files during its run. To ensure a clean test run, these are currently deleted before each group of tests. This is, in fact, a mistake, since if something goes wrong in the first group of tests,

Re: libfdt: Add functions for handling the compatible property

2007-10-16 Thread Jon Loeliger
So, like, the other day David Gibson mumbled: This patch adds functions for dealing with the compatible property. fdt_node_check_compatible() can be used to determine whether a node is compatible with a given string and fdt_node_offset_by_compatible() locates nodes with a given compatible

Re: dtc: Improve support for string escapes

2007-10-16 Thread Jon Loeliger
So, like, the other day David Gibson mumbled: dtc supports the use of C-style escapes (\n, \t and so forth) in string property definitions via the data_copy_escape_string() function. However, while it supports the most common escape characters, it doesn't support the full set that C does,

DTC Updates

2007-10-16 Thread Jon Loeliger
Guys and Dolls, I pushed out all of David's recent DTC and libfdt patches. I also jammed the following patch into the mix as well. HTH, jdl commit 9e32930ebcacfcf7cb7c1c2b8e776eb3957cf6cb Author: Jon Loeliger [EMAIL PROTECTED] Date: Tue Oct 16 07:35:38 2007 -0500 Restore warning

Re: Merge dtc

2007-10-16 Thread Kumar Gala
On Oct 16, 2007, at 8:17 AM, Grant Likely wrote: On 10/15/07, David Gibson [EMAIL PROTECTED] wrote: This very large patch incorporates a copy of dtc into the kernel source, in arch/powerpc/boot/dtc-src. This means that dtc is no longer an external dependency to build kernels with

Please pull from 'fixes-2.6.24'

2007-10-16 Thread Kumar Gala
(Note, this tree is based on a recent linus tree) Please pull from 'fixes-2.6.24' branch of master.kernel.org:/pub/scm/linux/kernel/git/galak/powerpc.git fixes-2.6.24 to receive the following updates: arch/powerpc/math-emu/math.c | 13 + arch/powerpc/sysdev/fsl_pci.c |

Serial init / BRG

2007-10-16 Thread Alan Bennett
For some reason I'm unable to track down the right sequence to initialize my serial ports. (note: my console is working, but attempts to initialize SCC1 and SCC4 fail). My console has the luxury of having uboot initialize it and its brg, but SCC1 / SCC4 aren't so lucky. A few questions: To

Re: [PATCH 4/5] ibmebus: Move to of_device and of_platform_driver, match eHCA and eHEA drivers

2007-10-16 Thread Joachim Fenkes
On Tuesday 09 October 2007 10:21, Jan-Bernd Themann wrote: Roland Dreier [EMAIL PROTECTED] wrote on 03.10.2007 20:05:44: Replace struct ibmebus_dev and struct ibmebus_driver with struct of_device and struct of_platform_driver, respectively. Match the external ibmebus interface

Re: [PATCH v2 4/7] bestcomm: core bestcomm support for Freescale MPC5200

2007-10-16 Thread tnt
On 10/15/07, Matt Sealey [EMAIL PROTECTED] wrote: My nits: Grant Likely wrote: From: Sylvain Munaut [EMAIL PROTECTED] +static int __devinit +bcom_engine_init(void) Why bcom and not bestcomm? I can type 'bcom' twice as fast. :-) bcom is a suitable shortening; I'm not concerned

ioctl32: Unknown cmd

2007-10-16 Thread Geert Uytterhoeven
Hi Arnd, Jens, The recent (post 2.6.23) changes to compat_ioctl made the reporting of unsupported ioctls more verbose. E.g. on the PS3 I get: | ioctl32(cdrom_id:608): Unknown cmd fd(3) cmd(5331){t:'S';sz:0} arg() on /dev/.tmp-11-0 | ioctl32(hdparm:1427): Unknown cmd fd(3)

Re: [PATCH 4/5] ibmebus: Move to of_device and of_platform_driver, match eHCA and eHEA drivers

2007-10-16 Thread Jeff Garzik
Joachim Fenkes wrote: On Tuesday 09 October 2007 10:21, Jan-Bernd Themann wrote: Roland Dreier [EMAIL PROTECTED] wrote on 03.10.2007 20:05:44: Replace struct ibmebus_dev and struct ibmebus_driver with struct of_device and struct of_platform_driver, respectively. Match the external

[PATCH 0/5] IB/ehca: SRQ and MR/MW fixes

2007-10-16 Thread Joachim Fenkes
Here are some more fixes for the eHCA driver, fixing some problems we found during internal system test. [1/5] fixes the QP pointer determination for SRQ base QPs [2/5] fixes a masking error in {,re}reg_phys_mr() [3/5] fixes a bug in alloc_fmr() and simplifies some code [4/5] refactors

[PATCH 1/5] IB/ehca: Supply QP token for SRQ base QPs

2007-10-16 Thread Joachim Fenkes
Because hardware reports the SRQ token in RWQEs of SRQ base QPs, supply the base QP token as SRQ token, so we can properly find the SRQ base QP. Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] --- drivers/infiniband/hw/ehca/ehca_qp.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)

[PATCH 2/5] IB/ehca: Fix masking error in {,re}reg_phys_mr()

2007-10-16 Thread Joachim Fenkes
Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] --- drivers/infiniband/hw/ehca/ehca_mrmw.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/ehca/ehca_mrmw.c b/drivers/infiniband/hw/ehca/ehca_mrmw.c index da88738..16c9efd 100644 ---

[PATCH 3/5] IB/ehca: Fix ehca_encode_hwpage_size() and alloc_fmr()

2007-10-16 Thread Joachim Fenkes
Simplify ehca_encode_hwpage_size(), fixing an infinite loop for pgsize == 0 in the process. Fix the bug in alloc_fmr() that triggered the loop. Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] --- drivers/infiniband/hw/ehca/ehca_mrmw.c | 15 --- 1 files changed, 4 insertions(+), 11

Re: [PATCH 4/5] ibmebus: Move to of_device and of_platform_driver, match eHCA and eHEA drivers

2007-10-16 Thread Stephen Rothwell
On Tue, 16 Oct 2007 23:20:00 +0200 Joachim Fenkes [EMAIL PROTECTED] wrote: One small change - I intend to remove the name and owner fields from struct of_platform_driver, so you should not bother initialising the name field and just initialise the name field of the embedded struct device_driver

[PATCH 4/5] IB/ehca: Change meaning of hca_cap_mr_pgsize

2007-10-16 Thread Joachim Fenkes
ehca_shca.hca_cap_mr_pgsize now contains all supported page sizes ORed together. This makes some checks easier to code and understand, plus we can return this value verbatim in query_hca(), fixing a problem with SRP (reported by Anton Blanchard -- thanks!). Signed-off-by: Joachim Fenkes [EMAIL

[PATCH 5/5] IB/ehca: Enable large page MRs by default

2007-10-16 Thread Joachim Fenkes
Signed-off-by: Joachim Fenkes [EMAIL PROTECTED] --- drivers/infiniband/hw/ehca/ehca_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/infiniband/hw/ehca/ehca_main.c b/drivers/infiniband/hw/ehca/ehca_main.c index d477dc3..2f51c13 100644 ---

Re: Serial init / BRG

2007-10-16 Thread Scott Wood
Alan Bennett wrote: A few questions: To use 3 brgs, should I have 1 brg entry and X reg values or three brg entries in my device tree? The current brg entry describes all 8 BRGs... don't touch it. :-) what are the third and fourth reg values of the brg item in the device tree. I'll need

Re: ioctl32: Unknown cmd

2007-10-16 Thread Jens Axboe
On Tue, Oct 16 2007, Geert Uytterhoeven wrote: Hi Arnd, Jens, The recent (post 2.6.23) changes to compat_ioctl made the reporting of unsupported ioctls more verbose. E.g. on the PS3 I get: | ioctl32(cdrom_id:608): Unknown cmd fd(3) cmd(5331){t:'S';sz:0} arg() on

RE: Refactor booting-without-of.txt

2007-10-16 Thread Stephen Neuendorffer
On a similar note, is there interest in actually factoring the device tree code out from the different architectures into a common codebase? Steve -Original Message- From: [EMAIL PROTECTED] g [mailto:[EMAIL PROTECTED] zlabs.org] On Behalf Of Grant Likely Sent: Monday, October

Re: Refactor booting-without-of.txt

2007-10-16 Thread Grant Likely
On 10/16/07, Stephen Neuendorffer [EMAIL PROTECTED] wrote: On a similar note, is there interest in actually factoring the device tree code out from the different architectures into a common codebase? It's already happened somewhat. (Look in drivers/of and include/linux/of*.h) However, I don't

RE: Refactor booting-without-of.txt

2007-10-16 Thread Stephen Neuendorffer
-Original Message- From: [EMAIL PROTECTED] g [mailto:[EMAIL PROTECTED] zlabs.org] On Behalf Of David Gibson Sent: Monday, October 15, 2007 8:24 PM To: Grant Likely Cc: Olof Johansson; linuxppc-dev; [EMAIL PROTECTED] Subject: Re: Refactor booting-without-of.txt On Mon, Oct

Re: [UNTESTED PATCH v2] 8xx: Convert mpc866ads to the new device binding.

2007-10-16 Thread Scott Wood
Vitaly Bordug wrote: Hi Scott, orig one does not build (not your issue apparently): CC drivers/net/fs_enet/fs_enet-main.o drivers/net/fs_enet/fs_enet-main.c: In function `fs_enet_probe': drivers/net/fs_enet/fs_enet-main.c:1252: error: implicit declaration of function

current -git adbhid.c build error

2007-10-16 Thread Joseph Fannin
Hello. Commit b981d8b3f5e008ff10d993be633ad00564fc22cd in Linus's git tree did this: @@ -374,9 +373,9 @@ adbhid_input_keycode(int id, int scancode, int repeat) #endif /* CONFIG_PPC_PMAC */ } - if (adbhid[id]-keycode[keycode]) { - input_report_key(adbhid[id]-input,

RE: [PATCH v2] Device tree bindings for Xilinx devices

2007-10-16 Thread Stephen Neuendorffer
+ n) Xilinx EMAC and Xilinx TEMAC + + Xilinx Ethernet devices. Uses common properties from other Ethernet + devices with the following constraints: + + Required properties: +- compatible : Must include one of: xilinx,plb-temac, + xilinx,plb-emac,

Describing devices in the device tree

2007-10-16 Thread Alan Bennett
I'm using a modified ep8248e.dts to describe my hardware and I want to enable the use of 3 standard interrupts. 1. irq5 2. timer1 3. timer2 How bad does this look? soc -- cpm -- timer { device_type = timer;

Re: [UNTESTED PATCH v2] 8xx: Convert mpc866ads to the new device binding.

2007-10-16 Thread Vitaly Bordug
Hi Scott, orig one does not build (not your issue apparently): CC drivers/net/fs_enet/fs_enet-main.o drivers/net/fs_enet/fs_enet-main.c: In function `fs_enet_probe': drivers/net/fs_enet/fs_enet-main.c:1252: error: implicit declaration of function `SET_MODULE_OWNER'

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Stephen Hemminger
On Tue, 16 Oct 2007 15:49:52 +1000 Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: net: Add __napi_sycnhronize() to sync with napi poll The EMAC driver which needs to handle multiple devices with one NAPI instance implements its own per-channel disable bit. However, when setting such a bit,

vga16fb doesn't build on powerpc (vgacon_remap_base)

2007-10-16 Thread Joseph Fannin
vga16fb is an available config option on powerpc, but it won't link with my .config: ERROR: vgacon_remap_base [drivers/video/vga16fb.ko] undefined! I'm guessing this is because include/asm-powerpc/vga.h declares vgacon_remap_base: extern unsigned long vgacon_remap_base; ...but

Re: [PATCH 2/2] i2c: Add devtree-aware iic support for PPC4xx

2007-10-16 Thread Jean Delvare
On Mon, 15 Oct 2007 22:21:38 -0600, Grant Likely wrote: On 10/15/07, David Gibson [EMAIL PROTECTED] wrote: In fact I think it may be acceptle to do the idx++ thing in this situation. Bus numbers are ugly, but it's not the worst ugliness in the horrible mess that is the Linux i2c subsystem.

Re: [PATCH v2] pasemi: process i2c device tree entries at boot

2007-10-16 Thread Scott Wood
Olof Johansson wrote: On Mon, Oct 15, 2007 at 05:54:51PM -0500, Scott Wood wrote: Olof Johansson wrote: Setup i2c_board_info based on device tree contents. This has to be a device_initcall since we need PCI to be probed by the time we run it, but before the actual driver is initialized. Can

Re: [PATCH v2] Device tree bindings for Xilinx devices

2007-10-16 Thread Grant Likely
On 10/16/07, Stephen Neuendorffer [EMAIL PROTECTED] wrote: + n) Xilinx EMAC and Xilinx TEMAC + + Xilinx Ethernet devices. Uses common properties from other Ethernet + devices with the following constraints: + + Required properties: +- compatible : Must include one of:

Re: Refactor booting-without-of.txt

2007-10-16 Thread Grant Likely
On 10/16/07, Stephen Neuendorffer [EMAIL PROTECTED] wrote: How about just 'device-tree', referring to any source, and then of-device-tree and flat-device-tree to document how the device tree is constructed. The fact that the API is poorly named is something that can always be fixed (and

Re: ioctl32: Unknown cmd

2007-10-16 Thread Arnd Bergmann
On Tuesday 16 October 2007, Geert Uytterhoeven wrote: The recent (post 2.6.23) changes to compat_ioctl made the reporting of unsupported ioctls more verbose. E.g. on the PS3 I get: | ioctl32(cdrom_id:608): Unknown cmd fd(3) cmd(5331){t:'S';sz:0} arg() on /dev/.tmp-11-0 |

Re: Describing devices in the device tree

2007-10-16 Thread Grant Likely
On 10/16/07, Alan Bennett [EMAIL PROTECTED] wrote: I'm using a modified ep8248e.dts to describe my hardware and I want to enable the use of 3 standard interrupts. 1. irq5 2. timer1 3. timer2 How bad does this look? soc -- cpm -- timer {

New time code miscalculates cpu usage

2007-10-16 Thread Olof Johansson
Hi, Not sure when this started happening, but I wanted to report it. I'll start bisecting in a day or two if noone else has gotten around to looking at it: $ echo int main(void) { while(1); } test.c ; gcc test.c $ time ./a.out sleep 2 ; killall a.out real0m2.008s user0m4.014s sys

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Benjamin Herrenschmidt
So this is really just like synchronize_irq()? Using msleep is bogus because you want to spin, you are only waiting for a softirq on the other cpu to finish. If you wait for a whole millisecond and sleep that is far longer than the napi routine should take. You could even optimize it like

Re: [Linux-fbdev-devel] vga16fb doesn't build on powerpc (vgacon_remap_base)

2007-10-16 Thread Antonino A. Daplas
On Tue, 2007-10-16 at 15:14 -0400, Joseph Fannin wrote: vga16fb is an available config option on powerpc, but it won't link with my .config: ERROR: vgacon_remap_base [drivers/video/vga16fb.ko] undefined! I'm guessing this is because include/asm-powerpc/vga.h declares

Re: Serial init / BRG

2007-10-16 Thread Alan Bennett
The lockup is only the shell. I echo from ttyCPM0 console and it locks then I echo from an ssh session and the kernel messages still output from the ttyCPM0 console. NOTE: I found a severe issue that seems to have unlocked ttyCPM2. BUT ttyCPM1 still hangs. (a while back, we commented out the

Re: [PATCH 1/2] bootwrapper: Allow wrapper script to execute verbosely

2007-10-16 Thread Andreas Schwab
Grant Likely [EMAIL PROTECTED] writes: +# Allow for verbose output +if [ $V == 1 ]; then The correct syntax is [ a = b ] (and the quotes around 1 are not needed, btw.). Andreas. -- Andreas Schwab, SuSE Labs, [EMAIL PROTECTED] SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg,

Re: [PATCH 1/2] bootwrapper: Allow wrapper script to execute verbosely

2007-10-16 Thread Grant Likely
On 10/16/07, Andreas Schwab [EMAIL PROTECTED] wrote: Grant Likely [EMAIL PROTECTED] writes: +# Allow for verbose output +if [ $V == 1 ]; then The correct syntax is [ a = b ] (and the quotes around 1 are not needed, btw.). heh; I guess I'm not much of a shell programmer. :-) I'll respin

Please pull linux-2.6-mpc52xx.git

2007-10-16 Thread Grant Likely
Paul, Here is the addition of the bestcomm driver. I think this is ready to go in. There are remaining outstanding comments; but my opinion is that they should be addressed in subsequent patches (performance optimization for mp5200b boards and making the sram management code a generic interface

Re: [PATCH 2/2] bootwrapper: Bail from script if any command fails

2007-10-16 Thread David Gibson
On Tue, Oct 16, 2007 at 03:31:07PM -0600, Grant Likely wrote: From: Grant Likely [EMAIL PROTECTED] Add the 'set -e' command to the wrapper script so that if any command fails then the script will automatically exit Ah.. this will conflict with my patch to merge dtc (because that touches the

Re: DTC Updates

2007-10-16 Thread David Gibson
On Tue, Oct 16, 2007 at 08:32:45AM -0500, Jon Loeliger wrote: Guys and Dolls, I pushed out all of David's recent DTC and libfdt patches. I also jammed the following patch into the mix as well. HTH, jdl commit 9e32930ebcacfcf7cb7c1c2b8e776eb3957cf6cb Author: Jon Loeliger [EMAIL

Re: [PATCH 2/2] i2c: Add devtree-aware iic support for PPC4xx

2007-10-16 Thread David Gibson
On Tue, Oct 16, 2007 at 09:19:39PM +0200, Jean Delvare wrote: On Mon, 15 Oct 2007 22:21:38 -0600, Grant Likely wrote: On 10/15/07, David Gibson [EMAIL PROTECTED] wrote: In fact I think it may be acceptle to do the idx++ thing in this situation. Bus numbers are ugly, but it's not the

Re: [PATCH] net: Fix new EMAC driver for NAPI changes

2007-10-16 Thread Jeff Garzik
Benjamin Herrenschmidt wrote: net: Fix new EMAC driver for NAPI changes This fixes the new EMAC driver for the NAPI updates. The previous patch by Roland Dreier (already applied) to do that doesn't actually work. This applies on top of it makes it work on my test Ebony machine. This patch

dtc: Restore missing code for testcases

2007-10-16 Thread David Gibson
Recent commits 333542fabf8720b881e992a5abca32ef4bcb841a and fd1bf3a5ae46962528ef89a824261a88830758a2 added new testcases to dtc. However, although the testcases were added to the Makefile and run_tests.sh, one of the .c files for the testcase was omitted from the patch in each case. This patch

Re: [PATCH] net: Fix new EMAC driver for NAPI changes

2007-10-16 Thread Benjamin Herrenschmidt
On Tue, 2007-10-16 at 21:16 -0400, Jeff Garzik wrote: Benjamin Herrenschmidt wrote: net: Fix new EMAC driver for NAPI changes This fixes the new EMAC driver for the NAPI updates. The previous patch by Roland Dreier (already applied) to do that doesn't actually work. This applies on

RE: [PATCH v2] Device tree bindings for Xilinx devices

2007-10-16 Thread Josh Boyer
On Tue, 2007-10-16 at 16:23 -0700, Stephen Neuendorffer wrote: It occurs to me that the 'compatible' bindings should probably be the name of the preferred driver for the device. + l) Xilinx ML300 Framebuffer +- compatible : Must include xilinx,ml300-fb Should probably be

Re: [PATCH 2/2] bootwrapper: Bail from script if any command fails

2007-10-16 Thread Grant Likely
On 10/16/07, David Gibson [EMAIL PROTECTED] wrote: On Tue, Oct 16, 2007 at 03:31:07PM -0600, Grant Likely wrote: From: Grant Likely [EMAIL PROTECTED] Add the 'set -e' command to the wrapper script so that if any command fails then the script will automatically exit Ah.. this will

dtc: Improve -Odts output

2007-10-16 Thread David Gibson
This patch makes improvements to the way properties are printed when in dtc is producing dts output. - Characters which need escaping are now properly handled when printing properties as strings - The heuristics for what format to use for a property are improved so that

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Benjamin Herrenschmidt
napi: use non-interruptible sleep in napi_disable The current napi_disable() uses msleep_interruptible() but doesn't (and can't) exit in case there's a signal, thus ending up doing a hot spin without a cpu_relax. Use uninterruptible sleep instead. Signed-off-by: Benjamin Herrenschmidt [EMAIL

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Benjamin Herrenschmidt
net: Add __napi_synchronize() to sync with napi poll The EMAC driver which needs to handle multiple devices with one NAPI instance implements its own per-channel disable bit. However, when setting such a bit, it needs to synchronize with the poller (that is make sure that any pending poller

Re: [PATCH v2] Device tree bindings for Xilinx devices

2007-10-16 Thread Grant Likely
On 10/16/07, Grant Likely [EMAIL PROTECTED] wrote: On 10/16/07, Stephen Neuendorffer [EMAIL PROTECTED] wrote: -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Grant Likely Sent: Tuesday, October 16, 2007 12:36 PM To: Stephen Neuendorffer

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Jeff Garzik
Benjamin Herrenschmidt wrote: net: Add __napi_synchronize() to sync with napi poll The EMAC driver which needs to handle multiple devices with one NAPI instance implements its own per-channel disable bit. However, when setting such a bit, it needs to synchronize with the poller (that is

Re: [PATCH/RFC] net: Add __napi_sycnhronize() to sync with napi poll

2007-10-16 Thread Benjamin Herrenschmidt
\ Note: unfortunately, Jeff already picked up the EMAC patch without waiting for this to be sorted out (oops...). So if you agree with this patch, it would be nice to have it go in quickly or maybe via Jeff's tree to avoid breakage ? Not terribly important tho. Sorry, I thought that

[PATCH 1/2] [SPARC/64] Consolidate of_register_driver

2007-10-16 Thread Stephen Rothwell
Also of_unregister_driver. These will be shortly also used by the PowerPC code. Signed-off-by: Stephen Rothwell [EMAIL PROTECTED] --- arch/sparc/kernel/of_device.c | 20 arch/sparc64/kernel/of_device.c | 20 drivers/of/platform.c |

[PATCH 2/2] [POWERPC] Use of_register_driver to implement of_register_platform_driver

2007-10-16 Thread Stephen Rothwell
Also of_unregister{,_platform}_driver. Signed-off-by: Stephen Rothwell [EMAIL PROTECTED] --- arch/powerpc/kernel/of_platform.c | 20 include/asm-powerpc/of_platform.h | 10 -- 2 files changed, 8 insertions(+), 22 deletions(-) This depends on the Sparc part

Re: current -git adbhid.c build error

2007-10-16 Thread Paul Mackerras
Joseph Fannin writes: Commit b981d8b3f5e008ff10d993be633ad00564fc22cd in Linus's git tree did this: @@ -374,9 +373,9 @@ adbhid_input_keycode(int id, int scancode, int repeat) #endif /* CONFIG_PPC_PMAC */ } - if (adbhid[id]-keycode[keycode]) { -

[PATCH] powerpc: Fix 64 bits vDSO dwarf info for CR register

2007-10-16 Thread Benjamin Herrenschmidt
The current DWARF info for CR are incorrect causing the gcc unwinder to go to lunch if taking a segfault in the vdso. This patch fixes it. Problem identified by Andrew Haley, and fix provided by Jakub Jelinek (thanks !). Unfortunately, a bug in gcc cause it to not quite work either, but that is

[PATCH] powerpc: Fix 64 bits vDSO dwarf info for CR register

2007-10-16 Thread Benjamin Herrenschmidt
The current DWARF info for CR are incorrect causing the gcc unwinder to go to lunch if taking a segfault in the vdso. This patch fixes it. Problem identified by Andrew Haley, and fix provided by Jakub Jelinek (thanks !). Unfortunately, a bug in gcc cause it to not quite work either, but that is

Re: current -git adbhid.c build error

2007-10-16 Thread Dmitry Torokhov
On Wednesday 17 October 2007, Paul Mackerras wrote: It was a mis-merge between 555ddbb4, which made that change and *did* add the declaration of key, and 9a402b64, which deleted the line that the declaration of key was added to. Oops, sorry. We need the patch below. Dmitry, will you

Re: Merge dtc

2007-10-16 Thread David Gibson
On Tue, Oct 16, 2007 at 07:17:01AM -0600, Grant Likely wrote: On 10/15/07, David Gibson [EMAIL PROTECTED] wrote: This very large patch incorporates a copy of dtc into the kernel source, in arch/powerpc/boot/dtc-src. This means that dtc is no longer an external dependency to build kernels