Thread (17 messages) 17 messages, 5 authors, 2015-01-07

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

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]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help