Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965161AbbEMNE6 (ORCPT ); Wed, 13 May 2015 09:04:58 -0400 Received: from bh-25.webhostbox.net ([208.91.199.152]:56195 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934201AbbEMNEz (ORCPT ); Wed, 13 May 2015 09:04:55 -0400 Message-ID: <55534BEF.7020409@roeck-us.net> Date: Wed, 13 May 2015 06:04:47 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Alexandre Belloni , "Opensource [Steve Twiss]" CC: Alessandro Zummo , LINUXKERNEL , RTCLINUX , DEVICETREE , David Dajun Chen , Dmitry Torokhov , Ian Campbell , Kumar Gala , LINUXINPUT , LINUXWATCHDOG , Lee Jones , Liam Girdwood , Mark Brown , Mark Rutland , Pawel Moll , Rob Herring , Samuel Ortiz , Support Opensource , Wim Van Sebroeck Subject: Re: [rtc-linux] [PATCH V1 3/6] rtc: da9062: DA9062 RTC driver References: <860a3b6dc7f8dac95c0e36967f24551af8805f7b.1429280614.git.stwiss.opensource@diasemi.com> <20150510095838.GF3338@piout.net> <6ED8E3B22081A4459DAC7699F3695FB7014B22A3F3@SW-EX-MBX02.diasemi.com> <20150513125858.GX3338@piout.net> In-Reply-To: <20150513125858.GX3338@piout.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=0.0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: authenticated_id: linux@roeck-us.net X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2079 Lines: 46 On 05/13/2015 05:58 AM, Alexandre Belloni wrote: > Hi, > > On 13/05/2015 at 12:31:36 +0000, Opensource [Steve Twiss] wrote : >> Something similar is being discussed for the OnKey component of the DA9062. >> https://lkml.org/lkml/2015/4/24/304 >> >> It is only the OnKey and RTC components that are similar to the DA9063 chip >> and I was hoping to keep the 62 RTC separate in this case ... however it should >> definitely be possible to re-use the DA9063 RTC driver if this is your requirement. >> If this will block my submission of the DA9062 then I will drop the RTC from my >> next patch set and try to re-work the existing 63 RTC driver accordingly. >> >> I think this sort of thing has been done before: >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/rtc/rtc-pm8xxx.c >> ... at first glance -- it seems to support that sort of thing. >> > > Isn't that exactly the point of using the MFD infrastructure? It allows > to reuse existing drivers even when the IP is part of a different chip. > The RTC inside the da9062 and the da9063 are obviously the same so there > is no point in duplicating the driver. > > I'm guessing using da9063-rtc instead of da9062-rtc is just working > fine. > >> I guess it would be possible to rename the da9063-rtc to something more sensible like >> da9xxx-rtc.c if this goes ahead? >> > > Sure, that can be done but this means that the module name will change. > if you feel that your current users can cope with that, I'm fine with it. > Don't know how this is handled for rtc drivers, but in other subsystems we just live with the original name. I don't see a need to rename a driver just because it starts supporting more hardware, and xxx is weird anyway since it suggests everything from 000 to 999, which is much worse than just sticking with 9063. Guenter -- 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/