[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