lkml.org 
[lkml]   [2012]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/8] ohci-nxp: Driver cleanup
Hi,

On 17/04/12 22:36, Arnd Bergmann wrote:
>> + hcd->regs = devm_request_and_ioremap(&pdev->dev, res);
>> + if (!hcd->regs) {
>> + err("Failed to devm_request_and_ioremap");
>> ret = -ENOMEM;
>> goto out4;
>> }
>> - hcd->regs = (void __iomem *)pdev->resource[0].start;
>> + hcd->rsrc_start = (u64)(u32)hcd->regs;
>> + hcd->rsrc_len = resource_size(res);
>
> This is wrong in multiple ways:
>
> * rsrc_start is a physical address, not an __iomem token.
> * you cannot cast a pointer to u32 in general, only to unsigned long.
>
> The fact that you need an ugly type cast like above could have told
> you that you are doing something wrong here. I guess it should
> be
>
> hcd->rsrc_start = res->start;

Right. Thanks for the hint!

Technically, I ported the bug, didn't introduce it ;-)
(pdev->resource[0].start was previously provided via resource already in
IO_ADDRESS (mapped) format). Nevertheless, it's a bug, of course, and it
didn't show up since the driver seems to not use hcd->rsrc_start anyway
(but hcd->regs).

Will post an update after making sure it's tested well.

Roland



\
 
 \ /
  Last update: 2012-04-18 22:37    [W:0.172 / U:0.228 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site