Date: Mon, 20 Apr 2009 19:20:45 +0200
From: Ingo Molnar <mingo@elte.hu>
To: Brian Maly <bmaly@redhat.com>
Cc: Andrew Morton <akpm@linux-foundation.org>, h.peter.anvin@intel.com,
       linux-kernel@vger.kernel.org, dannf@hp.com, ying.huang@intel.com
Subject: Re: [PATCH] rtc: add x86 support for rtc-efi
Message-ID: <20090420172045.GA17887@elte.hu>
References: <49DA40B9.2060804@redhat.com> <49DA817E.8040705@intel.com> <49DA8875.5050009@redhat.com> <20090409161803.f79c0b64.akpm@linux-foundation.org> <20090410135102.GB24809@elte.hu> <49ECADAB.4020609@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <49ECADAB.4020609@redhat.com>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 939
Lines: 26


* Brian Maly <bmaly@redhat.com> wrote:

>> Hm, it would be nice to first unify the relevant bits of 
>> arch/x86/kernel/time_{32|64}.c into arch/x86/kernel/time.c, and 
>> then we can apply such patches without duplicative effects.
>
> Ingo,
>
> Are you OK with consolidating this into arch/x86/kernel/rtc.c as 
> Huang Ying had suggested? This seems like the most logical place 
> for the rtc-efi init to happen, but your suggestion to consolidate 
> this into arch/x86/kernel.time.c may have advantages that I am not 
> aware of.  Anyway, I would appreciate any insight/opinions on this 
> if you have any.  Thanks.

Yes, that indeed sounds like an even better place for it.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/