Differential D5505

if_dwc add reset-gpio
ClosedPublic
Actions

Authored by manu_bidouilliste.com on Mar 1 2016, 11:58 AM.

Tags

Referenced Files

	Unknown Object (File)
	Thu, May 9, 7:44 PM

	Unknown Object (File)
	Wed, May 8, 5:04 AM

	Unknown Object (File)
	Wed, May 8, 5:04 AM

	Unknown Object (File)
	Tue, May 7, 6:50 AM

	Unknown Object (File)
	Sun, Apr 28, 1:44 AM

	Unknown Object (File)
	Feb 16 2024, 6:31 AM

	Unknown Object (File)
	Feb 16 2024, 5:30 AM

	Unknown Object (File)
	Dec 31 2023, 7:30 AM

Subscribers

Details

Reviewers

jmcneill
andrew
adrian

Group Reviewers

Commits

rS296365: Add support for resetting the PHY via GPIO.

Summary

The snps have some optional property for reseting the phy via gpios.
This patch implement the reset.

Test Plan

Apply patch and test on systems with dwc

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2714
Build 2734: arc lint + arc unit

Event Timeline

manu_bidouilliste.com updated this revision to Diff 13928.Mar 1 2016, 11:58 AM

manu_bidouilliste.com retitled this revision from to if_dwc add reset-gpio.

manu_bidouilliste.com updated this object.

manu_bidouilliste.com edited the test plan for this revision. (Show Details)

manu_bidouilliste.com added a reviewer: ARM.

manu_bidouilliste.com set the repository for this revision to rS FreeBSD src repository - subversion.

manu_bidouilliste.com added a project: ARM.

Herald added subscribers: loos, imp. · View Herald TranscriptMar 1 2016, 11:58 AM

jmcneill added a subscriber: jmcneill.Mar 1 2016, 12:52 PM

jmcneill added inline comments.

sys/dev/dwc/if_dwc.c
1082	Indent 4 spaces instead of a tab for multi-line if condition.
1086	4 spaces here too
1099	Instead of active_low = 0/1 it would simplify the GPIO_PIN_SET calls later if you had int value and set it to either GPIO_PIN_LOW/GPIO_PIN_HIGH
1102	4 spaces here too

Simplify the active_low logic.

manu_bidouilliste.com marked 4 inline comments as done.Mar 1 2016, 1:58 PM

Respect optional pin polarity.

br added a subscriber: br.Mar 1 2016, 6:06 PM

br added inline comments.

sys/dev/dwc/if_dwc.c
1041	line is too long

Fix style(9)

manu_bidouilliste.com marked an inline comment as done.Mar 2 2016, 7:05 AM

jmcneill accepted this revision.Mar 2 2016, 9:49 PM

jmcneill added a reviewer: jmcneill.

This revision is now accepted and ready to land.Mar 2 2016, 9:49 PM

andrew accepted this revision.Mar 3 2016, 11:08 AM

andrew added a reviewer: andrew.

adrian accepted this revision.Mar 3 2016, 9:54 PM

adrian added a reviewer: adrian.

Closed by commit rS296365: Add support for resetting the PHY via GPIO. (authored by jmcneill). · Explain WhyMar 3 2016, 10:26 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

dev/

dwc/

63 lines

Diff 13937

sys/dev/dwc/if_dwc.c

Loading...