diff mbox

mfd: vexpress-sysreg: initialize "site" variable

Message ID 1392309594-28229-1-git-send-email-semen.protsenko@linaro.org
State Accepted
Commit 63f5d2e8b227d54d6e0a750a4b9670a899fa1a05
Headers show

Commit Message

Sam Protsenko Feb. 13, 2014, 4:39 p.m. UTC
"site" variable should be initialized with 0 so that
when "site" property doesn't exist in DTB it can be handled correctly.

"0" value means board site number is motherboard (see
Documentation/devicetree/bindings/arm/vexpress.txt for details).

Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org>
---
 drivers/mfd/vexpress-sysreg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Lee Jones Feb. 14, 2014, 9:26 a.m. UTC | #1
> "site" variable should be initialized with 0 so that
> when "site" property doesn't exist in DTB it can be handled correctly.
> 
> "0" value means board site number is motherboard (see
> Documentation/devicetree/bindings/arm/vexpress.txt for details).
> 
> Signed-off-by: Semen Protsenko <semen.protsenko@linaro.org>
> ---
>  drivers/mfd/vexpress-sysreg.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.
diff mbox

Patch

diff --git a/drivers/mfd/vexpress-sysreg.c b/drivers/mfd/vexpress-sysreg.c
index 981bef4..35281e8 100644
--- a/drivers/mfd/vexpress-sysreg.c
+++ b/drivers/mfd/vexpress-sysreg.c
@@ -168,7 +168,7 @@  static void *vexpress_sysreg_config_func_get(struct device *dev,
 		struct device_node *node)
 {
 	struct vexpress_sysreg_config_func *config_func;
-	u32 site;
+	u32 site = 0;
 	u32 position = 0;
 	u32 dcc = 0;
 	u32 func_device[2];