Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: Ralf Baechle <ralf@linux-mips.org>
To: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: Atsushi Nemoto <anemo@mba.ocn.ne.jp>, linux-mips@linux-mips.org
Subject: Re: plat_timer_setup, mips_timer_ack, etc.
Date: Wed, 17 Oct 2007 18:40:12 +0100	[thread overview]
Message-ID: <20071017174012.GA11079@linux-mips.org> (raw)
In-Reply-To: <471639AC.8080301@ru.mvista.com>

On Wed, Oct 17, 2007 at 08:34:52PM +0400, Sergei Shtylyov wrote:

> >>$ git-grep mips_timer_ack arch/mips
> >>arch/mips/dec/time.c:   mips_timer_ack = dec_timer_ack;
> >>arch/mips/jmr3927/rbhma3100/setup.c:    mips_timer_ack = 
> >>jmr3927_timer_ack;
> 
>    TX3927 has three channel timer of which only channel 0 is used to 
> implement a clocksource -- however, clocksource code whould also need to be 
> changed since it's now jiffy-based and HRT doesn't tolerate this -- of 
> course, if anybody still cared about this boards
> 
> >>arch/mips/philips/pnx8550/common/time.c:        mips_timer_ack = 
> >>timer_ack;
> 
>    Here we have a case of a vendor abusing the count/compare register and 
> also adding 3 more of them. One pair can be used for clockevents, the other 
> for clocksource (its compare reg. being programmed to all ones).

Well, the TX3900 series is a bit of a frankenprocessor series.  Like take
32-bits from here, a limb from the R3000 and TLB from that other processor
and at the end shock it all well - at TTL levels that is ;-)  So it's not
quite obvious what to expect from that beast.

My question was mostly about the jmr3927 build failing with an undefined
reference to MIPS_CPU_IRQ_BASE.  For most other systems failing with the
same issue it made sense to glue that by converting the platform to
irq_cpu.  But if no device including the cp0 compare interrupt is directly
wired to the cp0 interrupt controller then enabling that doesn't make too
much sense.  So I guess jmr3927 and a hand full of other systems want a
different fix.

  Ralf

      parent reply	other threads:[~2007-10-17 17:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-16 15:52 plat_timer_setup, mips_timer_ack, etc Atsushi Nemoto
2007-10-16 16:36 ` Ralf Baechle
2007-10-16 17:01   ` Atsushi Nemoto
2007-10-17 16:28     ` Ralf Baechle
2007-10-17 16:34       ` Sergei Shtylyov
2007-10-17 16:58         ` Ralf Baechle
2007-10-17 17:40         ` Ralf Baechle [this message]

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=20071017174012.GA11079@linux-mips.org \
    --to=ralf@linux-mips.org \
    --cc=anemo@mba.ocn.ne.jp \
    --cc=linux-mips@linux-mips.org \
    --cc=sshtylyov@ru.mvista.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox