clk: mediatek: fhctl: Mark local variables static
authorTom Rix <trix@redhat.com>
Thu, 6 Apr 2023 01:09:35 +0000 (21:09 -0400)
committerStephen Boyd <sboyd@kernel.org>
Mon, 10 Apr 2023 23:57:12 +0000 (16:57 -0700)
smatch reports
drivers/clk/mediatek/clk-fhctl.c:17:27: warning: symbol
  'fhctl_offset_v1' was not declared. Should it be static?
drivers/clk/mediatek/clk-fhctl.c:30:27: warning: symbol
  'fhctl_offset_v2' was not declared. Should it be static?

These variables are only used in one file so should be static.

Signed-off-by: Tom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20230406010935.1944976-1-trix@redhat.com
Fixes: 8da312d6574c ("clk: mediatek: fhctl: Add support for older fhctl register layout")
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/mediatek/clk-fhctl.c

index 45e4842cbf04eb3aae67d3e460807c0edb3b2900..33b6ad8fdc2efa1386199e26fa1f337d51a3fd71 100644 (file)
@@ -14,7 +14,7 @@
 #define PERCENT_TO_DDSLMT(dds, percent_m10) \
        ((((dds) * (percent_m10)) >> 5) / 100)
 
-const struct fhctl_offset fhctl_offset_v1 = {
+static const struct fhctl_offset fhctl_offset_v1 = {
        .offset_hp_en = 0x0,
        .offset_clk_con = 0x4,
        .offset_rst_con = 0x8,
@@ -27,7 +27,7 @@ const struct fhctl_offset fhctl_offset_v1 = {
        .offset_mon = 0x10,
 };
 
-const struct fhctl_offset fhctl_offset_v2 = {
+static const struct fhctl_offset fhctl_offset_v2 = {
        .offset_hp_en = 0x0,
        .offset_clk_con = 0x8,
        .offset_rst_con = 0xc,