Return-path: Received: from lunge.queued.net ([173.255.254.236]:60560 "EHLO lunge.queued.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751088Ab2GSEgR (ORCPT ); Thu, 19 Jul 2012 00:36:17 -0400 Date: Wed, 18 Jul 2012 21:36:05 -0700 From: Andres Salomon To: Andrew Morton Cc: Paul Fox , Daniel Drake , "Richard A. Smith" , linux-kernel@vger.kernel.org, libertas-dev@lists.infradead.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, platform-driver-x86@vger.kernel.org, devel@driverdev.osuosl.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Dan Williams , "John W. Linville" , Matthew Garrett , Anton Vorontsov , David Woodhouse , Chris Ball , Jon Nettleton , Greg Kroah-Hartman Subject: Re: [PATCH 6/9] x86: OLPC: switch over to using new EC driver on x86 Message-ID: <20120718213605.1605ba91@dev.queued.net> (sfid-20120719_063637_284157_4B5220F9) In-Reply-To: <20120718212827.2080b28b@dev.queued.net> References: <20120718182144.2d7b0b50@dev.queued.net> <20120718212827.2080b28b@dev.queued.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: Damnit, my mailer mangled the resend. Okay, 3rd time's a charm.. (sorry for the noise!) On Wed, 18 Jul 2012 21:28:27 -0700 Andres Salomon wrote: > This uses the new EC driver framework in drivers/platform/olpc. The > XO-1 and XO-1.5-specific code is still in arch/x86, but the generic > stuff (including a new workqueue; no more running EC commands with > IRQs disabled!) can be shared with other architectures. > > Signed-off-by: Andres Salomon > --- > arch/x86/include/asm/olpc.h | 5 --- > arch/x86/platform/olpc/olpc.c | 53 > ++++++++++++++++++++------------------- > drivers/platform/olpc/olpc-ec.c | 5 --- 3 files changed, 27 > insertions(+), 36 deletions(-) > > diff --git a/arch/x86/include/asm/olpc.h b/arch/x86/include/asm/olpc.h > index 5b28f3e..72f9adf6 100644 > --- a/arch/x86/include/asm/olpc.h > +++ b/arch/x86/include/asm/olpc.h > @@ -100,11 +100,6 @@ extern void olpc_xo1_pm_wakeup_clear(u16 value); > > extern int pci_olpc_init(void); > > -/* EC related functions */ > - > -extern int olpc_ec_cmd_x86(unsigned char cmd, unsigned char *inbuf, > - size_t inlen, unsigned char *outbuf, size_t outlen); > - > /* SCI source values */ > > #define EC_SCI_SRC_EMPTY 0x00 > diff --git a/arch/x86/platform/olpc/olpc.c > b/arch/x86/platform/olpc/olpc.c index a3fa180..4590096 100644 > --- a/arch/x86/platform/olpc/olpc.c > +++ b/arch/x86/platform/olpc/olpc.c > @@ -14,7 +14,6 @@ > #include > #include > #include > -#include > #include > #include > #include > @@ -32,8 +31,6 @@ > struct olpc_platform_t olpc_platform_info; > EXPORT_SYMBOL_GPL(olpc_platform_info); > > -static DEFINE_SPINLOCK(ec_lock); > - > /* debugfs interface to EC commands */ > #define EC_MAX_CMD_ARGS (5 + 1) /* cmd byte + 5 args */ > #define EC_MAX_CMD_REPLY (8) > @@ -126,16 +123,13 @@ static int __wait_on_obf(unsigned int line, > unsigned int port, int desired) > * . Unfortunately, > while > * OpenFirmware's source is available, the EC's is not. > */ > -int olpc_ec_cmd_x86(unsigned char cmd, unsigned char *inbuf, size_t > inlen, > - unsigned char *outbuf, size_t outlen) > +static int olpc_xo1_ec_cmd(u8 cmd, u8 *inbuf, size_t inlen, u8 > *outbuf, > + size_t outlen, void *arg) > { > - unsigned long flags; > int ret = -EIO; > int i; > int restarts = 0; > > - spin_lock_irqsave(&ec_lock, flags); > - > /* Clear OBF */ > for (i = 0; i < 10 && (obf_status(0x6c) == 1); i++) > inb(0x68); > @@ -199,10 +193,8 @@ restart: > > ret = 0; > err: > - spin_unlock_irqrestore(&ec_lock, flags); > return ret; > } > -EXPORT_SYMBOL_GPL(olpc_ec_cmd_x86); > > void olpc_ec_wakeup_set(u16 value) > { > @@ -366,7 +358,7 @@ static void setup_debugfs(void) > &ec_debugfs_genops); > } > > -static int olpc_ec_suspend(void) > +static int olpc_ec_suspend(struct platform_device *pdev) > { > return olpc_ec_mask_write(ec_wakeup_mask); > } > @@ -425,8 +417,28 @@ static int __init add_xo1_platform_devices(void) > return 0; > } > > -static struct syscore_ops olpc_syscore_ops = { > +static int olpc_xo1_ec_probe(struct platform_device *pdev) > +{ > + /* get the EC revision */ > + olpc_ec_cmd(EC_FIRMWARE_REV, NULL, 0, > + (unsigned char *) &olpc_platform_info.ecver, > 1); + > + /* EC version 0x5f adds support for wide SCI mask */ > + if (olpc_platform_info.ecver >= 0x5f) > + olpc_platform_info.flags |= OLPC_F_EC_WIDE_SCI; > + > + pr_info("OLPC board revision %s%X (EC=%x)\n", > + ((olpc_platform_info.boardrev & 0xf) < 8) ? > "pre" : "", > + olpc_platform_info.boardrev >> 4, > + olpc_platform_info.ecver); > + > + return 0; > +} > + > +static struct olpc_ec_driver ec_xo1_driver = { > .suspend = olpc_ec_suspend, > + .probe = olpc_xo1_ec_probe, > + .ec_cmd = olpc_xo1_ec_cmd, > }; > > static int __init olpc_init(void) > @@ -436,16 +448,14 @@ static int __init olpc_init(void) > if (!olpc_ofw_present() || !platform_detect()) > return 0; > > - spin_lock_init(&ec_lock); > + /* register the XO-1 and 1.5-specific EC handler */ > + olpc_ec_driver_register(&ec_xo1_driver, NULL); > + platform_device_register_simple("olpc-ec", -1, NULL, 0); > > /* assume B1 and above models always have a DCON */ > if (olpc_board_at_least(olpc_board(0xb1))) > olpc_platform_info.flags |= OLPC_F_DCON; > > - /* get the EC revision */ > - olpc_ec_cmd(EC_FIRMWARE_REV, NULL, 0, > - (unsigned char *) &olpc_platform_info.ecver, > 1); - > #ifdef CONFIG_PCI_OLPC > /* If the VSA exists let it emulate PCI, if not emulate in > kernel. > * XO-1 only. */ > @@ -453,14 +463,6 @@ static int __init olpc_init(void) > !cs5535_has_vsa2()) > x86_init.pci.arch_init = pci_olpc_init; > #endif > - /* EC version 0x5f adds support for wide SCI mask */ > - if (olpc_platform_info.ecver >= 0x5f) > - olpc_platform_info.flags |= OLPC_F_EC_WIDE_SCI; > - > - printk(KERN_INFO "OLPC board revision %s%X (EC=%x)\n", > - ((olpc_platform_info.boardrev & 0xf) < 8) ? > "pre" : "", > - olpc_platform_info.boardrev >> 4, > - olpc_platform_info.ecver); > > if (olpc_platform_info.boardrev < olpc_board_pre(0xd0)) { /* > XO-1 */ r = add_xo1_platform_devices(); > @@ -468,7 +470,6 @@ static int __init olpc_init(void) > return r; > } > > - register_syscore_ops(&olpc_syscore_ops); > setup_debugfs(); > > return 0; > diff --git a/drivers/platform/olpc/olpc-ec.c > b/drivers/platform/olpc/olpc-ec.c index cfba41f..a3d32c2 100644 > --- a/drivers/platform/olpc/olpc-ec.c > +++ b/drivers/platform/olpc/olpc-ec.c > @@ -113,11 +113,6 @@ int olpc_ec_cmd(u8 cmd, u8 *inbuf, size_t inlen, > u8 *outbuf, size_t outlen) struct olpc_ec_priv *ec = ec_priv; > struct ec_cmd_desc desc; > > - /* XXX: this will be removed in later patches */ > - /* Are we using old-style callers? */ > - if (!ec_driver || !ec_driver->ec_cmd) > - return olpc_ec_cmd_x86(cmd, inbuf, inlen, outbuf, > outlen); - > /* Ensure a driver and ec hook have been registered */ > if (WARN_ON(!ec_driver || !ec_driver->ec_cmd)) > return -ENODEV;