From: Mark Brown <broonie@kernel.org>
To: Axel Haslam <ahaslam@baylibre.com>
Cc: Greg KH <gregkh@linuxfoundation.org>,
Johan Hovold <johan@kernel.org>,
robh+dt@kernel.org, Sekhar Nori <nsekhar@ti.com>,
Alan Stern <stern@rowland.harvard.edu>,
Kevin Hilman <khilman@baylibre.com>,
Sergei Shtylyov <sshtylyov@ru.mvista.com>,
David Lechner <david@lechnology.com>,
manjunath.goudar@linaro.org,
Alexandre Bailon <abailon@baylibre.com>,
linux-usb@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH/RFT v2 09/17] regulator: fixed: Add over current event
Date: Tue, 25 Oct 2016 15:33:04 +0100 [thread overview]
Message-ID: <20161025143304.GW17252@sirena.org.uk> (raw)
In-Reply-To: <CAKXjFTOHcqRFtbXMXSTyUZ3P00eRMz1N3d62Y3NWVZJDGeScZA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 791 bytes --]
On Tue, Oct 25, 2016 at 02:55:48PM +0200, Axel Haslam wrote:
> To be able to use regulator to handle the overcurrent pin, i need to be able
> to somehow retrieve the over current pin state from the regulator driver.
What makes you say that, none of the existing users need this?
> As i was trying your suggestion, i remembered why i thought i should use
> mode instead of status: Status seems to be for internal regulator driver use,
> there is no regulator_get_status, function and REGULATOR_STATUS_* are defined
> in driver.h and not in consumer.h as REGULATOR_MODE_*
> Would you be ok if i allow consumers to get the status via a new
> "regulator_get_status" call?
What would they do with this information that they can't do with the
existing error notification?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2016-10-25 14:33 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-24 16:46 [PATCH/RFT v2 00/17] Add DT support for ohci-da8xx ahaslam
2016-10-24 16:46 ` [PATCH/RFT v2 01/17] ARM: davinci: da8xx: add usb phy clocks ahaslam
[not found] ` <20161024164634.4330-1-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-24 16:46 ` [PATCH/RFT v2 02/17] ARM: davinci: da8xx: Add CFGCHIP syscon platform declaration ahaslam-rdvid1DuHRBWk0Htik3J/w
2016-10-25 8:10 ` Sekhar Nori
[not found] ` <7f9efe9d-0912-e10a-3e45-24c5d2b455ab-l0cyMroinI0@public.gmane.org>
2016-10-25 9:37 ` Axel Haslam
[not found] ` <CAKXjFTOVuWRL+yuk6sARa2PAd6j-V4Ej5AmPGz7F7JXKidvJrg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-10-25 10:17 ` Sekhar Nori
[not found] ` <758f206c-d0bd-6f70-da1e-42c88d6dd1f0-l0cyMroinI0@public.gmane.org>
2016-10-25 15:53 ` David Lechner
2016-10-26 8:56 ` Sekhar Nori
2016-10-24 16:46 ` [PATCH/RFT v2 03/17] ARM: davinci: da8xx: Add USB PHY " ahaslam-rdvid1DuHRBWk0Htik3J/w
2016-10-25 9:18 ` Sekhar Nori
[not found] ` <4129f1be-42f6-f182-c8a2-84befe01d298-l0cyMroinI0@public.gmane.org>
2016-10-25 9:37 ` Axel Haslam
2016-10-24 16:46 ` [PATCH/RFT v2 06/17] ARM: davinci: da8xx: Fix some redefined symbol warnings ahaslam-rdvid1DuHRBWk0Htik3J/w
2016-10-25 10:03 ` Sekhar Nori
[not found] ` <9fa01842-cf8e-87e9-75ce-5d4b2841fdfe-l0cyMroinI0@public.gmane.org>
2016-10-25 12:14 ` Alexandre Bailon
2016-10-24 16:46 ` [PATCH/RFT v2 09/17] regulator: fixed: Add over current event ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-10-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-24 17:43 ` Mark Brown
2016-10-24 17:53 ` Axel Haslam
2016-10-24 17:53 ` Mark Brown
[not found] ` <20161024175320.GO17252-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2016-10-24 18:11 ` Axel Haslam
2016-10-24 18:19 ` Mark Brown
2016-10-25 12:55 ` Axel Haslam
2016-10-25 14:33 ` Mark Brown [this message]
2016-10-25 14:57 ` Axel Haslam
2016-10-25 15:07 ` Axel Haslam
2016-10-30 20:42 ` Rob Herring
2016-10-24 16:46 ` [PATCH/RFT v2 10/17] USB: da8xx: use flag instead of bitmask for over current change ahaslam-rdvid1DuHRBWk0Htik3J/w
2016-10-24 16:46 ` [PATCH/RFT v2 11/17] USB: OHCI: make ohci-da8xx a separate driver ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-12-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 0:38 ` David Lechner
[not found] ` <403743d8-dff2-3389-105b-1082b674b0b8-nq/r/kbU++upp/zk7JDF2g@public.gmane.org>
2016-10-25 7:39 ` Axel Haslam
[not found] ` <CAKXjFTPvEXPRyq0HZiF7mcHWFaFoH7MjFqEKnTyuSCUOB=U7Aw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-10-25 16:12 ` David Lechner
[not found] ` <21019d11-3b01-3e0e-60bc-c4873cf20237-nq/r/kbU++upp/zk7JDF2g@public.gmane.org>
2016-10-25 16:21 ` Axel Haslam
[not found] ` <CAKXjFTM8RwsAut8cD0n0foxueSjR4VB4k4WJizXo9SyaW3viOA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-10-25 16:24 ` David Lechner
2016-10-24 16:46 ` [PATCH/RFT v2 12/17] USB: ochi-da8xx: Use a regulator for vbus/overcurrent ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-13-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 1:39 ` David Lechner
[not found] ` <672c1f1a-58b7-6947-7fb2-acd38fab8597-nq/r/kbU++upp/zk7JDF2g@public.gmane.org>
2016-10-25 8:24 ` Axel Haslam
[not found] ` <CAKXjFTOxQ58rThpPw9t73k-BBZ8YKGvMhWfAG1r+3kC=GJVxWg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-10-25 16:53 ` David Lechner
2016-10-25 17:32 ` Axel Haslam
2016-10-25 10:43 ` Sekhar Nori
[not found] ` <d7e2b356-4874-fdc1-c505-70e57e3908de-l0cyMroinI0@public.gmane.org>
2016-10-25 10:52 ` Axel Haslam
2016-10-24 16:46 ` [PATCH/RFT v2 13/17] USB: da8xx: use ohci priv data instead of globals ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-14-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 1:12 ` David Lechner
2016-10-25 9:56 ` Axel Haslam
2016-10-24 16:46 ` [PATCH/RFT v2 14/17] ARM: davinci: register the usb20_phy clock on the DT file ahaslam-rdvid1DuHRBWk0Htik3J/w
2016-10-24 16:46 ` [PATCH/RFT v2 15/17] usb: host: ohci-da8xx: Add devicetree bindings documentation ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-16-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 1:02 ` David Lechner
[not found] ` <523928a1-a321-abb6-504f-a1053c88ce53-nq/r/kbU++upp/zk7JDF2g@public.gmane.org>
2016-10-25 9:56 ` Axel Haslam
2016-10-24 16:46 ` [PATCH/RFT v2 16/17] USB: ohci-da8xx: Allow probing from DT ahaslam-rdvid1DuHRBWk0Htik3J/w
[not found] ` <20161024164634.4330-17-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 0:53 ` David Lechner
2016-10-25 8:10 ` Axel Haslam
2016-10-25 10:55 ` [PATCH/RFT v2 00/17] Add DT support for ohci-da8xx Sekhar Nori
2016-10-24 16:46 ` [PATCH/RFT v2 04/17] ARM: DTS: da850: Add cfgchip syscon node ahaslam
2016-10-24 16:46 ` [PATCH/RFT v2 05/17] ARM: DTS: da850: Add usb phy node ahaslam
2016-10-24 16:46 ` [PATCH/RFT v2 07/17] ARM: davinci: da8xx: Enable the usb20 "per" clk on phy_clk_enable ahaslam
2016-10-25 2:53 ` David Lechner
2016-10-25 10:01 ` Axel Haslam
2016-10-25 10:12 ` Sekhar Nori
[not found] ` <833b6b60-4504-dac7-ccbb-bc08c985ad01-l0cyMroinI0@public.gmane.org>
2016-10-25 16:05 ` David Lechner
2016-10-26 9:30 ` Sekhar Nori
2016-10-24 16:46 ` [PATCH/RFT v2 08/17] ARM: davinci: hawk: add full constraints for ohci plat boot ahaslam
[not found] ` <20161024164634.4330-9-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 10:28 ` Sekhar Nori
[not found] ` <522eeb9a-6fb1-88eb-5c58-7ee209a50fc3-l0cyMroinI0@public.gmane.org>
2016-10-25 10:31 ` Axel Haslam
2016-10-24 16:46 ` [PATCH/RFT v2 17/17] ARM: dts: da850: add usb device node ahaslam
[not found] ` <20161024164634.4330-18-ahaslam-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2016-10-25 0:48 ` David Lechner
[not found] ` <59ca6a1f-6db9-eed2-8e8a-77657c7febac-nq/r/kbU++upp/zk7JDF2g@public.gmane.org>
2016-10-25 8:03 ` Axel Haslam
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=20161025143304.GW17252@sirena.org.uk \
--to=broonie@kernel.org \
--cc=abailon@baylibre.com \
--cc=ahaslam@baylibre.com \
--cc=david@lechnology.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=johan@kernel.org \
--cc=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=manjunath.goudar@linaro.org \
--cc=nsekhar@ti.com \
--cc=robh+dt@kernel.org \
--cc=sshtylyov@ru.mvista.com \
--cc=stern@rowland.harvard.edu \
/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).