[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YRJPebbK3uQBU9K8@lahna>
Date: Tue, 10 Aug 2021 13:05:45 +0300
From: Mika Westerberg <mika.westerberg@...ux.intel.com>
To: Kate Hsuan <hpa@...hat.com>
Cc: Hans de Goede <hdegoede@...hat.com>,
Mark Gross <mgross@...ux.intel.com>,
Alex Hung <alex.hung@...onical.com>,
Sujith Thomas <sujith.thomas@...el.com>,
Rajneesh Bhardwaj <irenic.rajneesh@...il.com>,
David E Box <david.e.box@...el.com>,
Zha Qipeng <qipeng.zha@...el.com>,
Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>,
AceLan Kao <acelan.kao@...onical.com>,
Jithu Joseph <jithu.joseph@...el.com>,
Maurice Ma <maurice.ma@...el.com>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Dan Carpenter <dan.carpenter@...cle.com>,
Daniel Scally <djrscally@...il.com>,
linux-kernel@...r.kernel.org, Dell.Client.Kernel@...l.com,
platform-driver-x86@...r.kernel.org
Subject: Re: [PATCH 09/20] Move Intel SCU IPC of pdx86 to intel directory to
increase readability.
Hi,
No objections, I think this is good idea but..
On Tue, Aug 10, 2021 at 05:58:21PM +0800, Kate Hsuan wrote:
.. I suggest to add proper commit message here. Many maintainers ignore
patches that don't have one.
> Signed-off-by: Kate Hsuan <hpa@...hat.com>
> ---
> drivers/platform/x86/Kconfig | 47 -----------------
> drivers/platform/x86/Makefile | 6 +--
> drivers/platform/x86/intel/Kconfig | 1 +
> drivers/platform/x86/intel/Makefile | 5 ++
> drivers/platform/x86/intel/scu/Kconfig | 52 +++++++++++++++++++
> drivers/platform/x86/intel/scu/Makefile | 11 ++++
> .../x86/{ => intel/scu}/intel_scu_ipc.c | 0
> .../x86/{ => intel/scu}/intel_scu_ipcutil.c | 0
> .../x86/{ => intel/scu}/intel_scu_pcidrv.c | 0
> .../x86/{ => intel/scu}/intel_scu_pltdrv.c | 0
> .../x86/{ => intel/scu}/intel_scu_wdt.c | 0
> 11 files changed, 70 insertions(+), 52 deletions(-)
> create mode 100644 drivers/platform/x86/intel/scu/Kconfig
> create mode 100644 drivers/platform/x86/intel/scu/Makefile
> rename drivers/platform/x86/{ => intel/scu}/intel_scu_ipc.c (100%)
> rename drivers/platform/x86/{ => intel/scu}/intel_scu_ipcutil.c (100%)
> rename drivers/platform/x86/{ => intel/scu}/intel_scu_pcidrv.c (100%)
> rename drivers/platform/x86/{ => intel/scu}/intel_scu_pltdrv.c (100%)
> rename drivers/platform/x86/{ => intel/scu}/intel_scu_wdt.c (100%)
Powered by blists - more mailing lists