Re: [PATCH 1/5] ptp: Added a brand new class driver for ptp clocks.

2010-08-18 Thread Richard Cochran
On Tue, Aug 17, 2010 at 05:22:43PM -0700, john stultz wrote: Why would system time not be adjusted to the PTP time? This is my main concern, that we're presenting a fractured API to userland. Suddenly there isn't just system time, but ptp time as well, and possibly multiple different ptp

cpm_i2c: write OK but read fail

2010-08-18 Thread Shawn Jin
Hi, My system is MPC870 based and I'm enabling the I2C driver and the ds1339 RTC driver. When the kernel tries to probe the RTC chip (slave address 0x68), the first write seems OK but the subsequent read times out. I applied the I2C/SPI microcode patch, which doesn't seem necessary though. Below

Re: [PATCH 1/5] ptp: Added a brand new class driver for ptp clocks.

2010-08-18 Thread Richard Cochran
On Tue, Aug 17, 2010 at 01:36:29PM +0200, Arnd Bergmann wrote: On Tuesday 17 August 2010, Richard Cochran wrote: I've been looking at offering the PTP clock as a posix clock, and it is not as hard as I first thought. The PTP clock or clocks just have to be registered as one of the

Re: How to use mpc8xxx_gpio.c device driver

2010-08-18 Thread Ravi Gupta
Hi, Thanks for all your replies. Now I want to enable interrupts on some GPIO pins(GPIO pins number 224, 225, 226, 227, 228 and 229 i.e gpiochip224's pins 0 to 5). I configured these pins as input(GPIOF_IN) during gpio request. I am able to request gpio pins successfully. #define GPIO_CHIP0_BASE

Re: [PATCH 1/5] ptp: Added a brand new class driver for ptp clocks.

2010-08-18 Thread Arnd Bergmann
On Wednesday 18 August 2010, Richard Cochran wrote: On Tue, Aug 17, 2010 at 01:36:29PM +0200, Arnd Bergmann wrote: On Tuesday 17 August 2010, Richard Cochran wrote: I've been looking at offering the PTP clock as a posix clock, and it is not as hard as I first thought. The PTP clock or

[PATCH 1/4] powerpc/47x: Make sure mcsr is cleared before enabling machine check interrupts

2010-08-18 Thread Dave Kleikamp
Clear the machine check syndrom register before enabling machine check interrupts. The initial state of the tlb can lead to parity errors being flagged early after a cold boot. Signed-off-by: Dave Kleikamp sha...@linux.vnet.ibm.com --- arch/powerpc/kernel/head_44x.S |4 1 files

[PATCH 0/4] *** SUBJECT HERE ***

2010-08-18 Thread Dave Kleikamp
Josh, Here are some bug fixes for the powerpc-4xx tree. It'd be nice if they could make it into 2.6.46. Thanks, Shaggy Dave Kleikamp (4): powerpc/47x: Make sure mcsr is cleared before enabling machine check interrupts powerpc/47x: Remove redundant line from cputable.c powerpc/4xx:

[PATCH 4/4] powerpc/47x: Add an isync before the tlbivax instruction

2010-08-18 Thread Dave Kleikamp
Signed-off-by: Dave Kleikamp sha...@linux.vnet.ibm.com --- arch/powerpc/mm/tlb_nohash_low.S |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/mm/tlb_nohash_low.S b/arch/powerpc/mm/tlb_nohash_low.S index cfa7682..b9d9fed 100644 ---

[PATCH 3/4] powerpc/4xx: Index interrupt stacks by physical cpu

2010-08-18 Thread Dave Kleikamp
The interrupt stacks need to be indexed by the physical cpu since the critical, debug and machine check handlers use the contents of SPRN_PIR to index the critirq_ctx, dbgirq_ctx, and mcheckirq_ctx arrays. Signed-off-by: Dave Kleikamp sha...@linux.vnet.ibm.com --- arch/powerpc/kernel/irq.c

[PATCH 2/4] powerpc/47x: Remove redundant line from cputable.c

2010-08-18 Thread Dave Kleikamp
There are two entries for .cpu_user_features in arch/powerpc/kernel/cputable.c. Remove the one that doesn't belong Signed-off-by: Dave Kleikamp sha...@linux.vnet.ibm.com --- arch/powerpc/kernel/cputable.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git

Re: [PATCH 0/4] Some 47x patches for the powerpc-4xx tree

2010-08-18 Thread Dave Kleikamp
Sorry! Forgot to change the subject. Shaggy On Wed, 2010-08-18 at 11:44 -0500, Dave Kleikamp wrote: Josh, Here are some bug fixes for the powerpc-4xx tree. It'd be nice if they could make it into 2.6.46. Thanks, Shaggy Dave Kleikamp (4): powerpc/47x: Make sure mcsr is cleared

[PATCH] powerpc/pci: fix checking for child bridges in PCI code.

2010-08-18 Thread Grant Likely
pci_device_to_OF_node() can return null, and list_for_each_entry will never enter the loop when dev is NULL, so it looks like this test is a typo. Reported-by: Julia Lawall ju...@diku.dk Signed-off-by: Grant Likely grant.lik...@secretlab.ca --- arch/powerpc/kernel/pci_of_scan.c |2 +- 1

[PATCH] Correct rtas_data_buf locking in dlpar code

2010-08-18 Thread Nathan Fontenot
The dlpar code can cause a deadlock to occur when making the RTAS configure-connector call. This occurs because we make kmalloc calls, which can block, while parsing the rtas_data_buf and holding the rtas_data_buf_lock. This an cause issues if someone else attempts to grab the

[no subject]

2010-08-18 Thread Wolfgang Denk
Dear Rupjyoti, drivers/ata/sata_dwc_460ex.c fails to build in current mainline: ... CC drivers/ata/sata_dwc_460ex.o drivers/ata/sata_dwc_460ex.c:43:1: warning: DRV_NAME redefined In file included from drivers/ata/sata_dwc_460ex.c:38: drivers/ata/libata.h:31:1: warning: this is the

Re: [PATCH 1/5] ptp: Added a brand new class driver for ptp clocks.

2010-08-18 Thread john stultz
On Wed, 2010-08-18 at 09:19 +0200, Richard Cochran wrote: On Tue, Aug 17, 2010 at 05:22:43PM -0700, john stultz wrote: So while to me, it think it would be more ideal (or maybe just less different) to have a read-only interface (like the RTC), leaving PTPd to manage offset calculations and

Re:

2010-08-18 Thread Stephen Rothwell
Hi Wolfgang, On Wed, 18 Aug 2010 22:56:46 +0200 Wolfgang Denk w...@denx.de wrote: drivers/ata/sata_dwc_460ex.c: At top level: drivers/ata/sata_dwc_460ex.c:1592: warning: 'struct of_device' declared inside parameter list drivers/ata/sata_dwc_460ex.c:1592: warning: its scope is only this

Re: [PATCH 0/4] Some 47x patches for the powerpc-4xx tree

2010-08-18 Thread Benjamin Herrenschmidt
On Wed, 2010-08-18 at 11:45 -0500, Dave Kleikamp wrote: Sorry! Forgot to change the subject. Shaggy On Wed, 2010-08-18 at 11:44 -0500, Dave Kleikamp wrote: Josh, Here are some bug fixes for the powerpc-4xx tree. It'd be nice if they could make it into 2.6.46. Yeah and I'm sure they

[PATCH v2 1/3] Ramdisk address was not copied correctly on kexec'ed kernel

2010-08-18 Thread Matthew McClintock
Signed-off-by: Matthew McClintock m...@freescale.com --- kexec/arch/ppc/fixup_dtb.c |2 +- kexec/arch/ppc/kexec-ppc.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/kexec/arch/ppc/fixup_dtb.c b/kexec/arch/ppc/fixup_dtb.c index 26c23a3..09f9ac1 100644 ---

[PATCH v2 0/3] Misc. bug fixes for ppc32

2010-08-18 Thread Matthew McClintock
This patch series fixes a few issues I have discovered with my previous patch series. Nothing new has been added. v2: Missed signoff, removed initializing variable twice Matthew McClintock (3): Ramdisk address was not copied correctly on kexec'ed kernel Prevent multiple reservations for

[PATCH v2 3/3] Prevent initrd-start and initrd-end from appearing multiple times

2010-08-18 Thread Matthew McClintock
We always remove the old entry, and add it back if it is needed on for the kexec'ed kernel Signed-off-by: Matthew McClintock m...@freescale.com --- kexec/arch/ppc/fixup_dtb.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/kexec/arch/ppc/fixup_dtb.c

[PATCH v2 2/3] Prevent multiple reservations for cpu-release-addr

2010-08-18 Thread Matthew McClintock
Currently, we can add a lot of reservations over a small range, this does a simple check to verify the previous entry is not the same as the current one and skips it if so Signed-off-by: Matthew McClintock m...@freescale.com --- kexec/arch/ppc/fixup_dtb.c | 28 +--- 1

Re: 64-bit ppc rwsem

2010-08-18 Thread Benjamin Herrenschmidt
On Tue, 2010-08-17 at 22:28 -0700, David Miller wrote: From: Benjamin Herrenschmidt b...@kernel.crashing.org Date: Wed, 18 Aug 2010 15:03:23 +1000 I tried various tricks but so far they didn't work. I'll have another look tomorrow, but I may end up having to keep all the crap typecasts.

Re: 64-bit ppc rwsem

2010-08-18 Thread David Miller
From: Benjamin Herrenschmidt b...@kernel.crashing.org Date: Thu, 19 Aug 2010 15:23:23 +1000 Similar here, but using atomic_long_t instead so it works for 32-bit too for me. I suppose we could make that part common indeed. What about asm-generic/rwsem-atomic.h or rwsem-cmpxchg.h ? Using

Re: [PATCH 1/5] ptp: Added a brand new class driver for ptp clocks.

2010-08-18 Thread Richard Cochran
On Wed, Aug 18, 2010 at 05:12:56PM -0700, john stultz wrote: On Wed, 2010-08-18 at 09:19 +0200, Richard Cochran wrote: The timer/alarm stuff is ancillary and is not at all necessary. It is just a nice to have. I will happily remove it, if it is too troubling for people. If there's a