Re: [v2 11/16] lib/sha*.c: Update header list

2023-12-16 Thread Simon Glass
On Thu, 14 Dec 2023 at 11:18, Tom Rini  wrote:
>
> Cleanup the list of headers we include here. For the tools build we only
> need to exclude  as that's used by the target build for the
> prototype for schedule(), and we don't need to get that via
> . We can also make use of our  intentionally
> existing as a redirection to  to reduce ifdef'd lines.
>
> Signed-off-by: Tom Rini 
> ---
>  lib/sha1.c   | 7 ++-
>  lib/sha256.c | 7 ++-
>  lib/sha512.c | 6 +-
>  3 files changed, 5 insertions(+), 15 deletions(-)

Reviewed-by: Simon Glass 


[v2 11/16] lib/sha*.c: Update header list

2023-12-14 Thread Tom Rini
Cleanup the list of headers we include here. For the tools build we only
need to exclude  as that's used by the target build for the
prototype for schedule(), and we don't need to get that via
. We can also make use of our  intentionally
existing as a redirection to  to reduce ifdef'd lines.

Signed-off-by: Tom Rini 
---
 lib/sha1.c   | 7 ++-
 lib/sha256.c | 7 ++-
 lib/sha512.c | 6 +-
 3 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/lib/sha1.c b/lib/sha1.c
index 8d0740789349..7ef536f4b5db 100644
--- a/lib/sha1.c
+++ b/lib/sha1.c
@@ -17,12 +17,9 @@
 #endif
 
 #ifndef USE_HOSTCC
-#include 
-#include 
-#else
-#include 
+#include 
 #endif /* USE_HOSTCC */
-#include 
+#include 
 #include 
 
 #include 
diff --git a/lib/sha256.c b/lib/sha256.c
index 4d26aea1c8ce..665ba6f152e5 100644
--- a/lib/sha256.c
+++ b/lib/sha256.c
@@ -6,12 +6,9 @@
  */
 
 #ifndef USE_HOSTCC
-#include 
-#include 
-#else
-#include 
+#include 
 #endif /* USE_HOSTCC */
-#include 
+#include 
 #include 
 
 #include 
diff --git a/lib/sha512.c b/lib/sha512.c
index fbe8d5f5bfe9..ffe2c5cd9642 100644
--- a/lib/sha512.c
+++ b/lib/sha512.c
@@ -11,13 +11,9 @@
  */
 
 #ifndef USE_HOSTCC
-#include 
-#include 
-#else
-#include 
+#include 
 #endif /* USE_HOSTCC */
 #include 
-#include 
 #include 
 
 const uint8_t sha384_der_prefix[SHA384_DER_LEN] = {
-- 
2.34.1