Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752103AbcDRKju (ORCPT ); Mon, 18 Apr 2016 06:39:50 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:35308 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750717AbcDRKjs (ORCPT ); Mon, 18 Apr 2016 06:39:48 -0400 Date: Mon, 18 Apr 2016 12:39:45 +0200 From: Pavel Machek To: Felipe Balbi Cc: Baolin Wang , Greg KH , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Peter Chen , Alan Stern , r.baldyga@samsung.com, Yoshihiro Shimoda , Lee Jones , Mark Brown , Charles Keepax , patches@opensource.wolfsonmicro.com, Linux PM list , USB , device-mainlining@lists.linuxfoundation.org, LKML Subject: Re: [PATCH v7 1/4] gadget: Introduce the usb charger framework Message-ID: <20160418103945.GA29123@amd> References: <11ce6df3eb8a95cfed26f3321f15c98a934db642.1458128215.git.baolin.wang@linaro.org> <87h9foqnur.fsf@intel.com> <87poubgnbh.fsf@intel.com> <20160322113012.GC26924@xo-6d-61-c0.localdomain> <87fuujgwsm.fsf@intel.com> <20160418102316.GA27936@amd> <877ffvgqe9.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <877ffvgqe9.fsf@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3096 Lines: 67 On Mon 2016-04-18 13:30:54, Felipe Balbi wrote: > > Hi, > > Pavel Machek writes: > >> > Very often, you want to charge using 1.8A from an old desktop PC. > >> > >> if that old desktop's port is not a charging port, you shouldn't be > >> allowed to do that. Not ever. > > > > Yes, Felipe just decided that I should not be able to charge my N900 > > in useful way. > > you can do whatever you want with *your* kernel binary, but we're not > gonna ship something potentially dangerous. If that PC port is telling > you it can only allow 100mA, you should *not* be allowed to overcome > that limitation from the device side, sorry. Yes, and if your cpu tells you it can only run on 2GHz, you should not be able to run it on 2.1GHz. And you should not be able to use non-VESA VGA modes, because you could overheat coils in the monitor. You are shipping something potentially dangerous already, because you know, USB-A-to-micro-B cables with D+/D- connected to simulate charger are actually very common. The world did not end, yet, so it is clearly not as bad as you make it be. > >> >> a) you are connected to a dedicated charger > >> >> > >> >> In this case, you can get up to 2000mA depending on the charger. > >> >> > >> >> If $this charger can give you or not 2000mA is not detectable, > >> >> so what do charging ICs do ? They slowly increase the attached > >> >> load accross VBUS/GND and measure VBUS value. When IC notices > >> >> VBUS dropping bit, step back to previous load. > >> >> > >> >> This means you will always charger with maximum rating of DCP. > >> >> > >> >> Why would user change this ? More is unsafe, less is just > >> >> stupid. > >> > > >> > Actually, less is not stupid. Charging li-ion battery from li-ion battery might > >> > be stupid. Imagine I'm on train, with device like N900 (50% battery) and power bank > >> > (3Ah). I'm actively using the device. If I let it charge at full current, I'll waste > >> > energy. If I limit current to approximately the power consumption, it will run the > >> > powerbank empty, first, then empty the internal battery, maximizing total time I > >> > can use the device. > >> > >> why would you waste energy ? What the charger chip would do is charge > >> battery to maximum then just to maintenance charge from that point > >> on. Where is energy being wasted other than normal heat dissipation ? > > > > Physics 101, of course wasted energy goes to heat. Lets not waste > > energy by charging li-ion from li-ion when it is not required. > > your cellphone has no means to know that it's connected to a Li-Ion > battery. We don't have visibility on what we're connected to, just how > much it can source. But cellphone user knows what he connected his charger to, and that's why it is useful to be able to lower the current. Even when you said "less is just stupid" I demonstrated it is not, at least in case when your power source is a battery. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html