Re: [patch v8 2/3] platform/mellanox: mlxreg-hotplug: allow driver for ARM architecture

From: Darren Hart
Date: Mon Jan 15 2018 - 22:43:20 EST


On Thu, Jan 11, 2018 at 08:58:42AM +0000, Vadim Pasternak wrote:
> It allows driver to run on x86 and ARM architecture based systems.
> Also some unnecessary includes are removed.

Hi Vadim,

These changes are not all readily mapped to this description, nor to the
changelog below.

Only the kconfig change is obviously linked to building on ARM and not just x86.

Things like the removal of the dev_err messages and the relocation of the
create and destroy methods seem entirely unrelated to building on ARM.

I'd like to get this merged sooner rather than later as it has been lingering
for quite some time now - can you provide some more detail about how the changes
included in this patch are needed to make this driver build on ARM?

Thanks,

--
Darren Hart
VMware Open Source Technology Center