Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1379845

Re: [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API of extcon

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

On Fri, Apr 15, 2016 at 09:43:34AM +0900, Chanwoo Choi wrote:
> This patch removes the deprecated notifier API of extcon framework and then use
> the new extcon API[2] with the unique id[1] to indicate the each external
> connector. Alter deprecated API as following:
> - extcon_register_interest() -> extcon_register_notifier()
> - extcon_unregister_interest() -> extcon_unregister_notifier()
> 
> And, extcon alters the name of USB charger connector in patch[3] as following:
> - EXTCON_CHG_USB_SDP /* Standard Downstream Port */
> - EXTCON_CHG_USB_DCP /* Dedicated Charging Port */
> - EXTCON_CHG_USB_CDP /* Charging Downstream Port */
> - EXTCON_CHG_USB_ACA /* Accessory Charger Adapter */
> 
> So, the name of external charger connector are changed as following:
> - "USB" -> "SDP"
> - "TA" -> "DCP"
> 
> [1] Commit 2a9de9c0f08d61
> - ("extcon: Use the unique id for external connector instead of string)
> [2] Commit 046050f6e623e4
> - ("extcon: Update the prototype of extcon_register_notifier() with enum extcon
> [3] Commit 11eecf910bd81d
> - ("extcon: Modify the id and name of external connector")
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>  .../bindings/power_supply/charger-manager.txt      |  4 +--
>  drivers/power/charger-manager.c                    | 31 ++++++++++++++--------
>  include/linux/power/charger-manager.h              |  4 +--
>  3 files changed, 24 insertions(+), 15 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/power_supply/charger-manager.txt b/Documentation/devicetree/bindings/power_supply/charger-manager.txt
> index ec4fe9de3137..73193e380dc2 100644
> --- a/Documentation/devicetree/bindings/power_supply/charger-manager.txt
> +++ b/Documentation/devicetree/bindings/power_supply/charger-manager.txt
> @@ -65,13 +65,13 @@ Example :
>  		regulator@0 {
>  			cm-regulator-name = "chg-reg";
>  			cable@0 {
> -				cm-cable-name = "USB";
> +				cm-cable-id = 5; /* EXTCON_CHG_USB_SDP */
>  				cm-cable-extcon = "extcon-dev.0";
>  				cm-cable-min = <475000>;
>  				cm-cable-max = <500000>;
>  			};
>  			cable@1 {
> -				cm-cable-name = "TA";
> +				cm-cable-id = 6; /* EXTCON_CHG_USB_DCP */
>  				cm-cable-extcon = "extcon-dev.0";
>  				cm-cable-min = <650000>;
>  				cm-cable-max = <675000>;

This breaks DT ABI. Looks like charger-manager is not used in
mainline, but I guess ther should be an explicit Acked-By from
a DT binding maintainer. Also I think the defines should be in
some header includable from DTS, so that something like this
can be done:

cm-cable-id = <CHARGER_USB_DCP>;

Apart from that:

Acked-By: Sebastian Reichel <sre@kernel.org>

-- Sebastian

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RESEND PATCH 0/3] power: Remove the deprecated extcon functions Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-15 02:50 +0200
  [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API of  extcon Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-15 02:50 +0200
    Re: [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API  of extcon Sebastian Reichel <sre@kernel.org> - 2016-04-15 16:20 +0200
      Re: [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API  of extcon Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-18 07:50 +0200
      Re: [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API  of extcon Rob Herring <robh@kernel.org> - 2016-04-18 19:30 +0200
        Re: [RESEND PATCH 1/3] power: charger-manager: Replace deprecatd API  of extcon Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-19 01:40 +0200
  [RESEND PATCH 2/3] power: axp288_charger: Replace deprecatd API of  extcon Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-15 02:50 +0200
    Re: [RESEND PATCH 2/3] power: axp288_charger: Replace deprecatd API  of extcon Sebastian Reichel <sre@kernel.org> - 2016-04-15 16:30 +0200
      Re: [RESEND PATCH 2/3] power: axp288_charger: Replace deprecatd API of  extcon Chanwoo Choi <cw00.choi@samsung.com> - 2016-04-18 07:40 +0200

csiph-web