[U-Boot] [PATCH v3 16/17] board: toradex: use get_nand_dev_by_index()

Marcel Ziswiler marcel.ziswiler at toradex.com
Sat Feb 11 01:20:57 UTC 2017


On Fri, 2017-02-10 at 14:23 -0600, Grygorii Strashko wrote:
> As part of preparation for nand DM conversion the new API has been
> introduced to remove direct access to nand_info array. So, use it
> here
> instead of accessing to nand_info array directly.
> 
> Cc: Lucas Stach <dev at lynxeye.de>
> Cc: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> Signed-off-by: Grygorii Strashko <grygorii.strashko at ti.com>

Looks reasonable to me.

Reviewed-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>

> ---
>  board/toradex/colibri_t20/colibri_t20.c |  2 +-
>  board/toradex/common/tdx-cfg-block.c    | 12 ++++++++----
>  2 files changed, 9 insertions(+), 5 deletions(-)
> 
> diff --git a/board/toradex/colibri_t20/colibri_t20.c
> b/board/toradex/colibri_t20/colibri_t20.c
> index 7d574fb..71b8fd3 100644
> --- a/board/toradex/colibri_t20/colibri_t20.c
> +++ b/board/toradex/colibri_t20/colibri_t20.c
> @@ -69,7 +69,7 @@ int checkboard(void)
>  {
>  	printf("Model: Toradex Colibri T20 %dMB V%s\n",
>  	       (gd->ram_size == 0x10000000) ? 256 : 512,
> -	       (nand_info[0]->erasesize >> 10 == 512) ?
> +	       (get_nand_dev_by_index(0)->erasesize >> 10 == 512) ?
>  	       ((gd->ram_size == 0x10000000) ? "1.1B" : "1.1C") :
> "1.2A");
>  
>  	return 0;
> diff --git a/board/toradex/common/tdx-cfg-block.c
> b/board/toradex/common/tdx-cfg-block.c
> index 0014ce8..7722b92 100644
> --- a/board/toradex/common/tdx-cfg-block.c
> +++ b/board/toradex/common/tdx-cfg-block.c
> @@ -153,8 +153,10 @@ static int read_tdx_cfg_block_from_nand(unsigned
> char *config_block)
>  	size_t size = TDX_CFG_BLOCK_MAX_SIZE;
>  
>  	/* Read production parameter config block from NAND page */
> -	return nand_read_skip_bad(nand_info[0],
> CONFIG_TDX_CFG_BLOCK_OFFSET,
> -			 &size, NULL, TDX_CFG_BLOCK_MAX_SIZE,
> config_block);
> +	return nand_read_skip_bad(get_nand_dev_by_index(0),
> +				  CONFIG_TDX_CFG_BLOCK_OFFSET,
> +				  &size, NULL,
> TDX_CFG_BLOCK_MAX_SIZE,
> +				  config_block);
>  }
>  
>  static int write_tdx_cfg_block_to_nand(unsigned char *config_block)
> @@ -162,7 +164,8 @@ static int write_tdx_cfg_block_to_nand(unsigned
> char *config_block)
>  	size_t size = TDX_CFG_BLOCK_MAX_SIZE;
>  
>  	/* Write production parameter config block to NAND page */
> -	return nand_write_skip_bad(nand_info[0],
> CONFIG_TDX_CFG_BLOCK_OFFSET,
> +	return nand_write_skip_bad(get_nand_dev_by_index(0),
> +				   CONFIG_TDX_CFG_BLOCK_OFFSET,
>  				   &size, NULL,
> TDX_CFG_BLOCK_MAX_SIZE,
>  				   config_block, WITH_WR_VERIFY);
>  }
> @@ -425,7 +428,8 @@ static int do_cfgblock_create(cmd_tbl_t *cmdtp,
> int flag, int argc,
>  		 * empty (config block invalid...)
>  		 */
>  		printf("NAND erase block %d need to be erased before
> creating a Toradex config block\n",
> -		       CONFIG_TDX_CFG_BLOCK_OFFSET / nand_info[0]-
> >erasesize);
> +		       CONFIG_TDX_CFG_BLOCK_OFFSET /
> +		       get_nand_dev_by_index(0)->erasesize);
>  		goto out;
>  #elif defined(CONFIG_TDX_CFG_BLOCK_IS_IN_NOR)
>  		/*


More information about the U-Boot mailing list