All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ralf Baechle <ralf@linux-mips.org>
To: "Zhan, Rongkai" <rongkai.zhan@windriver.com>
Cc: linux-mips@linux-mips.org
Subject: Re: [PATCH 1/2] Wind River 4KC PPMC Eval Board Support
Date: Fri, 5 May 2006 17:07:26 +0100	[thread overview]
Message-ID: <20060505160726.GA9309@linux-mips.org> (raw)
In-Reply-To: <6A3254532ACD7A42805B4E1BFD18080EB307A0@ism-mail01.corp.ad.wrs.com>

On Fri, May 05, 2006 at 05:42:04PM +0200, Zhan, Rongkai wrote:

> Here is a patch to add the support for Wind River 4KC PPMC Evaluation
> board, which is based on the GT64120 bridge chip.

Standard problem: This patch has line-wrapped lines so can't be applied ...

> 1970-01-01 08:00:00.000000000 +0800
> +++ linux-2.6.16.11-ppmc/arch/mips/gt64120/wrppmc/int-handler.S
> 2006-05-05 16:38:12.000000000 +0800
> @@ -0,0 +1,37 @@
> +/*
> + * This file is subject to the terms and conditions of the GNU General
> Public
> + * License.  See the file "COPYING" in the main directory of this
> archive
> + * for more details.
> + *
> + * Copyright (C) 1995, 1996, 1997, 2003 by Ralf Baechle
> + * Copyright (C) Wind River System Inc. Rongkai.Zhan
> <rongkai.zhan@windriver.com>
> + */
> +#include <asm/asm.h>
> +#include <asm/mipsregs.h>
> +#include <asm/addrspace.h>
> +#include <asm/regdef.h>
> +#include <asm/stackframe.h>
> +
> +	.align	5
> +	.set	noat
> +NESTED(handle_IRQ, PT_SIZE, sp)
> +	SAVE_ALL
> +	CLI				# Important: mark KERNEL mode !
> +
> +	mfc0	t0, CP0_CAUSE		# get pending interrupts
> +	mfc0	t1, CP0_STATUS		# get enabled interrupts
> +	and	t0, t0, t1		# get allowed interrupts
> +	andi	t0, t0, 0xFF00
> +	beqz	t0, 1f
> +
> +	move	a0, sp			# Prepare 'struct pt_regs *regs'
> pointer
> +	jal	do_wrppmc_IRQ
> +	nop
> +	j	ret_from_irq
> +	nop
> +
> +	/* wrong alarm or masked ... */
> +1:	j	spurious_interrupt
> +	nop
> +END(handle_IRQ)

Changeset e4ac58afdfac792c0583af30dbd9eae53e24c78b rewrites all interrupt
handlers from assembler to C, so your patche does no longer work.  Can you
create a patch against the master branch, please?

> +	printk(KERN_NOTICE "You can safely turn off the power\n");

This looks sooo windowsy ;-)

  Ralf

  reply	other threads:[~2006-05-05 16:07 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-05 15:42 [PATCH 1/2] Wind River 4KC PPMC Eval Board Support Zhan, Rongkai
2006-05-05 15:42 ` Zhan, Rongkai
2006-05-05 16:07 ` Ralf Baechle [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-05-06  9:04 Mark.Zhan
2006-05-09 16:41 ` Ralf Baechle
2006-05-10  2:41   ` Mark.Zhan
2006-05-10 15:32     ` Ralf Baechle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060505160726.GA9309@linux-mips.org \
    --to=ralf@linux-mips.org \
    --cc=linux-mips@linux-mips.org \
    --cc=rongkai.zhan@windriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.