Hi Simon,
please delete unrelated text. I nearly missed the typo fix later.
> > - if (in_atomic() || irqs_disabled())
> > + if (in_atomic() || irqs_disabled()) {
> > + WARN(!adap->algo->master_xfer_atomic && !adap->algo->smbus_xfer_atomic,
> > + "No atomic I2C transfer handler for '%s'\n", dev_name(&adap->dev));
>
> Is WARN_ONCE more appropriate here?
Why? It could be multiple adapters or clients causing this?
> > + * The return codes from the @master_xfer{_atomic} field should indicate the
>
> I think "field" should be "fields" in the new text.
Fixed, thanks!
Regards,
Wolfram
Attachment:
signature.asc
Description: PGP signature
