Re: [PATCH RFC 3/3] net: vertexcom: Add MSE102x SPI support

From: Michael Heimpold
Date: Thu Sep 16 2021 - 07:26:27 EST


Hi Andrew,

Zitat von Andrew Lunn <andrew@xxxxxxx>:

+static int mse102x_probe_spi(struct spi_device *spi)
+{

...

+ netif_carrier_off(mse->ndev);
+ ndev->if_port = IF_PORT_10BASET;

That is not correct. Maybe you should add a IF_PORT_HOMEPLUG ?

Would a simple IF_PORT_HOMEPLUG be sufficient, or should it be
more precise as for Ethernet (10BASET, 100BASET...), e.g.
IF_PORT_HOMEPLUG_10
IF_PORT_HOMEPLUG_AV
IF_PORT_HOMEPLUG_AV2
IF_PORT_HOMEPLUG_GREENPHY

Thanks,
Michael

Attachment: bin_iBlYeKyZi.bin
Description: =?utf-8?b?w5ZmZmVudGxpY2hlciBQR1AtU2NobMO8c3NlbA==?=