RE: [PATCH v2 2/2] ARM: shmobile: defconfig: Enable Ethernet AVB

Hello Sergei, 

Thank you for your comments.

From: Sergei Shtylyov [mailto:sergei.shtylyov@cogentembedded.com]
Sent: 05 July 2017 17:14

Hello!

On 07/05/2017 06:56 PM, Biju Das wrote:

quoted

The iWave RZ/G1M Q7 SOM supports Gigabit Ethernet Phy (Micrel

KSZ9031MNX).

quoted

Gigabit Ethernet support is available in Renesas AVB driver.
To increase hardware support enable the driver in the
shmobile_defconfig multiplatform configuration.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Chris Paterson <redacted>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/configs/shmobile_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/shmobile_defconfig

b/arch/arm/configs/shmobile_defconfig
index 3c66a42..d992911 100644

--- a/arch/arm/configs/shmobile_defconfig
+++ b/arch/arm/configs/shmobile_defconfig

@@ -83,6 +83,7 @@ CONFIG_NETDEVICES=y
 # CONFIG_NET_VENDOR_MICREL is not set  #

CONFIG_NET_VENDOR_NATSEMI is

quoted

not set  CONFIG_SH_ETH=y
+CONFIG_RAVB=y

    Hm, I would enable it only as a module, given it's not easy to make use of
AVB on the 32-bit boards.

On the iWave-M platforms the only available Ethernet interface is AVB. Without including the RAVB driver in the Kernel we won't be able to use NFS.

I know adding CONFIG_RAVB for all shmobile devices may increase the Kernel size for everyone, especially when most cannot use AVB without an add-on board, but I don't think the increase will be big.


Kind regards, Chris 

[...]

MBR Sergei