Mostly good, except one thing:

> +config I2C_DESIGNWARE_BAYTRAIL
> +     bool "Intel Baytrail I2C semaphore support"
> +     depends on I2C_DESIGNWARE_PLATFORM
> +     select IOSF_MBI

This needs some dependency on something Baytrail specific. Otherwise, it
causes lots of build errors:

With ARM allnoconfig, I can select this option but there are no headers
for iosf.

With x86_64_defconfig and disabled ACPI, I can select this option but it
fails over "implicit declaration of function ‘acpi_evaluate_integer’".

An incremental patch will do for me, the rest looks good.

Attachment: signature.asc
Description: Digital signature

Reply via email to