On 26.09.2017 14:19, Thierry Reding wrote: > On Wed, Jul 05, 2017 at 07:29:45PM +0300, Dmitry Osipenko wrote: >> GART looses it's state only in case of a deepest suspend level. Let's not >> waste memory if machine doesn't support that suspend level. >> >> Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx> >> --- >> drivers/iommu/tegra-gart.c | 36 +++++++++++++++++++++++++----------- >> 1 file changed, 25 insertions(+), 11 deletions(-) > > I'm not sure about this. The savedata region uses 32 KiB of memory, > which is really not a whole lot. > > In addition, the suspend mode can be set in different places during the > boot process, some of which fairly late. I'm concerned that this boot > order dependency is going to come back to bite us eventually. > Indeed, suspend mode is set after drivers been probed, my bad. On the other hand LP0 support haven't been implemented for years, some of drivers do not implement suspend/resume. So maybe it's worth to give up on LP0 entirely... -- Dmitry -- 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
