Page MenuHomeFreeBSD
Differential D21631

powerpc/powernv: Add OPAL NVRAM driver
ClosedPublic
Actions

Authored by jhibbits on Sep 13 2019, 3:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 4 2024, 12:53 PM
Unknown Object (File)
Sep 25 2024, 6:04 AM
Unknown Object (File)
Sep 8 2024, 4:08 PM
Unknown Object (File)
Sep 7 2024, 4:01 PM
Unknown Object (File)
Aug 22 2024, 6:15 PM
Unknown Object (File)
Jul 16 2024, 7:33 AM
Unknown Object (File)
Jul 15 2024, 12:44 AM
Unknown Object (File)
Jul 9 2024, 6:16 PM
View All 21 Files
Subscribers
None

Details

Reviewers
luporl
bdragon
nwhitehorn
Summary

Add a very basic OPAL NVRAM driver, which can be used by the IBM
power-utils toolkit for reading and writing NVRAM on powernv systems.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26453
Build 24884: arc lint + arc unit

Event Timeline

jhibbits created this revision.Sep 13 2019, 3:11 AM
Harbormaster completed remote builds in B26453: Diff 62017.Sep 13 2019, 3:11 AM
luporl accepted this revision.Sep 13 2019, 7:47 PM

LGTM.

sys/powerpc/powernv/opal_nvram.c
148

Isn't a NULL check of contigmalloc() return value needed here?

This revision is now accepted and ready to land.Sep 13 2019, 7:47 PM
jhibbits added inline comments.Sep 13 2019, 8:22 PM
sys/powerpc/powernv/opal_nvram.c
148

You're right. Even though I specify M_WAITOK, from code inspection it only allows me 3 retries before it gives up and returns NULL.

jhibbits closed this revision.Sep 14 2019, 4:06 AM

r352313

Revision Contents
Changeset List

PathSize
sys/
conf/
1 line
modules/
3 lines
opal_nvram/
10 lines
powerpc/
powernv/
2 lines
272 lines

Diff 62017

View Options

sys/conf/files.powerpc

Loading...
View Options

sys/modules/Makefile

Loading...
View Options

sys/modules/opal_nvram/Makefile

Loading...
View Options

sys/powerpc/powernv/opal.h

Loading...
View Options

sys/powerpc/powernv/opal_nvram.c

Loading...