Commit 205e1776 authored by AngeloGioacchino Del Regno's avatar AngeloGioacchino Del Regno Committed by Krzysztof Kozlowski
Browse files

memory: mtk-smi: Use ARRAY_SIZE to define MTK_SMI_CLK_NR_MAX 



This definition is tied to the number of SMI common clocks (the array
mtk_smi_common_clks): improve the definition by using the ARRAY_SIZE
macro instead. That will also reduce room for mistakes when updating
the aforementioned array in the future.

Signed-off-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: default avatarYong Wu <yong.wu@mediatek.com>
Link: https://lore.kernel.org/r/20211015151557.510726-1-angelogioacchino.delregno@collabora.com


Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
parent e783362e

drivers/memory/mtk-smi.c

+1 −2
Original line number Diff line number Diff line
@@ -94,8 +94,6 @@ enum mtk_smi_type { 
	MTK_SMI_GEN2_SUB_COMM,	/* gen2 smi sub common */

};



#define MTK_SMI_CLK_NR_MAX			4



/* larbs: Require apb/smi clocks while gals is optional. */

static const char * const mtk_smi_larb_clks[] = {"apb", "smi", "gals"};

#define MTK_SMI_LARB_REQ_CLK_NR		2
@@ -106,6 +104,7 @@ static const char * const mtk_smi_larb_clks[] = {"apb", "smi", "gals"}; 
 * sub common: Require apb/smi/gals0 clocks in has_gals case. Otherwise, only apb/smi are required.

 */

static const char * const mtk_smi_common_clks[] = {"apb", "smi", "gals0", "gals1"};

#define MTK_SMI_CLK_NR_MAX		ARRAY_SIZE(mtk_smi_common_clks)

#define MTK_SMI_COM_REQ_CLK_NR		2

#define MTK_SMI_COM_GALS_REQ_CLK_NR	MTK_SMI_CLK_NR_MAX

#define MTK_SMI_SUB_COM_GALS_REQ_CLK_NR 3