From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christophe Ricard <christophe.ricard@gmail.com>
Date: Mon, 24 Aug 2015 22:20:45 +0200
Subject: [U-Boot] [PATCH v2 12/28] tpm: tpm_tis_i2c: Simplify init code
In-Reply-To: <1440289904-31280-13-git-send-email-sjg@chromium.org>
References: <1440289904-31280-1-git-send-email-sjg@chromium.org>
 <1440289904-31280-13-git-send-email-sjg@chromium.org>
Message-ID: <55DB7C9D.1020200@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

This one looks good to me.

Acked-by: Christophe Ricard<christophe-h.ricard@st.com>

Best Regards
Christophe

On 23/08/2015 02:31, Simon Glass wrote:
> Move all the init and uninit code into one place.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> Changes in v2: None
>
>   drivers/tpm/tpm_tis_i2c.c | 33 ++++++++++-----------------------
>   1 file changed, 10 insertions(+), 23 deletions(-)
>
> diff --git a/drivers/tpm/tpm_tis_i2c.c b/drivers/tpm/tpm_tis_i2c.c
> index 39652a9..9dd40dd 100644
> --- a/drivers/tpm/tpm_tis_i2c.c
> +++ b/drivers/tpm/tpm_tis_i2c.c
> @@ -608,27 +608,6 @@ out:
>   	return rc;
>   }
>   
> -static int tpm_open_dev(struct udevice *dev)
> -{
> -	int rc;
> -
> -	debug("%s: start\n", __func__);
> -	if (g_chip.is_open)
> -		return -EBUSY;
> -	rc = tpm_tis_i2c_init(dev);
> -	if (rc < 0)
> -		g_chip.is_open = 0;
> -	return rc;
> -}
> -
> -static void tpm_close(void)
> -{
> -	if (g_chip.is_open) {
> -		release_locality(&g_chip, g_chip.locality, 1);
> -		g_chip.is_open = 0;
> -	}
> -}
> -
>   /**
>    * Decode TPM configuration.
>    *
> @@ -712,7 +691,12 @@ int tis_open(void)
>   	if (!g_chip.inited)
>   		return -1;
>   
> -	rc = tpm_open_dev(g_chip.dev);
> +	debug("%s: start\n", __func__);
> +	if (g_chip.is_open)
> +		return -EBUSY;
> +	rc = tpm_tis_i2c_init(g_chip.dev);
> +	if (rc < 0)
> +		g_chip.is_open = 0;
>   
>   	return rc;
>   }
> @@ -722,7 +706,10 @@ int tis_close(void)
>   	if (!g_chip.inited)
>   		return -1;
>   
> -	tpm_close();
> +	if (g_chip.is_open) {
> +		release_locality(&g_chip, g_chip.locality, 1);
> +		g_chip.is_open = 0;
> +	}
>   
>   	return 0;
>   }