From: Michal Simek <michal.simek@amd.com>
To: "Jae Hyun Yoo" <quic_jaehyoo@quicinc.com>,
"Ovidiu Panait" <ovidiu.panait@windriver.com>,
"Simon Glass" <sjg@chromium.org>,
"Mario Six" <mario.six@gdsys.cc>,
"Masahisa Kojima" <masahisa.kojima@linaro.org>,
"Pali Rohár" <pali@kernel.org>,
"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
"Ashok Reddy Soma" <ashok.reddy.soma@xilinx.com>,
"Thomas Huth" <thuth@redhat.com>,
"Huang Jianan" <jnhuang95@gmail.com>,
"Chris Morgan" <macromorgan@hotmail.com>,
"Roland Gaudig" <roland.gaudig@weidmueller.com>,
"Patrick Delaunay" <patrick.delaunay@foss.st.com>,
"Alexandru Gagniuc" <mr.nuke.me@gmail.com>
Cc: "Jamie Iles" <quic_jiles@quicinc.com>,
"Graeme Gregory" <quic_ggregory@quicinc.com>,
"Cédric Le Goater" <clg@kaod.org>,
u-boot@lists.denx.de
Subject: Re: [PATCH v4 0/6] cmd/fru: move FRU handling support to common region
Date: Thu, 15 Sep 2022 16:38:31 +0200 [thread overview]
Message-ID: <ccc1b834-ca15-a736-1bb4-caf78e96ef3c@amd.com> (raw)
In-Reply-To: <db8f78dc-7f3e-62f2-97d6-cfae8e85267a@quicinc.com>
On 9/15/22 16:01, Jae Hyun Yoo wrote:
> Gentle ping for this series.
>
> On 8/25/2022 9:42 AM, Jae Hyun Yoo wrote:
>> Hello,
>>
>> The FRU handling was added as a Xilinx board dependent support but it
>> is also useful for other boards, so this commit moves the FRU handling
>> support to the common region so that it can be enabled by CONFIG_CMD_FRU.
>>
>> To provide manufacturer specific custom info fields and multi-records
>> parsing, it refactors the FRU handling logic using linked list so that each
>> board support can utilize them in their own custom way. This series adds
>> 'Product Info' parsing support, usage document and unit test script too.
>>
>> Please review!
>>
>> Thanks,
>> Jae
>>
>> Graeme Gregory (1):
>> cmd: fru: move FRU handling support to common region
>>
>> Jae Hyun Yoo (5):
>> xilinx: common: refactor FRU handling support
>> cmd: fru: fix a sandbox segfault issue
>> cmd: fru: add product info area parsing support
>> doc: fru: add documentation for the fru command and APIs
>> test: cmd: fru: add unit test for the fru command
>>
>> board/xilinx/Kconfig | 8 -
>> board/xilinx/common/Makefile | 3 -
>> board/xilinx/common/board.c | 68 ++-
>> board/xilinx/common/fru.h | 108 -----
>> board/xilinx/common/fru_ops.c | 415 -----------------
>> cmd/Kconfig | 8 +
>> cmd/Makefile | 1 +
>> {board/xilinx/common => cmd}/fru.c | 54 ++-
>> doc/usage/cmd/fru.rst | 144 ++++++
>> doc/usage/index.rst | 1 +
>> include/fru.h | 328 +++++++++++++
>> include/test/suites.h | 1 +
>> lib/Makefile | 1 +
>> lib/fru_ops.c | 724 +++++++++++++++++++++++++++++
>> test/cmd/Makefile | 1 +
>> test/cmd/fru.c | 84 ++++
>> test/cmd_ut.c | 6 +
>> 17 files changed, 1398 insertions(+), 557 deletions(-)
>> delete mode 100644 board/xilinx/common/fru.h
>> delete mode 100644 board/xilinx/common/fru_ops.c
>> rename {board/xilinx/common => cmd}/fru.c (50%)
>> create mode 100644 doc/usage/cmd/fru.rst
>> create mode 100644 include/fru.h
>> create mode 100644 lib/fru_ops.c
>> create mode 100644 test/cmd/fru.c
>>
Will look soon. Was busy with other stuff. Sorry for delay.
M
prev parent reply other threads:[~2022-09-15 14:38 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-25 16:42 [PATCH v4 0/6] cmd/fru: move FRU handling support to common region Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 1/6] xilinx: common: refactor FRU handling support Jae Hyun Yoo
2022-09-21 13:40 ` Michal Simek
2022-09-22 6:39 ` Jae Hyun Yoo
2022-09-22 7:29 ` Michal Simek
2022-09-22 16:22 ` Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 2/6] cmd: fru: move FRU handling support to common region Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 3/6] cmd: fru: fix a sandbox segfault issue Jae Hyun Yoo
2022-09-21 13:07 ` Michal Simek
2022-09-22 6:39 ` Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 4/6] cmd: fru: add product info area parsing support Jae Hyun Yoo
2022-09-21 13:52 ` Michal Simek
2022-09-22 6:39 ` Jae Hyun Yoo
2022-09-22 7:19 ` Michal Simek
2022-09-22 16:15 ` Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 5/6] doc: fru: add documentation for the fru command and APIs Jae Hyun Yoo
2022-09-21 13:54 ` Michal Simek
2022-09-22 6:40 ` Jae Hyun Yoo
2022-08-25 16:42 ` [PATCH v4 6/6] test: cmd: fru: add unit test for the fru command Jae Hyun Yoo
2022-09-15 14:01 ` [PATCH v4 0/6] cmd/fru: move FRU handling support to common region Jae Hyun Yoo
2022-09-15 14:38 ` Michal Simek [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=ccc1b834-ca15-a736-1bb4-caf78e96ef3c@amd.com \
--to=michal.simek@amd.com \
--cc=ashok.reddy.soma@xilinx.com \
--cc=clg@kaod.org \
--cc=jnhuang95@gmail.com \
--cc=macromorgan@hotmail.com \
--cc=mario.six@gdsys.cc \
--cc=masahisa.kojima@linaro.org \
--cc=mr.nuke.me@gmail.com \
--cc=ovidiu.panait@windriver.com \
--cc=pali@kernel.org \
--cc=patrick.delaunay@foss.st.com \
--cc=quic_ggregory@quicinc.com \
--cc=quic_jaehyoo@quicinc.com \
--cc=quic_jiles@quicinc.com \
--cc=roland.gaudig@weidmueller.com \
--cc=sjg@chromium.org \
--cc=thuth@redhat.com \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox