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
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to