On Wed, Dec 20, 2017 at 10:06:13AM +0000, Jon Hunter wrote: > > On 16/11/17 14:29, Peter De Schrijver wrote: > > Apply the memory built-in self test work around when ungating certain > > Tegra210 power domains. > > > > Signed-off-by: Peter De Schrijver <pdeschrijver@xxxxxxxxxx> > > --- > > drivers/soc/tegra/pmc.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c > > index 0453ff6..4c0582d 100644 > > --- a/drivers/soc/tegra/pmc.c > > +++ b/drivers/soc/tegra/pmc.c > > @@ -142,6 +142,7 @@ struct tegra_pmc_soc { > > > > bool has_tsense_reset; > > bool has_gpu_clamps; > > + bool needs_mbist_war; > > > > const struct tegra_io_pad_soc *io_pads; > > unsigned int num_io_pads; > > @@ -411,6 +412,9 @@ static int tegra_powergate_power_up(struct tegra_powergate *pg, > > > > usleep_range(10, 20); > > > > + if (pg->pmc->soc->needs_mbist_war) > > + tegra210_handle_mbist_war(pg->id); > > + > > if (disable_clocks) > > tegra_powergate_disable_clocks(pg); > > > > @@ -1712,6 +1716,7 @@ static int tegra_pmc_resume(struct device *dev) > > .cpu_powergates = tegra210_cpu_powergates, > > .has_tsense_reset = true, > > .has_gpu_clamps = true, > > + .needs_mbist_war = true, > > .num_io_pads = ARRAY_SIZE(tegra210_io_pads), > > .io_pads = tegra210_io_pads, > > }; > > After applying this series, tegra210-smaug no longer boots (no output > from the console). Reverting this patch allows it to boot again. So > something here is not working ... > Annoyingly I don't have smaug, only jetson TX1.. Did you try earlycon? Peter. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html