Received: by 10.213.65.68 with SMTP id h4csp88684imn; Wed, 21 Mar 2018 13:06:42 -0700 (PDT) X-Google-Smtp-Source: AG47ELvgVFn+8+RSMV3OPqnGRN7l6LDPejyPiZDJ3TT7WXuO3mWRyqk4cYulVLOLb6M4QhjM/m0b X-Received: by 10.98.159.85 with SMTP id g82mr17930696pfe.15.1521662802156; Wed, 21 Mar 2018 13:06:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521662802; cv=none; d=google.com; s=arc-20160816; b=jmbWKFTPNTP55Urc7luwcIvebDaSIKKJSBQ5EGw8DywIdnMj+bDmLU7Mj06tminu65 /yF6HnGYMO0qtezYOHfLbigBV/sqeS1kyxg5hOXl4kCneJm7dhs7dsn3xdoYGlDtxojG 5N8FG4b15VOyHlgq+mflGXUlBpFzXCE/BLuw78TQ7thGnA3ryDugCv1NRkjfKcxSIcaQ bffsOc2k402vYYIEl8wXliu8VjYFEUK7FIUY/Dnj3VFxdQDIdHgEtH8jXbAZXsK0keW1 rQmRCKVk5fHRzfNWNlqhx68Ja7SeJ482WAvmTz/9cXq1xBDMmxv8GeLAH3VG6elHjF+8 gKFQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:subject:user-agent:message-id :references:cc:in-reply-to:from:to:content-transfer-encoding :mime-version:dkim-signature:arc-authentication-results; bh=2GjJkDc2JdwVn1JXmW25cnM0iIqgGVk2Cltgel+sqVY=; b=RvD3HBRoW6zDeSWiTZ4H/uAF/2U2GLsa1vYtb3m5NLPVLhE+aqIQGeplLVj+vdP2tA /6KU+977LFo/icJu/bKtQyz4PxLJ2GQdPvtqAobRq/55J8W3KPfuVLqr7JYIAuGVSA58 bzfTHjbNYBU1SWKEG4OIcqjYdB0+UtpvwVlkr1y54LV/LeluMc+33mB3bNRaK7jEFng+ lrltx8c7giZ17CKexGl+j0USWXEW23B8fM9KXce8QwIe4eIV+NX8rmuE85Y2RDCizBpS pKioIGdFf0FlivLxRWsbhm8zycb+szAR0zRUl7B8WqjTle9TEHigcamHXC3NURTPkfpp rIsw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=U+3t32P+; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j62si3284086pgc.583.2018.03.21.13.06.27; Wed, 21 Mar 2018 13:06:42 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=U+3t32P+; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753436AbeCUUEn (ORCPT + 99 others); Wed, 21 Mar 2018 16:04:43 -0400 Received: from mail-pl0-f50.google.com ([209.85.160.50]:40072 "EHLO mail-pl0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753284AbeCUUEi (ORCPT ); Wed, 21 Mar 2018 16:04:38 -0400 Received: by mail-pl0-f50.google.com with SMTP id x4-v6so3801009pln.7 for ; Wed, 21 Mar 2018 13:04:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:content-transfer-encoding:to:from:in-reply-to:cc :references:message-id:user-agent:subject:date; bh=2GjJkDc2JdwVn1JXmW25cnM0iIqgGVk2Cltgel+sqVY=; b=U+3t32P+sJHZrgne949Wxu7VKqnIXSTausvAL0yk/nTaQ39WXdJ8V6wYKg3aMZYXDI 6GT5hNOQv67hpDPk7ZRJNDJvYaE0cfSgBRVdiaXWnoYQ6FuvMIKux6ie8y3vQ0QwE1QH d97Og5g4DgZy8/d20NXLNdWR46oINadt5rl7g= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:content-transfer-encoding:to:from :in-reply-to:cc:references:message-id:user-agent:subject:date; bh=2GjJkDc2JdwVn1JXmW25cnM0iIqgGVk2Cltgel+sqVY=; b=FfVhsOW1KH2Js8avSY4RTnoeDFGhEsnexfSUsEiMBsopGqNEGytxB9PGGrsqWUMJxc 2XFOwwIELQ1G7jlc39ZkDT75++gh2J6j/hsIHyaYFNuw2suhcs5Xnb0NIdy9rUJ0xvDa E88EiC/VogmEx74a5NVNQos0AtKH7YMs48FmSjfp6WLfflpKFLtqxo37s+3u+QrOVQFe unnGEWY5gzh+0xQGPxkPq1+/AdinHxjJJQH3F7Fn+tzD94Au0baCQZs4b0/WYXwVqvgF 63Jxi7iL5CYGOcWQynrAkQMHE5aTF6s+NLoKpbBJTC7/midm+uPk98sI0IymhmKXmHKF +boQ== X-Gm-Message-State: AElRT7Fu+nAxsYHEK5t+vfKOxSQYd5AeVZNEDu6HwQGPguWIMQMcVHTN oF4FUYctjI8ye6kJ5oST5WMc7Q== X-Received: by 2002:a17:902:6b81:: with SMTP id p1-v6mr22216351plk.181.1521662677859; Wed, 21 Mar 2018 13:04:37 -0700 (PDT) Received: from localhost ([2620:0:1000:1511:d30e:62c6:f82c:ff40]) by smtp.gmail.com with ESMTPSA id z13sm10662414pfk.129.2018.03.21.13.04.37 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 21 Mar 2018 13:04:37 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: Andy Shevchenko , Linus Walleij From: Stephen Boyd In-Reply-To: <1521655629.23017.84.camel@linux.intel.com> Cc: Stephen Boyd , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, Timur Tabi , Bjorn Andersson , Grant Likely , linux-gpio@vger.kernel.org References: <20180321165848.89751-1-swboyd@chromium.org> <20180321165848.89751-4-swboyd@chromium.org> <1521655629.23017.84.camel@linux.intel.com> Message-ID: <152166267662.91116.8188392814465278915@swboyd.mtv.corp.google.com> User-Agent: alot/0.7 Subject: Re: [PATCH v3 3/3] pinctrl: qcom: Don't allow protected pins to be requested Date: Wed, 21 Mar 2018 13:04:36 -0700 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Andy Shevchenko (2018-03-21 11:07:09) > On Wed, 2018-03-21 at 09:58 -0700, Stephen Boyd wrote: > > +static int msm_pinmux_request(struct pinctrl_dev *pctldev, unsigned > > offset) > > +{ > > + struct msm_pinctrl *pctrl =3D pinctrl_dev_get_drvdata(pctldev); > > + struct gpio_chip *chip =3D &pctrl->chip; > > + > > + if (gpiochip_line_is_valid(chip, offset)) > > + return 0; > > + > > + return -EINVAL; > = > Perhaps traditional pattern > = > if (!...) > return -EINVAL; > = > return 0; > = Or ternary? return gpiochip_line_is_valid(chip, offset) ? 0 : -EINVAL; > = > > +} > = > > seq_printf(s, " %dmA", msm_regval_to_drive(drive)); > > - seq_printf(s, " %s", pulls[pull]); > > + seq_printf(s, " %s\n", pulls[pull]); > = > I had commented this once, but you ignored by some reason. > = > I would rather just move = > seq_puts(s, "\n"); > here. > = > The rationale behind, besides making diff more neat, is to reduce > possible burden in the future if someone would like to squeeze more data > in between. Sure. > = > > + tmp =3D kmalloc_array(len, sizeof(tmp[0]), GFP_KERNEL); > = > sizeof(*tmp) ? > = Ok.