From patchwork Mon Jan 23 23:26:54 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Pavel Machek X-Patchwork-Id: 9533709 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 85D696042F for ; Mon, 23 Jan 2017 23:27:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7C80B2833F for ; Mon, 23 Jan 2017 23:27:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7072828365; Mon, 23 Jan 2017 23:27:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00 autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 08F702833F for ; Mon, 23 Jan 2017 23:27:22 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1cVo1B-0001dO-CY; Mon, 23 Jan 2017 23:27:21 +0000 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cVo16-0001bu-OT for linux-arm-kernel@lists.infradead.org; Mon, 23 Jan 2017 23:27:18 +0000 Received: by atrey.karlin.mff.cuni.cz (Postfix, from userid 512) id EEA90815A7; Tue, 24 Jan 2017 00:26:54 +0100 (CET) Date: Tue, 24 Jan 2017 00:26:54 +0100 From: Pavel Machek To: Guenter Roeck Subject: Re: v4.10-rc4 to v4.10-rc5: battery regression on Nokia N900 Message-ID: <20170123232654.GA19342@amd> References: <20170123124058.GA17134@amd> <20170123141331.GA11402@amd> <20170123144031.GA7870@amd> MIME-Version: 1.0 In-Reply-To: <20170123144031.GA7870@amd> User-Agent: Mutt/1.5.23 (2014-03-12) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170123_152717_138221_ABFA2547 X-CRM114-Status: GOOD ( 19.18 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: ivo.g.dimitrov.75@gmail.com, khilman@kernel.org, tony@atomide.com, aaro.koskinen@iki.fi, kernel list , sre@kernel.org, abcloriens@gmail.com, pali.rohar@gmail.com, fabio.estevam@nxp.com, rui.zhang@intel.com, linux-omap@vger.kernel.org, patrikbachan@gmail.com, linux-arm-kernel , serge@hallyn.com Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Hi! > > >I'll try to revert it on the top of v4.10-rc5 now... and yes, it fixes > > >the issue. > > > > > >Any idea what went wrong and how to fix that? > > > > > >Anyway as we are at -rc5 and this is warning fix that caused a > > >regression on different hardware... it should be reverted. > > > > > Agreed. > > > > What exactly does "stopped working" mean ? That might help understanding > > what went wrong. > > /sys files related to battery no longer appear. I beieve this has > something to do with it: > > [ 2.374877] of_get_named_gpiod_flags: parsed 'reset-gpios' property > of node '/ocp@68000000/spi@48098000/tsc2005@0[0]' - status (0) > [ 2.375946] input: TSC2005 touchscreen as > /devices/platform/68000000.ocp/48098000.spi/spi_master/spi1/spi1.0/input/input5 > [ 2.392120] rx51-battery: probe of n900-battery failed with error > -22 Mystery solved: pavel@n900:~$ dmesg | grep -5 Invalid [ 0.829650] of_get_named_gpiod_flags: parsed 'gpio-reset' property of node '/ocp@68000000/i2c@48072000/tlv320aic3x@19[0]' - status (0) [ 0.833831] tsl2563 2-0029: model 7, rev. 0 [ 0.837768] of_get_named_gpiod_flags: parsed 'enable-gpio' property of node '/ocp@68000000/i2c@48072000/lp5523@32[0]' - status (0) [ 1.921417] omap_i2c 48072000.i2c: controller timed out [ 2.056823] lp5523x 2-0032: lp5523 Programmable led chip found [ 2.064147] hwmon: Invalid character detected: bq27200-0 [ 2.064544] bq27xxx-battery 2-0055: failed to register battery [ 2.064605] bq27xxx-battery: probe of 2-0055 failed with error -22 [ 2.065368] of_get_named_gpiod_flags: parsed 'power-gpio' property of node '/ocp@68000000/i2c@48072000/tpa6130a2@60[0]' - status (0) [ 2.083221] bq2415x-charger 2-006b: automode supported, waiting for events [ 2.084442] bq2415x-charger 2-006b: driver registered --- [ 2.369842] g_ether gadget: g_ether ready [ 2.377197] tsc2005 spi1.0: GPIO lookup for consumer reset [ 2.377227] tsc2005 spi1.0: using device tree for GPIO lookup [ 2.377288] of_get_named_gpiod_flags: parsed 'reset-gpios' property of node '/ocp@68000000/spi@48098000/tsc2005@0[0]' - status (0) [ 2.378936] input: TSC2005 touchscreen as /devices/platform/68000000.ocp/48098000.spi/spi_master/spi1/spi1.0/input/input5 [ 2.395111] hwmon: Invalid character detected: rx51-battery [ 2.402587] rx51-battery: probe of n900-battery failed with error -22 [ 2.410247] omap_hsmmc 4809c000.mmc: GPIO lookup for consumer cd [ 2.410247] omap_hsmmc 4809c000.mmc: using device tree for GPIO lookup [ 2.410278] of_get_named_gpiod_flags: can't parse 'cd-gpios' property of node '/ocp@68000000/mmc@4809c000[0]' [ 2.410278] of_get_named_gpiod_flags: can't parse 'cd-gpio' property of node '/ocp@68000000/mmc@4809c000[0]' Pavel diff --git a/drivers/hwmon/hwmon.c b/drivers/hwmon/hwmon.c index 3932f92..fe5ec82 100644 --- a/drivers/hwmon/hwmon.c +++ b/drivers/hwmon/hwmon.c @@ -545,8 +545,10 @@ __hwmon_device_register(struct device *dev, const char *name, void *drvdata, int i, j, err, id; /* Do not accept invalid characters in hwmon name attribute */ - if (name && (!strlen(name) || strpbrk(name, "-* \t\n"))) + if (name && (!strlen(name) || strpbrk(name, "-* \t\n"))) { + printk("hwmon: Invalid character detected: %s\n", name); return ERR_PTR(-EINVAL); + } id = ida_simple_get(&hwmon_ida, 0, 0, GFP_KERNEL); if (id < 0)