diff mbox

[X/snapdragon,SRU] UBUNTU: [Config] Disable CONFIG_I2C_THUNDERX

Message ID 20170609080434.18561-1-kleber.souza@canonical.com
State New
Headers show

Commit Message

Kleber Sacilotto de Souza June 9, 2017, 8:04 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1696937

The support for IPMI system interface on Cavium ThunderX (bug #1688132)
has been added to Xenial kernel as a module compiled for arm64, so the
config for the -snapdragon kernel needs to be updated to explicitly
disable this module, otherwise the -snapdragon package build fails.

Signed-off-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
---
 debian.snapdragon/config/config.common.ubuntu | 1 +
 1 file changed, 1 insertion(+)

Comments

Andy Whitcroft June 9, 2017, 2:24 p.m. UTC | #1
On Fri, Jun 09, 2017 at 10:04:34AM +0200, Kleber Sacilotto de Souza wrote:
> BugLink: https://bugs.launchpad.net/bugs/1696937
> 
> The support for IPMI system interface on Cavium ThunderX (bug #1688132)
> has been added to Xenial kernel as a module compiled for arm64, so the
> config for the -snapdragon kernel needs to be updated to explicitly
> disable this module, otherwise the -snapdragon package build fails.
> 
> Signed-off-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
> ---
>  debian.snapdragon/config/config.common.ubuntu | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/debian.snapdragon/config/config.common.ubuntu b/debian.snapdragon/config/config.common.ubuntu
> index 70005869741c..c2df24cdbc65 100644
> --- a/debian.snapdragon/config/config.common.ubuntu
> +++ b/debian.snapdragon/config/config.common.ubuntu
> @@ -2425,6 +2425,7 @@ CONFIG_I2C_SMBUS=m
>  CONFIG_I2C_STUB=m
>  CONFIG_I2C_TAOS_EVM=m
>  # CONFIG_I2C_TEGRA is not set
> +# CONFIG_I2C_THUNDERX is not set
>  CONFIG_I2C_TINY_USB=m
>  CONFIG_I2C_VERSATILE=m
>  CONFIG_I2C_VIA=m

Seems reasonable.  This was not on before so noone can be using it, so
no chance of regression.

Acked-by: Andy Whitcroft <apw@canonical.com>

-apw
Thadeu Lima de Souza Cascardo June 9, 2017, 2:27 p.m. UTC | #2
Acked-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
Kleber Sacilotto de Souza June 9, 2017, 2:48 p.m. UTC | #3
Applied to xenial/snapdragon branch.
diff mbox

Patch

diff --git a/debian.snapdragon/config/config.common.ubuntu b/debian.snapdragon/config/config.common.ubuntu
index 70005869741c..c2df24cdbc65 100644
--- a/debian.snapdragon/config/config.common.ubuntu
+++ b/debian.snapdragon/config/config.common.ubuntu
@@ -2425,6 +2425,7 @@  CONFIG_I2C_SMBUS=m
 CONFIG_I2C_STUB=m
 CONFIG_I2C_TAOS_EVM=m
 # CONFIG_I2C_TEGRA is not set
+# CONFIG_I2C_THUNDERX is not set
 CONFIG_I2C_TINY_USB=m
 CONFIG_I2C_VERSATILE=m
 CONFIG_I2C_VIA=m