On Tue, 08 Aug 2017, Vadim Pasternak wrote: > > > > -----Original Message----- > > From: Pavel Machek [mailto:pavel@xxxxxx] > > Sent: Tuesday, August 08, 2017 11:17 AM > > To: Vadim Pasternak <vadimp@xxxxxxxxxxxx> > > Cc: Lee Jones <lee.jones@xxxxxxxxxx>; robh+dt@xxxxxxxxxx; > > devicetree@xxxxxxxxxxxxxxx; j.anaszewski@xxxxxxxxxxx; > > rpurdie@xxxxxxxxx; linux-leds@xxxxxxxxxxxxxxx; jiri@xxxxxxxxxxx; > > jacek.anaszewski@xxxxxxxxx > > Subject: Re: [patch v2 1/2] mfd: Add Mellanox regmap core driver > > > > Hi! > > > > > > > And such code can't be placed under drivers/platform/x86/. > > > > > > > > So aim for drivers/platform/mlx ...? > > > > > > > > > > So you suggest to move drivers/platform/x86/mlxcpld-hotplug.c to > > > drivers/platform/mellanox/mlxcpld-hotplug.c and extend thr driver with > > > DTS support (mlxreg_core_dev_enable/mlxreg_core_dev_disable)? > > > > If this leads to less code duplication -- yes, by all means do that. > > OK. > So, I will remove from mlxreg-core IC related part and send the patches for your > review. > In parallel I'll start to work with platform maintainers for moving mlxcpld-hotplug > and further extension for DTS support. After this is ready, I'll extend mlxreg-core > with IC activation. > > Is sounds OK with you? This sounds very reasonable to me, thanks. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
