Message ID | 20181005214507.26315-1-jae.hyun.yoo@linux.intel.com |
---|---|
Headers | show |
Series | i2c: aspeed: Add bus idle waiting logic for multi-master use cases | expand |
On Fri, Oct 5, 2018 at 2:45 PM Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> wrote: > > This commit removes hard-coded bus timeout value setting so that > it can be set by i2c-core-base. > > Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> > Reviewed-by: Joel Stanley <joel@jms.id.au> Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
On Fri, Oct 5, 2018 at 2:45 PM Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> wrote: > > In multi-master environment, this driver's master cannot know > exactly when peer master sends data to this driver's slave so a > case can be happened that this master tries to send data through > the master_xfer function but slave data from peer master is still > being processed by this driver. > > To prevent any state corruption in the case, this patch adds > checking code if any slave operation is ongoing and it waits up to > the bus timeout duration before starting a master_xfer operation. > > Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
On 10/9/2018 5:08 PM, Brendan Higgins wrote: > > Reviewed-by: Brendan Higgins <brendanhiggins@google.com> > Thanks for the review, Brendan!
On 10/9/2018 5:00 PM, Brendan Higgins wrote: > > Reviewed-by: Brendan Higgins <brendanhiggins@google.com> > Thank you!