From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BD7EEC001DB for ; Tue, 8 Aug 2023 14:27:21 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 19D1886297; Tue, 8 Aug 2023 16:27:20 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=collabora.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=collabora.com header.i=@collabora.com header.b="fe9L47f/"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 4927986297; Tue, 8 Aug 2023 16:27:19 +0200 (CEST) Received: from madras.collabora.co.uk (madras.collabora.co.uk [46.235.227.172]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 6D4ED8622C for ; Tue, 8 Aug 2023 16:27:17 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=collabora.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=detlev.casanova@collabora.com Received: from arisu.localnet (mtl.collabora.ca [66.171.169.34]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: detlev) by madras.collabora.co.uk (Postfix) with ESMTPSA id 7E10B6607207; Tue, 8 Aug 2023 15:27:16 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1691504837; bh=BRqA5qcAlQ9egRr48n/p4ak4WQyhSdgOKLuPvEAcHyA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fe9L47f/voJlFMN+jA/vBBbDbtASvlFjP1fmzIItULaqsFu4ZDW74OGGBri+Qxkvl COOqdcVDxE52cU0KmIUuD4v7OHZ/nb7dHR+bml0sIkc41Vgfgn44bqQPDUavlrCvjr r2pjKtflUK8/XfX7fosQLNLzzIMKmpSVOtNMXdT2lPYw4tbHRh0q/Jj2gbnUEqWbkp TSkwltVfvPgTlAPpQTeDlm3A+UFiX8wrabgLI9gy02v2jqgdbfQtkZ+omvRpGWnz3A SLo3qlMnbc2gfhavNGV8nDwz/uEKpDRLDgppaXoUVMyYQeKdXi+nwKW8FUz7yy2o5Q NxDneebm0Qmgg== From: Detlev Casanova To: Peter Robinson Cc: u-boot@lists.denx.de, Marek Vasut , Hai Pham , Tam Nguyen , Simon Glass Subject: Re: [PATCH v4 0/6] Introduce the sysinfo command Date: Tue, 08 Aug 2023 10:27:20 -0400 Message-ID: <12376909.O9o76ZdvQC@arisu> In-Reply-To: References: <20230720145010.84791-1-detlev.casanova@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean On Tuesday, August 8, 2023 10:21:11 A.M. EDT Peter Robinson wrote: > On Thu, Jul 20, 2023 at 3:50=E2=80=AFPM Detlev Casanova >=20 > wrote: > > The command can be used to show various information that can be used to > > identify the running system. > >=20 > > Currently supported subcommands are: > > * model: A string representing the model > > * id: The id of the board > > * revision: The revision of this board. >=20 > How does this differ from the bdinfo and some of the other commands > that provide similar info? If I'm not wrong, the bdinfo command is about showing what hardware is present on the board. The sysinfo command is about showing board identification information. =20 > > Changes since v3: > > - Fix documentation typo. > >=20 > > Changes since v2: > > - Fix code style. > > - Use printf() instead of debug(). > > - Clarify sysinfo new ids types (int). > > - Add a test for sysinfo command. > > - Add documentation for sysinfo command. > >=20 > > Changes since v1: > > - Removed shell function to select linux device tree. This will be > > =20 > > distributions job. > > =20 > > - Break revision in rev_major and rev_minor in the sysinfo driver. > >=20 > > Detlev Casanova (6): > > sysinfo: Add IDs for board id and revision > > cmd: Add a sysinfo command > > sysinfo: Add a test > > sysinfo: Add documentation > > sysinfo: rcar3: Use int instead of char for revision > > sysinfo: rcar3: Implement BOARD_ID and BOARD_REV_* > > =20 > > cmd/Kconfig | 6 ++ > > cmd/Makefile | 1 + > > cmd/sysinfo.c | 133 +++++++++++++++++++++++++++++++++++ > > configs/sandbox_defconfig | 1 + > > doc/usage/cmd/sysinfo.rst | 56 +++++++++++++++ > > drivers/sysinfo/rcar3.c | 141 ++++++++++++++++++++++++++------------ > > drivers/sysinfo/sandbox.c | 17 +++++ > > include/sysinfo.h | 5 ++ > > test/cmd/Makefile | 1 + > > test/cmd/test_sysinfo.c | 51 ++++++++++++++ > > 10 files changed, 370 insertions(+), 42 deletions(-) > > create mode 100644 cmd/sysinfo.c > > create mode 100644 doc/usage/cmd/sysinfo.rst > > create mode 100644 test/cmd/test_sysinfo.c > >=20 > > -- > > 2.41.0