Remove autotools

Remove autotools build in favor of meson.

Change-Id: Idb9c68dc79e6df21c01b01f7a323e6d281ca55d4
Signed-off-by: Kun Yi <kunyi731@gmail.com>
Signed-off-by: Patrick Williams <patrick@s

Remove autotools

Remove autotools build in favor of meson.

Change-Id: Idb9c68dc79e6df21c01b01f7a323e6d281ca55d4
Signed-off-by: Kun Yi <kunyi731@gmail.com>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

sdbusplus: replace message::variant with std::variant

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I10ffd8f063fb92a1b9a227b607d1ac439a8ed18e

sdbusplus: remove deprecated variant_ns

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6b522b72d990a1e9a4195a3f7c86d89e41b9961e

Add meson support

Add meson as an alternative build system.

Change-Id: If58674e6fd19d0165ecc73ae677c2a5ad6b30075
Signed-off-by: Kun Yi <kunyi731@gmail.com>

Work around warnings building with -Wall.

Mark several unused variables as such to avoid triggering build
failures.

Change-Id: Ie9b2b3a375afb7f7b48ccee926312a6fc640bd70
Signed-off-by: Kun Yi <kunyi

Work around warnings building with -Wall.

Mark several unused variables as such to avoid triggering build
failures.

Change-Id: Ie9b2b3a375afb7f7b48ccee926312a6fc640bd70
Signed-off-by: Kun Yi <kunyi731@gmail.com>

show more ...

main: Add verbose option

This makes it possible for the daemon to log each postcode it receives
so we can see the host making progress.

Change-Id: Ifc0c8de5c00a2871240f54cfd8dee5dc986d3c92
Signed-o

main: Add verbose option

This makes it possible for the daemon to log each postcode it receives
so we can see the host making progress.

Change-Id: Ifc0c8de5c00a2871240f54cfd8dee5dc986d3c92
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

main: Fixup non-blocking reads

Sometimes, we can get the notification of a read that ends up returning
EAGAIN or EWOULDBLOCK. We need to handle this as a non-error case as
this is normal for non-blo

main: Fixup non-blocking reads

Sometimes, we can get the notification of a read that ends up returning
EAGAIN or EWOULDBLOCK. We need to handle this as a non-error case as
this is normal for non-blocking file descriptors.

We can also optimize the routine by using edge triggering and reading
out all available post codes until we get an EAGAIN.

Change-Id: I16585cf70620ae8cd54ef284a0fe357401957f42
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

7seg: fix service file name

The service file name was missing an '@' at the end

Signed-off-by: Benjamin Fair <benjaminfair@google.com>
Change-Id: Ia0dee9d54c2934dcaf3f4891f81a1ca052e217b9

Add 7-segment display daemon

This daemon takes the path of a 7-segment display device node as its
argument and then writes POST codes which were received over D-Bus to
that device.

Signed-off-by: B

Add 7-segment display daemon

This daemon takes the path of a 7-segment display device node as its
argument and then writes POST codes which were received over D-Bus to
that device.

Signed-off-by: Benjamin Fair <benjaminfair@google.com>
Change-Id: Ic00548c3ebb5a7aa3c9a927c2de748595eb90e71

show more ...

main: Always report new post codes

Right now, we rely on the signals sent via the property value of the
post code changing in order to alert DBus clients of new post codes.
Unfortunately, the underl

main: Always report new post codes

Right now, we rely on the signals sent via the property value of the
post code changing in order to alert DBus clients of new post codes.
Unfortunately, the underlying code checks to see if the value has
actually changed before sending out a PropertiesChanged signal. This is
undesirable for post codes as the process could actually write the same
code multiple times over LPC, and we want to capure each of those
writes.

Avoid this behavior by updating the value to a known different value,
then perform an update with signal sending to the correct value.

Change-Id: I40f419ba8e63b8431c84a108c48f47980206eaa4
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

main: Improve support for existing and new driver semantics

The current Nuvoton driver will emit multiple post codes per read, but is buggy
and can send part of a post code if more than one is read

main: Improve support for existing and new driver semantics

The current Nuvoton driver will emit multiple post codes per read, but is buggy
and can send part of a post code if more than one is read at a time.
This means our current code will put the driver into a confused state if
we are using 4 byte post codes and the driver happens to have only
process 7 bytes of 8 bytes worth of codes in its internal fifo. This
leaves one byte in the fifo which cannot be read again until another 4
bytes are ready. Limiting our read size will prevent this issue as a
workaround until the new driver code can be submitted.

A driver re-work is in progress that will limit the output to a single
post code, and will only write the number of bytes needed for the
io_write transaction on the other end of the bus. Our change trivially
supports this by populating the post code as zero before writing out the
little endian value so that the upper bits are zero if a short post code
is emitted by the driver. Eventually we can get rid of the bytes runtime
flag altogether and rely on the driver returning a single post code per
read.

Change-Id: Ia2e6dc0761a6dd8ddb7cf1194a4230686a7a3d6d
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

main: Fix BMC system stuck when doing enumerate state after host shutdown

Root cause:
According original postd design, the postFd open without any file descriptor flag option.
This cause I/O operati

main: Fix BMC system stuck when doing enumerate state after host shutdown

Root cause:
According original postd design, the postFd open without any file descriptor flag option.
This cause I/O operation on the postFd will waiting process on wait_event_interruptible() in LPC BPC driver.
To avoid this stuck symptom when Host shutdown, postd daemon need to use nonblocking read.

Solution:
Change open fd flag to O_NONBLOCK avoid this kind of system stuck symptom.

Tested:
Using RunBMC-Olympus platform to verify it. BTW, this issue is 100% repo.
1. Power ON Host by WebUI.
2. Shutdown Host by WebUI.
3. Execute enumerate curl or busctl command as below to query state.
curl -b cjar -k https://${POLEG_IP}/xyz/openbmc_project/state/enumerate
busctl get-property xyz.openbmc_project.State.Boot.Raw /xyz/openbmc_project/state/boot/raw xyz.openbmc_project.State.Boot.Raw Value
4. BMC system get enumerate state normally without stuck symptom when Host shutdown.

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: Ibcaee07443d773c25f8d127f927a307bb0cb78e4

show more ...

build: install into bin instead of sbin

Installs into bin instead of sbin per guidelines.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id795cf2e720191e04641a2b12e802407e096ab2d

build: pkg anti-pattern: use defaults

Use the defaults in the pkg check where the default error message is
sufficient to identify which package is missing.

Change-Id: I3677cd4c57bba454ff5d705691947

build: pkg anti-pattern: use defaults

Use the defaults in the pkg check where the default error message is
sufficient to identify which package is missing.

Change-Id: I3677cd4c57bba454ff5d70569194757264a30a1e
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

build: drop sbindir variable

The sbindir variable is provided for free with autotools. For proper
replacement you must use @sbindir@.

Reference: www.gnu.org/prep/standards/html_node/Directory-Vari

build: drop sbindir variable

The sbindir variable is provided for free with autotools. For proper
replacement you must use @sbindir@.

Reference: www.gnu.org/prep/standards/html_node/Directory-Variables.html

Change-Id: I8847256a8ee5178f65427d986eaf54f5ba2bb7e1
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

Push service file into repo

Per systemd daemon guidelines, store the service file in the repository
and install it via the bitbake recipe.

Change-Id: I7da6f7a4d69513bb3358bb23a5ca63db789a9bab
Signe

Push service file into repo

Per systemd daemon guidelines, store the service file in the repository
and install it via the bitbake recipe.

Change-Id: I7da6f7a4d69513bb3358bb23a5ca63db789a9bab
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

build: check for experimental/filesystem

Check for experimental/filesystem instead of experimental/any

Change-Id: I21dbb225a4aae7f1f2bab01249d421cac0e3854e
Signed-off-by: Patrick Venture <venture@g

build: check for experimental/filesystem

Check for experimental/filesystem instead of experimental/any

Change-Id: I21dbb225a4aae7f1f2bab01249d421cac0e3854e
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

main: use sdeventplus wrapper

Convert sdevent to new C++ library wrappers.

Tested: ran on hardware platform and observed snooper able to collect
POST codes. Unit tests still pass.

Change-Id: Ic69a

main: use sdeventplus wrapper

Convert sdevent to new C++ library wrappers.

Tested: ran on hardware platform and observed snooper able to collect
POST codes. Unit tests still pass.

Change-Id: Ic69a41fc20dd77595f065600633a1dcfafd44a5a
Signed-off-by: Kun Yi <kunyi@google.com>

show more ...

Build as c++17

Change-Id: Ia6cfb3915174db1c55f1dd7adedd40c69000a217
Signed-off-by: Patrick Venture <venture@google.com>

Update .clang-format

Per sdbusplus's clang-format file.

Change-Id: Ib7868577dda1bc7ccb300a305e9a3c65efab136c
Signed-off-by: Patrick Venture <venture@google.com>

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

Change-Id: I43249

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

Change-Id: I432495c60d37d377612a7eda1f7169aafc68b59f
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

show more ...

main: reformat small code section

Change-Id: I5642469cdd352cc9635a29ccd28fd222d7d4589d
Signed-off-by: Patrick Venture <venture@google.com>

Merge "maintainers: Add new maintainers"

maintainers: Add new maintainers

Change-Id: I2afec1363741aef1c3921c7da4a50dd33053a464
Signed-off-by: Patrick Venture <venture@google.com>

snoop_listen: Add constructors for SnoopListen

Provide two additional constructors for SnoopListen, one that takes
any std::function that operates on sdbusplus message, and another one
that takes st

snoop_listen: Add constructors for SnoopListen

Provide two additional constructors for SnoopListen, one that takes
any std::function that operates on sdbusplus message, and another one
that takes std::function that operates on the uint64_t postcode. This
allows user to easily register a POST code handler without writing much
of the boiler plate code.

example.cpp is also converted to use the new constructor.

Tested:
'snooper' program compiled from example.cpp behaves correctly.

Change-Id: I20dc4fa5067e836d3cfa8ebb616a131f38a93a30
Signed-off-by: Kun Yi <kunyi731@gmail.com>

show more ...