| To: | sshtylyov@ru.mvista.com |
|---|---|
| Subject: | Re: [PATCH] rest of works for migration to GENERIC_TIME |
| From: | Atsushi Nemoto <anemo@mba.ocn.ne.jp> |
| Date: | Tue, 24 Oct 2006 00:38:45 +0900 (JST) |
| Cc: | linux-mips@linux-mips.org, ralf@linux-mips.org |
| In-reply-to: | <453CD3ED.8020005@ru.mvista.com> |
| Original-recipient: | rfc822;linux-mips@linux-mips.org |
| References: | <20061023.033407.104640794.anemo@mba.ocn.ne.jp> <453CD3ED.8020005@ru.mvista.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
On Mon, 23 Oct 2006 18:38:37 +0400, Sergei Shtylyov <sshtylyov@ru.mvista.com> wrote: > > * mips_hpt_mask variable to specify bitmask of hpt value. > > There's actually no need to introduce more variables. Just make > clocksource declaration public and override default mask if > necessary. > Also, I don't see much sense in further existence of > mips_hpt_read() -- it only causes each clocksource read go thru a > double indirection which is really ugly. The same approach shouyld > be used here. I agree with you that it would be a way to go. For now exporting clocksource_mips just to override the mask and keep using mips_hpt_read looks somewhat inconsistent, so I just added mips_hpt_mask variable. Replacing mips_hpt_read involves changes for _all_ platform code so I think it would be better to do it on next -rc1 stage. --- Atsushi Nemoto |
| Previous by Date: | Re: [PATCH] rest of works for migration to GENERIC_TIME, Atsushi Nemoto |
|---|---|
| Next by Date: | Re: [PATCH] rest of works for migration to GENERIC_TIME, Sergei Shtylyov |
| Previous by Thread: | Re: [PATCH] rest of works for migration to GENERIC_TIME, Sergei Shtylyov |
| Next by Thread: | Re: [PATCH] rest of works for migration to GENERIC_TIME, Sergei Shtylyov |
| Indexes: | [Date] [Thread] [Top] [All Lists] |