From: Suman Anna <s-anna@ti.com> To: Ohad Ben-Cohen <ohad@wizery.com>, Tony Lindgren <tony@atomide.com> Cc: Mark Rutland <mark.rutland@arm.com>, Rob Herring <robherring2@gmail.com>, Kumar Gala <galak@codeaurora.org>, Bjorn Andersson <bjorn@kryo.se>, Josh Cartwright <joshc@codeaurora.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, Rob Herring <robh+dt@kernel.org> Subject: Re: [PATCH v7 1/4] Documentation: dt: add common bindings for hwspinlock Date: Wed, 21 Jan 2015 11:56:37 -0600 [thread overview] Message-ID: <54BFE855.3090200@ti.com> (raw) In-Reply-To: <CAK=WgbZGGtwmQwzWumjpYOsYHTt8vPQpTraABwJBsv5VXOmJ_Q@mail.gmail.com> On 01/21/2015 06:41 AM, Ohad Ben-Cohen wrote: > On Tue, Jan 20, 2015 at 8:05 PM, Tony Lindgren <tony@atomide.com> wrote: >> How about default to Linux id space and allow overriding that with >> a module param option if needed? > > I'm not sure I'm following. > > If the main point of contention is the base_id field, I'm also fine > with removing it entirely, as I'm not aware of any actual user for it > (Suman please confirm?). Yeah, well the current implementations that I am aware of only have a single bank, so all of them would be using a value of 0. I am yet to see a platform with multiple instances where the property really makes a difference. v7 has the property mandatory, so all the implementations would need to define this value even if it is 0. regards Suman > > Mark? Rob? Will you accept Suman's patches if the base_id field is removed? > > Thanks, > Ohad. >
WARNING: multiple messages have this Message-ID (diff)
From: s-anna@ti.com (Suman Anna) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v7 1/4] Documentation: dt: add common bindings for hwspinlock Date: Wed, 21 Jan 2015 11:56:37 -0600 [thread overview] Message-ID: <54BFE855.3090200@ti.com> (raw) In-Reply-To: <CAK=WgbZGGtwmQwzWumjpYOsYHTt8vPQpTraABwJBsv5VXOmJ_Q@mail.gmail.com> On 01/21/2015 06:41 AM, Ohad Ben-Cohen wrote: > On Tue, Jan 20, 2015 at 8:05 PM, Tony Lindgren <tony@atomide.com> wrote: >> How about default to Linux id space and allow overriding that with >> a module param option if needed? > > I'm not sure I'm following. > > If the main point of contention is the base_id field, I'm also fine > with removing it entirely, as I'm not aware of any actual user for it > (Suman please confirm?). Yeah, well the current implementations that I am aware of only have a single bank, so all of them would be using a value of 0. I am yet to see a platform with multiple instances where the property really makes a difference. v7 has the property mandatory, so all the implementations would need to define this value even if it is 0. regards Suman > > Mark? Rob? Will you accept Suman's patches if the base_id field is removed? > > Thanks, > Ohad. >
next prev parent reply other threads:[~2015-01-21 17:57 UTC|newest] Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-14 20:58 [PATCH v7 0/4] hwspinlock core & omap dt support Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` [PATCH v7 1/4] Documentation: dt: add common bindings for hwspinlock Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-15 13:52 ` Mark Rutland 2015-01-15 13:52 ` Mark Rutland 2015-01-15 13:52 ` Mark Rutland 2015-01-15 13:55 ` Mark Rutland 2015-01-15 13:55 ` Mark Rutland 2015-01-15 13:55 ` Mark Rutland 2015-01-15 14:42 ` Rob Herring 2015-01-15 14:42 ` Rob Herring 2015-01-15 14:42 ` Rob Herring 2015-01-15 20:16 ` Suman Anna 2015-01-15 20:16 ` Suman Anna 2015-01-15 20:16 ` Suman Anna 2015-01-16 6:09 ` Ohad Ben-Cohen 2015-01-16 6:09 ` Ohad Ben-Cohen 2015-01-16 6:09 ` Ohad Ben-Cohen 2015-01-16 10:17 ` Mark Rutland 2015-01-16 10:17 ` Mark Rutland 2015-01-16 10:17 ` Mark Rutland 2015-01-17 0:46 ` Ohad Ben-Cohen 2015-01-17 0:46 ` Ohad Ben-Cohen 2015-01-17 0:46 ` Ohad Ben-Cohen 2015-01-20 18:05 ` Tony Lindgren 2015-01-20 18:05 ` Tony Lindgren 2015-01-20 18:05 ` Tony Lindgren 2015-01-21 12:41 ` Ohad Ben-Cohen 2015-01-21 12:41 ` Ohad Ben-Cohen 2015-01-21 12:41 ` Ohad Ben-Cohen 2015-01-21 17:56 ` Suman Anna [this message] 2015-01-21 17:56 ` Suman Anna 2015-01-21 17:56 ` Suman Anna 2015-01-22 18:56 ` Mark Rutland 2015-01-22 18:56 ` Mark Rutland 2015-01-22 18:56 ` Mark Rutland 2015-01-29 3:58 ` Suman Anna 2015-01-29 3:58 ` Suman Anna 2015-01-29 3:58 ` Suman Anna 2015-02-11 11:29 ` Mark Rutland 2015-02-11 11:29 ` Mark Rutland 2015-02-11 11:29 ` Mark Rutland 2015-02-16 18:06 ` Bjorn Andersson 2015-02-16 18:06 ` Bjorn Andersson 2015-02-16 18:06 ` Bjorn Andersson 2015-01-30 23:29 ` Bjorn Andersson 2015-01-30 23:29 ` Bjorn Andersson 2015-01-30 23:29 ` Bjorn Andersson 2015-01-31 5:41 ` Ohad Ben-Cohen 2015-01-31 5:41 ` Ohad Ben-Cohen 2015-01-31 5:41 ` Ohad Ben-Cohen 2015-02-01 11:00 ` Ohad Ben-Cohen 2015-02-01 11:00 ` Ohad Ben-Cohen 2015-02-01 11:00 ` Ohad Ben-Cohen 2015-02-02 21:14 ` Suman Anna 2015-02-02 21:14 ` Suman Anna 2015-02-02 21:14 ` Suman Anna 2015-02-01 17:55 ` Bjorn Andersson 2015-02-01 17:55 ` Bjorn Andersson 2015-02-01 17:55 ` Bjorn Andersson 2015-02-02 21:07 ` Suman Anna 2015-02-02 21:07 ` Suman Anna 2015-02-02 21:07 ` Suman Anna 2015-02-05 23:01 ` Bjorn Andersson 2015-02-05 23:01 ` Bjorn Andersson 2015-02-05 23:01 ` Bjorn Andersson 2015-02-06 0:11 ` Suman Anna 2015-02-06 0:11 ` Suman Anna 2015-02-06 0:11 ` Suman Anna 2015-02-06 0:34 ` Bjorn Andersson 2015-02-06 0:34 ` Bjorn Andersson 2015-02-06 0:34 ` Bjorn Andersson 2015-02-11 10:29 ` Ohad Ben-Cohen 2015-02-11 10:29 ` Ohad Ben-Cohen 2015-02-11 10:29 ` Ohad Ben-Cohen 2015-02-11 11:35 ` Mark Rutland 2015-02-11 11:35 ` Mark Rutland 2015-02-11 11:35 ` Mark Rutland 2015-02-16 20:30 ` Bjorn Andersson 2015-02-16 20:30 ` Bjorn Andersson 2015-02-16 20:30 ` Bjorn Andersson 2015-01-16 10:19 ` Mark Rutland 2015-01-16 10:19 ` Mark Rutland 2015-01-16 10:19 ` Mark Rutland 2015-01-16 17:49 ` Suman Anna 2015-01-16 17:49 ` Suman Anna 2015-01-16 17:49 ` Suman Anna 2015-01-14 20:58 ` [PATCH v7 2/4] Documentation: dt: add the omap hwspinlock bindings document Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` [PATCH v7 3/4] hwspinlock/core: add common OF helpers Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` [PATCH v7 4/4] hwspinlock/omap: add support for dt nodes Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-14 20:58 ` Suman Anna 2015-01-15 10:13 ` [PATCH v7 0/4] hwspinlock core & omap dt support Ohad Ben-Cohen 2015-01-15 10:13 ` Ohad Ben-Cohen
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=54BFE855.3090200@ti.com \ --to=s-anna@ti.com \ --cc=bjorn@kryo.se \ --cc=devicetree@vger.kernel.org \ --cc=galak@codeaurora.org \ --cc=joshc@codeaurora.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=ohad@wizery.com \ --cc=robh+dt@kernel.org \ --cc=robherring2@gmail.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: linkBe 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.