Re: [PATCH v6 2/5] i2c: Add STM32F4 I2C driver

Wolfram Sang
2016-12-15T19:42:12Z

[-- Attachment #1: Type: text/plain, Size: 290 bytes --]

> > + if (ret) {
> > + dev_err(i2c_dev->dev, "bus not free\n");
> > + ret = -EBUSY;
>
> I'm not sure if "bus not free" deserves an error message. Wolfram?

I tend to agree. But I never enforced it up to now, never found the time
to double check if I could/should enforce it.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]