From: MyungJoo Ham <myungjoo.ham@samsung.com>
To: Anton Vorontsov <cbouatmailru@gmail.com>,
"Tc, Jenny" <jenny.tc@intel.com>
Cc: 최찬우 <cw00.choi@samsung.com>,
"anish singh" <anish198519851985@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"myunjoo.ham@gmail.com" <myunjoo.ham@gmail.com>,
"lockwood@android.com" <lockwood@android.com>,
"peterhuewe@gmx.de" <peterhuewe@gmx.de>,
"broonie@opensource.wolfsonmicro.com"
<broonie@opensource.wolfsonmicro.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"lars@metafoo.de" <lars@metafoo.de>,
"jic23@kernel.org" <jic23@kernel.org>,
"Pallala, Ramakrishna" <ramakrishna.pallala@intel.com>
Subject: Re: Re: [PATCH] extcon : callback function to read cable property
Date: Tue, 20 Nov 2012 11:05:44 +0000 (GMT) [thread overview]
Message-ID: <32212704.282601353409543335.JavaMail.weblogic@epml17> (raw)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=euc-kr, Size: 2023 bytes --]
Anton Vorontsov wrote:
> The idea of using union seemed good to me, what happened to it?
>
> I mean, MyungJoo Ham wrote:
>
> | We may have:
> | enum extcon_cable_type {
> | EXTCON_CT_REGULATOR,
> | EXTCON_CT_PSY,
> | EXTCON_CT_CHARGER_CB,
> | ...
> | };
> | and have the following included at struct extcon_cable:
> | union {
> | struct regulator *reg;
> | struct power_supply *psy;
> | struct charger_cable *charger_cb;
> | ...
> | } cable data;
> | enum extcon_cable_type cable_type;
>
> This sounds good to me...
>
> > * extcon is not designed to support any cable state other than CONNECT/DISCONNECT
>
> Dunno for this one. Can we get these additional states via "properties" as
> described above?
>
> Thanks,
> Anton.
>
>
The "RFC" patch for this feature is now shown at:
http://git.kernel.org/?p=linux/kernel/git/mzx/extcon.git;a=commit;h=5655aeef83addaec77a3b9387a3dd18b6c146dd5
(Note that "for-add-feature" branch is sort of "RFC" branch)
I'm now considering relaying notifications of data updates possible via
the notifier block of register-interest. Any inputs are welcomed.
Anyway, for the USB issue of "suspend/resume at host-side and current-limit at
device-side", we will need to
1. update regulator subsystem to have notification for current change
(it does for enable/disable/voltage-changes)
2. let the charger use current-regulator
3. let the one who detects (usb driver?) changes at host-side change the
current limit of that current-regulator
4. let the event from current-regulator relayed via extcon.
We may use power-supply class as well for this issue. (may need to update
if power-supply class does not have notifications and suspend/resume states)
Cheers,
MyungJoo
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
next reply other threads:[~2012-11-20 11:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-20 11:05 MyungJoo Ham [this message]
2012-11-22 13:00 ` Re: [PATCH] extcon : callback function to read cable property Tc, Jenny
2012-11-22 20:03 ` Anton Vorontsov
-- strict thread matches above, loose matches on Subject: below --
2012-11-20 3:45 MyungJoo Ham
2012-10-25 8:37 MyungJoo Ham
2012-10-25 9:24 ` Tc, Jenny
2012-11-03 2:57 ` Tc, Jenny
2012-10-17 7:08 MyungJoo Ham
2012-10-19 3:13 ` Tc, Jenny
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=32212704.282601353409543335.JavaMail.weblogic@epml17 \
--to=myungjoo.ham@samsung.com \
--cc=anish198519851985@gmail.com \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=cbouatmailru@gmail.com \
--cc=cw00.choi@samsung.com \
--cc=gregkh@linuxfoundation.org \
--cc=jenny.tc@intel.com \
--cc=jic23@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-kernel@vger.kernel.org \
--cc=lockwood@android.com \
--cc=myunjoo.ham@gmail.com \
--cc=peterhuewe@gmx.de \
--cc=ramakrishna.pallala@intel.com \
/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).