| Message ID | 20170725161553.30147-3-privat@egil-hjelmeland.no |
|---|---|
| State | Changes Requested, archived |
| Delegated to: | David Miller |
| Headers | show |
On Tue, Jul 25, 2017 at 06:15:45PM +0200, Egil Hjelmeland wrote: > For some mysterious reason enable switch fabric port 0 TX fails to > work, when the TX has previous been disabled. Resolved by not > disable/enable switch fabric port 0 at startup. Port 1 and 2 are > still disabled in early init. > > Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no> > --- > drivers/net/dsa/lan9303-core.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c > index e622db586c3d..c2b53659f58f 100644 > --- a/drivers/net/dsa/lan9303-core.c > +++ b/drivers/net/dsa/lan9303-core.c > @@ -557,9 +557,6 @@ static int lan9303_disable_processing(struct lan9303 *chip) > { > int ret; > > - ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_0_OFFSET); > - if (ret) > - return ret; > ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_1_OFFSET); > if (ret) > return ret; > @@ -633,10 +630,6 @@ static int lan9303_setup(struct dsa_switch *ds) > if (ret) > dev_err(chip->dev, "failed to separate ports %d\n", ret); > > - ret = lan9303_enable_packet_processing(chip, LAN9303_PORT_0_OFFSET); > - if (ret) > - dev_err(chip->dev, "failed to re-enable switching %d\n", ret); > - Does this mean you are relying on something else enabling port 0? The bootloader? I'm wondering if it is better to keep the enable, but remove the disable? Andrew
On 26. juli 2017 18:58, Andrew Lunn wrote: > On Tue, Jul 25, 2017 at 06:15:45PM +0200, Egil Hjelmeland wrote: >> For some mysterious reason enable switch fabric port 0 TX fails to >> work, when the TX has previous been disabled. Resolved by not >> disable/enable switch fabric port 0 at startup. Port 1 and 2 are >> still disabled in early init. >> >> Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no> >> --- >> drivers/net/dsa/lan9303-core.c | 7 ------- >> 1 file changed, 7 deletions(-) >> >> diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c >> index e622db586c3d..c2b53659f58f 100644 >> --- a/drivers/net/dsa/lan9303-core.c >> +++ b/drivers/net/dsa/lan9303-core.c >> @@ -557,9 +557,6 @@ static int lan9303_disable_processing(struct lan9303 *chip) >> { >> int ret; >> >> - ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_0_OFFSET); >> - if (ret) >> - return ret; >> ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_1_OFFSET); >> if (ret) >> return ret; >> @@ -633,10 +630,6 @@ static int lan9303_setup(struct dsa_switch *ds) >> if (ret) >> dev_err(chip->dev, "failed to separate ports %d\n", ret); >> >> - ret = lan9303_enable_packet_processing(chip, LAN9303_PORT_0_OFFSET); >> - if (ret) >> - dev_err(chip->dev, "failed to re-enable switching %d\n", ret); >> - > > Does this mean you are relying on something else enabling port 0? The > bootloader? > > I'm wondering if it is better to keep the enable, but remove the > disable? > > Andrew > The (switch engine) ports are enabled by default. The only thing our bootloader does is to set gpo so the lan9303 is kept in reset until the linux driver starts. When I test with the next-next kernel I just specify the reset-gpo in DTS and the driver pulls it out of reset. Keeping the enable does no harm, as far as I recall, but I can double check that when I get time. I have no idea why the original mainline code does not work for me. Maybe it is a timing issue?
diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c index e622db586c3d..c2b53659f58f 100644 --- a/drivers/net/dsa/lan9303-core.c +++ b/drivers/net/dsa/lan9303-core.c @@ -557,9 +557,6 @@ static int lan9303_disable_processing(struct lan9303 *chip) { int ret; - ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_0_OFFSET); - if (ret) - return ret; ret = lan9303_disable_packet_processing(chip, LAN9303_PORT_1_OFFSET); if (ret) return ret; @@ -633,10 +630,6 @@ static int lan9303_setup(struct dsa_switch *ds) if (ret) dev_err(chip->dev, "failed to separate ports %d\n", ret); - ret = lan9303_enable_packet_processing(chip, LAN9303_PORT_0_OFFSET); - if (ret) - dev_err(chip->dev, "failed to re-enable switching %d\n", ret); - return 0; }
For some mysterious reason enable switch fabric port 0 TX fails to work, when the TX has previous been disabled. Resolved by not disable/enable switch fabric port 0 at startup. Port 1 and 2 are still disabled in early init. Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no> --- drivers/net/dsa/lan9303-core.c | 7 ------- 1 file changed, 7 deletions(-)