Differential D13357

bnxt: Correct the logic to report supported speeds
AbandonedPublic
Actions

Authored by siva.kallam_broadcom.com on Dec 4 2017, 6:16 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 18 2024, 1:13 PM

	Unknown Object (File)
	Jan 5 2024, 5:38 PM

	Unknown Object (File)
	Dec 27 2023, 9:56 PM

	Unknown Object (File)
	Dec 20 2023, 2:43 AM

	Unknown Object (File)
	Dec 12 2023, 1:42 AM

	Unknown Object (File)
	Nov 11 2023, 9:58 AM

	Unknown Object (File)
	Nov 7 2023, 6:17 PM

	Unknown Object (File)
	Oct 25 2023, 1:47 AM

View All 17 Files

Subscribers

Details

Reviewers

sbruno
shurd
bhargava.marreddy_broadcom.com

Summary

This patch has the below changes

phy_type UNKNOWN is treated as valid type and report auto if phy_type is unknown and log a debug message for user attention.
On Link up & down, update media types again. If Link state or media changes, user will observe correct supported media and speeds.
Removed work around in default case of assigning all the possible supported speeds. Just added debug message for user information.

Test Plan

test scenarios excuted:

If cable is not plugged in while driver load, making default supported speed as auto select
Once cable is plugged in and on Link up, depend on PHY type populating the real supported speeds.
If Cable already plugged in while loading driver, populating supported speeds(This support already present)

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

siva.kallam_broadcom.com created this revision.Dec 4 2017, 6:16 AM

Herald added a subscriber: imp. · View Herald TranscriptDec 4 2017, 6:16 AM

Context is not available. I will rework on this patch and resubmit.

Revision Contents
Changeset List

Path

Size

main/

Cumulus/

drivers/

freebsd/

35 lines

Diff 36174

main/Cumulus/drivers/freebsd/if_bnxt.c

Loading...