[U-Boot] [PATCH v2 1/2] Introduce generic TPM support in u-boot
Wolfgang Denk
wd at denx.de
Sun Oct 16 09:35:01 CEST 2011
Dear Vadim Bendebury,
In message <CANy1bu+DZmD_Z=AU8LdTUy7ewHfwMU-8PvH1OA1jba9Tz1xuxA at mail.gmail.com> you wrote:
>
> >> I am not sure what is wrong with a short macro in this case - is this
> >> against the coding style?
> >
> > It doesn't do typechecking.
>
> but the code around it does, doesn't it?
I explained this yesterday, too. Functions are preferred over macros.
In this case ther eis no reason not to use a function.
> Sorry, as I said, I am new here: how does this work on this project -
> does the submitter have to agree to all reviewer's comments? Can I ask
No, you don't have to agree. But we also don't have to accept code
that we don't like ;-)
> somebody else to confirm that using a macro in this case in
> inappropriate?
I already did.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The average woman would rather have beauty than brains, because the
average man can see better than he can think.
More information about the U-Boot
mailing list