Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422749AbcKAJXr (ORCPT ); Tue, 1 Nov 2016 05:23:47 -0400 Received: from mail-ve1eur01on0081.outbound.protection.outlook.com ([104.47.1.81]:35648 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1034787AbcKAJXp (ORCPT ); Tue, 1 Nov 2016 05:23:45 -0400 X-Greylist: delayed 65067 seconds by postgrey-1.27 at vger.kernel.org; Tue, 01 Nov 2016 05:23:45 EDT From: Noam Camus To: Vineet Gupta , Daniel Lezcano CC: "tglx@linutronix.de" , "linux-kernel@vger.kernel.org" , "linux-snps-arc@lists.infradead.org" , "Alexey.Brodkin@synopsys.com" Subject: RE: [PATCH 5/9] ARC: breakout aux handling into a seperate header Thread-Topic: [PATCH 5/9] ARC: breakout aux handling into a seperate header Thread-Index: AQHSM8jz42dAjZqzNE+tw2xXQwyM4aDDwEGg Date: Tue, 1 Nov 2016 07:49:57 +0000 Message-ID: References: <35bde193-8492-83e0-fb03-8385d8afd007@synopsys.com> <1477954096-770-1-git-send-email-vgupta@synopsys.com> <1477954096-770-6-git-send-email-vgupta@synopsys.com> In-Reply-To: <1477954096-770-6-git-send-email-vgupta@synopsys.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=noamca@mellanox.com; x-originating-ip: [212.179.42.66] x-ms-office365-filtering-correlation-id: 0d96aefa-da66-439a-b934-08d4022bace4 x-microsoft-exchange-diagnostics: 1;DB6PR0501MB2519;7:eG6H46by+9G8HQW3hp4FboKXzYJuq6J0GLsSC54knqb8Cb53J4gk74bAOy8kj2nMaIHdyTipCbAXeD0dAZtYqBNUMWhGLQpIjtZJV2V+7SUX+JOSSF5ym6jBPdiImyrGJQRioleSZ6p+uWoOYicIlJlYkw5BzB+bJHBfHvwRyCnYpfujwMiw4i7QVXN3gBuXAcOSXwYSDQ3SImaHmuyDuz9pFU5QObzEBVzsMDqSw0vLLZttZPN3b9UlXuvR/bjPrdRZCNApibtHmAtVZSth5uZGQV/LmgcJyyNwuDd19ZbwdOecBVKbqQy7rxF3KlxNEbLJEB45IxDQu6/v33iqBkkeA8yzfkCX9pDjVTdaQy8= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB6PR0501MB2519; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6055026);SRVR:DB6PR0501MB2519;BCL:0;PCL:0;RULEID:;SRVR:DB6PR0501MB2519; x-forefront-prvs: 01136D2D90 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(377454003)(199003)(189002)(189998001)(3660700001)(122556002)(86362001)(66066001)(92566002)(11100500001)(305945005)(81156014)(7696004)(2950100002)(7846002)(5001770100001)(5660300001)(97736004)(8676002)(19580395003)(19580405001)(7736002)(81166006)(3280700002)(68736007)(4326007)(105586002)(54356999)(76176999)(106116001)(106356001)(76576001)(50986999)(10400500002)(33656002)(5002640100001)(101416001)(2900100001)(9686002)(87936001)(586003)(2906002)(77096005)(102836003)(74316002)(3846002)(6116002)(8936002);DIR:OUT;SFP:1101;SCL:1;SRVR:DB6PR0501MB2519;H:DB6PR0501MB2518.eurprd05.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Nov 2016 07:49:57.1460 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0501MB2519 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uA19Nqb4001886 Content-Length: 983 Lines: 22 >From: Vineet Gupta [mailto:Vineet.Gupta1@synopsys.com] >Sent: Tuesday, November 1, 2016 12:48 AM >ARC timers use aux registers for programming and this paves way for moving ARC timer drivers into drivers/clocksource Maybe in this patch or just another one could you move from timer.c to the new soc header all timer related Aux registers definitions? This could be used by timer-nps driver. i.e.: /* Timer related Aux registers */ #define ARC_REG_TIMER0_LIMIT 0x23 /* timer 0 limit */ #define ARC_REG_TIMER0_CTRL 0x22 /* timer 0 control */ #define ARC_REG_TIMER0_CNT 0x21 /* timer 0 count */ #define ARC_REG_TIMER1_LIMIT 0x102 /* timer 1 limit */ #define ARC_REG_TIMER1_CTRL 0x101 /* timer 1 control */ #define ARC_REG_TIMER1_CNT 0x100 /* timer 1 count */ #define TIMER_CTRL_IE (1 << 0) /* Interrupt when Count reaches limit */ #define TIMER_CTRL_NH (1 << 1) /* Count only when CPU NOT halted */ #define ARC_TIMER_MAX 0xFFFFFFFF