Fix MISRA rule 8.3
authorRoberto Vargas <roberto.vargas@arm.com>
Mon, 12 Feb 2018 12:36:17 +0000 (12:36 +0000)
committerRoberto Vargas <roberto.vargas@arm.com>
Tue, 10 Jul 2018 10:17:51 +0000 (11:17 +0100)
Rule 8.3: All declarations of an object or function shall
              use the same names and type qualifiers.

Fixed for:
make DEBUG=1 PLAT=juno ARCH=aarch32 AARCH32_SP=sp_min RESET_TO_SP_MIN=1 JUNO_AARCH32_EL3_RUNTIME=1 bl32

Change-Id: Ia34f5155e1cdb67161191f69e8d1248cbaa39e1a
Signed-off-by: Roberto Vargas <roberto.vargas@arm.com>
include/lib/utils.h
lib/utils/mem_region.c
plat/arm/css/drivers/mhu/css_mhu_doorbell.c
plat/arm/css/drivers/scp/css_pm_scmi.c
plat/arm/css/drivers/scp/css_pm_scpi.c

index f367a1f4581c62c286dcb9b47bb03432533a7af4..5f13e993fd4ee5ac8d7e5c5d782dcd38491c796d 100644 (file)
@@ -37,10 +37,10 @@ void clear_mem_regions(mem_region_t *tbl, size_t nregions);
  * in a way that they minimize the number of entries used in the
  * translation tables.
  */
-void clear_map_dyn_mem_regions(mem_region_t *region,
+void clear_map_dyn_mem_regions(struct mem_region *regions,
                               size_t nregions,
                               uintptr_t va,
-                              size_t chunk_size);
+                              size_t chunk);
 
 /*
  * checks that a region (addr + nbytes-1) of memory is totally covered by
index e9541ba320edade7e552b3d88471ce5ef88861e1..a5c3c61700033d8a81118b29beff55a95f96adcf 100644 (file)
@@ -50,7 +50,7 @@ void clear_mem_regions(mem_region_t *tbl, size_t nregions)
  * be cleared, and chunk is the amount of memory mapped and
  * cleared in every iteration.
  */
-void clear_map_dyn_mem_regions(mem_region_t *regions,
+void clear_map_dyn_mem_regions(struct mem_region *regions,
                               size_t nregions,
                               uintptr_t va,
                               size_t chunk)
index b9faf679b340960011328e1fc5ff7694ebc95730..54f3e057d7e3af421ac3207b86ed74b77e38a990 100644 (file)
@@ -9,7 +9,7 @@
 #include "css_mhu_doorbell.h"
 #include "../scmi/scmi.h"
 
-void mhu_ring_doorbell(scmi_channel_plat_info_t *plat_info)
+void mhu_ring_doorbell(struct scmi_channel_plat_info *plat_info)
 {
        MHU_RING_DOORBELL(plat_info->db_reg_addr,
                        plat_info->db_modify_mask,
@@ -17,7 +17,7 @@ void mhu_ring_doorbell(scmi_channel_plat_info_t *plat_info)
        return;
 }
 
-void mhuv2_ring_doorbell(scmi_channel_plat_info_t *plat_info)
+void mhuv2_ring_doorbell(struct scmi_channel_plat_info *plat_info)
 {
        /* wake receiver */
        MHU_V2_ACCESS_REQUEST(MHUV2_BASE_ADDR);
index 91ea63a44e4f9916a58b135d05428094c2459739..0a5cff66859b5763f2f864ed4baa05f169cc5343 100644 (file)
@@ -142,7 +142,7 @@ void css_scp_suspend(const struct psci_power_state *target_state)
  * Helper function to turn off a CPU power domain and its parent power domains
  * if applicable.
  */
-void css_scp_off(const psci_power_state_t *target_state)
+void css_scp_off(const struct psci_power_state *target_state)
 {
        int lvl = 0, ret;
        uint32_t scmi_pwr_state = 0;
index 18e71f6e3ca3bbbcd3cf61dcfdf6ed442eb49809..123d54f4e5140b6f9266cd750c40e8c146fa8bc4 100644 (file)
@@ -47,7 +47,7 @@ void css_scp_suspend(const struct psci_power_state *target_state)
  * if applicable. Since SCPI doesn't differentiate between OFF and suspend, we
  * call the suspend helper here.
  */
-void css_scp_off(const psci_power_state_t *target_state)
+void css_scp_off(const struct psci_power_state *target_state)
 {
        css_scp_suspend(target_state);
 }