openbmc / linux

OpenBMC Linux kernel source tree

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Upstream miscellaneous control interfaces patch

amboar opened this issue · comments

We need a way to expose random bits in random registers in random devices in a unified way. This ran aground upstream previously due to the proposed devicetree bindings. Avoid that mess, implement support as a library, then implement basic drivers for the devices and expose the bits using the library.

@osenft just fyi I know you are interested in this work.

@amboar, can you explain more about the proposed solution?

I still feel its beneficial to have a clean Kernel<->userspace boundary that clearly defines which bits a userspace process can modify. I generally prefer a /sys based interface since the abstraction lives in the kernel. However, I appreciate that this means a lot of (potentially) duplicate work that each needs to be upstreamed separately ...

@osenft I think this was the most recent set of patches I sent:

https://lists.openwall.net/linux-kernel/2018/07/11/179

They need to be reworked as I note above and sent out again. Hopefully they'll get a bit more traction this way.

These patches were removed in favour of adding code to the relevant for the controls that were exposed. See 696029e for an example involving the display driver.