linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mfd: vexpress-sysreg: initialize "site" variable
@ 2014-02-13 16:39 Semen Protsenko
  2014-02-14  9:26 ` Lee Jones
  0 siblings, 1 reply; 2+ messages in thread
From: Semen Protsenko @ 2014-02-13 16:39 UTC (permalink / raw)
  To: linux-arm-kernel

"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(-)

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];
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH] mfd: vexpress-sysreg: initialize "site" variable
  2014-02-13 16:39 [PATCH] mfd: vexpress-sysreg: initialize "site" variable Semen Protsenko
@ 2014-02-14  9:26 ` Lee Jones
  0 siblings, 0 replies; 2+ messages in thread
From: Lee Jones @ 2014-02-14  9:26 UTC (permalink / raw)
  To: linux-arm-kernel

> "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.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-02-14  9:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-13 16:39 [PATCH] mfd: vexpress-sysreg: initialize "site" variable Semen Protsenko
2014-02-14  9:26 ` Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).