Message ID | 20100423171244.GB2140@oksana.dev.rtsoft.ru |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: Anton Vorontsov <avorontsov@mvista.com> Date: Fri, 23 Apr 2010 21:12:44 +0400 > gianfar driver may pass NULL pointer to the of_translate_address(), > which may lead to a kernel oops. Fix this by using of_iomap(), which > is also much simpler and shorter. > > Signed-off-by: Anton Vorontsov <avorontsov@mvista.com> Also applied to net-2.6, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index 080d1ce..df49af3 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c @@ -549,12 +549,8 @@ static int gfar_parse_group(struct device_node *np, struct gfar_private *priv, const char *model) { u32 *queue_mask; - u64 addr, size; - - addr = of_translate_address(np, - of_get_address(np, 0, &size, NULL)); - priv->gfargrp[priv->num_grps].regs = ioremap(addr, size); + priv->gfargrp[priv->num_grps].regs = of_iomap(np, 0); if (!priv->gfargrp[priv->num_grps].regs) return -ENOMEM;
gianfar driver may pass NULL pointer to the of_translate_address(), which may lead to a kernel oops. Fix this by using of_iomap(), which is also much simpler and shorter. Signed-off-by: Anton Vorontsov <avorontsov@mvista.com> --- drivers/net/gianfar.c | 6 +----- 1 files changed, 1 insertions(+), 5 deletions(-)