linux-kernel - RE: [patch v8 1/3] platform/x86: move Mellanox hardware platform hotplug driver to platform/mellanox

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM4PR05MB33305B31938957C720A3842DA2EA0@AM4PR05MB3330.eurprd05.prod.outlook.com>
Date:   Tue, 16 Jan 2018 08:06:18 +0000
From:   Vadim Pasternak <vadimp@...lanox.com>
To:     Darren Hart <dvhart@...radead.org>
CC:     "andy.shevchenko@...il.com" <andy.shevchenko@...il.com>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>,
        "jiri@...nulli.us" <jiri@...nulli.us>
Subject: RE: [patch v8 1/3] platform/x86: move Mellanox hardware platform
 hotplug driver to platform/mellanox



> -----Original Message-----
> From: Darren Hart [mailto:dvhart@...radead.org]
> Sent: Tuesday, January 16, 2018 5:43 AM
> To: Vadim Pasternak <vadimp@...lanox.com>
> Cc: andy.shevchenko@...il.com; gregkh@...uxfoundation.org; linux-
> kernel@...r.kernel.org; platform-driver-x86@...r.kernel.org; jiri@...nulli.us
> Subject: Re: [patch v8 1/3] platform/x86: move Mellanox hardware platform
> hotplug driver to platform/mellanox
> 
> On Thu, Jan 11, 2018 at 08:58:41AM +0000, Vadim Pasternak wrote:
> > It moves drivers/platform/x86/mlxcpld-hotplug.c to
> > drivers/platform/mellanox/mlxreg-hotplug.c and
> > include/linux/platform_data/mlxcpld-hotplug.h to
> > include/linux/platform_data/mlxreg.h for making hotplug driver usable
> > for the different machine architectures.
> >
> > Signed-off-by: Vadim Pasternak <vadimp@...lanox.com>
> > Acked-by: Andy Shevchenko <andy.shevchenko@...il.com>
> > ---
> > v7->v8:
> > v6->v7:
> >  Fixes added by Vadim:
> >  - remove dependency on X86_64 to allow also X86 architecture;
> > v5->v6:
> >  Fixes added by Vadim:
> >  - add SPD license record to Kconfig and Makefile;
> > v4->v5:
> >  Comments pointed out by Andy:
> >  - arrange MAINTAINERS in correct alphabetic order;
> > v3->v4:
> >  Comments pointed out by Darren:
> >  - Refactor the patches to provide the changes in patchset in incremental
> >    order;
> >  - Modify MAINTAINERS records;
> >  - Use git-mv for the replaced files;
> >   Commnets pointed out by Colin:
> >  - Make structures mlxplat_dev and mlxplat_hotplug static;
> > ---
> 
> ...
> 
> >  config MLX_PLATFORM
> >  	tristate "Mellanox Technologies platform support"
> > -	depends on X86_64
> 
> This seems like it would be more appropriately applied to 2/3 as this patch is
> supposed to be just about the renaming of the files, methods, and defines from
> CPLD to reg, and the creation of the mellanox platform.
> 
> Any reason this needs to be in this patch?
> 

Hi Darren,
Thank you very much for reviews.

I removed X86_64 dependency in this patch to allow the driver also for
32 bit architecture.

> --
> Darren Hart
> VMware Open Source Technology Center