Re: [PATCH 1/2] powerpc: i2c-mpc: preserve I2C clocking

On Tue, Mar 31, 2009 at 02:43:39PM +0200, Wolfgang Grandegger wrote:
> The I2c node property "fsl,preserve-clocking" allows to overtake the
> clock settings from the boot loader and avoids the hard-coded setting.

Hrm.  This is dubious.  The device tree should generally describe
hardware, not OS/driver behaviour which is what this appears to be
doing.  There are exceptions, but you need to justify them.

--

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Re: [PATCH 6/8] --- arch/powerpc/boot/dts/socrates.dts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)

On Tue, Mar 31, 2009 at 02:37:33PM +0200, Wolfgang Grandegger wrote:
> Index: linux-2.6/arch/powerpc/boot/dts/socrates.dts
> ===================================================================
> --- linux-2.6.orig/arch/powerpc/boot/dts/socrates.dts	2009-03-31 13:24:38.078719288 +0200
> +++ linux-2.6/arch/powerpc/boot/dts/socrates.dts	2009-03-31 13:47:56.986720719 +0200
>  <at>  <at>  -52,6 +52,7  <at>  <at> 
>  	soc8544 <at> e0000000 {
>  		#address-cells = <1>;
>  		#size-cells = <1>;
> +		device_type = "soc";

You should not be adding device_type to a new board's dts.  You
*especially* shouldn't be doing that with no patch comment explaining
why.

--

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Re: using IRQ1 in mpc5121ads

On Mon, Mar 30, 2009 at 10:33:11AM -0400, sylvain louchez wrote:
> Hello, I'm a newbie looking for where the documentation and implementation
> model can be found.
> 
>  
> 
> My custom driver is looking for an interrupt notification from the kernel -
> and it registers in the /proc/interrupts file as expected when installed,
> i.e.
> 
> $insmod custom_driver.ko gps_irq=<n>
> 
> **Note - this part's OK, I see the instance in /proc/interrupts after I load
> it**
> 
>  
> 
> On the hardware side the actual signal is a 1 Hz pulse (from GPS) routed to
> IRQ1 through the board's CPLD. This is the only interrupt on IRQ_1 so we can
> keep the logic simple i.e. on IRQ1 we can just jump to the service routine
> in a custom driver.
> 
>  
> 
> I understand 2 things have to be done:
> 
>  
> 
> 1. create an entry in the dts file; I think we've got this understood
> through advice already received - for example the new interrupt entry can be

Re: 8641D with linux 2.6.29 did'n show second cpu

On Mar 31, 2009, at 10:06 AM, Eduard Fuchs wrote:

> Hi,
>
> Am Dienstag 31 März 2009 15:20:21 schrieb Kumar Gala:
>> On Mar 31, 2009, at 5:15 AM, Eduard Fuchs wrote:
>>> Hi,
>>>
>>> I'm testing the evaluation board from freescale (MPC8641DHPCN) with
>>> the latest
>>> linux kernel (2.6.29) and the second core seems disabled. CONFIG_SPM
>>> is
>>> enabled but the /proc/cpuinfo shows only one cpu. With original
>>> 2.6.27 kernel
>>> both cpu cores appears in /proc/cpuinfo.
>>>
>>> Has anyone an idea how can i enable the second core?
>>
>> Are you using u-boot?  I can't remember if you need a newer u-boot to
>> deal with changes in how we bring up the second core.
>
> Yes. My u-boot version is 2009.03. Are there some additional config  
> keys for
> SMP support in u-boot?

It appears we introduced a bug in v2009.03 of u-boot.

- k

Re: [PATCH] Add hvc_console to MAINTAINERS

On Tue, 2009-03-31 at 09:29 -0400, Josh Boyer wrote:
> Add a MAINTAINERS entry for the hypervisor virtual console support
> used on IBM POWER servers.
> 
> Signed-off-by: Josh Boyer <jwboyer <at> linux.vnet.ibm.com>
> ---
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c5f4e9d..387ad45 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
>  <at>  <at>  -2176,6 +2176,11  <at>  <at>  W:       http://www.ia64-linux.org/
>  T:     git kernel.org:/pub/scm/linux/kernel/git/aegl/linux-2.6.git
>  S:     Maintained
>  
> +IBM HYPERVISOR VIRTUAL CONSOLE
> +P:     Several
> +L:     linuxppc-dev <at> ozlabs.org
> +S:     Maintained

I don't think it's the "IBM" Hypervisor virtual console driver anymore.
There are backends for Xen, Beat (Toshiba) and virtio. 

cheers

--

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au

Re: [PATCH] Add hvc_console to MAINTAINERS

On Wed, Apr 01, 2009 at 10:44:00AM +1100, Michael Ellerman wrote:
>On Tue, 2009-03-31 at 09:29 -0400, Josh Boyer wrote:
>> Add a MAINTAINERS entry for the hypervisor virtual console support
>> used on IBM POWER servers.
>> 
>> Signed-off-by: Josh Boyer <jwboyer <at> linux.vnet.ibm.com>
>> ---
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index c5f4e9d..387ad45 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>>  <at>  <at>  -2176,6 +2176,11  <at>  <at>  W:       http://www.ia64-linux.org/
>>  T:     git kernel.org:/pub/scm/linux/kernel/git/aegl/linux-2.6.git
>>  S:     Maintained
>>  
>> +IBM HYPERVISOR VIRTUAL CONSOLE
>> +P:     Several
>> +L:     linuxppc-dev <at> ozlabs.org
>> +S:     Maintained
>
>I don't think it's the "IBM" Hypervisor virtual console driver anymore.
>There are backends for Xen, Beat (Toshiba) and virtio. 

Was just going by the copyright in the files, and Documentation/powerpc/hvc*

Also, if there are other backends, then is the list specified really
correct?  Perhaps it should just be linux-kernel.

Guess that's the problem with trying to add a maintainers entry for something

Re: kexec: proc/device-tree/chosen/linux,kernel-end: No such file or directory

On Tue, 2009-03-31 at 18:25 +0200, Geert Uytterhoeven wrote:
> With today's kernel, I can no longer kexec any kernel on PS3:
> 
> | /proc/device-tree/chosen/linux,kernel-end: No such file or directory
> | Could not get memory layout
> 
> Today's bad version: 15f7176eb1cccec0a332541285ee752b935c1c85
> Yesterday's good version: 0d34fb8e93ceba7b6dad0062dbb4a0813bacd75b
> 
> Before I start bisecting (tomorrow or the day thereafter), anyone with an idea?

Hmm, not really.

Looking at the code the only obvious way you could end up without that
property is if you didn't have a /chosen?

cheers

--

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

_______________________________________________

Re: [PATCH] Add hvc_console to MAINTAINERS

On Tue, Mar 31, 2009 at 02:40:58PM -0700, Joe Perches wrote:
>On Tue, 2009-03-31 at 14:22 -0700, Andrew Morton wrote:
>> On Tue, 31 Mar 2009 14:10:23 -0700
>> Joe Perches <joe <at> perches.com> wrote:
>> > On Tue, 2009-03-31 at 09:29 -0400, Josh Boyer wrote:
>> > > Add a MAINTAINERS entry for the hypervisor virtual console support
>> > > used on IBM POWER servers.
>> > > diff --git a/MAINTAINERS b/MAINTAINERS
>> > > index c5f4e9d..387ad45 100644
>> > > --- a/MAINTAINERS
>> > > +++ b/MAINTAINERS
>> > >  <at>  <at>  -2176,6 +2176,11  <at>  <at>  W:       http://www.ia64-linux.org/
>> > >  T:     git kernel.org:/pub/scm/linux/kernel/git/aegl/linux-2.6.git
>> > >  S:     Maintained
>> > >  
>> > > +IBM HYPERVISOR VIRTUAL CONSOLE
>> > > +P:     Several
>> > > +L:     linuxppc-dev <at> ozlabs.org
>> > > +S:     Maintained
>> > > +
>> > >  IBM MCA SCSI SUBSYSTEM DRIVER
>> > >  P:     Michael Lang
>> > >  M:     langa2 <at> kph.uni-mainz.de
>> > 
>> > I think this sort of entry should not be acceptable.
>> > Maintainers should have individual names and contact addresses.
>> 
>> There isn't a maintainer.
>> 
>> But people send patches to the wrong list, so the ppc guys (who use

PowerPC 460EX AD7416 Temperature Sensor

Does anyone know if the I2C temperature sensor is functioning on the AMCC 
460EX?
When I do a cat /proc/ad7416 I get the following crash.

Unable to handle kernel paging request for data at address 0x00000000
Faulting instruction address: 0xc02cbe28
Oops: Kernel access of bad area, sig: 11 [#1]
PowerPC 44x Platform
NIP: c02cbe28 LR: c023cec8 CTR: c023d314
REGS: ef107d90 TRAP: 0300   Not tainted  (2.6.28.7)
MSR: 00029000 <EE,ME>  CR: 88000444  XER: 00000000
DEAR: 00000000, ESR: 00000000
TASK = ef8590c0[2611] 'cat' THREAD: ef106000
GPR00: 00000000 ef107e40 ef8590c0 00000000 ef107e9c 00000000 00000c00 
ef107e98
GPR08: 00000000 c0330000 ffffffff c0330000 48000448 1001cb7c 100042bc 
100df49c
GPR16: 00000002 00000400 c032f014 c032efe4 ef107e9c ef107e98 bfffffff 
efb43a00
GPR24: ef61d000 00000000 ef107f20 ef107e98 00000c00 c0400000 00001000 
efb43a00
NIP [c02cbe28] mutex_lock+0x0/0x1c
LR [c023cec8] ad7416_read_temp+0x24/0x80
Call Trace:
[ef107e40] [00000400] 0x400 (unreliable)
[ef107e70] [c023d334] i2c_ad7416_read_proc+0x20/0x70
[ef107e90] [c00e8018] proc_file_read+0x108/0x334
[ef107ee0] [c00e2d3c] proc_reg_read+0x4c/0x70
[ef107ef0] [c00a54e8] vfs_read+0xb4/0x16c
[ef107f10] [c00a58e0] sys_read+0x4c/0x90

Re: [PATCH] Add hvc_console to MAINTAINERS

On Tue, 2009-03-31 at 19:50 -0400, Josh Boyer wrote:
> On Wed, Apr 01, 2009 at 10:44:00AM +1100, Michael Ellerman wrote:
> >On Tue, 2009-03-31 at 09:29 -0400, Josh Boyer wrote:
> >> Add a MAINTAINERS entry for the hypervisor virtual console support
> >> used on IBM POWER servers.
> >> 
> >> Signed-off-by: Josh Boyer <jwboyer <at> linux.vnet.ibm.com>
> >> ---
> >> 
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index c5f4e9d..387ad45 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >>  <at>  <at>  -2176,6 +2176,11  <at>  <at>  W:       http://www.ia64-linux.org/
> >>  T:     git kernel.org:/pub/scm/linux/kernel/git/aegl/linux-2.6.git
> >>  S:     Maintained
> >>  
> >> +IBM HYPERVISOR VIRTUAL CONSOLE
> >> +P:     Several
> >> +L:     linuxppc-dev <at> ozlabs.org
> >> +S:     Maintained
> >
> >I don't think it's the "IBM" Hypervisor virtual console driver anymore.
> >There are backends for Xen, Beat (Toshiba) and virtio. 
> 
> Was just going by the copyright in the files, and Documentation/powerpc/hvc*
> 
> Also, if there are other backends, then is the list specified really
> correct?  Perhaps it should just be linux-kernel.