Re: [PATCH v4 1/2] ARM: socfpga: support suspend to ram

[atull <atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>]

On Tue, 26 May 2015, Pavel Machek wrote:

> Hi!
> 
> > Thanks for the feedback.
> > 
> > Yes I agree.
> > 
> > One possibility is that I could add a suspend function to EDAC that
> > always fails with a helpful message like "Suspend with EDAC not
> > supported."  If someone wants to suspend, they'll have to take the
> > EDAC out of the device tree so its probe will bomb out.  I'm not
> > sure how normal it is to have suspend functions in the kernel
> > that always fail, though.
> 
> "Interesting" solution, but better than config exclusion, I'd say.
> 
> > > > +	/*
> > > > +	 * This code assumes that the bootloader has already
> > > > +	 * properly configured the sdram controller:
> > > 
> > > Which bootloaders can do that? 
> > 
> > Whatever bootloader is configuring the sdram controller for the
> > board.
> 
> Ok, so direct question: will mainline u-boot 2015.04 configure the
> controller correctly?

Should be in 2015.07

Alan Tull

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html