Differential D33798

LinuxKPI: add hex2bin()
ClosedPublic
Actions

Authored by bz on Jan 9 2022, 1:10 AM.

Details

Reviewers

• hselasky

Group Reviewers

linuxkpi

Commits

rGf88fcc52d888: LinuxKPI: add hex2bin()
rGdeb9bfbd5bce: LinuxKPI: add hex2bin()

Summary

Add a hex2bin() implementation needed by a driver's debugfs code.

MFC after: 3 days

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 43792
Build 40680: arc lint + arc unit

Event Timeline

bz created this revision.Jan 9 2022, 1:10 AM

Herald added subscribers: emaste, imp. · View Herald TranscriptJan 9 2022, 1:10 AM

bz requested review of this revision.Jan 9 2022, 1:10 AM

Harbormaster completed remote builds in B43792: Diff 101149.Jan 9 2022, 1:10 AM

I couldn't find any in libkern to my surprise. Let me know if we do have something more native.

• hselasky added a subscriber: • hselasky.Jan 9 2022, 9:18 AM

• hselasky added inline comments.

sys/compat/linuxkpi/common/include/linux/kernel.h
652	Char is not always signed?? Use int8_t ?

• hselasky added inline comments.Jan 9 2022, 10:27 AM

sys/compat/linuxkpi/common/include/linux/kernel.h
652	Functions that return error codes should use "int" type. How do you know EINVAL fits into char?

Use int instead of char as requested though I'd hope we never change EINVAL.

Harbormaster completed remote builds in B43798: Diff 101165.Jan 9 2022, 1:55 PM

• hselasky accepted this revision.Jan 9 2022, 1:58 PM

This revision is now accepted and ready to land.Jan 9 2022, 1:58 PM

Closed by commit rGdeb9bfbd5bce: LinuxKPI: add hex2bin() (authored by bz). · Explain WhyJan 9 2022, 2:38 PM

This revision was automatically updated to reflect the committed changes.

bz added a commit: rGdeb9bfbd5bce: LinuxKPI: add hex2bin().

bz added a commit: rGf88fcc52d888: LinuxKPI: add hex2bin().Jan 16 2022, 11:21 PM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

kernel.h

31 lines

Diff 101149

View Options

sys/compat/linuxkpi/common/include/linux/kernel.h

LinuxKPI: add hex2bin()ClosedPublicActions