From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2D384C433F5 for ; Mon, 6 Dec 2021 20:44:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349761AbhLFUrp (ORCPT ); Mon, 6 Dec 2021 15:47:45 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51620 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239617AbhLFUro (ORCPT ); Mon, 6 Dec 2021 15:47:44 -0500 Received: from mail-oi1-x22d.google.com (mail-oi1-x22d.google.com [IPv6:2607:f8b0:4864:20::22d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3E4EEC061746 for ; Mon, 6 Dec 2021 12:44:15 -0800 (PST) Received: by mail-oi1-x22d.google.com with SMTP id q25so23826470oiw.0 for ; Mon, 06 Dec 2021 12:44:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:in-reply-to:references:from:user-agent:date:message-id :subject:to:cc; bh=Kd/VW0k24ZiHeQ745g+bN45hbG/acmtRc9TNwBDa86g=; b=buSDdWYwB4NAhrdftCBmPm5r+hmeyl+5sNoYCON9535QpKwENQ3W6DGlahDKYsvbk0 UfASPUJHNi4MPAUQwE72yTiJTkf4HQ9UnIRo61L83Qa27ACTh8j0hrzlZM4uwu/RgRL4 FwW2xz19d5eRj55VHKpMOrwAg26R4WeepTS94= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:in-reply-to:references:from :user-agent:date:message-id:subject:to:cc; bh=Kd/VW0k24ZiHeQ745g+bN45hbG/acmtRc9TNwBDa86g=; b=j0TXY0rV0O+priciWPu6nB3qR6vrlzkU3sEfRFEBm6XnjdIj+ybcReiS+ZY7u2sCbB rOor2XeEzCEGc9G2Fgi6lf/hnmJ5ppxyqRacfQ8lHgt7hhf8u0kXwvp3QH/Cm36uiqvI XApbea9p/CsFBQwQk8bTi5DlH5CQjNtrsE7iARjdKbCsx/i1qfTBxMvYckSvcbzKuGP6 THF2l/aM+1t9S3yw62dOUMBrOjEiFPozqHXaXHssdUZ0Ai5yg+b/SzKAbjRu17cPzvZP I3WsSmcvEBuweGLG7X2Lui1xJB9Nlr69juBB9JhMzaebwuNweoXt5JFzR5Tqd/eEjoeP dChw== X-Gm-Message-State: AOAM532C3h+4V83uSwIFAzwggNpaDjH5/b+EBKgGiqwXValbyo0o9Gip iEGGpLteZViJYxaG3l18CULqJdKuArDegujeVpOxVA== X-Google-Smtp-Source: ABdhPJy1aPu4PZyyqqf9TaHxIlYtZXNcCxpyocIp3nHBma+pzCidUmdUcGRG4ZTRJNottzSjYp80KeFFzHWJjk3tVXQ= X-Received: by 2002:aca:2319:: with SMTP id e25mr1037302oie.164.1638823454543; Mon, 06 Dec 2021 12:44:14 -0800 (PST) Received: from 753933720722 named unknown by gmailapi.google.com with HTTPREST; Mon, 6 Dec 2021 12:44:14 -0800 MIME-Version: 1.0 In-Reply-To: <20211205191009.32454-1-nishadkamdar@gmail.com> References: <20211205191009.32454-1-nishadkamdar@gmail.com> From: Stephen Boyd User-Agent: alot/0.9.1 Date: Mon, 6 Dec 2021 12:44:14 -0800 Message-ID: Subject: Re: [PATCH] mmc: core: Add support for the eMMC RTC feature in mmc_ops To: Adrian Hunter , Avri Altman , Bean Huo , Huijin Park , Jens Axboe , Nishad Kamdar , Shawn Lin , Ulf Hansson , Wolfram Sang , Yue Hu Cc: Greg Kroah-Hartman , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Nishad Kamdar (2021-12-05 11:10:08) > diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c > index d63d1c735335..490372341b3b 100644 > --- a/drivers/mmc/core/mmc_ops.c > +++ b/drivers/mmc/core/mmc_ops.c > @@ -1063,3 +1063,62 @@ int mmc_sanitize(struct mmc_card *card, unsigned int timeout_ms) > return err; > } > EXPORT_SYMBOL_GPL(mmc_sanitize); > + > +int mmc_set_time(struct mmc_card *card, struct mmc_host *host, > + u8 rtc_info_type, u64 seconds) > +{ > + struct mmc_request mrq = {}; > + struct mmc_command cmd = {}; > + struct mmc_data data = {}; > + struct scatterlist sg; > + int err = 0; > + u8 *data_buf; > + > + data_buf = kzalloc(512, GFP_KERNEL); Use some #define for 512 because it's used three times in here? > + if (!data_buf) > + return -ENOMEM; > + > + if (rtc_info_type == 0x01 || rtc_info_type == 0x02 || > + rtc_info_type == 0x03) { > + data_buf[0] = 0x01; > + data_buf[1] = rtc_info_type; > + memcpy(&data_buf[2], &seconds, sizeof(u64)); Use sizeof(seconds) instead? > + } else { > + pr_err("%s: invalid rtc_info_type %d\n", > + mmc_hostname(host), rtc_info_type); > + kfree(data_buf); > + return -EINVAL; > + } > + > + mrq.cmd = &cmd; > + mrq.data = &data; > + > + cmd.opcode = MMC_SET_TIME; > + cmd.arg = 0; > + cmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_ADTC; > + > + data.blksz = 512; > + data.blocks = 1; > + data.flags = MMC_DATA_WRITE; > + data.sg = &sg; > + data.sg_len = 1; > + sg_init_one(&sg, data_buf, 512); > + > + mmc_set_data_timeout(&data, card); > + > + mmc_wait_for_req(host, &mrq); > + > + if (cmd.error) { > + err = cmd.error; > + goto out; > + } > + > + if (data.error) { > + err = data.error; > + goto out; > + } Why not if (cmd.error) { err = cmd.error; } else if (data.error) { err = data.error; } else { err = 0; } > +out: And then drop out: and the assignment of err to 0 up above? > + kfree(data_buf); > + return err; > +} > +EXPORT_SYMBOL_GPL(mmc_set_time);