[PATCH 04/15] ARM: OMAP2+: hwmod: Update the reset API for AM33XX
Hiremath, Vaibhav
hvaibhav at ti.com
Tue Nov 6 01:06:20 EST 2012
On Mon, Nov 05, 2012 at 23:27:52, Bedia, Vaibhav wrote:
> On Mon, Nov 05, 2012 at 12:28:36, Hiremath, Vaibhav wrote:
> [...]
> > > - u32 mask = 1 << shift;
> > > -
> > > - /* Check the current status to avoid de-asserting the line twice */
> > > - if (am33xx_prm_is_hardreset_asserted(shift, inst, rstctrl_offs) == 0)
> > > - return -EEXIST;
> >
> > Any specific reason why you have removed this check?
>
> During bootup the hardreset line is asserted, so wouldn't that check lead
> to the function always returning without doing anything?
>
The check is,
/* Check the current status to avoid de-asserting the line twice */
if (am33xx_prm_is_hardreset_asserted(shift, inst, rstctrl_offs) == 0)
return -EEXIST;
The code is checking whether the line is already de-asserted (== 0), so I am
not sure how this will change if hardreset line is asserted during bootup.
Thanks,
Vaibhav
More information about the linux-arm-kernel
mailing list