Hi Ellen,
On Mon, 20 Apr 2015 15:52:47 -0700, Ellen Wang wrote:
> This leads to a related question: If the driver is serialized, then the
> (status = priv->status) inside wait_event_timeout() isn't strictly
> necessary, correct? It can just be priv->status. I'm just want to
> double check that there's no race and I can add a priv->auxsts and not
> have to stick something like this inside the wait_event_timeout():
>
> (auxsts = priv->auxsts, status = priv->status)
To be honest I'm not 100% certain. I think this was only a minor
optimization, but I may be wrong. Adding Daniel to Cc, he added that
code to the i2c-i801 driver. Daniel, any comment on this?
Equally mysterious to me is:
priv->status |= status;
in i801_isr() where it would seem that a simple "=" would suffice.
--
Jean Delvare
SUSE L3 Support
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
