Re: [PATCH net-next v1 2/3] ARM: imx: add FEC sleep mode callback function
From: Fabio Estevam <festevam@gmail.com>
Date: 2015-01-07 03:44:17
From: Fabio Estevam <festevam@gmail.com>
Date: 2015-01-07 03:44:17
On Wed, Jan 7, 2015 at 12:33 AM, Shawn Guo [off-list ref] wrote:
The GPR difference between SoCs can be encoded in device tree as well. It's pointless to repeat the same code pattern for every single platform, that need to set up GPR bits for enabling magic packet wake up, while the only difference is the register and bit offset. The platform code will become quite messy and unmaintainable if every device driver dump their GPR register setup code into platform. Sorry, but it's NACK from me.
This patch has already reached linux-next via Dave's tree as commit 456062b3ec6f5. Should a revert patch be sent? Also, this commit causes a build warning in linux-next: arch/arm/mach-imx/mach-imx6sx.c:59:20: warning: 'imx6sx_enet_plt_init' defined but not used [-Wunused-function]