Hi Guenter, On 2016년 08월 02일 03:18, Guenter Roeck wrote: > On Sun, Jul 31, 2016 at 10:50 PM, Chanwoo Choi <cw00.c...@samsung.com> wrote: >> This patch-set add the support the extcon type, extcon property >> and the synchronization functions. >> >> The each external connector has the common characters. So, the external >> connectors are able to gather in the specific type. And the each external >> connectors has the specific H/W desigin to support the multiple features >> throught h/w lines. There are the requirement to express the each h/w >> character of each external connector. Lastly, when the state and property >> are changed, the extcon notify the extcon client driver of the changed >> information. To support the notification on extcon provider drivers, >> this patches support the three sync functions. >> >> Changes from v1: >> - Expand the size (+1) of the property array for each extcon type. >> - Use the memset() to initialize the property when connector is detached. >> - Wrap the data of struct extcon_dev in the lock mechanism. >> - Don't send the notification if connector state is not changed in >> extcon_set_state_sync() >> - Fix the minor issue. >> - Add Tested-by tag of Chris Zhong for these patches. >> - Add Signed-off tag of Myungjoo Ham for patch1. >> - Add Reviewed-by tag of Guenter Roeck for patch6. >> >> Depends on: >> This patch depend on the extcon git repository[2]. (branch : extcon-next) >> > > For the series: > > Tested-by: Guenter Roeck <gro...@chromium.org>
Thanks for your test and review. I'll modify the minor issue of your comment. I'll send next version(v3) and then merged patches on extcon-next branch. Regards, Chanwoo Choi > >> [1] https://en.wikipedia.org/wiki/DisplayPort >> [2] https://git.kernel.org/cgit/linux/kernel/git/chanwoo/extcon.git/ >> >> [Detailed description of these patches] >> 1. Add the extcon type to group the each external connector. >> There are five categories unitl now as following: >> - EXTCON_TYPE_USB : USB connector >> - EXTCON_TYPE_CHG : Charger connector >> - EXTCON_TYPE_JACK : Jack connector >> - EXTCON_TYPE_DISP : Display connector >> - EXTCON_TYPE_MISC : Miscellaneous connector >> >> 2. Add the extcon property to support the multiple characteristic >> for the specific H/W design. >> - EXTCON_PROP_USB_[property name] >> - EXTCON_PROP_CHG_[property name] >> - EXTCON_PROP_JACK_[property name] >> - EXTCON_PROP_DISP_[property name] >> e.g., EXTCON_PROP_USB_ID and EXTCON_PROP_USB_VBUS >> The list of the new extcon APIs for the property as following: >> - int extcon_get_property(struct extcon_dev *edev, >> unsigned int id, unsigned int prop, >> union extcon_property_value *prop_val) >> - int extcon_set_property(struct extcon_dev *edev, >> unsigned int id, unsigned int prop, >> union extcon_property_value prop_val) >> - int extcon_get_property_capability(struct extcon_dev *edev, >> unsigned int id, unsigned int prop); >> - int extcon_set_property_capability(struct extcon_dev *edev, >> unsigned int id, unsigned int prop); >> >> 3. Add the sync functions to synchronize the data of each external connector >> between an extcon provider driver and the extcon client drivers. >> The list of the new extcon sync APIs as following: >> - extcon_sync() : Send the notification for each external connector to >> synchronize the information between and extcon provider >> driver >> and the extcon client drivers. >> - extcon_set_state_sync() : Set the state of external connector with noti. >> - extcon_set_property_sync() : Set the property of external connector with >> noti. >> >> 4. Add the new external connector definition. The EXTCON_DISP_DP >> means the Display external connector[1]. >> The list of new external connector as following: >> - EXTCON_DISP_DP >> The list of new property of USB connector as following: >> - EXTCON_PROP_USB_TYPEC_POLARITY >> >> 5. Rename the renames the existing extcon_get/set_cable_state_() >> to maintain the function naming pattern like as extcon APIs for property. >> - extcon_set_cable_state_() -> extcon_set_state() >> - extcon_get_cable_state_() -> extcon_get_state() >> >> For example, >> case 1, change the state of external connector and synchronized the data. >> extcon_set_state_sync(edev, EXTCON_USB, 1); >> case 2, change both the state and property of external connector >> and synchronized the data. >> extcon_set_state(edev, EXTCON_USB, 1); >> extcon_set_property(edev, EXTCON_USB, EXTCON_PROP_USB_ID, 1); >> extcon_set_property(edev, EXTCON_USB, EXTCON_PROP_USB_VBUS, 0); >> extcon_sync(edev, EXTCON_USB); >> case 3, change the property of external connector and synchronized the data. >> extcon_set_property(edev, EXTCON_USB, EXTCON_PROP_USB_VBUS, 0); >> extcon_set_property(edev, EXTCON_USB, EXTCON_PROP_USB_ID, 1); >> extcon_sync(edev, EXTCON_USB); >> case 4, change the property of external connector and synchronized the data. >> extcon_set_property_sync(edev, EXTCON_USB, EXTCON_PROP_USB_VBUS, 0); >> >> >> Chanwoo Choi (5): >> extcon: Add the extcon_type to gather each connector into five category >> extcon: Add the support for extcon property according to extcon type >> extcon: Add the support for the capability of each property >> extcon: Rename the extcon_set/get_state() to maintain the function naming >> pattern >> extcon: Add the synchronization extcon APIs to support the notification >> >> Chris Zhong (1): >> extcon: Add EXTCON_DISP_DP and the property for USB Type-C >> >> drivers/extcon/extcon.c | 730 >> ++++++++++++++++++++++++++++++++++++++++-------- >> include/linux/extcon.h | 176 +++++++++++- >> 2 files changed, 786 insertions(+), 120 deletions(-) >> >> -- >> 1.9.1 >> > > >