nvme sensors

[Benjamin Fair]

On Fri, 17 Apr 2020 at 11:42, Vijay Khemka <vijaykhemka at fb.com> wrote:
>
>
>
> On 4/17/20, 11:11 AM, "Benjamin Fair" <benjaminfair at google.com> wrote:
>
>     On Fri, 17 Apr 2020 at 11:05, Vijay Khemka <vijaykhemka at fb.com> wrote:
>     >
>     >
>     >
>     > On 4/16/20, 4:35 PM, "Benjamin Fair" <benjaminfair at google.com> wrote:
>     >
>     >     On Thu, 16 Apr 2020 at 16:15, Vijay Khemka <vijaykhemka at fb.com> wrote:
>     >     >
>     >     > Currently, dbus-sensors support reading NVME sensors via mctp only. It uses
>     >     > libmctp and some of smbus patch from Intel. It also uses slave mqueue mctp
>     >     > kernel driver which is not upstream.
>     >     >
>     >     > So currently it is not matured and fully upstreamed.
>     >     >
>     >     > James, how can we use this without mctp. I don't see any non mctp support like
>     >     > Sending direct smbus command, please advise if I missed it.
>     >
>     >     If you want to use NVME-MI without MCTP, this use case is supported by
>     >     phosphor-nvme. It makes direct SMBus calls.
>     >
>     > Ben, thanks I see that but it needs certain GPIOs like PwrGd, presence and I am trying to find if
>     > Our platform support any such GPIOs.
>
>     If your platform doesn't have these GPIOs, we should be able to make
>     phosphor-nvme more flexible and not require them. It was designed for
>     one system at first with the expectation that it could become more
>     general over time.
>
> Cool, That will be great. I also don't see recipe file for this.

The recipe file is at
meta-phosphor/recipes-phosphor/sensors/phosphor-nvme_git.bb

> And how easy/hard to integrate this
> with entity manager as we use entity manager to list all sensors.

I'm not familiar enough with entity-manager to say for sure.

>
>     >
>     >     >
>     >     > Regards
>     >     > -Vijay
>     >     >
>     >     >
>     >
>     >
>
>