[U-Boot] [PATCH v2 2/2] Add a cli command to test the TPM device.
Mike Frysinger
vapier at gentoo.org
Sat Oct 15 22:01:54 CEST 2011
On Saturday 15 October 2011 15:44:04 Wolfgang Denk wrote:
> Vadim Bendebury wrote:
> > > Two underscores aren't a good practice.
> >
> > I did this as a result of a previous review. Do you have a suggestion
> > how this should be done instead?
>
> First, and most important, __u_boot_cmd_tpm appears to be undefined in
> your two patches, so it looks to be a real bug.
>
> Second, please read the C standard's section about reserved
> identifiers.
no, this is coming from common u-boot code. look at
include/command.h:U_BOOT_CMD defines.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111015/4d36d91f/attachment.pgp
More information about the U-Boot
mailing list