linux-kernel - Re: [PATCHv4 0/8] Resource-managed extcon device register function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-id: <5355018D.9050402@samsung.com>
Date:	Mon, 21 Apr 2014 20:31:25 +0900
From:	Chanwoo Choi <cw00.choi@...sung.com>
To:	Sangjung Woo <sangjung.woo@...sung.com>
Cc:	MyungJoo Ham <myungjoo.ham@...sung.com>,
	linux-kernel@...r.kernel.org,
	Krzysztof Kozlowski <k.kozlowski@...sung.com>,
	Seung-Woo Kim <sw0312.kim@...sung.com>
Subject: Re: [PATCHv4 0/8] Resource-managed extcon device register function

Hi Sangjung,

On 04/21/2014 07:10 PM, Sangjung Woo wrote:
> These patches add resource-managed extcon device register functions for
> developers' convenience and apply them to related device driver files.
> This work can make the code more tidy since extcon device is automatically
> unregistered on driver detach so tiresome managing codes could be removed.
> 
> 
> Changelog
> 
> v4:
> * fix the memory leak bug becuase of single pointer
> * clean up the unnecessary code and make proper indentation
> 
> v3:
> * send the right version instead of previous v1
> * add the credit for reviewers according to the review rules
> 
> v2:
> * modify and clean up all unnecessary code reported by Chanwoo
> * fix the bug reported by Seung-Woo
> * add the credit for reviewers
> 
> v1:
> * initial version
> 
> Sangjung Woo (8):
>   extcon: Add resource-managed extcon register function
>   extcon: adc-jack: Use devm_extcon_dev_register()
>   extcon: gpio: Use devm_extcon_dev_register()
>   extcon: max14577: Use devm_extcon_dev_register()
>   extcon: max77693: Use devm_extcon_dev_register()
>   extcon: max8997: Use devm_extcon_dev_register()
>   extcon: palmas: Use devm_extcon_dev_register()
>   extcon: arizona: Use devm_extcon_dev_register()
> 
>  drivers/extcon/extcon-adc-jack.c |   30 ++++++------------
>  drivers/extcon/extcon-arizona.c  |   12 +++----
>  drivers/extcon/extcon-class.c    |   64 ++++++++++++++++++++++++++++++++++++++
>  drivers/extcon/extcon-gpio.c     |   16 +++-------
>  drivers/extcon/extcon-max14577.c |    9 ++----
>  drivers/extcon/extcon-max77693.c |    7 ++---
>  drivers/extcon/extcon-max8997.c  |    4 +--
>  drivers/extcon/extcon-palmas.c   |   15 +++------
>  include/linux/extcon.h           |   17 ++++++++++
>  9 files changed, 108 insertions(+), 66 deletions(-)
> 

Applied this patchset.

But, applied below patch [1] instead of patch[2] on v4.

[1]
[PATCHv5] extcon: Add resource-managed extcon register function
- https://lkml.org/lkml/2014/4/21/140

[2] 
[PATCHv4 1/8] extcon: Add resource-managed extcon register function
- https://lkml.org/lkml/2014/4/21/128

Thanks,
Chanwoo Choi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/