From: Suman Anna <s-anna@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: Tony Lindgren <tony@atomide.com>,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/2] Couple of DRA7 hwmod patches on Timers
Date: Mon, 23 Mar 2015 11:30:13 -0500 [thread overview]
Message-ID: <55103F95.5070100@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1503211939340.2512@utopia.booyaka.com>
Hi Paul,
>> Following are couple of DRA7 hwmod patches for the GPTimers.
>> Patches based on 4.0-rc1.
>>
>> The first patch adds the data for timers 13 through 16, the DT
>> nodes are already present, and when enabled without the hwmod
>> data triggers a l3_noc interrupt and hangs the kernel boot [1].
>> The boot hang can also be fixed by checking the return status
>> of pm_runtime_get_sync() in the OMAP dmtimer probe, I will post
>> a separate fix for that.
>>
>> Second patch is a minor fix.
>
> Thanks. Sounds like the first one should go in for v4.0-rc fixes, and the
> second one can wait for v4.1?
>
> If so then could you repost the first fix to include the description of
> why it should go in early in the patch description ("the DT nodes are
> already present, and when enabled without the hwmod data triggers a l3_noc
> interrupt and hangs the kernel boot"). That should avoid anyone
> questioning why it would go in as a v4.0-rc fix at this point, and should
> help the -stable crew out.
Actually, both of them can be queued for v4.1. Tony has already sent a
pull request with the fixes [1] in the driver. The driver fixes will
scale for all SoCs, so will fix this boot issue as well if anyone tries
to enable these timers.
regards
Suman
[1] http://marc.info/?l=linux-omap&m=142661461902725&w=2
WARNING: multiple messages have this Message-ID (diff)
From: s-anna@ti.com (Suman Anna)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] Couple of DRA7 hwmod patches on Timers
Date: Mon, 23 Mar 2015 11:30:13 -0500 [thread overview]
Message-ID: <55103F95.5070100@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1503211939340.2512@utopia.booyaka.com>
Hi Paul,
>> Following are couple of DRA7 hwmod patches for the GPTimers.
>> Patches based on 4.0-rc1.
>>
>> The first patch adds the data for timers 13 through 16, the DT
>> nodes are already present, and when enabled without the hwmod
>> data triggers a l3_noc interrupt and hangs the kernel boot [1].
>> The boot hang can also be fixed by checking the return status
>> of pm_runtime_get_sync() in the OMAP dmtimer probe, I will post
>> a separate fix for that.
>>
>> Second patch is a minor fix.
>
> Thanks. Sounds like the first one should go in for v4.0-rc fixes, and the
> second one can wait for v4.1?
>
> If so then could you repost the first fix to include the description of
> why it should go in early in the patch description ("the DT nodes are
> already present, and when enabled without the hwmod data triggers a l3_noc
> interrupt and hangs the kernel boot"). That should avoid anyone
> questioning why it would go in as a v4.0-rc fix at this point, and should
> help the -stable crew out.
Actually, both of them can be queued for v4.1. Tony has already sent a
pull request with the fixes [1] in the driver. The driver fixes will
scale for all SoCs, so will fix this boot issue as well if anyone tries
to enable these timers.
regards
Suman
[1] http://marc.info/?l=linux-omap&m=142661461902725&w=2
next prev parent reply other threads:[~2015-03-23 16:30 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-16 20:54 [PATCH 0/2] Couple of DRA7 hwmod patches on Timers Suman Anna
2015-03-16 20:54 ` Suman Anna
2015-03-16 20:54 ` [PATCH 1/2] ARM: DRA7: hwmod: Add data for GPTimers 13 through 16 Suman Anna
2015-03-16 20:54 ` Suman Anna
2015-03-24 18:05 ` Paul Walmsley
2015-03-24 18:05 ` Paul Walmsley
2015-03-16 20:54 ` [PATCH 2/2] ARM: DRA7: hwmod: Fix the hwmod class for GPTimer4 Suman Anna
2015-03-16 20:54 ` Suman Anna
2015-03-24 18:06 ` Paul Walmsley
2015-03-24 18:06 ` Paul Walmsley
2015-03-24 19:23 ` Suman Anna
2015-03-24 19:23 ` Suman Anna
2015-03-21 19:42 ` [PATCH 0/2] Couple of DRA7 hwmod patches on Timers Paul Walmsley
2015-03-21 19:42 ` Paul Walmsley
2015-03-23 16:30 ` Suman Anna [this message]
2015-03-23 16:30 ` Suman Anna
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=55103F95.5070100@ti.com \
--to=s-anna@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
--cc=tony@atomide.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.