From: Heikki Krogerus <heikki.krogerus@nokia.com>
To: Grazvydas Ignotas <notasas@gmail.com>
Cc: balbi@ti.com, linux-omap@vger.kernel.org, cbouatmailru@gmail.com
Subject: Re: [PATCH 0/1] power_supply: twl4030_charger modification
Date: Mon, 28 Feb 2011 08:05:33 +0200 [thread overview]
Message-ID: <20110228060533.GA2181@esdhcp034230> (raw)
In-Reply-To: <AANLkTik_N0Kk7putk+mb__S7NJxCYHSrxCH=DofduGpX@mail.gmail.com>
On Sat, Feb 26, 2011 at 08:10:17PM +0200, ext Grazvydas Ignotas wrote:
> On Fri, Feb 25, 2011 at 2:12 PM, Felipe Balbi <balbi@ti.com> wrote:
> > On Fri, Feb 25, 2011 at 01:56:35PM +0200, Heikki Krogerus wrote:
> >> Hi guys,
> >>
> >> Grazvydas, I don't have hardware to test this with. If this is OK to
> >> you, could you test it?
>
> Sure, replied to your patch.
Thanks!
> >> Felipe, was the idea that you take this with the atomic notifier
> >> support for otg patches? I guess this is how it needs to be done in
> >> order to keep things bisectable, right? Or does this simply go to
> >> Anton?
> >
> > We can't change patches anymore :-( So as long as it goes on the merge
> > window, we should be ok, hopefully.
>
> Talking about notifiers, do we have away to find the amount of current
> host can provide yet? I haven't followed USB development closely
> lately.
There is nothing to support this yet. I'm only assuming the ENUMERATED
event will be used for this. Now with the atomic notifiers it should
be OK for the controller drivers to send this notification always when
usb_gadget_vbus_draw() is called.
But Felipe is working with the udc class. Maybe he has some other
method in mind? For now I think we could send the ENUMERATED
event from all the controller drivers. Felipe?
--
heikki
prev parent reply other threads:[~2011-02-28 6:05 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-25 11:56 [PATCH 0/1] power_supply: twl4030_charger modification Heikki Krogerus
2011-02-25 11:56 ` [PATCH 1/1] twl4030_charger: Make the driver atomic notifier safe Heikki Krogerus
2011-02-26 17:53 ` Grazvydas Ignotas
2011-02-28 6:29 ` Heikki Krogerus
2011-02-28 14:32 ` Anton Vorontsov
2011-02-25 12:12 ` [PATCH 0/1] power_supply: twl4030_charger modification Felipe Balbi
2011-02-26 18:10 ` Grazvydas Ignotas
2011-02-28 6:05 ` Heikki Krogerus [this message]
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=20110228060533.GA2181@esdhcp034230 \
--to=heikki.krogerus@nokia.com \
--cc=balbi@ti.com \
--cc=cbouatmailru@gmail.com \
--cc=linux-omap@vger.kernel.org \
--cc=notasas@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox