ANDROID: Use -Wall and more when building libfdt am: 170bf1d4ce am: 350d496637 am: 98f5673dc6 am: 7a55909ac6

Original change: https://android-review.googlesource.com/c/platform/external/dtc/+/2784258

Change-Id: I0ee033f59b97bc940f04ed95f4461350ddb65019
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Android.bp

@@ -44,14 +44,21 @@ license {
 }
 
 cc_defaults {
-    name: "dt_defaults",
+    name: "dtc_cflags_defaults",
     cflags: [
         "-Wall",
         "-Werror",
         "-Wno-macro-redefined",
-        "-Wno-sign-compare",
         "-Wno-missing-field-initializers",
+        "-Wno-sign-compare",
         "-Wno-unused-parameter",
+    ],
+}
+
+cc_defaults {
+    name: "dt_defaults",
+    defaults: ["dtc_cflags_defaults"],
+    cflags: [
         "-DNO_YAML"
     ],
 

BUILD.bazel

@@ -1,26 +1,22 @@
+COPTS = [
+    "-Wall",
+    "-Werror",
+    "-Wno-macro-redefined",
+    "-Wno-missing-field-initializers",
+    "-Wno-sign-compare",
+    "-Wno-unused-parameter",
+]
+
 cc_library(
     name = "libfdt",
     srcs = glob([
         "libfdt/*.h",
         "libfdt/*.c",
     ]),
-    copts = [
+    copts = COPTS,
-        "-Werror",
-        "-Wno-macro-redefined",
-        "-Wno-sign-compare",
-    ],
     includes = ["libfdt"],
 )
 
-COPTS = [
-    "-Wall",
-    "-Werror",
-    "-Wno-macro-redefined",
-    "-Wno-sign-compare",
-    "-Wno-missing-field-initializers",
-    "-Wno-unused-parameter",
-]
-
 genrule(
     name = "lexer",
     srcs = [

libfdt/Android.bp

@@ -7,12 +7,7 @@ package {
 cc_library {
     name: "libfdt",
     host_supported: true,
-
+    defaults: ["dtc_cflags_defaults"],
-    cflags: [
-        "-Werror",
-        "-Wno-macro-redefined",
-        "-Wno-sign-compare",
-    ],
     srcs: [
         "fdt.c",
         "fdt_check.c",