All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mahesh Kurapati <mahesh.kurapati@keysight.com>
To: Ed Tanous <edtanous@google.com>
Cc: "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>
Subject: RE: add a new yaml interface definition
Date: Wed, 6 Jan 2021 00:23:00 +0000	[thread overview]
Message-ID: <SN6PR17MB2558B1D463069389138AF0DA96D00@SN6PR17MB2558.namprd17.prod.outlook.com> (raw)
In-Reply-To: <CAH2-KxBV9_0Dt79Quy0f4HkXXPdHfBw9FsG=4KwdWXBYNEA-ew@mail.gmail.com>

Hello Ed, 

Thank you for the guidelines.  I am also not planning to expose the GPIOs as is.  As you said I am implementing them as a service, for e.g., alarm service, and exposing the Alarm raise/clear as dbus methods. 

Regards
Mahesh 

-----Original Message-----
From: Ed Tanous <edtanous@google.com> 
Sent: Tuesday, January 5, 2021 11:28 AM
To: Mahesh Kurapati <mahesh.kurapati@keysight.com>
Cc: openbmc@lists.ozlabs.org
Subject: Re: add a new yaml interface definition

On Mon, Jan 4, 2021 at 1:42 PM Mahesh Kurapati <mahesh.kurapati@keysight.com> wrote:
>
> Hello,
>
>
>
> Thank you for the video on the phosphor-dbus-interfaces architecture.  It clarified the development flow.
>
>
>
> I am trying to expose some of the discrete GPIO signals,

We should think very carefully about if we want to expose raw GPIOs directly to dbus.  In practice, almost all GPIOs need some kind of filtering (debounce, power state filtering, minor state machine of other GPIOs) and that tends to be very difficult to model directly in a generic way.  In general, it's a much better idea to model your GPIO as a high level concept, like an LED, or a Power state controller, then expose a well defined API to dbus for that device.  That means that downstream clients can identify the GPIO interface reasonably and expose the appropriate APIs to the user.

> and methods to generate audio and visual alarms for our management software.  I will define two new yaml files describing these interfaces.  From the training video I understood that I should use the sbus++ to generate the cpp boilerplate code and make it part of the library.  I will extend my daemon code to implement the actual functionality as explained in the video.  Where I am stuck is on how do I add my yaml files to the phosphor-dbus-interfaces infrastructure?  How to do this in my yocto environment?  Please help.
>
>
>
> Thank you,
>
> Mahesh

      reply	other threads:[~2021-01-06  0:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-04 21:14 add a new yaml interface definition Mahesh Kurapati
2021-01-04 22:22 ` Patrick Williams
2021-01-04 23:05   ` Mahesh Kurapati
2021-01-04 23:15     ` Patrick Williams
2021-01-05  0:42       ` Mahesh Kurapati
2021-01-05 14:10         ` Patrick Williams
2021-01-06  0:28           ` Mahesh Kurapati
2021-01-05 17:28 ` Ed Tanous
2021-01-06  0:23   ` Mahesh Kurapati [this message]

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=SN6PR17MB2558B1D463069389138AF0DA96D00@SN6PR17MB2558.namprd17.prod.outlook.com \
    --to=mahesh.kurapati@keysight.com \
    --cc=edtanous@google.com \
    --cc=openbmc@lists.ozlabs.org \
    /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.