From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v3 3/3] ACPI / PMIC: Add opregion driver for Intel Dollar Cove TI PMIC Date: Fri, 25 Aug 2017 17:38:11 +0200 Message-ID: References: <20170825134443.14843-1-tiwai@suse.de> <20170825134443.14843-4-tiwai@suse.de> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org To: Andy Shevchenko Cc: Lee Jones , Darren Hart , Andy Shevchenko , "Rafael J . Wysocki" , Mika Westerberg , Johannes Stezenbach , Hans de Goede , Platform Driver , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: platform-driver-x86.vger.kernel.org On Fri, 25 Aug 2017 17:24:19 +0200, Andy Shevchenko wrote: > > On Fri, Aug 25, 2017 at 4:44 PM, Takashi Iwai wrote: > > One really minor nit (fine if it will be ignored): > > > +config CHTDC_TI_PMIC_OPREGION > > CHTDC_TI_PMIC_OPREGION -> CHT_DC_TI_PMIC_OPREGION Ah, the world is inconsistent... > > obj-$(CONFIG_XPOWER_PMIC_OPREGION) += pmic/intel_pmic_xpower.o > > obj-$(CONFIG_BXT_WC_PMIC_OPREGION) += pmic/intel_pmic_bxtwc.o > > obj-$(CONFIG_CHT_WC_PMIC_OPREGION) += pmic/intel_pmic_chtwc.o > > +obj-$(CONFIG_CHTDC_TI_PMIC_OPREGION) += pmic/intel_pmic_chtdc_ti.o > > Ditto (as you can notice by above context). ... where the file name is combined. I really don't mind, too. Rafael, please let me know if you want a resubmission with this correction. thanks, Takashi