From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Mon, 7 Sep 2020 08:31:15 +0200 Subject: [PATCH v3 01/12] binman: Allow entry args to be required In-Reply-To: References: <20200901051400.v3.1.I658eaf634ae1882a849d51ca76de38cdd2f7ff40@changeid> <20200901111405.358391-1-sjg@chromium.org> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Simon, On 05. 09. 20 23:10, Simon Glass wrote: > If an entry argument is needed by an entry but the entry argument is not > present, then a strange error can occur when trying to read the file. > > Fix this by allowing arguments to be required. Select this option for the > cros-ec-rw entry. If a filename is provided in the node, allow that to be > used. > > Also tidy up a few related tests to make the error string easier to find, > and fully ignore unused return values. > > Signed-off-by: Simon Glass > --- > > (no changes since v1) > > tools/binman/README.entries | 7 ++++++- > tools/binman/etype/blob_named_by_arg.py | 10 ++++++---- > tools/binman/etype/cros_ec_rw.py | 3 +-- > tools/binman/ftest.py | 15 +++++++++++---- > 4 files changed, 24 insertions(+), 11 deletions(-) > > Applied to u-boot-dm/next, thanks! Did you start to use patman for sending this? Just curios because I am missing indentation in origin message. Thanks, Michal