[PATCH v1] arch/arm: sm: introduce efusedump command
Tom Rini
trini at konsulko.com
Tue Dec 13 15:06:25 CET 2022
On Tue, Dec 13, 2022 at 01:59:01PM +0000, Alexey Romanov wrote:
> Hello!
>
> On Mon, Dec 12, 2022 at 11:08:34AM -0500, Tom Rini wrote:
> > On Fri, Dec 09, 2022 at 03:52:04PM +0300, Alexey Romanov wrote:
> >
> > > Using this command user can print efuse memory:
> > >
> > > $ sm efusedump 0 10
> > > 00000000: ff 00 31 00 00 ff 66 00 00 00 ..1...f...
> > >
> > > Signed-off-by: Alexey Romanov <avromanov at sberdevices.ru>
> > > ---
> > > arch/arm/mach-meson/sm.c | 36 +++++++++++++++++++++++++++++++++++-
> > > 1 file changed, 35 insertions(+), 1 deletion(-)
> >
> > This needs to be moved done under cmd/ somewhere, perhaps cmd/meson/
> >
> > --
> > Tom
>
>
> Do you mean it must be moved by a different patchset? From my point of view,
> your comment is totally right, because arch dependent commands are located at
> cmd/ARCH path. But I'm afraid my patchset is just about new helper command.
>
> Anyway, I can send a new different patchset with refactoring suggested by you.
> But, please, comment out some words about new efuse command if possible.
I think your changes are fine, I'd like to see this as a 2 part patch
where part 1 is just moving the code and 2 is adding your changes.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20221213/ce37bc39/attachment.sig>
More information about the U-Boot
mailing list