linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
To: "Andrew F. Davis" <afd@ti.com>
Cc: "Pali Rohár" <pali.rohar@gmail.com>,
	"Sebastian Reichel" <sre@kernel.org>,
	"Dmitry Eremin-Solenikov" <dbaryshkov@gmail.com>,
	"David Woodhouse" <dwmw2@infradead.org>,
	linux-pm@vger.kernel.org, "Pavel Machek" <pavel@ucw.cz>
Subject: Re: Should 9aafabc7fece be reverted?
Date: Thu, 19 May 2016 09:44:04 +0300	[thread overview]
Message-ID: <573D60B4.7010906@gmail.com> (raw)
In-Reply-To: <573C9F45.5090109@ti.com>

Hi,

On 18.05.2016 19:58, Andrew F. Davis wrote:
> Hello all,
>
> Why do we need this patch? I removed giving each battery a number in its
> name intentionally. It doesn't seem to be needed anymore, and only two
> old drivers out of the ~70 power supply drivers do this.
>

The patch is needed because commit <703df6c09795> breaks existing 
userspace on Nokia N900, which counts on name numbering. Also, commit 
<703df6c09795> description says nothing about name changing, so I 
assumed it is an unintentional side-effect of splitting the code into 
modules.

> Also I don't see this in my mailbox archive, was I CC'd on this patch?
>

You were not, get_maintainer.pl on v4.4-rc7 (the tree I was working on 
by the time I've sent <9aafabc7fece>, iirc) gives:

Sebastian Reichel <xxx@xxx.org> (maintainer:POWER SUPPLY CLASS/SUBSYSTEM 
and DRIVERS)
Dmitry Eremin-Solenikov <xxx@xxx.org> (maintainer:POWER SUPPLY 
CLASS/SUBSYSTEM and DRIVERS)
David Woodhouse <xxx@xxx.org> (maintainer:POWER SUPPLY CLASS/SUBSYSTEM 
and DRIVERS)
"Pali Rohár" <xxx@xxx.org> (maintainer:NOKIA N900 POWER SUPPLY DRIVERS)
linux-pm@vger.kernel.org (open list:POWER SUPPLY CLASS/SUBSYSTEM and 
DRIVERS)
linux-kernel@vger.kernel.org (open list)
(mail addresses masked by me)

You are not listed and as I try to avoid making more noise than needed 
when sending patches, I didn't include you in CC. Apologies if I should 
have CC-ed you.

Ivo

  reply	other threads:[~2016-05-19  6:44 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-18 16:58 Should 9aafabc7fece be reverted? Andrew F. Davis
2016-05-19  6:44 ` Ivaylo Dimitrov [this message]
2016-05-19  9:17   ` Pali Rohár
2016-05-19  9:34     ` Ivaylo Dimitrov
2016-05-19 15:38     ` Andrew F. Davis
2016-05-19 15:44       ` Pali Rohár
2016-05-19 15:48         ` Andrew F. Davis
2016-05-19 15:51           ` Pali Rohár
2016-05-19 15:57             ` Andrew F. Davis
2016-05-19 16:06               ` Pali Rohár
2016-05-19 16:11                 ` Andrew F. Davis
2016-05-19 16:18                   ` Pali Rohár
2016-05-19 16:46                     ` Andrew F. Davis
2016-05-20  9:29                       ` Pavel Machek
2016-05-23 15:55                         ` Andrew F. Davis
2016-05-23 16:10                           ` Pali Rohár
2016-05-23 16:29                             ` Andrew F. Davis
2016-05-20  9:34                       ` Pavel Machek
2016-05-20 10:35                         ` Pali Rohár
2016-05-23 16:05                           ` Andrew F. Davis
2016-05-20  9:22           ` Pavel Machek

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=573D60B4.7010906@gmail.com \
    --to=ivo.g.dimitrov.75@gmail.com \
    --cc=afd@ti.com \
    --cc=dbaryshkov@gmail.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=pali.rohar@gmail.com \
    --cc=pavel@ucw.cz \
    --cc=sre@kernel.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).