From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [PATCH 03/15] extcon: cht-wc: Add Intel Cherry Trail Whiskey Cove PMIC extcon driver Date: Tue, 21 Mar 2017 12:54:16 +0900 Message-ID: <58D0A3E8.6090609@samsung.com> References: <20170317095527.10487-1-hdegoede@redhat.com> <20170317095527.10487-4-hdegoede@redhat.com> <58CF3183.4090101@samsung.com> <1490014841.19767.106.camel@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Return-path: In-reply-to: <1490014841.19767.106.camel@linux.intel.com> Sender: linux-pm-owner@vger.kernel.org To: Andy Shevchenko , Hans de Goede , "Rafael J . Wysocki" , Len Brown , Wolfram Sang , Lee Jones , Sebastian Reichel , MyungJoo Ham Cc: linux-acpi@vger.kernel.org, Takashi Iwai , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org List-Id: linux-acpi@vger.kernel.org On 2017년 03월 20일 22:00, Andy Shevchenko wrote: > On Mon, 2017-03-20 at 10:33 +0900, Chanwoo Choi wrote: >> On 2017년 03월 17일 18:55, Hans de Goede wrote: > >>> +static const struct platform_device_id cht_wc_extcon_table[] = { >>> + { .name = "cht_wcove_pwrsrc" }, >> >> You use the 'cht_wc' word instead of 'cht_wcove_pwrsrc'. >> So, To maintain the consistency, you better to use the 'cht-wc' as the >> name. >> - I prefer to use '-' instead of '_' in the name. >> .name ="cht-wc" > > I would keep as Hans did for the sake of consistency among all Whiskey > Cove device drivers (and predecessors like Crystal Cove). The 'wcove' short word is not used in this patch. If the author want to use the 'wcove', I recommend that you should use the 'wcove' instead of 'wc' in this patch. And, I think that 'pwrsrc' is not ambiguous. Hans might use the 'pwrsrc' as the 'Power Source'. But, this driver is not power source. Instead, this driver supports the detection of external connector. I think 'power source' means the power supply instead of detector. > > I understand your point from extcon subsystem view, but PMICs like > Whiskey Cove are multi-functional devices, and thus naming across them > (same prefix in use to be precise) is better idea. > >> >>> + {}, >>> +}; >>> +MODULE_DEVICE_TABLE(platform, cht_wc_extcon_table); > -- Best Regards, Chanwoo Choi Samsung Electronics