04.10.2021 12:09, Krzysztof Kozlowski пишет: >> static int emc_setup_hw(struct tegra_emc *emc) >> { >> + u32 emc_cfg, emc_dbg, emc_fbio, emc_adr_cfg; >> u32 intmask = EMC_REFRESH_OVERFLOW_INT; >> - u32 emc_cfg, emc_dbg, emc_fbio; >> + static bool print_sdram_info_once; > How about moving print_sdram_info_once to emc_read_lpddr_sdram_info()? > Less code here. > The SDRAM info is printed out for each attached SDRAM chip. There are two prints if two chips are attached to memory controller. Hence this print_once flag should cover both prints.