linux-mips
[Top] [All Lists]

Re: [PATCH 2/2] [MIPS] add DECstation DS1287 clockevent

To: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Subject: Re: [PATCH 2/2] [MIPS] add DECstation DS1287 clockevent
From: Dmitri Vorobiev <dmitri.vorobiev@gmail.com>
Date: Wed, 23 Apr 2008 22:44:54 +0400
Cc: yoichi_yuasa@tripeaks.co.jp, ralf@linux-mips.org, linux-mips@linux-mips.org
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:organization:user-agent:mime-version:to:cc:subject:references:in-reply-to:content-type:content-transfer-encoding; bh=Pvzy8l5pav9yxoSjGNNtexejPzk2r/qxbVVKWCz62uw=; b=V1r2pRVzAvzmTeQAfMASRSOvmcpb5ioUgdqTdfcTiBZaiyWT3R7Z9Zse28N9Ry8f5xSO5d4fkZFxL59jw9dMN8ZY0AutNlYO++NH8BP6aV3VjFP1TdNV5s1a86RoZWN9H5myQIuk/Sv3BE1T+w4CR4MXNFZ6KfY6tJ0aiOGS2Kw=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:organization:user-agent:mime-version:to:cc:subject:references:in-reply-to:content-type:content-transfer-encoding; b=mt18noNfN/q701Su/2HT2u9VvY2onadi1dyZ2aitjLJIDMWNCmGIu8V9oxodfZp8dXv3to5FvCSgh+VQs8MElXtUmcDvDzN28WFfmLtBEMwiNSYSKjApDNTSdeGrm49FwTEoAUl3VSKPMs7FgwO/UC5GEX4qS08Vfosd/f6tJ/c=
In-reply-to: <20080423.115528.59033169.nemoto@toshiba-tops.co.jp>
Organization: DmVo Home
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <20080423085140.a693b2e5.yoichi_yuasa@tripeaks.co.jp> <200804222353.m3MNr8m4025538@po-mbox303.hop.2iij.net> <20080423.115528.59033169.nemoto@toshiba-tops.co.jp>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Thunderbird 1.5.0.14ubu (X11/20080306)
Atsushi Nemoto пишет:
> On Wed, 23 Apr 2008 08:52:45 +0900, Yoichi Yuasa 
> <yoichi_yuasa@tripeaks.co.jp> wrote:
>> add DECstation DS1287 clockevent
>>
>> Signed-off-by: Yoichi Yuasa <yoichi_yuasa@tripeaks.co.jp>
> ...
>> --- linux-orig/arch/mips/kernel/Makefile     2008-04-22 19:01:43.957134178 
>> +0900
>> +++ linux/arch/mips/kernel/Makefile  2008-04-22 18:03:05.427649422 +0900
>> @@ -9,8 +9,9 @@ obj-y                += cpu-probe.o branch.o entry.o g
>>                 time.o topology.o traps.o unaligned.o
>>  
>>  obj-$(CONFIG_CEVT_BCM1480)  += cevt-bcm1480.o
>> -obj-$(CONFIG_CEVT_R4K)              += cevt-r4k.o
>> +obj-$(CONFIG_CEVT_DS1287)   += cevt-ds1287.o
>>  obj-$(CONFIG_CEVT_GT641XX)  += cevt-gt641xx.o
>> +obj-$(CONFIG_CEVT_R4K)              += cevt-r4k.o
>>  obj-$(CONFIG_CEVT_SB1250)   += cevt-sb1250.o
>>  obj-$(CONFIG_CEVT_TXX9)             += cevt-txx9.o
>>  obj-$(CONFIG_CSRC_BCM1480)  += csrc-bcm1480.o
> 
> Why CONFIG_CEVT_R4K line was moved?  The order is important?
> 
>> --- linux-orig/arch/mips/kernel/cevt-ds1287.c        1970-01-01 
>> 09:00:00.000000000 +0900
>> +++ linux/arch/mips/kernel/cevt-ds1287.c     2008-04-22 18:03:05.455637018 
>> +0900
> ...
>> +static void ds1287_set_mode(enum clock_event_mode mode,
>> +                        struct clock_event_device *evt)
>> +{
>> +    unsigned long flags;
>> +    u8 val;
>> +
>> +    spin_lock_irqsave(&rtc_lock, flags);
> 
> You do not have to use irqsave here, while set_mode is always called
> with interrupts disabled.  And for rtc_lock ... I don't know if this
> code could be used on SMP :-)
> 

After I had saved a DECstation 5000/200 from a junkyard a few years
ago (very neat hardware, actually), I did some research on this brand.
AFAICT, all MIPS-based DECstation models were UP.

Dmitri

<Prev in Thread] Current Thread [Next in Thread>