Doug, On Mon, Jul 14, 2014 at 10:30 PM, 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) > > 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. Sure, will repost both pit and pi by reordering the pin as you suggested. -- 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