From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750863AbbFLEne (ORCPT ); Fri, 12 Jun 2015 00:43:34 -0400 Received: from mailout4.samsung.com ([203.254.224.34]:50570 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750732AbbFLEnc (ORCPT ); Fri, 12 Jun 2015 00:43:32 -0400 X-AuditID: cbfee691-f79ca6d00000456a-b5-557a63723a01 Message-id: <557A6371.40300@samsung.com> Date: Fri, 12 Jun 2015 13:43:29 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130106 Thunderbird/17.0.2 MIME-version: 1.0 To: Greg KH Cc: balbi@ti.com, myungjoo.ham@samsung.com, k.kozlowski@samsung.com, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH] extcon: Redefine the unique id of supported external connectors without 'enum extcon' type References: <1434075006-2931-1-git-send-email-cw00.choi@samsung.com> <20150612042741.GA10691@kroah.com> In-reply-to: <20150612042741.GA10691@kroah.com> Content-type: text/plain; charset=us-ascii Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprOIsWRmVeSWpSXmKPExsWyRsSkULcouSrUoGEGp8XB+/UWzYvXs1m8 fmFocXnXHDaLRctamS1uN65gc2Dz2D93DbtH35ZVjB7Hb2xn8vi8SS6AJYrLJiU1J7MstUjf LoErY8mCT8wFf7krrn5rZGlgvMPZxcjBISFgIrFuKWMXIyeQKSZx4d56ti5GLg4hgaWMEofv zWCGSJhI3Oi9AZVYxCix4uodZgjnAaPEuUmTWEGqeAU0JHb//AnWwSKgKrHifRvYWDYBLYn9 L0C6OTlEBcIkVk6/wgJRLyjxY/I9MFsEqPfl0VtgNrNAnUTv/tlg9cIChRL3WyYzgdhCApkS 105eAKvhFNCXmLT+J1S9lsT6nceZIGx5ic1r3oIdJyGwiV1i9vm9bBAHCUh8m3yIBeJlWYlN B6A+k5Q4uOIGywRGsVlITpqFZOwsJGMXMDKvYhRNLUguKE5KLzLVK07MLS7NS9dLzs/dxAiM r9P/nk3cwXj/gPUhRgEORiUe3gStqlAh1sSy4srcQ4ymQFdMZJYSTc4HRnFeSbyhsZmRhamJ qbGRuaWZkjivjvTPYCGB9MSS1OzU1ILUovii0pzU4kOMTBycUg2M26e1TlBQ4DjPp7SNLTtK 63GyRW95qinvHOFzBWK3p/jo3Sm6/dDha9U3ybqmQ/M+e3FWh/qYvnmg9iQgZeoXYWeH5Nvl s0MO6pXFZHtN4PBYtvzwskDL+y7d817uV5nhulKSqXHFviaeM4kdDe2/klKdHnjlrWd+/WyK fn6o74RE9wTuM0uVWIozEg21mIuKEwGizbnxqgIAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrEIsWRmVeSWpSXmKPExsVy+t9jAd2i5KpQgx2TWCwO3q+3aF68ns3i 9QtDi8u75rBZLFrWymxxu3EFmwObx/65a9g9+rasYvQ4fmM7k8fnTXIBLFENjDYZqYkpqUUK qXnJ+SmZeem2St7B8c7xpmYGhrqGlhbmSgp5ibmptkouPgG6bpk5QLuVFMoSc0qBQgGJxcVK +naYJoSGuOlawDRG6PqGBMH1GBmggYQ1jBlLFnxiLvjLXXH1WyNLA+Mdzi5GTg4JAROJG703 2CBsMYkL99YD2VwcQgKLGCVWXL3DDOE8YJQ4N2kSK0gVr4CGxO6fP5lBbBYBVYkV79sYQWw2 AS2J/S8gJokKhEmsnH6FBaJeUOLH5HtgtghQ78ujt8BsZoE6id79s8HqhQUKJe63TGYCsYUE MiWunbwAVsMpoC8xaf1PqHotifU7jzNB2PISm9e8ZZ7AKDALyYpZSMpmISlbwMi8ilE0tSC5 oDgpPddQrzgxt7g0L10vOT93EyM4fp9J7WBc2WBxiFGAg1GJh/ehf1WoEGtiWXFl7iFGCQ5m JRFeE1ugEG9KYmVValF+fFFpTmrxIUZTYAhMZJYSTc4Hppa8knhDYxMzI0sjc0MLI2NzJXHe k/k+oUIC6YklqdmpqQWpRTB9TBycUg2MKUYrb27vUZZNP+pxSEZl+UOTMp6Eq9emW+mpne9c 26l81fSWX7wv80Ily6jXl4ULEll/nYra3+955/CBdv/rbDsjGvzuLxHbo3RYd36N+dtiSaPV i7ZZvf1wNvQLD/urZLV7pzaWtlwTmJgmuuWV6Y6baz7smvcjx+XtZR2lJoeatBf3a1h/KbEU ZyQaajEXFScCAGj8kkD1AgAA DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Greg, On 06/12/2015 01:27 PM, Greg KH wrote: > On Fri, Jun 12, 2015 at 11:10:06AM +0900, Chanwoo Choi wrote: >> This patch just redefine the unique id of supported external connectors without >> 'enum extcon' type. Because unique id would be used on devictree file(*.dts) to >> indicate the specific external connectors like key number of input framework. >> So, I have the plan to move this definitions to following header file which >> includes the unique id of supported external connectors. >> - include/dt-bindings/extcon/extcon.h >> >> Fixes: 2a9de9c0f08d ("extcon: Use the unique id for external connector instead of string") >> Signed-off-by: Chanwoo Choi >> --- >> Dear Greg, >> >> I sent pull-request[1] of extcon for 4.2. This patch[2] in the pull-request[2] >> includes the one issue about 'enum extcon'. The 'enum extcon' type is ambiguous >> and not clear. So, if you possible, I hope you to pick this patch on char-misc >> git repository. > > I don't understand, I took that pull request, right? > > What does that have to do with this patch? This patch is based on patch[1] and then patch[1] was merged on char-misc-next branch (char-misc.git) through pull request. [1] 2a9de9c0f08d ("extcon: Use the unique id for external connector instead of string") So, after reviewing this patch by you, if this patch is ok, I hope that this patch will be applied to char-misc-next branch. I'm sorry about un-clear explanation. Best Regards, Chanwoo Choi