lkml.org 
[lkml]   [2021]   [Aug]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 09/20] Move Intel SCU IPC of pdx86 to intel directory to increase readability.
On Tue, Aug 10, 2021 at 07:43:42PM +0800, Kate Hsuan wrote:
> Hi Mika,
>
> Thanks for your suggestion.
> I'll add them to the patches and resend them again.
>
> Thank you.

Please, do not top post!

> On Tue, Aug 10, 2021 at 6:06 PM Mika Westerberg
> <mika.westerberg@linux.intel.com> wrote:
> >
> > 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.

> > > 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%)

Besides what Mika said this should follow the example of pmt, like dropping the
intel_scu prefix from the files, but leave it for the module (see Makefile
tricks for the details how to achieve this).

--
With Best Regards,
Andy Shevchenko


\
 
 \ /
  Last update: 2021-08-10 15:28    [W:0.049 / U:0.540 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site