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
prev 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 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.