All of lore.kernel.org
 help / color / mirror / Atom feed
From: nm@ti.com (Nishanth Menon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/4] firmware: ti_sci: Add helper apis to mange resources
Date: Sat, 6 Oct 2018 11:34:32 -0500	[thread overview]
Message-ID: <20181006163432.z5skpgfm7oqoiuaf@akan> (raw)
In-Reply-To: <176f77cf-d295-0f37-ea9d-26f76f6d79c7@ti.com>

On 13:08-20181006, Lokesh Vutla wrote:
> 
> 
> On 10/6/2018 1:44 AM, Nishanth Menon wrote:
> > NAK. Where is the binding for this? What is this decoding? Please don't
> > decode binding without having an official binding.
> 
> This is a helper library for the clients to get resource range from DT
> and is not compulsory for clients to use this library to managing these
> resources. So in my understanding DT bindings should go into the client
> DT doc as posted here[1].  If you really insist I can add these details

Thanks.. Please get the binding approved before we discuss
implementation in driver.

> in ti,sci documentation but not sure if we are doing the same for all
> other helper apis.

the core TISCI device does'nt require any ranges property. these need to
be parsed and handled in corresponding tisci drivers? I don't see the
benefit in dealing with OF parsing in generic TISCI firmware driver.
yes, there might be a little similarities, but really we'd have to argue
strongly why it needs to belong to the firmware driver. So far it looks
like just a single irq-chip driver seems to need it, I cant see the
argument for it getting caught up in that path.

--
Regards,
Nishanth Menon

  reply	other threads:[~2018-10-06 16:34 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-05 14:30 [PATCH 0/4] firmware: ti_sci: Introduce RM support Lokesh Vutla
2018-10-05 14:30 ` [PATCH 1/4] firmware: ti_sci: Add support to get TISCI handle using of_phandle Lokesh Vutla
2018-10-05 14:30 ` [PATCH 2/4] firmware: ti_sci: Add support for RM core ops Lokesh Vutla
2018-10-05 14:30 ` [PATCH 3/4] firmware: ti_sci: Add support for IRQ management Lokesh Vutla
2018-10-05 14:30 ` [PATCH 4/4] firmware: ti_sci: Add helper apis to mange resources Lokesh Vutla
2018-10-05 20:14   ` Nishanth Menon
2018-10-06  7:38     ` Lokesh Vutla
2018-10-06 16:34       ` Nishanth Menon [this message]
2018-10-10 17:00         ` Santosh Shilimkar

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=20181006163432.z5skpgfm7oqoiuaf@akan \
    --to=nm@ti.com \
    --cc=linux-arm-kernel@lists.infradead.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.