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)
Sat, May 9, 10:43 AM
Unknown Object (File)
Sat, Apr 25, 8:33 PM
Unknown Object (File)
Fri, Apr 24, 11:49 PM
Unknown Object (File)
Apr 13 2026, 5:25 AM
Unknown Object (File)
Apr 5 2026, 3:07 AM
Unknown Object (File)
Mar 24 2026, 1:51 AM
Unknown Object (File)
Mar 24 2026, 1:36 AM
Unknown Object (File)
Mar 20 2026, 5:36 AM
View All 60 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...