Fix clang warnings

ble/ble.mk

@@ -27,4 +27,4 @@ BLE_LINKERFILE		:=	$(BLE_PATH)/ble.ld.S
 FORCE:
 
 $(MV_DDR_LIB): FORCE
-	@+make -C $(MV_DDR_PATH) --no-print-directory PLAT_INCLUDES="$(PLAT_INCLUDES)" PLATFORM=$(PLAT) ARCH=AARCH64 OBJ_DIR=$(CURDIR)/$(BUILD_PLAT)/ble
+	@+$(MAKE) -C $(MV_DDR_PATH) --no-print-directory PLAT_INCLUDES="$(PLAT_INCLUDES)" PLATFORM=$(PLAT) ARCH=AARCH64 OBJ_DIR=$(CURDIR)/$(BUILD_PLAT)/ble

drivers/marvell/comphy/phy-comphy-cp110.c

@@ -80,7 +80,10 @@ enum pcie_link_width {
 	PCIE_LNK_WIDTH_UNKNOWN  = 0xFF,
 };
 
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wmissing-braces"
 _Bool rx_trainng_done[AP_NUM][CP_NUM][MAX_LANE_NR] = {0};
+#pragma clang diagnostic pop
 
 static void mvebu_cp110_get_ap_and_cp_nr(uint8_t *ap_nr, uint8_t *cp_nr, uint64_t comphy_base)
 {

drivers/marvell/i2c/a8k_i2c.c

@@ -95,7 +95,7 @@ static int marvell_i2c_interrupt_get(void)
 	/* get the interrupt flag bit */
 	reg = mmio_read_32((uintptr_t)&base->control);
 	reg &= I2C_CONTROL_IFLG;
-	return reg && I2C_CONTROL_IFLG;
+	return reg & I2C_CONTROL_IFLG;
 }
 
 static int marvell_i2c_wait_interrupt(void)

lib/locks/bakery/bakery_lock_normal.c

@@ -67,6 +67,7 @@ IMPORT_SYM(uintptr_t, __PERCPU_BAKERY_LOCK_SIZE__, PERCPU_BAKERY_LOCK_SIZE);
 					    dccivac((uintptr_t)addr)
 
 /* Helper function to check if the lock is acquired */
+__attribute__((unused))
 static inline int is_lock_acquired(const bakery_info_t *my_bakery_info,
 							int is_cached)
 {

plat/common/plat_bl_common.c

@@ -18,8 +18,10 @@
 #pragma weak bl2_el3_plat_prepare_exit
 #pragma weak plat_error_handler
 #pragma weak bl2_plat_preload_setup
+#if LOAD_IMAGE_V2
 #pragma weak bl2_plat_handle_pre_image_load
 #pragma weak bl2_plat_handle_post_image_load
+#endif
 #pragma weak plat_try_next_boot_source
 
 void bl2_el3_plat_prepare_exit(void)

plat/marvell/a8k/common/plat_pm.c

@@ -761,6 +761,8 @@ __dead2 a8k_pwr_domain_pwr_down_wfi(const psci_power_state_t *target_state)
 	 * At this stage any access to DRAM (procedure call) will
 	 * release it from the self-refresh mode
 	 */
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wasm-operand-widths"
 	__asm__ volatile (
 		/* Align to a cache line */
 		"	.balign 64\n\t"
@@ -784,6 +786,7 @@ __dead2 a8k_pwr_domain_pwr_down_wfi(const psci_power_state_t *target_state)
 		: : [srcmd] "r" (srcmd), [sdram_reg] "r" (sdram_reg),
 		    [gpio_addr] "r" (gpio_addr),  [gpio_data] "r" (gpio_data)
 		: "x1");
+#pragma clang diagnostic pop
 
 	panic();
 }

plat/marvell/common/marvell_bl2_setup.c

@@ -47,7 +47,9 @@ static bl2_to_bl31_params_mem_t bl31_params_mem;
 #pragma weak bl2_plat_flush_bl31_params
 #pragma weak bl2_plat_set_bl31_ep_info
 #pragma weak bl2_plat_get_scp_bl2_meminfo
+#ifdef BL32_BASE
 #pragma weak bl2_plat_get_bl32_meminfo
+#endif
 #pragma weak bl2_plat_set_bl32_ep_info
 #pragma weak bl2_plat_get_bl33_meminfo
 #pragma weak bl2_plat_set_bl33_ep_info