From: Daniel Mack <zonque@gmail.com>
To: Vaibhav Hiremath <hvaibhav@ti.com>
Cc: rtc-linux@googlegroups.com, a.zummo@towertech.it,
koen@dominion.thruhere.net, david-b@pacbell.net,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/3] RTC: omap-rtc: enable pm_runtime
Date: Thu, 18 Oct 2012 18:19:44 +0200 [thread overview]
Message-ID: <50802C20.9040408@gmail.com> (raw)
In-Reply-To: <50802A87.9010409@ti.com>
On 18.10.2012 18:12, Vaibhav Hiremath wrote:
>
>
> On 10/18/2012 9:23 PM, Daniel Mack wrote:
>> This is needed as preparation for platforms where using pm runtime usage
>> is mandatory.
>>
>> Signed-off-by: Daniel Mack <zonque@gmail.com>
>
> It looks like, you just duplicated effort here.
> RTC patches have been already submitted quite some time back for AM33xx,
> probably you missed to do google before spending time on this.
>
> Patch Series -
> http://www.mail-archive.com/davinci-linux-open-source@linux.davincidsp.com/msg23253.html
Heh, damn :) I would have thought that so shortly after the merge
window, that code either went in already or is yet missing. Oh well.
> Denial,
> It would be really helpful if you could test these patches and ack them.
Ok, will do tomorrow. What's missing there is support for writing to the
OMAP_RTC_OSC_REG (path 3/3 in my series). Would like me to rebase the
functionality? I need thatkind of setup for my board ...
> Just FYI, we have bunch of parallel activities going inside team to
> upstream all the patches/module/drivers/features, so to avoid such
> duplication of effort, I suggest you to google for the patches or ping
> on mailing list.
Yes, it's more difficult to participate in the AM33xx code than in other
platforms. I'm not complaining about other doing all the hard work
though, it's just a little different.
Anyway - thanks for the heads-up!
Daniel
WARNING: multiple messages have this Message-ID (diff)
From: zonque@gmail.com (Daniel Mack)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/3] RTC: omap-rtc: enable pm_runtime
Date: Thu, 18 Oct 2012 18:19:44 +0200 [thread overview]
Message-ID: <50802C20.9040408@gmail.com> (raw)
In-Reply-To: <50802A87.9010409@ti.com>
On 18.10.2012 18:12, Vaibhav Hiremath wrote:
>
>
> On 10/18/2012 9:23 PM, Daniel Mack wrote:
>> This is needed as preparation for platforms where using pm runtime usage
>> is mandatory.
>>
>> Signed-off-by: Daniel Mack <zonque@gmail.com>
>
> It looks like, you just duplicated effort here.
> RTC patches have been already submitted quite some time back for AM33xx,
> probably you missed to do google before spending time on this.
>
> Patch Series -
> http://www.mail-archive.com/davinci-linux-open-source at linux.davincidsp.com/msg23253.html
Heh, damn :) I would have thought that so shortly after the merge
window, that code either went in already or is yet missing. Oh well.
> Denial,
> It would be really helpful if you could test these patches and ack them.
Ok, will do tomorrow. What's missing there is support for writing to the
OMAP_RTC_OSC_REG (path 3/3 in my series). Would like me to rebase the
functionality? I need thatkind of setup for my board ...
> Just FYI, we have bunch of parallel activities going inside team to
> upstream all the patches/module/drivers/features, so to avoid such
> duplication of effort, I suggest you to google for the patches or ping
> on mailing list.
Yes, it's more difficult to participate in the AM33xx code than in other
platforms. I'm not complaining about other doing all the hard work
though, it's just a little different.
Anyway - thanks for the heads-up!
Daniel
next prev parent reply other threads:[~2012-10-18 16:19 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-18 15:53 [PATCH 1/3] RTC: omap-rtc: enable for SOC_AM33XX Daniel Mack
2012-10-18 15:53 ` Daniel Mack
2012-10-18 15:53 ` [PATCH 2/3] RTC: omap-rtc: enable pm_runtime Daniel Mack
2012-10-18 15:53 ` Daniel Mack
2012-10-18 16:12 ` Vaibhav Hiremath
2012-10-18 16:12 ` Vaibhav Hiremath
2012-10-18 16:19 ` Daniel Mack [this message]
2012-10-18 16:19 ` Daniel Mack
2012-10-18 16:33 ` Hiremath, Vaibhav
2012-10-18 16:33 ` Hiremath, Vaibhav
2012-10-19 10:58 ` Mohammed, Afzal
2012-10-19 10:58 ` Mohammed, Afzal
2012-10-21 19:51 ` Daniel Mack
2012-10-21 19:51 ` Daniel Mack
2012-10-22 5:59 ` Hiremath, Vaibhav
2012-10-22 5:59 ` Hiremath, Vaibhav
2012-10-18 15:53 ` [PATCH 3/3] RTC: omap-rtc: add DT bindings Daniel Mack
2012-10-18 15:53 ` Daniel Mack
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50802C20.9040408@gmail.com \
--to=zonque@gmail.com \
--cc=a.zummo@towertech.it \
--cc=david-b@pacbell.net \
--cc=hvaibhav@ti.com \
--cc=koen@dominion.thruhere.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=rtc-linux@googlegroups.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.