From: Dong Aisheng <dongas86@gmail.com> To: Gary Bisson <gary.bisson@boundarydevices.com> Cc: Dong Aisheng <aisheng.dong@nxp.com>, linux-mmc@vger.kernel.org, ulf.hansson@linaro.org, chris@printf.net, haibo.chen@nxp.com, adrian.hunter@intel.com, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [V2, 10/15] mmc: sdhci-esdhc-imx: enable hw auto retuning for MAN_TUNING Date: Tue, 16 Aug 2016 18:18:18 +0800 [thread overview] Message-ID: <20160816101818.GD9211@shlinux2> (raw) In-Reply-To: <20160815145941.GA23663@t450s.lan> On Mon, Aug 15, 2016 at 04:59:41PM +0200, Gary Bisson wrote: > Dong Aisheng, All, > > On Tue, Jul 12, 2016 at 03:46:19PM +0800, Dong Aisheng wrote: > > Indicating hw auto retuning support for mx6qdl in the fake caps_1 > > register and enable auto retuning in post_tuning process after > > tuning completes. > > Have you tried this change with a SDIO3.0 device? I'm asking because a Tested with SD3.0 device. Not really for SDIO3.0 device since there's no SDIO3.0 device on my available MX6Q/DL boards. > similar change in latest NXP 4.1.15 kernel broke SDIO3.0 support. > With this tuning, the kernel would report: > mmc2: Got data interrupt 0x00000002 even though no data operation was in > progress. > Can you provide the full enumeration log? > Doing a 'git bisect' and then reverting this patch fixed the issue. > Although I haven't tested that change on mainline kernel, I wanted you > to know about this observation before it gets merged. > > As a FYI, the issue has been reported to NXP community: > https://community.nxp.com/thread/431316 > Thanks for nofity. I will check it on my side. Regards Dong Aisheng > Regards, > Gary
WARNING: multiple messages have this Message-ID (diff)
From: dongas86@gmail.com (Dong Aisheng) To: linux-arm-kernel@lists.infradead.org Subject: [V2, 10/15] mmc: sdhci-esdhc-imx: enable hw auto retuning for MAN_TUNING Date: Tue, 16 Aug 2016 18:18:18 +0800 [thread overview] Message-ID: <20160816101818.GD9211@shlinux2> (raw) In-Reply-To: <20160815145941.GA23663@t450s.lan> On Mon, Aug 15, 2016 at 04:59:41PM +0200, Gary Bisson wrote: > Dong Aisheng, All, > > On Tue, Jul 12, 2016 at 03:46:19PM +0800, Dong Aisheng wrote: > > Indicating hw auto retuning support for mx6qdl in the fake caps_1 > > register and enable auto retuning in post_tuning process after > > tuning completes. > > Have you tried this change with a SDIO3.0 device? I'm asking because a Tested with SD3.0 device. Not really for SDIO3.0 device since there's no SDIO3.0 device on my available MX6Q/DL boards. > similar change in latest NXP 4.1.15 kernel broke SDIO3.0 support. > With this tuning, the kernel would report: > mmc2: Got data interrupt 0x00000002 even though no data operation was in > progress. > Can you provide the full enumeration log? > Doing a 'git bisect' and then reverting this patch fixed the issue. > Although I haven't tested that change on mainline kernel, I wanted you > to know about this observation before it gets merged. > > As a FYI, the issue has been reported to NXP community: > https://community.nxp.com/thread/431316 > Thanks for nofity. I will check it on my side. Regards Dong Aisheng > Regards, > Gary
next prev parent reply other threads:[~2016-08-16 10:27 UTC|newest] Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-07-12 7:46 [PATCH V2 00/15] a few sdhci/imx clean up and fix patches Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 01/15] mmc: sdhci: using common mmc_regulator_set_vqmmc() Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 7:06 ` Adrian Hunter 2016-07-21 7:06 ` Adrian Hunter 2016-07-12 7:46 ` [PATCH V2 02/15] mmc: sdhci-esdhc-imx: remove SDHCI_QUIRK_BROKEN_TIMEOUT_VAL Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 03/15] mmc: sdhci-esdhc-imx: add esdhc specific suspend resume callback Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 04/15] mmc: sdhci-esdhc-imx: restore watermark level setting after resume Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 7:18 ` Adrian Hunter 2016-07-21 7:18 ` Adrian Hunter 2016-07-12 7:46 ` [PATCH V2 05/15] mmc: sdhci-esdhci-imx: disable DLL delay line settings explicitly Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 06/15] mmc: sdhci-esdhc-imx: support setting tuning start point Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 07/15] doc: dt: fsl-imx-esdhc: add set tuning start point binding Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 13:59 ` Andrew Lunn 2016-07-12 13:59 ` Andrew Lunn 2016-07-13 9:10 ` Dong Aisheng 2016-07-13 9:10 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 08/15] mmc: sdhci: add standard hw auto retuning support Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 7:46 ` Adrian Hunter 2016-07-21 7:46 ` Adrian Hunter 2016-07-12 7:46 ` [PATCH V2 09/15] mmc: sdhci-esdhc-imx: enable hw auto retuning for STD_TUNING Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 10/15] mmc: sdhci-esdhc-imx: enable hw auto retuning for MAN_TUNING Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-08-15 14:59 ` [V2, " Gary Bisson 2016-08-15 14:59 ` Gary Bisson 2016-08-16 10:18 ` Dong Aisheng [this message] 2016-08-16 10:18 ` Dong Aisheng 2016-08-16 12:44 ` Gary Bisson 2016-08-16 12:44 ` Gary Bisson 2016-08-17 14:31 ` Dong Aisheng 2016-08-17 14:31 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 11/15] mmc: sdhci-esdhc-imx: fix strobe DLL lock wrong clock issue Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 12/15] mmc: sdhci-esdhc-imx: factor out hw related initialization into function Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 8:11 ` Adrian Hunter 2016-07-21 8:11 ` Adrian Hunter 2016-07-12 7:46 ` [PATCH V2 13/15] mmc: sdhci-esdhc-imx: move tuning static configuration into hwinit function Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-12 7:46 ` [PATCH V2 14/15] mmc: sdhci-esdhci-imx: re-initialize hw state after resume Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 8:11 ` Adrian Hunter 2016-07-21 8:11 ` Adrian Hunter 2016-07-12 7:46 ` [PATCH V2 15/15] mmc: sdhci-esdhc-imx: clear tuning bits during driver probe Dong Aisheng 2016-07-12 7:46 ` Dong Aisheng 2016-07-21 8:33 ` Adrian Hunter 2016-07-21 8:33 ` Adrian Hunter 2016-07-12 13:54 ` [PATCH V2 00/15] a few sdhci/imx clean up and fix patches Andrew Lunn 2016-07-12 13:54 ` Andrew Lunn 2016-07-13 8:55 ` Dong Aisheng 2016-07-13 8:55 ` Dong Aisheng 2016-07-13 13:40 ` Andrew Lunn 2016-07-13 13:40 ` Andrew Lunn 2016-07-21 9:32 ` Adrian Hunter 2016-07-21 9:32 ` Adrian Hunter 2016-07-21 10:23 ` Dong Aisheng 2016-07-21 10:23 ` Dong Aisheng 2016-07-23 9:37 ` Ulf Hansson 2016-07-23 9:37 ` Ulf Hansson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160816101818.GD9211@shlinux2 \ --to=dongas86@gmail.com \ --cc=adrian.hunter@intel.com \ --cc=aisheng.dong@nxp.com \ --cc=chris@printf.net \ --cc=gary.bisson@boundarydevices.com \ --cc=haibo.chen@nxp.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-mmc@vger.kernel.org \ --cc=shawnguo@kernel.org \ --cc=ulf.hansson@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.