media: atomisp: comment an unused code
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sun, 31 May 2020 09:39:17 +0000 (11:39 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Thu, 11 Jun 2020 17:04:26 +0000 (19:04 +0200)
There's a different table for some BYT variants that depend
on something inside a FIXME ifdef.

Place this also inside it, just to shut up a clang-11 warning.

Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/atomisp/pci/atomisp_v4l2.c

index 0c90fc84f7c13e45444324e58228d89a0acc69c8..27d1e9f6a93388bacb67070b844e243956b2a1d2 100644 (file)
@@ -392,6 +392,13 @@ static const struct atomisp_freq_scaling_rule dfs_rules_byt_cr[] = {
        },
 };
 
+#ifdef FIXME
+/*
+ * Disable this, as it is used only when this is true:
+ *     INTEL_MID_BOARD(3, TABLET, BYT, BLK, PRO, CRV2) ||
+ *     INTEL_MID_BOARD(3, TABLET, BYT, BLK, ENG, CRV2))
+ * However, the original code is commented
+ */
 static const struct atomisp_dfs_config dfs_config_byt_cr = {
        .lowest_freq = ISP_FREQ_200MHZ,
        .max_freq_at_vmin = ISP_FREQ_320MHZ,
@@ -399,6 +406,7 @@ static const struct atomisp_dfs_config dfs_config_byt_cr = {
        .dfs_table = dfs_rules_byt_cr,
        .dfs_table_size = ARRAY_SIZE(dfs_rules_byt_cr),
 };
+#endif
 
 static const struct atomisp_freq_scaling_rule dfs_rules_cht[] = {
        {