* [U-Boot] [RFC 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata
@ 2018-03-18 14:25 Heinrich Schuchardt
2018-03-18 16:12 ` [U-Boot] [U-Boot, RFC, " Philipp Tomsich
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Heinrich Schuchardt @ 2018-03-18 14:25 UTC (permalink / raw)
To: u-boot
The error checking should also catch the case that no range has beeen
defined.
syscon_get_first_range() returns NULL if no range is defined.
cf. rk3399_mipi.c
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
I do not possess the hardware for testing. But the coding is not consistent
with the coding used in rk3399_mipi.c.
Why did we duplicate the coding instead of using a common c file?
---
drivers/video/rockchip/rk3288_mipi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/rockchip/rk3288_mipi.c b/drivers/video/rockchip/rk3288_mipi.c
index 953b47fb8c..27d125f77e 100644
--- a/drivers/video/rockchip/rk3288_mipi.c
+++ b/drivers/video/rockchip/rk3288_mipi.c
@@ -136,7 +136,7 @@ static int rk_mipi_ofdata_to_platdata(struct udevice *dev)
struct rk_mipi_priv *priv = dev_get_priv(dev);
priv->grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
- if (IS_ERR(priv->grf)) {
+ if (!priv->grf || IS_ERR(priv->grf)) {
debug("%s: Get syscon grf failed (ret=%p)\n",
__func__, priv->grf);
return -ENXIO;
--
2.16.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [U-Boot, RFC, 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata
2018-03-18 14:25 [U-Boot] [RFC 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata Heinrich Schuchardt
@ 2018-03-18 16:12 ` Philipp Tomsich
2018-03-18 16:13 ` Philipp Tomsich
2018-03-18 23:48 ` Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Philipp Tomsich @ 2018-03-18 16:12 UTC (permalink / raw)
To: u-boot
> The error checking should also catch the case that no range has beeen
> defined.
> syscon_get_first_range() returns NULL if no range is defined.
> cf. rk3399_mipi.c
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> I do not possess the hardware for testing. But the coding is not consistent
> with the coding used in rk3399_mipi.c.
>
> Why did we duplicate the coding instead of using a common c file?
> ---
> drivers/video/rockchip/rk3288_mipi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [U-Boot, RFC, 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata
2018-03-18 14:25 [U-Boot] [RFC 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata Heinrich Schuchardt
2018-03-18 16:12 ` [U-Boot] [U-Boot, RFC, " Philipp Tomsich
@ 2018-03-18 16:13 ` Philipp Tomsich
2018-03-18 23:48 ` Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Philipp Tomsich @ 2018-03-18 16:13 UTC (permalink / raw)
To: u-boot
> The error checking should also catch the case that no range has beeen
> defined.
> syscon_get_first_range() returns NULL if no range is defined.
> cf. rk3399_mipi.c
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> I do not possess the hardware for testing. But the coding is not consistent
> with the coding used in rk3399_mipi.c.
>
> Why did we duplicate the coding instead of using a common c file?
> ---
> drivers/video/rockchip/rk3288_mipi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [U-Boot, RFC, 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata
2018-03-18 14:25 [U-Boot] [RFC 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata Heinrich Schuchardt
2018-03-18 16:12 ` [U-Boot] [U-Boot, RFC, " Philipp Tomsich
2018-03-18 16:13 ` Philipp Tomsich
@ 2018-03-18 23:48 ` Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Philipp Tomsich @ 2018-03-18 23:48 UTC (permalink / raw)
To: u-boot
> The error checking should also catch the case that no range has beeen
> defined.
> syscon_get_first_range() returns NULL if no range is defined.
> cf. rk3399_mipi.c
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
> I do not possess the hardware for testing. But the coding is not consistent
> with the coding used in rk3399_mipi.c.
>
> Why did we duplicate the coding instead of using a common c file?
> ---
> drivers/video/rockchip/rk3288_mipi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Applied to u-boot-rockchip, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-18 23:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-18 14:25 [U-Boot] [RFC 1/1] video: rk3288_mipi: check in rk_mipi_ofdata_to_platdata Heinrich Schuchardt
2018-03-18 16:12 ` [U-Boot] [U-Boot, RFC, " Philipp Tomsich
2018-03-18 16:13 ` Philipp Tomsich
2018-03-18 23:48 ` Philipp Tomsich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.