All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andrew F. Davis" <afd@ti.com>
To: Liam Breck <liam@networkimprov.net>, linux-pm@vger.kernel.org
Subject: Re: [PATCH v12.6 0/11] bq27xxx_battery partial series
Date: Mon, 10 Apr 2017 09:46:47 -0500	[thread overview]
Message-ID: <21b8db86-358f-f3ed-0e4c-e98fb8bed164@ti.com> (raw)
In-Reply-To: <20170410050712.930-1-liam@networkimprov.net>

On 04/10/2017 12:07 AM, Liam Breck wrote:
> Latest changes:
> 
> New patch to disable updates to flash/NVM. The purpose of this series has always
> been to update NVM or RAM via DT. It was Sebastian's idea to site the properties
> in a fixed/simple-battery node representing the battery to the fuel gauge (i.e.
> your characterization that I "misused" the battery node is mistaken). I relent
> only because I've switched from a NVM to RAM gauge. I expect others will succeed
> in reverting this. Also, I have no need for a knob which lets userspace trigger
> NVM update, so I leave that to others.
> 
> Add module param use_monitored_battery to the power_supply_battery_info patch;
> it is dropped in the above patch. Allowing users to ignore the DT properties
> is the correct solution IMO.
> 
> Add Copyright notice
> 
> unseal() returns error on key==0
> 
> Skip reset on non-cfgupdate chips if not updated
> 
> Emit warning if no dm_regs table
> 
> Map orig ID to chip ID via bq27xxx_chips[]
> 
> Drop "prototypes" in enum comments
> 
> Rename dupes detector bq27xxx_battery_dbg_regs_dupes()
> 
> ^.^ ^.^ - We can has test?
> 

Can you post the whole series against v4.11-rc6?

      parent reply	other threads:[~2017-04-10 14:46 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-10  5:07 [PATCH v12.6 0/11] bq27xxx_battery partial series Liam Breck
2017-04-10  5:07 ` [PATCH v12.6 05/11] power: bq27xxx_battery: Add chip data memory read/write support Liam Breck
2017-04-10 14:47   ` Andrew F. Davis
2017-04-10  5:07 ` [PATCH v12.6 06/11] power: bq27xxx_battery: Add power_supply_battery_info support Liam Breck
2017-04-10  5:07 ` [PATCH v12.6 07/11] power: bq27xxx_battery: Enable data memory update for certain chips Liam Breck
2017-04-10  5:07 ` [PATCH v12.6 08/11] devicetree: power: bq27xxx: Add monitored-battery documentation Liam Breck
2017-04-13 20:15   ` Rob Herring
2017-04-10  5:07 ` [PATCH v12.6 09/11] power: bq27xxx_battery: Flag identical register maps when in debug mode Liam Breck
2017-04-10  5:07 ` [PATCH v12.6 10/11] power: bq27xxx_battery: Remove duplicate register arrays Liam Breck
2017-04-10  5:07 ` [PATCH v12.6 11/11] power: bq27xxx_battery: Disable flash/NVM data memory update Liam Breck
2017-04-10 14:46 ` Andrew F. Davis [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=21b8db86-358f-f3ed-0e4c-e98fb8bed164@ti.com \
    --to=afd@ti.com \
    --cc=liam@networkimprov.net \
    --cc=linux-pm@vger.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.