pci: Use a switch statement when reading ivars
ClosedPublic
Actions

Authored by andrew on Dec 12 2024, 9:24 AM.

Tags

None

Referenced Files

	F109075905: D48045.diff
	Fri, Jan 31, 11:36 AM

	Unknown Object (File)
	Fri, Jan 24, 5:47 PM

	Unknown Object (File)
	Fri, Jan 24, 9:46 AM

	Unknown Object (File)
	Sun, Jan 19, 8:13 AM

	Unknown Object (File)
	Thu, Jan 16, 3:06 PM

	Unknown Object (File)
	Sun, Jan 5, 5:11 AM

	Unknown Object (File)
	Dec 30 2024, 8:56 AM

	Unknown Object (File)
	Dec 28 2024, 9:34 PM

View All 11 Files

Subscribers

imp

Details

Reviewers

jhb
jkim
cperciva
imp

Commits

rGfafb43abd0dc: pci: Use a switch statement when reading ivars

Summary

In pci_host_generic.c use a switch statement rather than a series
of if statements.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

andrew created this revision.Dec 12 2024, 9:24 AM

Herald added a subscriber: imp. · View Herald TranscriptDec 12 2024, 9:24 AM

andrew requested review of this revision.Dec 12 2024, 9:24 AM

Harbormaster completed remote builds in B61099: Diff 147843.Dec 12 2024, 9:24 AM

andrew added a parent revision: D48044: acpica: Extract _OSC parsing to a common file.Dec 12 2024, 9:24 AM

andrew added a child revision: D48046: pci: Make generic_pcie_read_ivar non static.

imp accepted this revision.Dec 12 2024, 2:33 PM

imp added inline comments.

sys/dev/pci/pci_host_generic.c
383	That could be default: case too, if you want

This revision is now accepted and ready to land.Dec 12 2024, 2:33 PM

Closed by commit rGfafb43abd0dc: pci: Use a switch statement when reading ivars (authored by andrew). · Explain WhyDec 12 2024, 6:19 PM

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rGfafb43abd0dc: pci: Use a switch statement when reading ivars.

Revision Contents
Changeset List

Path

Size

sys/

dev/

pci/

pci_host_generic.c

8 lines

Diff 147885

View Options

sys/dev/pci/pci_host_generic.c

pci: Use a switch statement when reading ivarsClosedPublicActions