From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by arago-project.org (Postfix) with ESMTPS id A8C6F529A7 for ; Thu, 5 Feb 2015 18:14:45 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id t15IEjs5000995 for ; Thu, 5 Feb 2015 12:14:45 -0600 Received: from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id t15IEhDj019112 for ; Thu, 5 Feb 2015 12:14:43 -0600 Received: from dflp32.itg.ti.com (10.64.6.15) by DLEE70.ent.ti.com (157.170.170.113) with Microsoft SMTP Server id 14.3.224.2; Thu, 5 Feb 2015 12:14:43 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp32.itg.ti.com (8.14.3/8.13.8) with ESMTP id t15IEhPK004148; Thu, 5 Feb 2015 12:14:43 -0600 Date: Thu, 5 Feb 2015 13:14:43 -0500 From: Denys Dmytriyenko To: "Cooper Jr., Franklin" Message-ID: <20150205181443.GP9943@edge> References: <1423152936-3396-1-git-send-email-sam.nelson@ti.com> <8F29D6B095ED194EA1980491A5E029710C7F91C3@DFLE08.ent.ti.com> <20150205165237.GJ9943@edge> <8F29D6B095ED194EA1980491A5E029710C7F926D@DFLE08.ent.ti.com> <20150205171423.GL9943@edge> <8F29D6B095ED194EA1980491A5E029710C7F92B2@DFLE08.ent.ti.com> <20150205174958.GN9943@edge> MIME-Version: 1.0 In-Reply-To: <20150205174958.GN9943@edge> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" Subject: Re: [PATCH] sysvinit: Changing default setting to add HWCLOCKACCESS=no X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Feb 2015 18:14:45 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Thu, Feb 05, 2015 at 12:49:58PM -0500, Denys Dmytriyenko wrote: > On Thu, Feb 05, 2015 at 12:19:30PM -0500, Cooper Jr., Franklin wrote: > > > On Thu, Feb 05, 2015 at 12:08:09PM -0500, Cooper Jr., Franklin wrote: > > > > > On Thu, Feb 05, 2015 at 04:45:43PM +0000, Cooper Jr., Franklin wrote: > > > > > > I see references to several boards that have enabled rtc in its > > > > > > dts file. I didn't check all the boards we support in meta-ti but > > > > > > additional boards may use it. So we shouldn't disable it for these boards. > > > > > > > > > > Exactly, so the proposal was to disable it by default and enable > > > > > through machine overrides for those boards that have it. > > > > > > > [Franklin] I'm assuming there was some conversation in the background > > > > regarding this patch > > > > > > Well, not about the patch, but about the problem this patch is trying to fix :) > > > > > > > > > > I wasn't aware of since the above two points weren't clear. But no > > > > problem doing with what you suggested as long as enabling it for the > > > > right boards is being done as part of a patch series with this one. > > > > > > Do we have a list of boards that have RTC? Just glancing over what I have I > > > don't see any... > > [Franklin] am437x sk, am437x gp evm and am57x based on looking at the dt. > > Hmm, I don't see any batteries on am437x sk. But x15 does have a battery next > to Eth port. > > > > > As an alternative, we can keep it enabled by default and disable it just for > > > Keystone to make Sam happy. But I've seen that error on pretty much every > > > platform lately. > > > [Franklin] I would prefer that approach instead. And true about the error > > but it could be a bug or error that needs to be fixed so I rather not hide > > it especially since I haven't received any questions regarding it before. > > Ok I just sent a tiny patch to meta-ti that disables rtc and dependent hwclock on keystone platforms. Anyone who needs to do the same for other platforms, feel free to send patches. -- Denys