lkml.org 
[lkml]   [2015]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ARM: OMAP1: PM: fix some build warnings on 1510-only Kconfigs
On Wed, 11 Feb 2015, Tony Lindgren wrote:

> * Paul Walmsley <paul@pwsan.com> [150210 18:28]:
> > On Tue, 10 Feb 2015, Jon Hunter wrote:
> > > On 07/02/2015 00:23, Paul Walmsley wrote:
> >
> > > Unfortunately, there is not a single TRM for the omap5910 but individual
> > > documents for each chapter in the original TRM. Check out the "OMAP5910
> > > Dual-Core Processor Timer Reference Guide" and possibly the "OMAP5910
> > > Dual-Core Processor Clock Generation and System Reset Management
> > > Reference Guide"
> > >
> > > The omap15xx/5910 did have a 32k timer but as you can see it appears it
> > > was never supported by the kernel for this device (not sure why). I do
> > > recall that there is some errata regarding the 32k timer, if you look at
> > > the omap5910 errata document and search for 32k you should find it.
> >
> > OK thanks for the context. I probably am not going to investigate adding
> > support for this timer on OMAP1510/5910 - am primarily trying to avoid
> > causing a regression on the existing platforms.
>
> At least I've never seen the 32KiHz timer registers in any 15xx
> documentation. Jon are you sure you're not mixing up 5910 (15xx)
> and 5912 (16xx)?

It's documented in the OMAP5910 Timer Reference Guide (SPRU682A) Section 3
"32-kHz Timer", at the link Jon mentioned. Have not checked the errata
that Jon mentioned though.

Regarding the patch: I'd suggest keeping the compilation warning fixes
(which was the original purpose of the patch) from anything that changes
the logic too much. That way if there's an error in the patch that
changes the logic and it needs to be reverted, it won't also revert the
warning fixes.


- Paul


\
 
 \ /
  Last update: 2015-02-11 22:21    [W:0.075 / U:5.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site