Hi,

thanks for the submission!

> +#define CDNS_I2C_BROKEN_HOLD_BIT     0x00000001

BIT(0) maybe?

> +static const struct cdns_platform_data r1p10_i2c_def = {
> +                             .quirks = CDNS_I2C_BROKEN_HOLD_BIT, };

Closing '}' should be on seperate line.

And what Mark said.

Other than that, looks okay I'd say.

Attachment: signature.asc
Description: Digital signature

Reply via email to