* [PATCH v3] mmc: tegra: fix SDR50 tuning override
@ 2020-01-07 9:47 Michał Mirosław
2020-01-16 14:39 ` Ulf Hansson
0 siblings, 1 reply; 5+ messages in thread
From: Michał Mirosław @ 2020-01-07 9:47 UTC (permalink / raw)
To: Adrian Hunter, Ulf Hansson, Thierry Reding, Jonathan Hunter, Lucas Stach
Cc: linux-mmc, linux-tegra, linux-kernel
Commit 7ad2ed1dfcbe inadvertently mixed up a quirk flag's name and
broke SDR50 tuning override. Use correct NVQUIRK_ name.
Fixes: 7ad2ed1dfcbe ("mmc: tegra: enable UHS-I modes")
Cc: <stable@vger.kernel.org> # 4f6aa3264af4: mmc: tegra: Only advertise UHS modes if IO regulator is present
Cc: <stable@vger.kernel.org>
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
Reviewed-by: Thierry Reding <treding@nvidia.com>
Tested-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
---
v3: added Thierry's signs that were missing in v2
v2: converted 'Depends-On' tag to proper 'Cc: stable' lines
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
---
drivers/mmc/host/sdhci-tegra.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c
index 7bc950520fd9..403ac44a7378 100644
--- a/drivers/mmc/host/sdhci-tegra.c
+++ b/drivers/mmc/host/sdhci-tegra.c
@@ -386,7 +386,7 @@ static void tegra_sdhci_reset(struct sdhci_host *host, u8 mask)
misc_ctrl |= SDHCI_MISC_CTRL_ENABLE_DDR50;
if (soc_data->nvquirks & NVQUIRK_ENABLE_SDR104)
misc_ctrl |= SDHCI_MISC_CTRL_ENABLE_SDR104;
- if (soc_data->nvquirks & SDHCI_MISC_CTRL_ENABLE_SDR50)
+ if (soc_data->nvquirks & NVQUIRK_ENABLE_SDR50)
clk_ctrl |= SDHCI_CLOCK_CTRL_SDR50_TUNING_OVERRIDE;
}
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v3] mmc: tegra: fix SDR50 tuning override
2020-01-07 9:47 [PATCH v3] mmc: tegra: fix SDR50 tuning override Michał Mirosław
@ 2020-01-16 14:39 ` Ulf Hansson
2020-01-17 14:11 ` Michał Mirosław
0 siblings, 1 reply; 5+ messages in thread
From: Ulf Hansson @ 2020-01-16 14:39 UTC (permalink / raw)
To: Michał Mirosław
Cc: Adrian Hunter, Thierry Reding, Jonathan Hunter, Lucas Stach,
linux-mmc, linux-tegra, Linux Kernel Mailing List
On Tue, 7 Jan 2020 at 10:47, Michał Mirosław <mirq-linux@rere.qmqm.pl> wrote:
>
> Commit 7ad2ed1dfcbe inadvertently mixed up a quirk flag's name and
> broke SDR50 tuning override. Use correct NVQUIRK_ name.
>
> Fixes: 7ad2ed1dfcbe ("mmc: tegra: enable UHS-I modes")
> Cc: <stable@vger.kernel.org> # 4f6aa3264af4: mmc: tegra: Only advertise UHS modes if IO regulator is present
I am dropping this tag, simply because I don't understand what it should tell.
Instead, please monitor responses from stable maintainers, to see if
there is failure to apply this for stable and then send a manual
backport.
> Cc: <stable@vger.kernel.org>
> Acked-by: Adrian Hunter <adrian.hunter@intel.com>
> Reviewed-by: Thierry Reding <treding@nvidia.com>
> Tested-by: Thierry Reding <treding@nvidia.com>
> Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Applied for fixes, thanks!
Kind regards
Uffe
>
> ---
> v3: added Thierry's signs that were missing in v2
> v2: converted 'Depends-On' tag to proper 'Cc: stable' lines
>
> Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
> ---
> drivers/mmc/host/sdhci-tegra.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c
> index 7bc950520fd9..403ac44a7378 100644
> --- a/drivers/mmc/host/sdhci-tegra.c
> +++ b/drivers/mmc/host/sdhci-tegra.c
> @@ -386,7 +386,7 @@ static void tegra_sdhci_reset(struct sdhci_host *host, u8 mask)
> misc_ctrl |= SDHCI_MISC_CTRL_ENABLE_DDR50;
> if (soc_data->nvquirks & NVQUIRK_ENABLE_SDR104)
> misc_ctrl |= SDHCI_MISC_CTRL_ENABLE_SDR104;
> - if (soc_data->nvquirks & SDHCI_MISC_CTRL_ENABLE_SDR50)
> + if (soc_data->nvquirks & NVQUIRK_ENABLE_SDR50)
> clk_ctrl |= SDHCI_CLOCK_CTRL_SDR50_TUNING_OVERRIDE;
> }
>
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v3] mmc: tegra: fix SDR50 tuning override
2020-01-16 14:39 ` Ulf Hansson
@ 2020-01-17 14:11 ` Michał Mirosław
2020-01-17 15:41 ` Dmitry Osipenko
2020-01-20 9:43 ` Ulf Hansson
0 siblings, 2 replies; 5+ messages in thread
From: Michał Mirosław @ 2020-01-17 14:11 UTC (permalink / raw)
To: Ulf Hansson
Cc: Adrian Hunter, Thierry Reding, Jonathan Hunter, Lucas Stach,
linux-mmc, linux-tegra, Linux Kernel Mailing List
On Thu, Jan 16, 2020 at 03:39:54PM +0100, Ulf Hansson wrote:
> On Tue, 7 Jan 2020 at 10:47, Michał Mirosław <mirq-linux@rere.qmqm.pl> wrote:
> >
> > Commit 7ad2ed1dfcbe inadvertently mixed up a quirk flag's name and
> > broke SDR50 tuning override. Use correct NVQUIRK_ name.
> >
> > Fixes: 7ad2ed1dfcbe ("mmc: tegra: enable UHS-I modes")
> > Cc: <stable@vger.kernel.org> # 4f6aa3264af4: mmc: tegra: Only advertise UHS modes if IO regulator is present
>
> I am dropping this tag, simply because I don't understand what it should tell.
It tells the maintainer that he needs to cherry-pick this commit if
its not in particular stable version already. I guess this is only
for v4.4, as v4.9+ already have it, and v3.16 does not include the
Fixed commit.
Best Regards,
Michał Mirosław
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v3] mmc: tegra: fix SDR50 tuning override
2020-01-17 14:11 ` Michał Mirosław
@ 2020-01-17 15:41 ` Dmitry Osipenko
2020-01-20 9:43 ` Ulf Hansson
1 sibling, 0 replies; 5+ messages in thread
From: Dmitry Osipenko @ 2020-01-17 15:41 UTC (permalink / raw)
To: Michał Mirosław, Ulf Hansson
Cc: Adrian Hunter, Thierry Reding, Jonathan Hunter, Lucas Stach,
linux-mmc, linux-tegra, Linux Kernel Mailing List
17.01.2020 17:11, Michał Mirosław пишет:
> On Thu, Jan 16, 2020 at 03:39:54PM +0100, Ulf Hansson wrote:
>> On Tue, 7 Jan 2020 at 10:47, Michał Mirosław <mirq-linux@rere.qmqm.pl> wrote:
>>>
>>> Commit 7ad2ed1dfcbe inadvertently mixed up a quirk flag's name and
>>> broke SDR50 tuning override. Use correct NVQUIRK_ name.
>>>
>>> Fixes: 7ad2ed1dfcbe ("mmc: tegra: enable UHS-I modes")
>>> Cc: <stable@vger.kernel.org> # 4f6aa3264af4: mmc: tegra: Only advertise UHS modes if IO regulator is present
>>
>> I am dropping this tag, simply because I don't understand what it should tell.
>
> It tells the maintainer that he needs to cherry-pick this commit if
> its not in particular stable version already. I guess this is only
> for v4.4, as v4.9+ already have it, and v3.16 does not include the
> Fixed commit.
I guess it could be: Cc: <stable@vger.kernel.org> # v4.4+
And then you could email Greg KH or Sasha Levin, asking to pick up the
additional commit to the stable kernel, or they will ask you by
themselves about why the patch isn't applying and how to handle it.
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v3] mmc: tegra: fix SDR50 tuning override
2020-01-17 14:11 ` Michał Mirosław
2020-01-17 15:41 ` Dmitry Osipenko
@ 2020-01-20 9:43 ` Ulf Hansson
1 sibling, 0 replies; 5+ messages in thread
From: Ulf Hansson @ 2020-01-20 9:43 UTC (permalink / raw)
To: Michał Mirosław
Cc: Adrian Hunter, Thierry Reding, Jonathan Hunter, Lucas Stach,
linux-mmc, linux-tegra, Linux Kernel Mailing List
On Fri, 17 Jan 2020 at 15:11, Michał Mirosław <mirq-linux@rere.qmqm.pl> wrote:
>
> On Thu, Jan 16, 2020 at 03:39:54PM +0100, Ulf Hansson wrote:
> > On Tue, 7 Jan 2020 at 10:47, Michał Mirosław <mirq-linux@rere.qmqm.pl> wrote:
> > >
> > > Commit 7ad2ed1dfcbe inadvertently mixed up a quirk flag's name and
> > > broke SDR50 tuning override. Use correct NVQUIRK_ name.
> > >
> > > Fixes: 7ad2ed1dfcbe ("mmc: tegra: enable UHS-I modes")
> > > Cc: <stable@vger.kernel.org> # 4f6aa3264af4: mmc: tegra: Only advertise UHS modes if IO regulator is present
> >
> > I am dropping this tag, simply because I don't understand what it should tell.
>
> It tells the maintainer that he needs to cherry-pick this commit if
> its not in particular stable version already. I guess this is only
> for v4.4, as v4.9+ already have it, and v3.16 does not include the
> Fixed commit.
Alright, I updated myself on stable rules and found this, thanks!
So I have amended the patch to add back the tag, let's give this a try.
Kind regards
Uffe
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-01-20 9:44 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-07 9:47 [PATCH v3] mmc: tegra: fix SDR50 tuning override Michał Mirosław
2020-01-16 14:39 ` Ulf Hansson
2020-01-17 14:11 ` Michał Mirosław
2020-01-17 15:41 ` Dmitry Osipenko
2020-01-20 9:43 ` Ulf Hansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).