LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* Re: [Alsa-devel] [RFC 01/12] snd-powermac: no longer handle anything with a layout-id property
From: Johannes Berg @ 2006-07-07  8:40 UTC (permalink / raw)
  To: Benjamin Herrenschmidt
  Cc: alsa-devel, Takashi Iwai, netstar, linuxppc-dev, Lee Revell
In-Reply-To: <1152260861.9862.59.camel@localhost.localdomain>

[-- Attachment #1: Type: text/plain, Size: 336 bytes --]

On Fri, 2006-07-07 at 18:27 +1000, Benjamin Herrenschmidt wrote:

> Well, when the master volume shows up, softvol can "unplug" itself :)

Hmm, not a bad idea, that way there's always a master volume. And when
some cards have an unusable master volume then softvol could be forced
to superimpose via the config file.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply

* Re: snd-aoa: g5 tas codec problems
From: Johannes Berg @ 2006-07-07  8:49 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev list, alsa-devel
In-Reply-To: <1152258426.9862.44.camel@localhost.localdomain>

[-- Attachment #1: Type: text/plain, Size: 331 bytes --]

On Fri, 2006-07-07 at 17:47 +1000, Benjamin Herrenschmidt wrote:
> Also, we should try
> (if not already the case) to cache our clock/i2s state so that
> subsequent prepare() don't try to change things that are already ok. 

I do that, the function reads the dws and sfr registers and exits early
if they match.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply

* Re: G5 troubles booting powerpc-git (July 6)
From: Andrew Morton @ 2006-07-07  8:52 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev
In-Reply-To: <1152260824.9862.57.camel@localhost.localdomain>

On Fri, 07 Jul 2006 18:27:04 +1000
Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:

> On Fri, 2006-07-07 at 01:23 -0700, Andrew Morton wrote:
> > On Fri, 07 Jul 2006 08:56:27 +1000
> > Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> > 
> > > If you look at arch/powerpc/kernel/irq.c from line 479:
> > > 
> > > 	/* Check if mapping already exist, if it does, call
> > > 	 * host->ops->map() to update the flags
> > > 	 */
> > > 	virq = irq_find_mapping(host, hwirq);
> > > 	if (virq != IRQ_NONE) {
> > > 		pr_debug("irq: -> existing mapping on virq %d\n", virq);
> > > 		host->ops->map(host, virq, hwirq, flags);
> > > 		return virq;
> > > 	}
> > > 
> > > What if you comment out the host->ops->map(...) call in there ? Does it
> > > help ?
> > 
> > No, there's no change in behaviour.
> 
> Doh ! Ok, looks like a different issue. Can you try this patch ? Please,
> do _not_ apply it to your tree as, as it is it will break non-powermac.
> It's some work I'm doing to clean up a couple of rough edges in my irq
> rework and fix a little misdesign.
> 
> If it doesn't fix it, then it's indeed a completely different issue that I'll have to track down tomorrow
> hopefully.

Needs this to compile:

diff -puN arch/powerpc/platforms/pseries/ras.c~a-fix arch/powerpc/platforms/pseries/ras.c
--- a/arch/powerpc/platforms/pseries/ras.c~a-fix
+++ a/arch/powerpc/platforms/pseries/ras.c
@@ -93,8 +93,7 @@ static void request_ras_irqs(struct devi
 		for (i = 0; i < opicplen; i++) {
 			if (count > 15)
 				break;
-			virqs[count] = irq_create_mapping(NULL, *(opicprop++),
-							 IRQ_TYPE_NONE);
+			virqs[count] = irq_create_mapping(NULL, *(opicprop++));
 			if (virqs[count] == NO_IRQ)
 				printk(KERN_ERR "Unable to allocate interrupt "
 				       "number for %s\n", np->full_name);
diff -puN arch/powerpc/platforms/pseries/xics.c~a-fix arch/powerpc/platforms/pseries/xics.c
--- a/arch/powerpc/platforms/pseries/xics.c~a-fix
+++ a/arch/powerpc/platforms/pseries/xics.c
@@ -757,7 +757,7 @@ void xics_request_IPIs(void)
 {
 	unsigned int ipi;
 
-	ipi = irq_create_mapping(xics_host, XICS_IPI, 0);
+	ipi = irq_create_mapping(xics_host, XICS_IPI);
 	BUG_ON(ipi == NO_IRQ);
 
 	/*
diff -puN drivers/char/hvsi.c~a-fix drivers/char/hvsi.c
--- a/drivers/char/hvsi.c~a-fix
+++ a/drivers/char/hvsi.c
@@ -1299,7 +1299,7 @@ static int __init hvsi_console_init(void
 		hp->inbuf_end = hp->inbuf;
 		hp->state = HVSI_CLOSED;
 		hp->vtermno = *vtermno;
-		hp->virq = irq_create_mapping(NULL, irq[0], 0);
+		hp->virq = irq_create_mapping(NULL, irq[0]);
 		if (hp->virq == NO_IRQ) {
 			printk(KERN_ERR "%s: couldn't create irq mapping for 0x%x\n",
 				__FUNCTION__, irq[0]);
_

But it still doesn't help.


These:

arch/powerpc/sysdev/i8259.c:222: warning: initialization from incompatible pointer type
arch/powerpc/platforms/pseries/xics.c:555: warning: initialization from incompatible pointer type
arch/powerpc/platforms/pseries/xics.c:561: warning: initialization from incompatible pointer type

look like super-serious box-killers.  Hope I'm not using either of those.

^ permalink raw reply

* Re: snd-aoa: g5 tas codec problems
From: Benjamin Herrenschmidt @ 2006-07-07  8:56 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linuxppc-dev list, alsa-devel
In-Reply-To: <1152262148.15068.30.camel@localhost>

On Fri, 2006-07-07 at 10:49 +0200, Johannes Berg wrote:
> On Fri, 2006-07-07 at 17:47 +1000, Benjamin Herrenschmidt wrote:
> > Also, we should try
> > (if not already the case) to cache our clock/i2s state so that
> > subsequent prepare() don't try to change things that are already ok. 
> 
> I do that, the function reads the dws and sfr registers and exits early
> if they match.

Ok. Be careful that I've removed the initial init of TAS thinking we
always get to clock restart to do it in prepare()... might need to be
put back.

Ben.

^ permalink raw reply

* RE: [PATCH 1/3] Freescale QE UCC gigabit ethernet driver
From: Li Yang-r58472 @ 2006-07-07  8:59 UTC (permalink / raw)
  To: Kumar Gala; +Cc: Andrew Morton, netdev, jgarzik, linuxppc-dev
In-Reply-To: <FA983307-6A01-4FCB-A15A-ACC0E7B068A9@kernel.crashing.org>

> -----Original Message-----
> From: Kumar Gala [mailto:galak@kernel.crashing.org]
> Sent: Thursday, July 06, 2006 9:45 PM
> To: Li Yang-r58472
> Cc: Andrew Morton; jgarzik@pobox.com; netdev@vger.kernel.org;
> linuxppc-dev@ozlabs.org
> Subject: Re: [PATCH 1/3] Freescale QE UCC gigabit ethernet driver
>=20
> Nack.  Here are some high level things that should be addressed:
> * There is a ton of debug code that should probably be stripped out,
> such as dump_regs, etc..

The reason I left these debug code in is that, QE is very flexible.
User probably needs to fine tune parameters for their own application.
It will be good to give them some clue doing that.  It's fine to remove
them, if you do think they are too verbose.
> * The phy support should use the phylib instead

I'll do this when I do get some time. ;)
> * convert from being a platform_device to of_device

I'm not up against moving to use of_device.  But why are we putting
extra effort in closing the door to backward compatibility with ppc
arch, and doesn't provide any new feature. ;)
>=20
> - kumar
>=20

^ permalink raw reply

* Re: G5 troubles booting powerpc-git (July 6)
From: Benjamin Herrenschmidt @ 2006-07-07  8:57 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linuxppc-dev
In-Reply-To: <20060707015237.68c768f2.akpm@osdl.org>


> > If it doesn't fix it, then it's indeed a completely different issue that I'll have to track down tomorrow
> > hopefully.
> 
> Needs this to compile:

Yeah, it's a powermac-only patch at the moment... thanks for doing part
of my work tho :)

Cheers,
Ben.

> diff -puN arch/powerpc/platforms/pseries/ras.c~a-fix arch/powerpc/platforms/pseries/ras.c
> --- a/arch/powerpc/platforms/pseries/ras.c~a-fix
> +++ a/arch/powerpc/platforms/pseries/ras.c
> @@ -93,8 +93,7 @@ static void request_ras_irqs(struct devi
>  		for (i = 0; i < opicplen; i++) {
>  			if (count > 15)
>  				break;
> -			virqs[count] = irq_create_mapping(NULL, *(opicprop++),
> -							 IRQ_TYPE_NONE);
> +			virqs[count] = irq_create_mapping(NULL, *(opicprop++));
>  			if (virqs[count] == NO_IRQ)
>  				printk(KERN_ERR "Unable to allocate interrupt "
>  				       "number for %s\n", np->full_name);
> diff -puN arch/powerpc/platforms/pseries/xics.c~a-fix arch/powerpc/platforms/pseries/xics.c
> --- a/arch/powerpc/platforms/pseries/xics.c~a-fix
> +++ a/arch/powerpc/platforms/pseries/xics.c
> @@ -757,7 +757,7 @@ void xics_request_IPIs(void)
>  {
>  	unsigned int ipi;
>  
> -	ipi = irq_create_mapping(xics_host, XICS_IPI, 0);
> +	ipi = irq_create_mapping(xics_host, XICS_IPI);
>  	BUG_ON(ipi == NO_IRQ);
>  
>  	/*
> diff -puN drivers/char/hvsi.c~a-fix drivers/char/hvsi.c
> --- a/drivers/char/hvsi.c~a-fix
> +++ a/drivers/char/hvsi.c
> @@ -1299,7 +1299,7 @@ static int __init hvsi_console_init(void
>  		hp->inbuf_end = hp->inbuf;
>  		hp->state = HVSI_CLOSED;
>  		hp->vtermno = *vtermno;
> -		hp->virq = irq_create_mapping(NULL, irq[0], 0);
> +		hp->virq = irq_create_mapping(NULL, irq[0]);
>  		if (hp->virq == NO_IRQ) {
>  			printk(KERN_ERR "%s: couldn't create irq mapping for 0x%x\n",
>  				__FUNCTION__, irq[0]);
> _
> 
> But it still doesn't help.
> 
> 
> These:
> 
> arch/powerpc/sysdev/i8259.c:222: warning: initialization from incompatible pointer type
> arch/powerpc/platforms/pseries/xics.c:555: warning: initialization from incompatible pointer type
> arch/powerpc/platforms/pseries/xics.c:561: warning: initialization from incompatible pointer type
> 
> look like super-serious box-killers.  Hope I'm not using either of those.

As I said, it's powermac only for now. I need to fix the above among
others. It's simple changes in most case. In the meantime, try on your
quad using a g5_defconfig

Ben.

^ permalink raw reply

* Re: snd-aoa: g5 tas codec problems
From: Johannes Berg @ 2006-07-07  9:04 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev list, alsa-devel
In-Reply-To: <1152262585.9862.61.camel@localhost.localdomain>

[-- Attachment #1: Type: text/plain, Size: 778 bytes --]

On Fri, 2006-07-07 at 18:56 +1000, Benjamin Herrenschmidt wrote:
> On Fri, 2006-07-07 at 10:49 +0200, Johannes Berg wrote:
> > On Fri, 2006-07-07 at 17:47 +1000, Benjamin Herrenschmidt wrote:
> > > Also, we should try
> > > (if not already the case) to cache our clock/i2s state so that
> > > subsequent prepare() don't try to change things that are already ok. 
> > 
> > I do that, the function reads the dws and sfr registers and exits early
> > if they match.
> 
> Ok. Be careful that I've removed the initial init of TAS thinking we
> always get to clock restart to do it in prepare()... might need to be
> put back.

Oh yes, we do, wonder why it even worked then since most of the time
we'll be using whatever the firmware does (44.1KHz,16bit).

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply

* SIU_INT_IRQ4 in MPC 8260
From: jagannathanjay @ 2006-07-07  9:05 UTC (permalink / raw)
  To: linuxppc-embedded

Hi all

I am trying to set up a interrupt handler for SIU_INT_IRQ4.

The target I am using is MPC 8260 , emmbedded planet Evaluation board.

The OS is Embedded Linux .
-------------------------------------------------------------------------
------------
void my_interrupt_handler(int irq, void *dev_id, struct pt_regs *regs)
{

 printk("*********** \n");

}


 if( request_irq(SIU_INT_IRQ4,my_interrupt_handler,0,0,0) < 0)
 {
     printk("\n unable to register the interrupt handler \n");
 }

-------------------------------------------------------------------------
------------

But my_interrupt_handler isn't getting called .

Can anyone help me to fix this problem

Regards
Jagan





________________________________________________________________________
Check Out the new free AIM(R) Mail -- 2 GB of storage and 
industry-leading spam and email virus protection.

^ permalink raw reply

* Re: G5 troubles booting powerpc-git (July 6)
From: Andrew Morton @ 2006-07-07  9:11 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev
In-Reply-To: <1152262667.9862.64.camel@localhost.localdomain>

On Fri, 07 Jul 2006 18:57:47 +1000
Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:

> 
> In the meantime, try on your quad using a g5_defconfig

I tried it with your recently-sent patch applied.  It still dies in
smu_sensors_init().

^ permalink raw reply

* RE: SIU_INT_IRQ4 in MPC 8260
From: Li Yang-r58472 @ 2006-07-07  9:18 UTC (permalink / raw)
  To: jagannathanjay, linuxppc-embedded
In-Reply-To: <8C86FBB9DCF99D2-1D1C-336A@FWM-D43.sysops.aol.com>

Are you sure the IRQ4 line is generating irqs?  Did you measure the line
using an oscilloscope?

Best Regards,
Leo
> -----Original Message-----
> From: linuxppc-embedded-bounces+leoli=3Dfreescale.com@ozlabs.org
> [mailto:linuxppc-embedded-bounces+leoli=3Dfreescale.com@ozlabs.org] On
Behalf
> Of jagannathanjay@aim.com
> Sent: Friday, July 07, 2006 5:05 PM
> To: linuxppc-embedded@ozlabs.org
> Subject: SIU_INT_IRQ4 in MPC 8260
>=20
> Hi all
>=20
> I am trying to set up a interrupt handler for SIU_INT_IRQ4.
>=20
> The target I am using is MPC 8260 , emmbedded planet Evaluation board.
>=20
> The OS is Embedded Linux .
>
------------------------------------------------------------------------
-
> ------------
> void my_interrupt_handler(int irq, void *dev_id, struct pt_regs *regs)
> {
>=20
>  printk("*********** \n");
>=20
> }
>=20
>=20
>  if( request_irq(SIU_INT_IRQ4,my_interrupt_handler,0,0,0) < 0)
>  {
>      printk("\n unable to register the interrupt handler \n");
>  }
>=20
>
------------------------------------------------------------------------
-
> ------------
>=20
> But my_interrupt_handler isn't getting called .
>=20
> Can anyone help me to fix this problem
>=20
> Regards
> Jagan
>=20
>=20
>=20
>=20
>=20
>
________________________________________________________________________
> Check Out the new free AIM(R) Mail -- 2 GB of storage and
> industry-leading spam and email virus protection.
>=20
> _______________________________________________
> Linuxppc-embedded mailing list
> Linuxppc-embedded@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-embedded

^ permalink raw reply

* Re: G5 troubles booting powerpc-git (July 6)
From: Benjamin Herrenschmidt @ 2006-07-07  9:24 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linuxppc-dev
In-Reply-To: <20060707021137.7f5dd182.akpm@osdl.org>

On Fri, 2006-07-07 at 02:11 -0700, Andrew Morton wrote:
> On Fri, 07 Jul 2006 18:57:47 +1000
> Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> 
> > 
> > In the meantime, try on your quad using a g5_defconfig
> 
> I tried it with your recently-sent patch applied.  It still dies in
> smu_sensors_init().

Ok, so there's something else. I'll let you know.

Ben.

^ permalink raw reply

* RE: SIU_INT_IRQ4 in MPC 8260
From: Li Yang-r58472 @ 2006-07-07  9:31 UTC (permalink / raw)
  To: jagannathanjay; +Cc: linuxppc-embedded
In-Reply-To: <8C86FBE3C676D29-1D1C-33C6@FWM-D43.sysops.aol.com>

Interrupt is signal sending from external devices to CPU notifying
something has happened.  So if you want your interrupt handler be
called, first you have to make sure the device does send such signals.

Best Regards,
Leo
> -----Original Message-----
> From: jagannathanjay@aim.com [mailto:jagannathanjay@aim.com]
> Sent: Friday, July 07, 2006 5:24 PM
> To: Li Yang-r58472
> Subject: Re: SIU_INT_IRQ4 in MPC 8260
>=20
> No I haven't measured the IRQ4 line
>=20
> I am newbie into MPC8260 .
>=20
> Can u guide me as to what way I should proceed to get my interrupt
> handler working?
>=20
> Regards
> Jagan
>=20
>=20
>=20
> -----Original Message-----
> From: Li Yang-r58472 <LeoLi@freescale.com>
> To: jagannathanjay@aim.com; linuxppc-embedded@ozlabs.org
> Sent: Fri, 7 Jul 2006 17:18:54 +0800
> Subject: RE: SIU_INT_IRQ4 in MPC 8260
>=20
>    Are you sure the IRQ4 line is generating irqs?  Did you measure the
> line
> using an oscilloscope?
>=20
> Best Regards,
> Leo
> > -----Original Message-----
> > From: linuxppc-embedded-bounces+leoli=3Dfreescale.com@ozlabs.org
> > [mailto:linuxppc-embedded-bounces+leoli=3Dfreescale.com@ozlabs.org] =
On
> Behalf
> > Of jagannathanjay@aim.com
> > Sent: Friday, July 07, 2006 5:05 PM
> > To: linuxppc-embedded@ozlabs.org
> > Subject: SIU_INT_IRQ4 in MPC 8260
> >
> > Hi all
> >
> > I am trying to set up a interrupt handler for SIU_INT_IRQ4.
> >
> > The target I am using is MPC 8260 , emmbedded planet Evaluation
board.
> >
> > The OS is Embedded Linux .
> >
>
------------------------------------------------------------------------
> -
> > ------------
> > void my_interrupt_handler(int irq, void *dev_id, struct pt_regs
*regs)
> > {
> >
> >  printk("***********
> ");
> >
> > }
> >
> >
> >  if( request_irq(SIU_INT_IRQ4,my_interrupt_handler,0,0,0) < 0)
> >  {
> >      printk("
>  unable to register the interrupt handler
> ");
> >  }
> >
> >
>
------------------------------------------------------------------------
> -
> > ------------
> >
> > But my_interrupt_handler isn't getting called .
> >
> > Can anyone help me to fix this problem
> >
> > Regards
> > Jagan
> >
> >
> >
> >
> >
> >
>
________________________________________________________________________
> > Check Out the new free AIM(R) Mail -- 2 GB of storage and
> > industry-leading spam and email virus protection.
> >
> > _______________________________________________
> > Linuxppc-embedded mailing list
> > Linuxppc-embedded@ozlabs.org
> > https://ozlabs.org/mailman/listinfo/linuxppc-embedded
>=20
>=20
>
________________________________________________________________________
> Check Out the new free AIM(R) Mail -- 2 GB of storage and
> industry-leading spam and email virus protection.

^ permalink raw reply

* Re: SIU_INT_IRQ4 in MPC 8260
From: Laurent Pinchart @ 2006-07-07  9:39 UTC (permalink / raw)
  To: linuxppc-embedded; +Cc: jagannathanjay
In-Reply-To: <8C86FBB9DCF99D2-1D1C-336A@FWM-D43.sysops.aol.com>

> I am trying to set up a interrupt handler for SIU_INT_IRQ4.
>
> The target I am using is MPC 8260 , emmbedded planet Evaluation board.
> The OS is Embedded Linux .

[snip]

> But my_interrupt_handler isn't getting called .

Make sure the CPM IRQ controller is properly configured. Here's the code I 
used to configure IRQ5 and IRQ6 in edge sensitive mode. Read the MPC8260 
datasheet to check how to set other options. If IRQ4 is multiplexed with 
other signals, make sure you configured the MPC8260 to use the external pin 
as IRQ4.

static void __init tbox_cp11_init_irq(void)
{
        volatile intctl_cpm2_t *ic = &cpm2_immr->im_intctl;

        init_irq();

        /* Make IRQ5 and IRQ6 edge sensitive */
        ic->ic_siexr |= 1 << (14 - (SIU_INT_IRQ5 - SIU_INT_IRQ1));
        irq_desc[SIU_INT_IRQ5].status &= ~IRQ_LEVEL;

        ic->ic_siexr |= 1 << (14 - (SIU_INT_IRQ6 - SIU_INT_IRQ1));
        irq_desc[SIU_INT_IRQ6].status &= ~IRQ_LEVEL;
}

void __init m82xx_board_init(void)
{
        init_irq = ppc_md.init_IRQ;
        ppc_md.init_IRQ = tbox_cp11_init_irq;

}

Laurent Pinchart

^ permalink raw reply

* Re: [Alsa-devel] [RFC 01/12] snd-powermac: no longer handle anything with a layout-id property
From: Takashi Iwai @ 2006-07-07  9:51 UTC (permalink / raw)
  To: Benjamin Herrenschmidt
  Cc: alsa-devel, netstar, linuxppc-dev, Lee Revell, Johannes Berg
In-Reply-To: <1152250720.9862.32.camel@localhost.localdomain>

At Fri, 07 Jul 2006 15:38:40 +1000,
Benjamin Herrenschmidt wrote:
> 
> On Fri, 2006-06-30 at 01:16 +0200, Andreas Schwab wrote:
> > Lee Revell <rlrevell@joe-job.com> writes:
> > 
> > > What is the content of /proc/asound/cards?  The patches must have
> > > changed the card name and failed to create a matching config file
> > > in /usr/share/alsa/cards.
> > 
> > Thanks for the hint.  I have added 'AppleOnbdAudio cards.PMac' to
> > /usr/share/alsa/cards/aliases.conf, and it works again.
> 
> There is still an issue... wether to enable softvol or not... With
> snd-aoa, the card name is always the same but wether it has hardware
> volume control or not (Toonie doesn't) is not known... Alsa should have
> ways to add softvol automatically if no master volume control exist.

Currently, it's aliased to PMacToonie.conf, so that "PCM" volume is
added automatically when no corresponding h/w volume eixsts.


Takashi

^ permalink raw reply

* Re: [Alsa-devel] [RFC 01/12] snd-powermac: no longer handle anything with a layout-id property
From: Johannes Berg @ 2006-07-07  9:55 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: alsa-devel, netstar, linuxppc-dev, Lee Revell
In-Reply-To: <s5hpsghwxny.wl%tiwai@suse.de>

[-- Attachment #1: Type: text/plain, Size: 275 bytes --]

On Fri, 2006-07-07 at 11:51 +0200, Takashi Iwai wrote:

> Currently, it's aliased to PMacToonie.conf, so that "PCM" volume is
> added automatically when no corresponding h/w volume eixsts.

Will submit a patch to name it "PCM" then on tas instead of "PCM1"

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply

* Re: [PATCH 15/20] [powerpc, netdevices] Constify & voidify get_property()
From: Paul Mackerras @ 2006-07-07 10:35 UTC (permalink / raw)
  To: Jeremy Kerr; +Cc: linuxppc-dev, cbe-oss-dev, netdev
In-Reply-To: <98457831@toto.iv>

Jeremy Kerr writes:

> powerpc-specific network device driver changes.

For some reason, the whitespace got munged on this one.  There were
some rejects on some of the other patches due to some of Ben H's
patches.  Please fix them and move the include/linux/platform_device.h
change into the same patch as the drivers/base/platform.c change.

I have applied patches 1-3.  Please resubmit the rest.

Regards,
Paul.

^ permalink raw reply

* Re: [1/5][POWERPC] boot: prepare for zImage.kexec
From: Paul Mackerras @ 2006-07-07 10:35 UTC (permalink / raw)
  To: Milton Miller; +Cc: linuxppc-dev, Sam Ravnborg
In-Reply-To: <200607012346.k61NkWXL006760@sullivan.realtime.net>

Milton Miller writes:

> This patch moves the initializations of prom variables to boot/prom.c
> to allow it to be replaced for kexec.   Kexec does not care about
> the stdout handle, but will care about the location of the device
> tree and will search for RTAS.

Unfortunately it doesn't apply now because of other changes to
boot.h.  Please respin the series against the powerpc.git tree.

Thanks,
Paul.

^ permalink raw reply

* patches added to powerpc.git tree
From: Paul Mackerras @ 2006-07-07 11:29 UTC (permalink / raw)
  To: linuxppc-dev

The following patches have been pushed to the powerpc.git tree (master
branch):

Benjamin Herrenschmidt:
      [POWERPC] Workaround Pegasos incorrect ISA "ranges"
      [POWERPC] Fix 32 bits warning in prom_init.c
      [POWERPC] Fix non-MPIC CHRPs with CONFIG_SMP set
      [POWERPC] Fix default clock for udbg_16550
      [POWERPC] Fix legacy_serial.c error handling on 32 bits
      [POWERPC] Add briq support to CHRP
      [POWERPC] Add support for briq front panel
      [POWERPC] More offb/bootx fixes
      [POWERPC] Xserve G5 thermal control fixes
      [POWERPC] Add cpufreq support for Xserve G5

Jeremy Kerr:
      [POWERPC] Remove linux,device properties
      [POWERPC] Use const qualifiers for prom parsing utilites
      [POWERPC] Remove linux,pci-domain properties

Johannes Berg:
      [POWERPC] fix up front-LED Kconfig

Michael Ellerman:
      [POWERPC] Fix mem= handling when the memory limit is > RMO size

Niels Kristian Bech Jensen:
      [POWERPC] Add -fno-stack-protector to BOOTCFLAGS in arch/powerpc/boot/Makefile.

 arch/powerpc/Kconfig                         |    2 
 arch/powerpc/boot/Makefile                   |    4 
 arch/powerpc/configs/pmac32_defconfig        |  112 ++++++++-
 arch/powerpc/kernel/legacy_serial.c          |   13 +
 arch/powerpc/kernel/of_device.c              |   24 --
 arch/powerpc/kernel/pci_64.c                 |   39 ---
 arch/powerpc/kernel/prom_init.c              |   42 ++++
 arch/powerpc/kernel/prom_parse.c             |   22 +-
 arch/powerpc/kernel/smp.c                    |   24 +-
 arch/powerpc/kernel/udbg_16550.c             |    6 -
 arch/powerpc/mm/lmb.c                        |    3 
 arch/powerpc/platforms/chrp/pci.c            |   42 +++-
 arch/powerpc/platforms/chrp/setup.c          |   39 +++
 arch/powerpc/platforms/powermac/bootx_init.c |   35 ++-
 arch/powerpc/platforms/powermac/cpufreq_64.c |   78 ++++---
 drivers/char/Kconfig                         |   14 +
 drivers/char/Makefile                        |    1 
 drivers/char/briq_panel.c                    |  268 +++++++++++++++++++++++
 drivers/ide/Kconfig                          |   14 -
 drivers/macintosh/Kconfig                    |    9 +
 drivers/macintosh/therm_pm72.c               |  218 +++++++++++++++++-
 drivers/macintosh/therm_pm72.h               |   33 +++
 drivers/macintosh/via-pmu-led.c              |    2 
 drivers/video/offb.c                         |  307 ++++++++++++++------------
 include/asm-powerpc/processor.h              |    1 
 include/asm-powerpc/prom.h                   |    2 
 26 files changed, 1023 insertions(+), 331 deletions(-)
 create mode 100644 drivers/char/briq_panel.c

^ permalink raw reply

* Re: patches added to powerpc.git tree
From: Johannes Berg @ 2006-07-07 11:39 UTC (permalink / raw)
  To: Paul Mackerras; +Cc: linuxppc-dev
In-Reply-To: <17582.17835.641485.233064@cargo.ozlabs.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 297 bytes --]

On Fri, 2006-07-07 at 21:29 +1000, Paul Mackerras wrote:

> Johannes Berg:
>       [POWERPC] fix up front-LED Kconfig

Thanks for picking this up.

I notice you haven't picked up Andreas' irq conversion fix, was that
intentional? Or did I miss it going in elsewhere?

Thanks,
johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply

* Re: G5 troubles booting powerpc-git (July 6)
From: Benjamin Herrenschmidt @ 2006-07-07 11:45 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linuxppc-dev
In-Reply-To: <20060707021137.7f5dd182.akpm@osdl.org>

On Fri, 2006-07-07 at 02:11 -0700, Andrew Morton wrote:
> On Fri, 07 Jul 2006 18:57:47 +1000
> Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> 
> > 
> > In the meantime, try on your quad using a g5_defconfig
> 
> I tried it with your recently-sent patch applied.  It still dies in
> smu_sensors_init().

Do you have lockdep ? There have been some changes to the completion
stuff, I'm wondering if the DECLARE_COMPLETION in windfarm_smu_sensors.c
need to be change to DECLARE_COMLETION_ONSTACK... (And other similars in
windfarm_*).

I haven't had a chance to test yet but I'll audit through all the
powerpc uses of DECLARE_COMPLETION since I use DECLARE_COMPLETION on
stack variables often.

Ben.

^ permalink raw reply

* Re: Xilinx BSP for linux 2.6
From: Ming Liu @ 2006-07-07 11:46 UTC (permalink / raw)
  To: ammubhai; +Cc: linuxppc-embedded
In-Reply-To: <44AD20F6.8020401@gmail.com>

Dear Ameet,
A good news is that now my 2.6 kernel is running in my ML403 board! 
According your guidance, first I made the kernel as simple as possible and 
it works well. Then I included the patch for SystemACE and here is the 
information shown on the hyper teminal:


Linux/PPC load: console=ttyS0,9600 root=/dev/xsa3 
[    0.000148] Console: colour dummy device 80x25
[    0.000919] Dentry cache hash table entries: 16384 (order: 4, 65536 
bytes)
[    0.002277] Inode-cache hash table entries: 8192 (order: 3, 32768 bytes)
[    0.012802] Memory: 63104k available (1340k kernel code, 384k data, 76k 
init,
 0k highmem)
[    0.230310] Mount-cache hash table entries: 512
[    0.238305] VFS: Disk quotas dquot_6.5.1
[    0.238516] Dquot-cache hash table entries: 1024 (order 0, 4096 bytes)
[    0.238817] JFFS2 version 2.2. (NAND) (C) 2001-2003 Red Hat, Inc.
[    0.239435] JFS: nTxBlock = 493, nTxLock = 3944
[    0.241883] Initializing Cryptographic API
[    0.241940] io scheduler noop registered (default)
[    0.322864] Serial: 8250/16550 driver $Revision: 1.90 $ 4 ports, IRQ 
sharing
disabled
[    0.324759] serial8250.0: ttyS0 at MMIO 0x40401003 (irq = 1) is a 16550A
[    1.653644] RAMDISK driver initialized: 16 RAM disks of 65536K size 1024 
bloc
ksize
[    1.745079] loop: loaded (max 8 devices)
[    1.793392]  xsa: xsa1 xsa2 xsa3
[    1.839262] mice: PS/2 mouse device common for all mice
[    1.935059] VFS: Mounted root (ext2 filesystem) readonly.
[    2.000037] Freeing unused kernel memory: 76k init
[    2.075656] Warning: unable to open an initial console.

I use xsa3 as the partition to store the file system. Now the file system 
is the old one for my 2.4 kernel generated by busybox. Then I think that's 
why the problem "unable to open an initial console" is generated. Do you 
have some suggestion on how to solve it? Thanks for your opinion. 

Also, when I compile the kernel, there is a error. That's in adapter.c 
file, undefined reference to "XAssert". I checked the source code and found 
that the function of XAssert is defined in the file xbasic_types.c, not the 
xbasic_types.h. So it cannot be included into adapter.c. And also in the 
makefile there is no dependency on xbasic_types.c. So I added the 
defination for this function in adapter.c, copying it from xbasic_types.c. 
It then worked well. 

Waiting for your guidance. You know, an expert's opinion is always useful 
for a novice, like me. :)

Regards
Ming

_________________________________________________________________
免费下载 MSN Explorer:   http://explorer.msn.com/lccn/  

^ permalink raw reply

* Re: Xilinx BSP for linux 2.6
From: Ameet Patil @ 2006-07-07 12:42 UTC (permalink / raw)
  To: Ming Liu; +Cc: linuxppc-embedded
In-Reply-To: <BAY110-F7612E5FEF57E56118641EB2740@phx.gbl>

Hi Ming,

Excellent News!

Don't worry... the problem you are facing could be because of missing
device files in your /dev folder.

Do as follows:
mount the CF card onto your PC and then goto the /dev folder on your
root filesystem in the card. Here run the following commands as root:
# mknod -m 660 console c 5 1
# mknod -m 660 xsa b 254 0
# mknod -m 660 xsa1 b 254 1
# mknod -m 660 xsa2 b 254 2
# mknod -m 660 xsa3 b 254 3
Now there should be a directory tts in your dev. If not create one
"mkdir tts". Then do this in the dev/tts folder,
# mknod -m 660 0 b 4 64

> Also, when I compile the kernel, there is a error. That's in adapter.c
> file, undefined reference to "XAssert". I checked the source code and
I think you have the old patch. If you download the new one, the
compilation problem should not occur. But you resolved it yourself,...
which is good.

Let me know if this works. I shall add this stuff in PART II of the article.

-Ameet

Ming Liu wrote:
> Dear Ameet,
> A good news is that now my 2.6 kernel is running in my ML403 board! 
> According your guidance, first I made the kernel as simple as possible 
> and it works well. Then I included the patch for SystemACE and here is 
> the information shown on the hyper teminal:
> 
> 
> Linux/PPC load: console=ttyS0,9600 root=/dev/xsa3 [    0.000148] 
> Console: colour dummy device 80x25
> [    0.000919] Dentry cache hash table entries: 16384 (order: 4, 65536 
> bytes)
> [    0.002277] Inode-cache hash table entries: 8192 (order: 3, 32768 bytes)
> [    0.012802] Memory: 63104k available (1340k kernel code, 384k data, 
> 76k init,
> 0k highmem)
> [    0.230310] Mount-cache hash table entries: 512
> [    0.238305] VFS: Disk quotas dquot_6.5.1
> [    0.238516] Dquot-cache hash table entries: 1024 (order 0, 4096 bytes)
> [    0.238817] JFFS2 version 2.2. (NAND) (C) 2001-2003 Red Hat, Inc.
> [    0.239435] JFS: nTxBlock = 493, nTxLock = 3944
> [    0.241883] Initializing Cryptographic API
> [    0.241940] io scheduler noop registered (default)
> [    0.322864] Serial: 8250/16550 driver $Revision: 1.90 $ 4 ports, IRQ 
> sharing
> disabled
> [    0.324759] serial8250.0: ttyS0 at MMIO 0x40401003 (irq = 1) is a 16550A
> [    1.653644] RAMDISK driver initialized: 16 RAM disks of 65536K size 
> 1024 bloc
> ksize
> [    1.745079] loop: loaded (max 8 devices)
> [    1.793392]  xsa: xsa1 xsa2 xsa3
> [    1.839262] mice: PS/2 mouse device common for all mice
> [    1.935059] VFS: Mounted root (ext2 filesystem) readonly.
> [    2.000037] Freeing unused kernel memory: 76k init
> [    2.075656] Warning: unable to open an initial console.
> 
> I use xsa3 as the partition to store the file system. Now the file 
> system is the old one for my 2.4 kernel generated by busybox. Then I 
> think that's why the problem "unable to open an initial console" is 
> generated. Do you have some suggestion on how to solve it? Thanks for 
> your opinion.
> Also, when I compile the kernel, there is a error. That's in adapter.c 
> file, undefined reference to "XAssert". I checked the source code and 
> found that the function of XAssert is defined in the file 
> xbasic_types.c, not the xbasic_types.h. So it cannot be included into 
> adapter.c. And also in the makefile there is no dependency on 
> xbasic_types.c. So I added the defination for this function in 
> adapter.c, copying it from xbasic_types.c. It then worked well.
> Waiting for your guidance. You know, an expert's opinion is always 
> useful for a novice, like me. :)
> 
> Regards
> Ming
> 
> _________________________________________________________________
> 免费下载 MSN Explorer:   http://explorer.msn.com/lccn/ 
> 

^ permalink raw reply

* Re: [PATCH 1/3] Freescale QE UCC gigabit ethernet driver
From: Kumar Gala @ 2006-07-07 13:19 UTC (permalink / raw)
  To: Li Yang-r58472; +Cc: Andrew Morton, netdev, jgarzik, linuxppc-dev
In-Reply-To: <4879B0C6C249214CBE7AB04453F84E4DF278@zch01exm20.fsl.freescale.net>


On Jul 7, 2006, at 3:59 AM, Li Yang-r58472 wrote:

>> -----Original Message-----
>> From: Kumar Gala [mailto:galak@kernel.crashing.org]
>> Sent: Thursday, July 06, 2006 9:45 PM
>> To: Li Yang-r58472
>> Cc: Andrew Morton; jgarzik@pobox.com; netdev@vger.kernel.org;
>> linuxppc-dev@ozlabs.org
>> Subject: Re: [PATCH 1/3] Freescale QE UCC gigabit ethernet driver
>>
>> Nack.  Here are some high level things that should be addressed:
>> * There is a ton of debug code that should probably be stripped out,
>> such as dump_regs, etc..
>
> The reason I left these debug code in is that, QE is very flexible.
> User probably needs to fine tune parameters for their own application.
> It will be good to give them some clue doing that.  It's fine to  
> remove
> them, if you do think they are too verbose.

Then I suggest you look at debugfs.

>> * The phy support should use the phylib instead
>
> I'll do this when I do get some time. ;)
>> * convert from being a platform_device to of_device
>
> I'm not up against moving to use of_device.  But why are we putting
> extra effort in closing the door to backward compatibility with ppc
> arch, and doesn't provide any new feature. ;)

Because the intent is for arch/ppc to go away in the future.  You are  
introducing a new driver and a new platform, this is the ideal time  
for us to use of_device.  The benefits are that the driver is the  
ideal place to parse the details of the device node rather than  
platform code in arch/powerpc.

- kumar

^ permalink raw reply

* looking for ppc_md.restart implementation
From: Eran Ben-Avi @ 2006-07-07 13:15 UTC (permalink / raw)
  To: linuxppc-embedded

[-- Attachment #1: Type: text/plain, Size: 141 bytes --]

Hi,
 
I am looking for an implementation for the ppc_md.restart hook api under my LSP.
I'm using IBM750Gl and MPC7448 CPU's.
 
Thanks.

[-- Attachment #2: Type: text/html, Size: 605 bytes --]

^ permalink raw reply

* Re: [PATCH] dtc: fix endian issue when reading blobs
From: Michael Neuling @ 2006-07-07 13:28 UTC (permalink / raw)
  To: linuxppc-dev; +Cc: Jon Loeliger, miltonm
In-Reply-To: <20060707055313.E6FC067A2E@ozlabs.org>

The reserve mem regions are screwy if you read a blob on x86.  I'm
guessing there may be a few more of these lurking in the code. 

Signed-off-by: Michael Neuling <mikey@neuling.org>

---
Oops.. wrong way around.. thanks Milton.

 flattree.c |    2 ++
 1 file changed, 2 insertions(+)

Index: dtc/flattree.c
===================================================================
--- dtc.orig/flattree.c
+++ dtc/flattree.c
@@ -619,6 +619,8 @@ static struct reserve_info *flat_read_me
 	p = inb->ptr;
 	while (1) {
 		flat_read_chunk(inb, &re, sizeof(re));
+		re.address  = be64_to_cpu(re.address);
+		re.size = be64_to_cpu(re.size);
 		if (re.size == 0)
 			break;
 

^ permalink raw reply


This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox