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)
Wed, Sep 9, 7:38 PM
Unknown Object (File)
Mon, Sep 7, 5:27 PM
Unknown Object (File)
Sat, Sep 5, 12:15 PM
Unknown Object (File)
Fri, Sep 4, 10:34 PM
Unknown Object (File)
Fri, Sep 4, 2:44 PM
Unknown Object (File)
Fri, Sep 4, 12:56 AM
Unknown Object (File)
Wed, Sep 2, 10:20 PM
Unknown Object (File)
Tue, Sep 1, 7:19 PM
View All 79 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...