From: Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>
To: Paul Walmsley <paul-DWxLp4Yu+b8AvxtiuMwx3w@public.gmane.org>
Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>,
Benoit Cousson <bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCHv2 0/9] omap hwspinlock dt support
Date: Mon, 30 Sep 2013 10:56:51 -0500 [thread overview]
Message-ID: <52499F43.3010007@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1309300309530.17980-rwI8Ez+7Ko+d5PgPZx9QOdBPR1lH4CV8@public.gmane.org>
On 09/29/2013 10:12 PM, Paul Walmsley wrote:
> Hi
>
> On Fri, 27 Sep 2013, Suman Anna wrote:
>
>> Paul,
>> The hwmod data patches needs to be merged only after the respective DT
>> node patches are merged, without which the hwmod entry will not have a
>> base address while enabling and idling (using sysc) the hwmod during
>> hwmod initialization.
>
> Hmm, ideally there shouldn't be a merge dependency here. Could you please
> send a patch to the hwmod core that logs a warning and skips the operation
> in these circumstances? Or maybe just skips the registration of the
> device?
>
Yes, will look into this. This is not specific to hwspinlock, but may be
seen with others as well if the omap_hwmod_addr_space or its equivalent
is not populated.
regards
Suman
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Suman Anna <s-anna@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: Ohad Ben-Cohen <ohad@wizery.com>,
Benoit Cousson <bcousson@baylibre.com>,
Tony Lindgren <tony@atomide.com>,
Kumar Gala <galak@codeaurora.org>, <linux-kernel@vger.kernel.org>,
<linux-omap@vger.kernel.org>, <devicetree@vger.kernel.org>
Subject: Re: [PATCHv2 0/9] omap hwspinlock dt support
Date: Mon, 30 Sep 2013 10:56:51 -0500 [thread overview]
Message-ID: <52499F43.3010007@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1309300309530.17980@utopia.booyaka.com>
On 09/29/2013 10:12 PM, Paul Walmsley wrote:
> Hi
>
> On Fri, 27 Sep 2013, Suman Anna wrote:
>
>> Paul,
>> The hwmod data patches needs to be merged only after the respective DT
>> node patches are merged, without which the hwmod entry will not have a
>> base address while enabling and idling (using sysc) the hwmod during
>> hwmod initialization.
>
> Hmm, ideally there shouldn't be a merge dependency here. Could you please
> send a patch to the hwmod core that logs a warning and skips the operation
> in these circumstances? Or maybe just skips the registration of the
> device?
>
Yes, will look into this. This is not specific to hwspinlock, but may be
seen with others as well if the omap_hwmod_addr_space or its equivalent
is not populated.
regards
Suman
next prev parent reply other threads:[~2013-09-30 15:56 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-17 19:30 [PATCHv2 0/9] omap hwspinlock dt support Suman Anna
2013-09-17 19:30 ` Suman Anna
2013-09-17 19:30 ` [PATCHv2 1/9] hwspinlock/core: add common dt bindings and OF helpers Suman Anna
2013-09-17 19:30 ` Suman Anna
2013-09-27 16:04 ` Kumar Gala
2013-09-27 16:04 ` Kumar Gala
[not found] ` <14DB8294-1148-446A-A6D3-34E66BA8732C-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2013-09-27 16:48 ` Suman Anna
2013-09-27 16:48 ` Suman Anna
[not found] ` <5245B6F9.7030505-l0cyMroinI0@public.gmane.org>
2013-09-27 17:14 ` Kumar Gala
2013-09-27 17:14 ` Kumar Gala
[not found] ` <3C582FCF-7A24-48AC-B98B-AF1D18C1E22B-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2013-09-27 19:26 ` Suman Anna
2013-09-27 19:26 ` Suman Anna
2013-10-01 8:36 ` Mark Rutland
2013-10-01 8:36 ` Mark Rutland
2013-10-03 4:04 ` Suman Anna
[not found] ` <524CECBF.2070805-l0cyMroinI0@public.gmane.org>
2013-10-09 21:40 ` Mark Rutland
2013-10-09 21:40 ` Mark Rutland
2013-09-17 19:30 ` [PATCHv2 2/9] hwspinlock/omap: add support for dt nodes Suman Anna
2013-09-17 19:30 ` Suman Anna
2013-09-27 16:06 ` Kumar Gala
2013-09-27 16:06 ` Kumar Gala
2013-09-27 16:21 ` Suman Anna
2013-09-27 16:21 ` Suman Anna
[not found] ` <5245B076.1090008-l0cyMroinI0@public.gmane.org>
2013-09-27 17:14 ` Kumar Gala
2013-09-27 17:14 ` Kumar Gala
[not found] ` <AE597DCC-83D7-4BCD-8990-3D63540F32EA-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2013-10-01 8:40 ` Mark Rutland
2013-10-01 8:40 ` Mark Rutland
2013-10-03 4:12 ` Suman Anna
[not found] ` <524CEE9F.8060407-l0cyMroinI0@public.gmane.org>
2013-10-09 21:46 ` Mark Rutland
2013-10-09 21:46 ` Mark Rutland
2013-10-10 20:29 ` Suman Anna
2013-10-10 20:29 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 3/9] ARM: dts: OMAP4: Add hwspinlock node Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 4/9] ARM: OMAP5: hwmod data: Add spinlock data Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-10-09 7:12 ` Paul Walmsley
[not found] ` <alpine.DEB.2.02.1310090712440.30199-rwI8Ez+7Ko+d5PgPZx9QOdBPR1lH4CV8@public.gmane.org>
2013-10-09 17:49 ` Suman Anna
2013-10-09 17:49 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 5/9] ARM: dts: OMAP5: Add hwspinlock node Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 6/9] hwspinlock/omap: support AM33xx Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 7/9] hwspinlock/omap: enable module before reading SYSSTATUS register Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 8/9] ARM: dts: AM33XX: Add hwspinlock node Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-09-17 19:31 ` [PATCHv2 9/9] ARM: AM33xx: hwmod_data: add the sysc configuration for spinlock Suman Anna
2013-09-17 19:31 ` Suman Anna
2013-10-09 7:12 ` Paul Walmsley
[not found] ` <cover.1379445653.git.s-anna-l0cyMroinI0@public.gmane.org>
2013-09-27 15:48 ` [PATCHv2 0/9] omap hwspinlock dt support Suman Anna
2013-09-27 15:48 ` Suman Anna
[not found] ` <5245A8CA.4020805-l0cyMroinI0@public.gmane.org>
2013-09-30 3:12 ` Paul Walmsley
2013-09-30 3:12 ` Paul Walmsley
[not found] ` <alpine.DEB.2.02.1309300309530.17980-rwI8Ez+7Ko+d5PgPZx9QOdBPR1lH4CV8@public.gmane.org>
2013-09-30 15:56 ` Suman Anna [this message]
2013-09-30 15:56 ` 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=52499F43.3010007@ti.com \
--to=s-anna-l0cymroini0@public.gmane.org \
--cc=bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org \
--cc=paul-DWxLp4Yu+b8AvxtiuMwx3w@public.gmane.org \
--cc=tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org \
/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.