linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: rklein@nvidia.com (Rhyland Klein)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: tegra: Add charger subnode to tps65090 node
Date: Mon, 20 May 2013 11:35:03 -0400	[thread overview]
Message-ID: <519A42A7.5080208@nvidia.com> (raw)
In-Reply-To: <519A4108.2040606@wwwdotorg.org>

On 5/20/2013 11:28 AM, Stephen Warren wrote:
> On 05/20/2013 09:24 AM, Rhyland Klein wrote:
>> On 5/17/2013 7:57 PM, Stephen Warren wrote:
>>> On 04/10/2013 01:51 PM, Rhyland Klein wrote:
>>>> The charger is now represented by a distinct subnode of the tps65090
>>>> device. Add this node and enable low current charging with it.
>>>
>>> What's the status of the TPS60590 bindings; are they agreed upon by
>>> NVIDIA, TI, and SlimLogic yet? In other words, is this patch still
>>> something I should apply for 3.11, or does it need to be reworked?
>>
>> I haven't seen any discussion with slimlogic or TI about the tps65090.
>> As far as I know the bindings for this driver haven't changed.
>>
>> Laxman, do you think the work on the palmas driver will impact the
>> design of the bindings for the tps65090?
> 
> Sorry, perhaps I'm confusing two different chips. If TPS65090 isn't
> Palmas, then ignore my question. In which case, I suppose I should just
> apply your patch then?
> 

I would say yes. The design of having the child node this way was how we
had agreed worked best. The only reason I would see a significant reason
to change this, is if something was decided that all mfd devices should
start to follow some pattern which differed, which would mean changing
existing bindings and therefore is unlikely.

-rhyland

-- 
nvpublic

  reply	other threads:[~2013-05-20 15:35 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-10 19:51 [PATCH] ARM: tegra: Add charger subnode to tps65090 node Rhyland Klein
2013-04-10 22:30 ` Stephen Warren
2013-04-11 15:38   ` Rhyland Klein
2013-04-11 16:17     ` Stephen Warren
2013-04-11 16:27       ` Rhyland Klein
2013-05-17 23:57 ` Stephen Warren
2013-05-20 15:24   ` Rhyland Klein
2013-05-20 15:28     ` Stephen Warren
2013-05-20 15:35       ` Rhyland Klein [this message]
2013-05-20 19:15         ` Stephen Warren
2013-05-20 20:45           ` Rhyland Klein

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=519A42A7.5080208@nvidia.com \
    --to=rklein@nvidia.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).