[D,E,SRU,PATCHv2,0/1] Enable Armada SOCs and MVPP2 NIC driver for disco/generic arm64
mbox series

Message ID 20190715070922.15443-1-po-hsu.lin@canonical.com
Headers show
Series
  • Enable Armada SOCs and MVPP2 NIC driver for disco/generic arm64
Related show

Message

Po-Hsu Lin July 15, 2019, 7:09 a.m. UTC
V2: drop CONFIG_ARCH_MULTI_V7 and add configs with updateconfigs as
suggested by ppisati.

== SRU Justification ==
The mvpp2 is a driver for network controllers(s) in Marvell SOCs,
particularly Armada 8040.

However this driver was neither enabled as a module or a built-in for
Disco arm64.

== Fix ==
Just like what we did for armhf, set CONFIG_MVPP2 to "m" for arm64 can
solve this problem.

We need to enable the Armada SOCs support (CONFIG_ARCH_MVEBU) as well
to meet the dependency requirement.

Other configs were added by the updateconfigs process.

== Test ==
A test kernel for Disco ARM64 could be found here:
https://people.canonical.com/~phlin/kernel/lp-1835054-mvpp2/V2/

User has confirmed that the V1 kernel can fix this missing driver
issue (V2 is just with CONFIG_ARCH_MULTI_V7 dropped and some configs
added explicitly, I have it tested on an ARM64 node, the mvpp2 module
can be loaded without any issue).

== Regression Potential ==
Low, this patch just enable the Armada SOCs and make this mvpp2 driver
to be built as a module on ARM64, and we already have those for armhf.

Po-Hsu Lin (1):
  UBUNTU: [Config] Enable CONFIG_MVPP2 for arm64

 debian.master/config/annotations               |  4 ++--
 debian.master/config/arm64/config.common.arm64 |  1 -
 debian.master/config/armhf/config.common.armhf |  1 -
 debian.master/config/config.common.ubuntu      | 14 ++++++++++++++
 4 files changed, 16 insertions(+), 4 deletions(-)

Comments

Seth Forshee July 17, 2019, 4:40 p.m. UTC | #1
On Mon, Jul 15, 2019 at 03:09:21PM +0800, Po-Hsu Lin wrote:
> V2: drop CONFIG_ARCH_MULTI_V7 and add configs with updateconfigs as
> suggested by ppisati.
> 
> == SRU Justification ==
> The mvpp2 is a driver for network controllers(s) in Marvell SOCs,
> particularly Armada 8040.
> 
> However this driver was neither enabled as a module or a built-in for
> Disco arm64.
> 
> == Fix ==
> Just like what we did for armhf, set CONFIG_MVPP2 to "m" for arm64 can
> solve this problem.
> 
> We need to enable the Armada SOCs support (CONFIG_ARCH_MVEBU) as well
> to meet the dependency requirement.
> 
> Other configs were added by the updateconfigs process.
> 
> == Test ==
> A test kernel for Disco ARM64 could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1835054-mvpp2/V2/
> 
> User has confirmed that the V1 kernel can fix this missing driver
> issue (V2 is just with CONFIG_ARCH_MULTI_V7 dropped and some configs
> added explicitly, I have it tested on an ARM64 node, the mvpp2 module
> can be loaded without any issue).
> 
> == Regression Potential ==
> Low, this patch just enable the Armada SOCs and make this mvpp2 driver
> to be built as a module on ARM64, and we already have those for armhf.

Positive testing and smoke tested for regressions.

Acked-by: Seth Forshee <seth.forshee@canonical.com>

Applied to eoan/master-next, thanks!
Khaled Elmously July 23, 2019, 5:48 a.m. UTC | #2
On 2019-07-15 15:09:21 , Po-Hsu Lin wrote:
> V2: drop CONFIG_ARCH_MULTI_V7 and add configs with updateconfigs as
> suggested by ppisati.
> 
> == SRU Justification ==
> The mvpp2 is a driver for network controllers(s) in Marvell SOCs,
> particularly Armada 8040.
> 
> However this driver was neither enabled as a module or a built-in for
> Disco arm64.
> 
> == Fix ==
> Just like what we did for armhf, set CONFIG_MVPP2 to "m" for arm64 can
> solve this problem.
> 
> We need to enable the Armada SOCs support (CONFIG_ARCH_MVEBU) as well
> to meet the dependency requirement.
> 
> Other configs were added by the updateconfigs process.
> 
> == Test ==
> A test kernel for Disco ARM64 could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1835054-mvpp2/V2/
> 
> User has confirmed that the V1 kernel can fix this missing driver
> issue (V2 is just with CONFIG_ARCH_MULTI_V7 dropped and some configs
> added explicitly, I have it tested on an ARM64 node, the mvpp2 module
> can be loaded without any issue).
> 
> == Regression Potential ==
> Low, this patch just enable the Armada SOCs and make this mvpp2 driver
> to be built as a module on ARM64, and we already have those for armhf.
> 
> Po-Hsu Lin (1):
>   UBUNTU: [Config] Enable CONFIG_MVPP2 for arm64
> 
>  debian.master/config/annotations               |  4 ++--
>  debian.master/config/arm64/config.common.arm64 |  1 -
>  debian.master/config/armhf/config.common.armhf |  1 -
>  debian.master/config/config.common.ubuntu      | 14 ++++++++++++++
>  4 files changed, 16 insertions(+), 4 deletions(-)
> 
> -- 
> 2.7.4
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team