Messages in this thread | | | Date | Wed, 8 Jul 2015 12:55:31 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH v1 0/4] iosf_mbi: clean up |
| |
* Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> On Wed, 2015-07-08 at 11:36 +0200, Ingo Molnar wrote: > > * Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > > > > > There are few patches to clean up (with small fixes) iosf_mbi > > > driver along with > > > extension it to cover Intel Tangier. > > > > > > Andy Shevchenko (4): > > > iosf_mbi: check result for all calls of debugfs API > > > iosf_mbi: pci_dev_put() is NULL-proof > > > iosf_mbi: group global variables > > > iosf_mbi: append Intel Tangier ID > > > > > > arch/x86/include/asm/iosf_mbi.h | 8 ++++---- > > > arch/x86/kernel/iosf_mbi.c | 31 +++++++++++++++-------------- > > > -- > > > 2 files changed, 19 insertions(+), 20 deletions(-) > > > > Could you please also move it from arch/x86/kernel/ to > > arch/x86/platform/[intel-quark?] while at it? > > I will think about proper folder (apparently not quark, since it is > specific to man different Intel SoCs).
Yeah, you could make it arch/x86/platform/iosf/, and the file would be arch/x86/platform/iosf/mbi.c? Or something like that?
> > > > arch/x86/kernel/ is the wrong place for this driver. > > Would be okay to send an additional patch or better to regenerate the whole > series?
So it might be better to first do the file movement, then the fixes. It should be relatively easy to regenerate the patches like that, and if any failure is bisected back to a commit it's easy to revert - or if some new review observation comes in then it will be easy to adjust the series.
Thanks,
Ingo
| |