From: Patrick Williams <patrick@stwcx.xyz>
To: Ben_Pai@wistron.com
Cc: openbmc@lists.ozlabs.org, Claire_Ku@wistron.com,
wangat@tw.ibm.com, bradleyb@fuzziesquirrel.com
Subject: Re: phosphor-bittware repository
Date: Tue, 12 May 2020 07:20:12 -0500 [thread overview]
Message-ID: <20200512122012.GG10214@heinlein> (raw)
In-Reply-To: <822cfe8a5ec94973903534f7932d9049@wistron.com>
[-- Attachment #1: Type: text/plain, Size: 1723 bytes --]
Hello Ben,
On Tue, May 12, 2020 at 09:09:50AM +0000, Ben_Pai@wistron.com wrote:
> I want to implement related functions for the bittware 250-SoC card.
> For example: Sensor reading, VPD information, led control and Brick
> Protection mechanism.
I looked briefly at the datasheet for this hardware [1]. It appears to
expose an SMBus interface for the features you mentioned. The most
straight-forward way to get this implemented is to create a kernel
driver for most of the features you mentioned. If you implement a
driver for this hardware that interacts with the hwmon, eeprom, and LED/GPIO
subsystems in the kernel(*), you'll be able to reuse a lot of existing
OpenBMC functionality without rewriting any userspace code.
- Sensor
- Kernel: hwmon
- Userspace: phosphor-hwmon or dbus-sensors
- VPD
- Kernel: eeprom
- Userspace: entity-manager (I think)
- LED control
- Kernel: LED / GPIO
- Userspace: phosphor-led-manager
The only part that wouldn't be covered would be the "Brick Protection".
We'd need to see some more information on how this is exposed but you
might be able to work it into the existing phosphor-bmc-code-mgmt
repository. For power supplies, they did create a separate repository
(phosphor-psu-code-mgmt) but I think they relied on some kernel APIs for
doing part of the work.
(*) Depending on how the bittware hardware is implemented at an SMBus
level you may end up with multiple smaller drivers (this is better).
If there is a single SMBus address for all these functions, you'll
likely end up with one big driver.
1. https://www.bittware.com/wp-content/uploads/datasheets/ds-250-soc.pdf
--
Patrick Williams
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2020-05-12 12:20 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-12 9:09 phosphor-bittware repository Ben_Pai
2020-05-12 12:20 ` Patrick Williams [this message]
-- strict thread matches above, loose matches on Subject: below --
2020-05-19 6:47 Ben_Pai
2020-05-19 15:44 ` Patrick Williams
2020-05-07 6:07 Ben_Pai
2020-05-11 13:05 ` Patrick Williams
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200512122012.GG10214@heinlein \
--to=patrick@stwcx.xyz \
--cc=Ben_Pai@wistron.com \
--cc=Claire_Ku@wistron.com \
--cc=bradleyb@fuzziesquirrel.com \
--cc=openbmc@lists.ozlabs.org \
--cc=wangat@tw.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.