From 0b2ee9391079a5092eb99dce764e3836eeb92cd9 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Sat, 15 Jun 2013 19:58:35 +0200 Subject: pnp: Remove now plain wrappers for default PnP functions After removing the enter()/exit() functions for configuration mode, most wrappers for our standard PnP functions just call the underlying default implementation. Remove those with a little cocci: @ op_match @ identifier op; identifier pnp_op =~ "^pnp_((alt_|)enable|(set|enable)_resources)$"; type device_t; identifier dev; @@ static void op(device_t dev) { pnp_op(dev); } @@ identifier op_match.op; @@ -op(...) {...} /* Three rules to match the alignment, hmmp... */ @@ identifier op_match.op, op_match.pnp_op; identifier ops; @@ struct device_operations ops = { - .set_resources = op, + .set_resources = pnp_op, }; @@ identifier op_match.op, op_match.pnp_op; identifier ops; @@ struct device_operations ops = { - .enable_resources = op, + .enable_resources = pnp_op, }; @@ identifier op_match.op, op_match.pnp_op; identifier ops; @@ struct device_operations ops = { - .enable = op, + .enable = pnp_op, }; Change-Id: Idc0e52c7e3600a01f3b6a4e17763557b271b481e Signed-off-by: Nico Huber Reviewed-on: http://review.coreboot.org/3483 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Marc Jones Reviewed-by: Stefan Reinauer --- src/superio/winbond/w83627hf/superio.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) (limited to 'src/superio/winbond/w83627hf') diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c index 625f149ec7..4d68c35033 100644 --- a/src/superio/winbond/w83627hf/superio.c +++ b/src/superio/winbond/w83627hf/superio.c @@ -134,11 +134,6 @@ static void w83627hf_init(device_t dev) } } -static void w83627hf_pnp_set_resources(device_t dev) -{ - pnp_set_resources(dev); -} - static void w83627hf_pnp_enable_resources(device_t dev) { pnp_enable_resources(dev); @@ -153,11 +148,6 @@ static void w83627hf_pnp_enable_resources(device_t dev) pnp_exit_conf_mode(dev); } -static void w83627hf_pnp_enable(device_t dev) -{ - pnp_alt_enable(dev); -} - static const struct pnp_mode_ops pnp_conf_mode_ops = { .enter_conf_mode = pnp_enter_ext_func_mode, .exit_conf_mode = pnp_exit_ext_func_mode, @@ -165,9 +155,9 @@ static const struct pnp_mode_ops pnp_conf_mode_ops = { static struct device_operations ops = { .read_resources = pnp_read_resources, - .set_resources = w83627hf_pnp_set_resources, + .set_resources = pnp_set_resources, .enable_resources = w83627hf_pnp_enable_resources, - .enable = w83627hf_pnp_enable, + .enable = pnp_alt_enable, .init = w83627hf_init, .ops_pnp_mode = &pnp_conf_mode_ops, }; -- cgit v1.2.3