From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752961AbbEATlM (ORCPT ); Fri, 1 May 2015 15:41:12 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:53752 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752911AbbEATlJ (ORCPT ); Fri, 1 May 2015 15:41:09 -0400 From: Arnd Bergmann To: One Thousand Gnomes Cc: Jonathan Richardson , Scott Branden , Darren Edamura , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Ray Jui , Greg Kroah-Hartman , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus Date: Fri, 01 May 2015 21:40:40 +0200 Message-ID: <9193036.7UYD2OGJHk@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20150501203004.3add2c95@lxorguk.ukuu.org.uk> References: <1429744923-2007-1-git-send-email-jonathar@broadcom.com> <5543CD73.2030902@broadcom.com> <20150501203004.3add2c95@lxorguk.ukuu.org.uk> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:HO/5NZwHufJuGShi3TXa30C0IKZQOtqmjkvcd1HGHM6rXdqRrjN inMiX3AYmL7HeLVl+dLb6Mfpa5kpAN79hJAeLZ5YVNG+Q+dsQswndOzXFDmG002EZAbUQQn EWx9eNlNhT1ofpdABaxDQWpWfIF75V0aAcbgZ70s1L8ekuDJuPG9iY7a5lu2HBt1F0yskRg K6ziq1nev2er7LKaxevpA== X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 01 May 2015 20:30:04 One Thousand Gnomes wrote: > > > channel control, unless I'm missing something. If people are flexible > > with extending that I could propose something. Let me know which way you > > prefer to go. Thanks. > > I would strongly favour fixing PTP to do this right. Otherwise we will > have 2 or 3 adhoc drivers, then end up moving them to PTP and then end up > dealing with the compat mess. Agreed. I was hoping that there was already a subsystem in the kernel that could be extended to work with this driver. If all that is needed is to pass more fields of the existing timex to ptp drivers, that should be fairly easy to do. We also have some padding bytes available in struct timex in case some extra data needs to be passed, or we could add another clock_* system call if it's absolutely required to have another entry into the driver, and connect that through struct k_clock and posix_clock_operations(). I would hope we don't even need that. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Fri, 01 May 2015 21:40:40 +0200 Subject: [PATCH 2/2] misc: Add initial Digital Timing Engine (DTE) driver for cygnus In-Reply-To: <20150501203004.3add2c95@lxorguk.ukuu.org.uk> References: <1429744923-2007-1-git-send-email-jonathar@broadcom.com> <5543CD73.2030902@broadcom.com> <20150501203004.3add2c95@lxorguk.ukuu.org.uk> Message-ID: <9193036.7UYD2OGJHk@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Friday 01 May 2015 20:30:04 One Thousand Gnomes wrote: > > > channel control, unless I'm missing something. If people are flexible > > with extending that I could propose something. Let me know which way you > > prefer to go. Thanks. > > I would strongly favour fixing PTP to do this right. Otherwise we will > have 2 or 3 adhoc drivers, then end up moving them to PTP and then end up > dealing with the compat mess. Agreed. I was hoping that there was already a subsystem in the kernel that could be extended to work with this driver. If all that is needed is to pass more fields of the existing timex to ptp drivers, that should be fairly easy to do. We also have some padding bytes available in struct timex in case some extra data needs to be passed, or we could add another clock_* system call if it's absolutely required to have another entry into the driver, and connect that through struct k_clock and posix_clock_operations(). I would hope we don't even need that. Arnd