Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp3820976imu; Mon, 10 Dec 2018 08:19:41 -0800 (PST) X-Google-Smtp-Source: AFSGD/UK9iBDM/E6bTWbSeCUHlLf2d2yRpeU7MhQa4N7A2LOWPqAGctFe99iu30rTLluSjJL8iw0 X-Received: by 2002:a62:870e:: with SMTP id i14mr13257363pfe.41.1544458781381; Mon, 10 Dec 2018 08:19:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1544458781; cv=none; d=google.com; s=arc-20160816; b=LrgH/vSPaShIF36uHnSCTXkXPLFuFNfzPS1Mou+1hJjOy7LBUi/N559e/J6L7Ypw0O J7IggFTYGk4w3CAenoir4vAXjYYjb++a3h+vs+j0SykuIqMiNANISYrrNOGlKmWf1IDF csdrI5PENwg46Cg6atKi7G907d7/ZIBeX3UqrkRAcc6BCLJsEn1Euv4xxKk15Oy2hpQZ 2fcw8Ey0G69iAYFvdDpX4kBSYonh6mxfTG5rCO128l/azpA/mEU/ab1ewDeUd1jMUEYM FfN4bNm3ZoDPKs+ZPHF10lAByFWmijPI2lYvJ9JNDHCpcCQpibISYGE+3JP/CP1cqVbk EZeg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=8v8joAtLtbQz45WSH3svSJ/xIYvrhVCTJ3ubUyBE/Mo=; b=LaeLI6uS6bISTOFb8df+6Kj3TNYGgC4q1e1vtJ6wZRVzcQMZnZ/0p54M4ZIUbkTXR2 4CDD22dXLZxNqjIPfSnKpifPAsaqgXrJ5M0vsctP/qJgUo7xLlhffAi4vhYGhtv5MrdZ K+pg7oz6Q1LtmL/7y/oiVkIVWOwIAbKOuM5oeo3dOTRDDmgzaTdTR4/XsD0v7V/c+WqY FGp/TcnVmg3vdsNeTFgsBv6bxdKeAQ2dv1W+K5IRbqw8YwzM91iwrtjhsMQsRFZKBz0i 9ZbGWbg/Bf54XYksWoDhVnxAD1z7Ap31AZNQHSAoZXtlXhm/eSxzZr5vefiuGwgcdve/ VgXQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=kfiIj21i; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 123si10904505pfx.109.2018.12.10.08.19.26; Mon, 10 Dec 2018 08:19:41 -0800 (PST) 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=@gmail.com header.s=20161025 header.b=kfiIj21i; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728029AbeLJOq0 (ORCPT + 99 others); Mon, 10 Dec 2018 09:46:26 -0500 Received: from mail-ot1-f68.google.com ([209.85.210.68]:35919 "EHLO mail-ot1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726888AbeLJOqZ (ORCPT ); Mon, 10 Dec 2018 09:46:25 -0500 Received: by mail-ot1-f68.google.com with SMTP id k98so10610989otk.3; Mon, 10 Dec 2018 06:46:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=8v8joAtLtbQz45WSH3svSJ/xIYvrhVCTJ3ubUyBE/Mo=; b=kfiIj21iB0BbSIRMDFWsdDq30wwYgfnq9BnMwYlpg8SjTzHBDCs3J6h6dcMszOu1mv aVePY6cN7pyhyy08EclWMS+jKouoVMIhOS88af/o2kbREQ4irBeNJQY1tUnwWBQ6OfFs E9Z+VehTAP8G+P6O7Fw8mpsoi6B21OhRiV9w9hLkRMP0pbRdBw/7K32lkPwuXvxGjDTj 0XbTaNoRL2Xjw37Owl1IkuhXz6YEGkAox5r7GhfbcSbGKfHGBNLehNH4i/fyX+h32QEx BVAPNPrr5+tTaO3O/9ODxSW4MbLq1bVK5EWjeP1kptcIfIQOm0PKY0RTgkQhu03dPqSW 1q2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=8v8joAtLtbQz45WSH3svSJ/xIYvrhVCTJ3ubUyBE/Mo=; b=HDkwYbXSysMTaH/OdQ8aaIrpb+EBBa7jQH9RhxktZhWUW9jamA+6suoe8KN2W66UJM IMZLfqQuTYliSbsxi+jvrvAmlcIDlpVksWQaCJ0cVcOmqlOT1B5GqUBZVMUc8juebRnc VzFNk/hzskvCV50D+kkCyXEDIIpZ174c9N+P+5ja3t/oQVXIU3z1eZcN7VrVFgR42gV5 omZfnQ2TRVflWcR3WHcl8A8NrxitW3BPpHjS1Nd2PGZXCEPIblV2z2GqU1TKdo47Tht5 4XEEc92KxYZ7VDrhLEcnn4n0YxnuWEokJXzislHXkNPYQ9yBxAjbsUk3V2zLZxqFRwci yYtw== X-Gm-Message-State: AA+aEWa4E46KiWBOlFuw4FPPTtQJiOXYu/9NTSsVNbL4tf/BXimyN6Ri 1wRHml+zjE7dBL5ypRQdxKidn0fl3mXUqHTvErs= X-Received: by 2002:a9d:7e87:: with SMTP id m7mr9513039otp.225.1544453184260; Mon, 10 Dec 2018 06:46:24 -0800 (PST) MIME-Version: 1.0 References: <1544426368-16415-1-git-send-email-Anson.Huang@nxp.com> <1544426368-16415-2-git-send-email-Anson.Huang@nxp.com> In-Reply-To: <1544426368-16415-2-git-send-email-Anson.Huang@nxp.com> From: Fabio Estevam Date: Mon, 10 Dec 2018 12:46:17 -0200 Message-ID: Subject: Re: [PATCH V2 2/2] iio: accell: mma8452: add optional vdd/vddio regulator operation support To: Yongcai Huang Cc: Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Rob Herring , Mark Rutland , rtresidd@electromag.com.au, "Gustavo A. R. Silva" , Greg Kroah-Hartman , Leonard Crestez , martink@posteo.de, linux-iio@vger.kernel.org, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-kernel , NXP Linux Team Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 10, 2018 at 5:25 AM Anson Huang wrote: > @@ -1533,6 +1536,27 @@ static int mma8452_probe(struct i2c_client *client, > data->client = client; > mutex_init(&data->lock); > data->chip_info = match->data; > + data->vdd_reg = devm_regulator_get_optional(&client->dev, "vdd"); > + if (!IS_ERR(data->vdd_reg)) { > + ret = regulator_enable(data->vdd_reg); > + if (ret) { > + dev_err(&client->dev, "failed to enable VDD regulator\n"); > + return ret; > + } > + } else if (data->vdd_reg == ERR_PTR(-EPROBE_DEFER)) { > + return -EPROBE_DEFER; > + } > + > + data->vddio_reg = devm_regulator_get_optional(&client->dev, "vddio"); > + if (!IS_ERR(data->vddio_reg)) { > + ret = regulator_enable(data->vddio_reg); > + if (ret) { > + dev_err(&client->dev, "failed to enable VDDIO regulator\n"); > + return ret; As mentioned in your other patch: you should disable the previously enabled regulator in the error path.