All of lore.kernel.org
 help / color / mirror / Atom feed
From: Grant Grundler <grundler@parisc-linux.org>
To: John David Anglin <dave@hiauly1.hia.nrc.ca>
Cc: parisc-linux@lists.parisc-linux.org
Subject: Re: [parisc-linux] Re: 2.6.14-rc2-pa2 ccio patch
Date: Sat, 1 Oct 2005 00:50:49 -0600	[thread overview]
Message-ID: <20051001065049.GA3308@colo.lackof.org> (raw)
In-Reply-To: <200509292321.j8TNLc60012238@hiauly1.hia.nrc.ca>

On Thu, Sep 29, 2005 at 07:21:38PM -0400, John David Anglin wrote:
> > Yes and Yes. But my reading of the PA1.1 arch says it *only* has a 2-bit
> > s field.
> 
> Oops, my mix up (fic/fdc confusion).

no problem - I've done worse.

>   However, don't add sr0 to the instruction.

Why not? I added it because I didn't like assumption the caller
to *io_pdir_entry was passing KERNEL_SPACE argument. If that's going
to forever be true, then we may as well hardcode KERNEL_SPACE
in both ccio and sba io_pdir_entry() routines instead of
taking a space_id parameter. Would save us a few cycles.

>   Specifying %r0 instead of '0' is the issue.  Strictly,
> the immediate form is only allowed in PA 2.0.  However, if the assembler
> doesn't accept it let me know.

That's what it looked like to me too.
It now uses %r0.

> I'm going to take a crack at trying to fix PA 1.0 to 1.1 promotion
> this weekend.  This is currently tied to the the lax matching used
> with PA 1.0 instructions.

Like I said, this isn't a problem that bothers me.

thanks!
grant
_______________________________________________
parisc-linux mailing list
parisc-linux@lists.parisc-linux.org
http://lists.parisc-linux.org/mailman/listinfo/parisc-linux

  reply	other threads:[~2005-10-01  6:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20050929063554.51BCD494003@palinux.hppa>
     [not found] ` <20050929064709.GA22206@colo.lackof.org>
     [not found]   ` <200509291513.j8TFDjxN010783@hiauly1.hia.nrc.ca>
2005-09-29 15:43     ` [parisc-linux] Re: 2.6.14-rc2-pa2 ccio patch Grant Grundler
2005-09-29 23:21       ` John David Anglin
2005-10-01  6:50         ` Grant Grundler [this message]
2005-10-08 19:30         ` John David Anglin
2005-10-08 19:50         ` John David Anglin
2005-09-29 18:04   ` Joel Soete
     [not found] <IO7AYH$A873EF9A4A4D74961990E3C295F405D8@scarlet.be>
2005-10-11 16:04 ` John David Anglin

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=20051001065049.GA3308@colo.lackof.org \
    --to=grundler@parisc-linux.org \
    --cc=dave@hiauly1.hia.nrc.ca \
    --cc=parisc-linux@lists.parisc-linux.org \
    /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.