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 X-Spam-Level: X-Spam-Status: No, score=-11.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 63D99C388F9 for ; Wed, 11 Nov 2020 08:00:37 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B4BFD206B5 for ; Wed, 11 Nov 2020 08:00:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Ge9CnHvz"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=xilinx.onmicrosoft.com header.i=@xilinx.onmicrosoft.com header.b="QcCwT+hh" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B4BFD206B5 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xilinx.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=WUlAgY07T9pKrkwTESG8yAMrgnwjXG5wEy6RlDUfxB0=; b=Ge9CnHvzBcbX8s1jk3UsQHwMs T7akPNpJeqlUj9BBXcYnVh16tveX+ntKYolUEh+IfZQtqKrWTciS7Tdv6p4Hg06hyQTH/Q3knXtq5 c4SSpABNmpy4g2FJKP9eGd07dfm57DchuLOdEMDVj5kKRaHO55qMpQ+k0369/H36uA+8mVDGudjyy lXbl+9IS10ZfjEqD+LR9/t9xA9AVs/HTD1yNbEYnFKHi0cTgR03TcV5qGw6NTEt9v4HistG6rRImT Z3jggWljs6IUrGui7ZEcGivJmHyvkW7zyk+uwE+0K3byEZl2tQbYjV1R0WyqWFE8AwysG6aS13iB2 e40qkF76A==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kcl2v-0000R5-FJ; Wed, 11 Nov 2020 08:00:02 +0000 Received: from mail-dm6nam10on2081.outbound.protection.outlook.com ([40.107.93.81] helo=NAM10-DM6-obe.outbound.protection.outlook.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kcl2o-0000QJ-AF for linux-arm-kernel@lists.infradead.org; Wed, 11 Nov 2020 07:59:56 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VjXjjtD9Q7V5gcp6IDGpUkMdf1Qwfp6oNo5TDUjlJRYk102+hoImsdY+dsPHc0WVo1CEvp4VUjs9VVMvDrYQdFoo0605e0wlYn2W3C78dP607AD2Xr5Nm54MoayZvn2RtRTpIzSCeq7h6p/O0bn7d2KxGikIXQEJKFOXHb6saqeYw/VoQJHrjGDa21ChuuHA1axsw6OoNSvjk1E+LdTcAa02BfKVuGZywfENQo5i4COmFBannCaDNuguM9GgU1U/CM2haIS6ms/1a70QIjFuG/VWqLw0aYAdA50+WuhFbIfAXGFg7P2NgEbk5EuyzMvO9XnwY7hXQFQZxu8GhIizzg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=tq2Ay8xZzTKEuRpxfTimdd/Hj17tLv04TcB9X9IIU84=; b=js985ENX2e0bmjHfyc+CSjufQrNatBlpKxg3PaLbe/34BxIwDq65h+9tn4hHQ5hgK9NZeeEqmIGlXiPzGgR3ngaFwPh3+J8s/wzOrlAAUPBKVFRi9VkSDMyiDOyxNzgbqFgSMhGZAYp7ZnzXyW3Tm3a2ToUs9G515JImX9eaEbVXKzATDLnexxL7EIWFDI6pr9mmOj0iGd4VmjLD3HW41yK1p1RkIBbe6UC6JsAX8sn0Wf+Qhxc4fpj1ivIc8hw62vHSGXVRWotrmCvPf1EQVoWmWwkSXogF3YScItNh57vjH/vPfdCgzfC0dBQNSgTIrUYbh01fxqlrQnCu/EsCoQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=denx.de smtp.mailfrom=xilinx.com; dmarc=bestguesspass action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=tq2Ay8xZzTKEuRpxfTimdd/Hj17tLv04TcB9X9IIU84=; b=QcCwT+hhtSUlCep46cbEWASEUTfuMXKPVprJNZeBhhme+2nyfs/MN4WedDev8RdkJBOhsHnh1rIoBYXjrX5Ny6xI8aI4TP6vZr41iaLa+IqA0h/OMA49tcF7d04F14EPpzcmTcbsFasXF8yNu+hp7KnRhGzgFc7gdADwQyI6tOw= Received: from CY4PR20CA0012.namprd20.prod.outlook.com (2603:10b6:903:98::22) by DM6PR02MB5130.namprd02.prod.outlook.com (2603:10b6:5:47::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3541.25; Wed, 11 Nov 2020 07:59:50 +0000 Received: from CY1NAM02FT039.eop-nam02.prod.protection.outlook.com (2603:10b6:903:98:cafe::e9) by CY4PR20CA0012.outlook.office365.com (2603:10b6:903:98::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3541.25 via Frontend Transport; Wed, 11 Nov 2020 07:59:50 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; denx.de; dkim=none (message not signed) header.d=none;denx.de; dmarc=bestguesspass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch01.xlnx.xilinx.com; Received: from xsj-pvapexch01.xlnx.xilinx.com (149.199.62.198) by CY1NAM02FT039.mail.protection.outlook.com (10.152.75.140) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.3541.17 via Frontend Transport; Wed, 11 Nov 2020 07:59:49 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch01.xlnx.xilinx.com (172.19.86.40) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1913.5; Tue, 10 Nov 2020 23:59:43 -0800 Received: from smtp.xilinx.com (172.19.127.96) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.1913.5 via Frontend Transport; Tue, 10 Nov 2020 23:59:43 -0800 Received: from [172.30.17.110] (port=60358) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1kcl2c-0006P2-S2; Tue, 10 Nov 2020 23:59:43 -0800 Subject: Re: [PATCH] mmc: sdhci-of-arasan: Allow configuring zero tap values To: Manish Narani , , , References: <1605030122-52196-1-git-send-email-manish.narani@xilinx.com> From: Michal Simek Message-ID: Date: Wed, 11 Nov 2020 08:59:39 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.2 MIME-Version: 1.0 In-Reply-To: <1605030122-52196-1-git-send-email-manish.narani@xilinx.com> Content-Language: en-US X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4095f067-f089-40de-588e-08d88617c377 X-MS-TrafficTypeDiagnostic: DM6PR02MB5130: X-Microsoft-Antispam-PRVS: X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-MS-Oob-TLC-OOBClassifiers: OLM:6430; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: lYRKgaqmdj9+kHj0l4LopnQUs4RL50q/5z61YoIDMRcFKqsIh8Y52s8wHASJ+H8YPd43s8LUvnU0RKT/XRFP259zj+lrfGitpCH1xBoHEULVclyOrxiQ3Ohe1c3oNoQL/0UezApr3J4wOd5g0Gp5TPa3Rfn54vWXgMlIiRIY9HwTIWVkzYHl/PzAEp57bQmXlPbd4i6FlMTsJ0le3/Sqa54ymaK+8Ibt+yppQW3SKBHcS4QyEVx3BahxT4ypiUaZ5nyXwUJCjWm6AhQoV/jsK+NB3IxnikIPOnSvlLFitDqDGurMFEUQ+ga4Msajo7T/f8N4HEq1fa3o6oRetisKHM6MogOlwx8KoM0uJ5jb5zDLiSm8HiWe+Oo+B4tiS6ZWDx3E7NB6hLgoBZ4P/N6M9/GSkLCvfff0YGoHZdgSgRqqCcUM8wjRRPwceB3SzY+kbGZt6YmZQY3WZCpqbWFf0g== X-Forefront-Antispam-Report: CIP:149.199.62.198; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:xsj-pvapexch01.xlnx.xilinx.com; PTR:unknown-62-198.xilinx.com; CAT:NONE; SFS:(4636009)(136003)(396003)(376002)(346002)(39860400002)(46966005)(478600001)(44832011)(70206006)(336012)(9786002)(110136005)(2906002)(83380400001)(8936002)(36906005)(54906003)(70586007)(26005)(8676002)(316002)(31686004)(4326008)(426003)(6666004)(31696002)(186003)(5660300002)(2616005)(7636003)(82740400003)(47076004)(82310400003)(356005)(36756003)(50156003)(43740500002); DIR:OUT; SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Nov 2020 07:59:49.4044 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 4095f067-f089-40de-588e-08d88617c377 X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c; Ip=[149.199.62.198]; Helo=[xsj-pvapexch01.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: CY1NAM02FT039.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR02MB5130 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201111_025954_516035_54663491 X-CRM114-Status: GOOD ( 20.46 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Marek Vasut , Sai Krishna Potthuri , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, git@xilinx.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org +Marek On 10. 11. 20 18:42, Manish Narani wrote: > Allow configuring the Output and Input tap values with zero to avoid > failures in some cases (one of them is SD boot mode) where the output > and input tap values may be already set to non-zero. > > Signed-off-by: Sai Krishna Potthuri > Signed-off-by: Manish Narani > --- > drivers/mmc/host/sdhci-of-arasan.c | 40 ++++++------------------------ > 1 file changed, 8 insertions(+), 32 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c > index 829ccef87426..100621e55427 100644 > --- a/drivers/mmc/host/sdhci-of-arasan.c > +++ b/drivers/mmc/host/sdhci-of-arasan.c > @@ -600,14 +600,8 @@ static int sdhci_zynqmp_sdcardclk_set_phase(struct clk_hw *hw, int degrees) > u8 tap_delay, tap_max = 0; > int ret; > > - /* > - * This is applicable for SDHCI_SPEC_300 and above > - * ZynqMP does not set phase for <=25MHz clock. > - * If degrees is zero, no need to do anything. > - */ > - if (host->version < SDHCI_SPEC_300 || > - host->timing == MMC_TIMING_LEGACY || > - host->timing == MMC_TIMING_UHS_SDR12 || !degrees) > + /* This is applicable for SDHCI_SPEC_300 and above */ > + if (host->version < SDHCI_SPEC_300) > return 0; > > switch (host->timing) { > @@ -668,14 +662,8 @@ static int sdhci_zynqmp_sampleclk_set_phase(struct clk_hw *hw, int degrees) > u8 tap_delay, tap_max = 0; > int ret; > > - /* > - * This is applicable for SDHCI_SPEC_300 and above > - * ZynqMP does not set phase for <=25MHz clock. > - * If degrees is zero, no need to do anything. > - */ > - if (host->version < SDHCI_SPEC_300 || > - host->timing == MMC_TIMING_LEGACY || > - host->timing == MMC_TIMING_UHS_SDR12 || !degrees) > + /* This is applicable for SDHCI_SPEC_300 and above */ > + if (host->version < SDHCI_SPEC_300) > return 0; > > switch (host->timing) { > @@ -733,14 +721,8 @@ static int sdhci_versal_sdcardclk_set_phase(struct clk_hw *hw, int degrees) > struct sdhci_host *host = sdhci_arasan->host; > u8 tap_delay, tap_max = 0; > > - /* > - * This is applicable for SDHCI_SPEC_300 and above > - * Versal does not set phase for <=25MHz clock. > - * If degrees is zero, no need to do anything. > - */ > - if (host->version < SDHCI_SPEC_300 || > - host->timing == MMC_TIMING_LEGACY || > - host->timing == MMC_TIMING_UHS_SDR12 || !degrees) > + /* This is applicable for SDHCI_SPEC_300 and above */ > + if (host->version < SDHCI_SPEC_300) > return 0; > > switch (host->timing) { > @@ -804,14 +786,8 @@ static int sdhci_versal_sampleclk_set_phase(struct clk_hw *hw, int degrees) > struct sdhci_host *host = sdhci_arasan->host; > u8 tap_delay, tap_max = 0; > > - /* > - * This is applicable for SDHCI_SPEC_300 and above > - * Versal does not set phase for <=25MHz clock. > - * If degrees is zero, no need to do anything. > - */ > - if (host->version < SDHCI_SPEC_300 || > - host->timing == MMC_TIMING_LEGACY || > - host->timing == MMC_TIMING_UHS_SDR12 || !degrees) > + /* This is applicable for SDHCI_SPEC_300 and above */ > + if (host->version < SDHCI_SPEC_300) > return 0; > > switch (host->timing) { > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel