| Message ID | 1598653517-13658-2-git-send-email-skomatineni@nvidia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Fix timeout clock used by hardware data timeout | expand |
On Fri, Aug 28, 2020 at 03:25:11PM -0700, Sowjanya Komatineni wrote: >commit b5a84ecf025a ("mmc: tegra: Add Tegra210 support") What does this line above represent? >SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK is set for Tegra210 from the >beginning of Tegra210 support in the driver. > >Tegra210 SDMMC hardware by default uses timeout clock (TMCLK) >instead of SDCLK and this quirk should not be set. > >So, this patch remove this quirk for Tegra210. > >Fixes: b5a84ecf025a ("mmc: tegra: Add Tegra210 support") >Cc: stable <stable@vger.kernel.org> # 4.19 >Tested-by: Jon Hunter <jonathanh@nvidia.com> >Reviewed-by: Jon Hunter <jonathanh@nvidia.com> >Acked-by: Adrian Hunter <adrian.hunter@intel.com> >Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com>
On 8/28/20 4:15 PM, Sasha Levin wrote: > On Fri, Aug 28, 2020 at 03:25:11PM -0700, Sowjanya Komatineni wrote: >> commit b5a84ecf025a ("mmc: tegra: Add Tegra210 support") > > What does this line above represent? > SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK is set incorrectly in above commit when Tegra210 support was added. >> SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK is set for Tegra210 from the >> beginning of Tegra210 support in the driver. >> >> Tegra210 SDMMC hardware by default uses timeout clock (TMCLK) >> instead of SDCLK and this quirk should not be set. >> >> So, this patch remove this quirk for Tegra210. >> >> Fixes: b5a84ecf025a ("mmc: tegra: Add Tegra210 support") >> Cc: stable <stable@vger.kernel.org> # 4.19 >> Tested-by: Jon Hunter <jonathanh@nvidia.com> >> Reviewed-by: Jon Hunter <jonathanh@nvidia.com> >> Acked-by: Adrian Hunter <adrian.hunter@intel.com> >> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com> >
On Fri, Aug 28, 2020 at 04:23:48PM -0700, Sowjanya Komatineni wrote: > > On 8/28/20 4:15 PM, Sasha Levin wrote: > > On Fri, Aug 28, 2020 at 03:25:11PM -0700, Sowjanya Komatineni wrote: > > > commit b5a84ecf025a ("mmc: tegra: Add Tegra210 support") > > > > What does this line above represent? > > > SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK is set incorrectly in above commit > > when Tegra210 support was added. Odd, that's a new format to send to us to apply :) Can you please provide the git commit id of the original commit in Linus's tree, as per the documentation, so we know what this is, and can document that? Look at all of the commits in the stable trees for examples of how to do this. Can y ou fix that up and resend this whole series? thanks, greg k-h
diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c index 27bdf6d..731956e 100644 --- a/drivers/mmc/host/sdhci-tegra.c +++ b/drivers/mmc/host/sdhci-tegra.c @@ -423,7 +423,6 @@ static const struct sdhci_tegra_soc_data soc_data_tegra124 = { static const struct sdhci_pltfm_data sdhci_tegra210_pdata = { .quirks = SDHCI_QUIRK_BROKEN_TIMEOUT_VAL | - SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK | SDHCI_QUIRK_SINGLE_POWER_WRITE | SDHCI_QUIRK_NO_HISPD_BIT | SDHCI_QUIRK_BROKEN_ADMA_ZEROLEN_DESC |
commit b5a84ecf025a ("mmc: tegra: Add Tegra210 support") SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK is set for Tegra210 from the beginning of Tegra210 support in the driver. Tegra210 SDMMC hardware by default uses timeout clock (TMCLK) instead of SDCLK and this quirk should not be set. So, this patch remove this quirk for Tegra210. Fixes: b5a84ecf025a ("mmc: tegra: Add Tegra210 support") Cc: stable <stable@vger.kernel.org> # 4.19 Tested-by: Jon Hunter <jonathanh@nvidia.com> Reviewed-by: Jon Hunter <jonathanh@nvidia.com> Acked-by: Adrian Hunter <adrian.hunter@intel.com> Signed-off-by: Sowjanya Komatineni <skomatineni@nvidia.com> --- drivers/mmc/host/sdhci-tegra.c | 1 - 1 file changed, 1 deletion(-)