Vikas, On Mon, Jul 14, 2014 at 10:00 AM, Doug Anderson <dianders@xxxxxxxxxxxx> wrote: > Vikas, > > On Sun, Jul 13, 2014 at 11:33 PM, Vikas Sajjan <vikas.sajjan@xxxxxxxxxxx> wrote: >> From: Doug Anderson <dianders@xxxxxxxxxxxx> >> >> The mask-tpm-reset GPIO is used by the kernel to prevent the TPM from >> being reset across sleep/wake. If we don't set it to anything then >> the TPM will be reset. U-Boot will detect this as invalid >> and will reset the system on resume time. This GPIO can always be low >> and not hurt anything. It will get pulled back high again during a >> normal warm reset when it will default back to an input. >> >> To properly preserve the TPM state across suspend/resume and to make >> the chrome U-Boot happy, properly set the GPIO to mask the >> reset to the TPM. >> >> Signed-off-by: Doug Anderson <dianders@xxxxxxxxxxxx> >> Signed-off-by: Vikas Sajjan <vikas.sajjan@xxxxxxxxxxx> >> --- >> changes since v1: >> - removed fixed regulator and used hogs instead. >> >> arch/arm/boot/dts/exynos5420-peach-pit.dts | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) > > This looks reasonable to me except for two things: > > 1. You need to apply this to pit and pi (not just pi) Sorry, just realized that you had two patches and they weren't just revisions (I got confused when one was v1 and one was v2). I would have just put them in the same patch, but two patches is OK. > 2. The proper sort ordering for elements within a pin control group is > by pin number, so this pin should be _after_ gpx0-2, not before. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |