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 0B0ABC4332F for ; Tue, 15 Mar 2022 23:03:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349790AbiCOXEy (ORCPT ); Tue, 15 Mar 2022 19:04:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58752 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344119AbiCOXEx (ORCPT ); Tue, 15 Mar 2022 19:04:53 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E793E2AC7; Tue, 15 Mar 2022 16:03:39 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 93079B8190D; Tue, 15 Mar 2022 23:03:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DD318C340E8; Tue, 15 Mar 2022 23:03:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1647385417; bh=TzB71EHsQSxi9ycyQW305aKb38kU4PoZa65+ZUbL93c=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=gxcMWeCpBQjsYX9XnajV96+qEiT05QvvAx+rpLrcAIzeNueu3XWZgwe1bu+82MSBo 8QtaTDP9fJx5OgfCXARIFUexLMi0jPJs/vNLyeFZqi1jS8CkFc0ovgc57dSyCUNkOW EaGMwjv3hwlNYVX/NmwlmTf7otRdnACrkJEFDFsl2nqjvl01R5QjRk3a5rO2gsXXPP rAY9iKskfNSakrlT0SzC7D+j1L3AG8RJRn1nH9ENRUJ+KeoaiPR+qQyQt2K4Jcmx28 W7FavvnKY13dgKnurZpxAFG/STrJFbgl9ESVSWN0O3sFE1hLCsPzbip00dWz2/B2Z0 jKUsesoPwf9cQ== Received: by pali.im (Postfix) id F1C44824; Wed, 16 Mar 2022 00:03:33 +0100 (CET) Date: Wed, 16 Mar 2022 00:03:33 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Marcin Wojtas Cc: Ulf Hansson , Linux Kernel Mailing List , "linux-mmc@vger.kernel.org" , Ziji Hu , Adrian Hunter , Grzegorz Jaszczyk , Tomasz Nowicki , Kostya Porotchkin , Alex Leibovich , "# 4.0+" Subject: Re: [PATCH] mmc: sdhci-xenon: fix 1.8v regulator stabilization Message-ID: <20220315230333.eyznbu5tuxneizbs@pali> References: <20201211141656.24915-1-mw@semihalf.com> <20220314154033.4x74zscayee32rrj@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20180716 Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org Hello! On Monday 14 March 2022 16:51:25 Marcin Wojtas wrote: > Hi Pali, > > > pon., 14 mar 2022 o 16:40 Pali Rohár napisał(a): > > > > On Monday 11 January 2021 19:06:24 Ulf Hansson wrote: > > > On Fri, 11 Dec 2020 at 15:17, Marcin Wojtas wrote: > > > > > > > > From: Alex Leibovich > > > > > > > > Automatic Clock Gating is a feature used for the power > > > > consumption optimisation. It turned out that > > > > during early init phase it may prevent the stable voltage > > > > switch to 1.8V - due to that on some platfroms an endless > > > > printout in dmesg can be observed: > > > > "mmc1: 1.8V regulator output did not became stable" > > > > Fix the problem by disabling the ACG at very beginning > > > > of the sdhci_init and let that be enabled later. > > > > > > > > Fixes: 3a3748dba881 ("mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality") > > > > Signed-off-by: Alex Leibovich > > > > Signed-off-by: Marcin Wojtas > > > > Cc: stable@vger.kernel.org > > > > > > Applied for fixes (by fixing the typos), thanks! > > > > Hello! > > > > Is not this patch address same issue which was fixed by patch which was > > merged earlier? > > > > bb32e1987bc5 ("mmc: sdhci-xenon: fix annoying 1.8V regulator warning") > > https://lore.kernel.org/linux-mmc/CAPDyKFqAsvgAjfL-c9ukFNWeGJmufQosR2Eg9SKjXMVpNitdkA@mail.gmail.com/ > > > > This indeed look similar. This fix was originally developed for CN913x > platform without the mentioned patch (I'm wondering if it would also > suffice to fix A3k board's problem). Anyway, I don't think we have an > issue here, as everything seems to work fine on top of mainline Linux > with both changes. Yea, there should be no issue. Just question is if we need _both_ fixes. I could probably try to revert bb32e1987bc5 and check what happens on A3k board. > Best regards, > Marcin > > > > Kind regards > > > Uffe > > > > > > > > > > --- > > > > drivers/mmc/host/sdhci-xenon.c | 7 ++++++- > > > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/drivers/mmc/host/sdhci-xenon.c b/drivers/mmc/host/sdhci-xenon.c > > > > index c67611fdaa8a..4b05f6fdefb4 100644 > > > > --- a/drivers/mmc/host/sdhci-xenon.c > > > > +++ b/drivers/mmc/host/sdhci-xenon.c > > > > @@ -168,7 +168,12 @@ static void xenon_reset_exit(struct sdhci_host *host, > > > > /* Disable tuning request and auto-retuning again */ > > > > xenon_retune_setup(host); > > > > > > > > - xenon_set_acg(host, true); > > > > + /* > > > > + * The ACG should be turned off at the early init time, in order > > > > + * to solve a possile issues with the 1.8V regulator stabilization. > > > > + * The feature is enabled in later stage. > > > > + */ > > > > + xenon_set_acg(host, false); > > > > > > > > xenon_set_sdclk_off_idle(host, sdhc_id, false); > > > > > > > > -- > > > > 2.29.0 > > > >