Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1370540 > unrolled thread

Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework

Started byFelipe Balbi <balbi@kernel.org>
First post2016-04-04 13:00 +0200
Last post2016-04-04 20:50 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework Felipe Balbi <balbi@kernel.org> - 2016-04-04 13:00 +0200
    Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework Mark Brown <broonie@kernel.org> - 2016-04-04 18:10 +0200
      Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework Greg KH <gregkh@linuxfoundation.org> - 2016-04-04 20:50 +0200

#1370540 — Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework

FromFelipe Balbi <balbi@kernel.org>
Date2016-04-04 13:00 +0200
SubjectRe: [PATCH v7 1/4] gadget: Introduce the usb charger framework
Message-ID<rkaoG-1Mz-1@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Hi,

Mark Brown <broonie@kernel.org> writes:
> On Fri, Apr 01, 2016 at 08:43:10AM +0300, Felipe Balbi wrote:
>> Mark Brown <broonie@kernel.org> writes:
>
>> > IIRC Greg didn't want new classes?
>
>> good point. Still, this doesn't seem to fit a but_type IMO.
>
> It does in this new world order.  IIRC on an earlier round of review
> there was some code that didn't use a bus but that got complaints that
> it was trying to reimplement the bus functionality.

fair enough, I'll wait for Greg to have some time to comment on
this. Bottomline is that there is *no* real bus. Charger ICs will use
SPI or I2C and that's a real bus, $subject is not.

-- 
balbi

[toc] | [next] | [standalone]


#1370724

FromMark Brown <broonie@kernel.org>
Date2016-04-04 18:10 +0200
Message-ID<rkfeG-5Fz-15@gated-at.bofh.it>
In reply to#1370540

[Multipart message — attachments visible in raw view] — view raw

On Mon, Apr 04, 2016 at 01:47:50PM +0300, Felipe Balbi wrote:
> Mark Brown <broonie@kernel.org> writes:

> > It does in this new world order.  IIRC on an earlier round of review
> > there was some code that didn't use a bus but that got complaints that
> > it was trying to reimplement the bus functionality.

> fair enough, I'll wait for Greg to have some time to comment on
> this. Bottomline is that there is *no* real bus. Charger ICs will use

I've got a feeling Greg is zoning this out by now...

> SPI or I2C and that's a real bus, $subject is not.

Well, there's the physical connection between the system power supply
and the USB port if you're very keen on looking for hardware.

[toc] | [prev] | [next] | [standalone]


#1370817

FromGreg KH <gregkh@linuxfoundation.org>
Date2016-04-04 20:50 +0200
Message-ID<rkhJv-7va-1@gated-at.bofh.it>
In reply to#1370724
On Mon, Apr 04, 2016 at 09:04:48AM -0700, Mark Brown wrote:
> On Mon, Apr 04, 2016 at 01:47:50PM +0300, Felipe Balbi wrote:
> > Mark Brown <broonie@kernel.org> writes:
> 
> > > It does in this new world order.  IIRC on an earlier round of review
> > > there was some code that didn't use a bus but that got complaints that
> > > it was trying to reimplement the bus functionality.
> 
> > fair enough, I'll wait for Greg to have some time to comment on
> > this. Bottomline is that there is *no* real bus. Charger ICs will use
> 
> I've got a feeling Greg is zoning this out by now...

Very true :)

I have a lot of other patches to go through this week, it will be a
while before I get to these...

thanks,

greg k-h

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web