On 11/28/2017 01:07 PM, Chris Brandt wrote:
> On Monday, November 27, 2017 1, Marek Vasut wrote:
>>> wait_for_bit() wants you to pass a direct address of a 32-bit register.
>>> The register I am waiting for is a 16-bit register and the hardware
>>> manual doesn't say 32-bit is allowed.
>>> When I do a 32-bit read on that address, I actually get a different
>>> value.
>>>
>>> For example:
>>>
>>> => md.w E8010060 1
>>> e8010060: 0040
>>> => md.l E8010060 1
>>> e8010060: 00400000
>>>
>>>
>>> So, I can't use wait_for_bit()
>>
>> Can we somehow extend wait_for_bit() ?
> 
> It's a nice function, so it would be good if it worked with more than 
> just 32-bit registers.
> 
> However, it's used in 98 places at the moment, so coordinating modifying
> all those changes at once might be an issue.

What about readb_poll_timeout() , would that work ?

> Chris
> 


-- 
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to