This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: atomisp: make dfs_config_merr_117a struct const
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Sat May 9 09:48:37 2020 +0200

This setting is used only for one of te Merryfield PCI IDs.

As this is an ISP2400, we can just get rid of a version
test, writing the right value directly inside the struct.

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/staging/media/atomisp/pci/atomisp_v4l2.c | 17 ++---------------
 1 file changed, 2 insertions(+), 15 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c 
b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
index b5c3724047c5..9914e05d4fe4 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
@@ -217,7 +217,7 @@ static const struct atomisp_dfs_config dfs_config_merr_1179 
= {
        .dfs_table_size = ARRAY_SIZE(dfs_rules_merr_1179),
 };
 
-static struct atomisp_freq_scaling_rule dfs_rules_merr_117a[] = {
+static const struct atomisp_freq_scaling_rule dfs_rules_merr_117a[] = {
        {
                .width = 1920,
                .height = 1080,
@@ -229,11 +229,7 @@ static struct atomisp_freq_scaling_rule 
dfs_rules_merr_117a[] = {
                .width = 1080,
                .height = 1920,
                .fps = 30,
-               /*
-                * FIXME: this is weird, but .isp_freq depends on
-                * the chip being ISP2400 or ISP2401. So, this should be
-                * initialized on runtime.
-                */
+               .isp_freq = ISP_FREQ_266MHZ,
                .run_mode = ATOMISP_RUN_MODE_VIDEO,
        },
        {
@@ -1629,15 +1625,6 @@ static int atomisp_pci_probe(struct pci_dev *dev,
                        isp->dfs = &dfs_config_merr_1179;
                        break;
                case ATOMISP_PCI_DEVICE_SOC_MRFLD_117A:
-                       /*
-                        * FIXME: This should likely be uneeded. Either one
-                        * value is likely the correct one for this resolution
-                        */
-                       if (!atomisp_hw_is_isp2401)
-                               dfs_rules_merr_117a[1].isp_freq = 
ISP_FREQ_266MHZ;
-                       else
-                               dfs_rules_merr_117a[1].isp_freq = 
ISP_FREQ_400MHZ;
-
                        isp->dfs = &dfs_config_merr_117a;
 
                        break;

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to