Hi Yannic, On Tue, 27 May 2025 at 14:24, Yannic Moog <y.m...@phytec.de> wrote: > > _entries, as highlighted by leading _, is a private member and thus > should not be accessed directly. As a result use the GetEntries function > instead. > GetEntries() returns _entries. No functional change introduced in this > commit. > > Signed-off-by: Yannic Moog <y.m...@phytec.de> > --- > tools/binman/etype/atf_fip.py | 4 ++-- > tools/binman/etype/blob_phase.py | 4 ++-- > tools/binman/etype/cbfs.py | 12 +++++----- > tools/binman/etype/efi_capsule.py | 2 +- > tools/binman/etype/fit.py | 4 ++-- > tools/binman/etype/mkimage.py | 4 ++-- > tools/binman/etype/nxp_imx8mcst.py | 2 +- > tools/binman/etype/nxp_imx8mimage.py | 2 +- > tools/binman/etype/section.py | 44 > +++++++++++++++++------------------ > tools/binman/etype/ti_board_config.py | 8 +++---- > tools/binman/ftest.py | 2 +- > 11 files changed, 44 insertions(+), 44 deletions(-)
Are you sure this is right? The classes which use _entries are subclasses, except for ftest.py which is test code where we should probably add: #pylint: disable=protected-access Regards, Simon
