CC: [email protected] CC: Kalle Valo <[email protected]> CC: [email protected] CC: [email protected] TO: Anilkumar Kolli <[email protected]> CC: Kalle Valo <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git pending head: 34cbb4043dca455fca888e1ced323e588912b6a2 commit: c18d89a2ec8070e904a6db3b111a761a5becd620 [46/56] ath11k: Use reserved host DDR addresses from DT for PCI devices :::::: branch date: 25 hours ago :::::: commit date: 33 hours ago config: riscv-randconfig-m031-20211210 (https://download.01.org/0day-ci/archive/20211211/[email protected]/config) compiler: riscv64-linux-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> New smatch warnings: drivers/net/wireless/ath/ath11k/qmi.c:1891 ath11k_qmi_assign_target_mem_chunk() error: uninitialized symbol 'ret'. Old smatch warnings: drivers/net/wireless/ath/ath11k/qmi.c:2161 ath11k_qmi_load_file_target_mem() error: uninitialized symbol 'ret'. vim +/ret +1891 drivers/net/wireless/ath/ath11k/qmi.c 6eb6ea51382873 Govind Singh 2020-08-14 1875 6eb6ea51382873 Govind Singh 2020-08-14 1876 static int ath11k_qmi_assign_target_mem_chunk(struct ath11k_base *ab) d5c65159f28953 Kalle Valo 2019-11-23 1877 { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1878 struct device *dev = ab->dev; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1879 struct device_node *hremote_node = NULL; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1880 struct resource res; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1881 u32 host_ddr_sz; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1882 int i, idx, ret; d5c65159f28953 Kalle Valo 2019-11-23 1883 d5c65159f28953 Kalle Valo 2019-11-23 1884 for (i = 0, idx = 0; i < ab->qmi.mem_seg_count; i++) { d5c65159f28953 Kalle Valo 2019-11-23 1885 switch (ab->qmi.target_mem[i].type) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1886 case HOST_DDR_REGION_TYPE: c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1887 hremote_node = of_parse_phandle(dev->of_node, "memory-region", 0); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1888 if (!hremote_node) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1889 ath11k_dbg(ab, ATH11K_DBG_QMI, c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1890 "qmi fail to get hremote_node\n"); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 @1891 return ret; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1892 } c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1893 c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1894 ret = of_address_to_resource(hremote_node, 0, &res); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1895 if (ret) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1896 ath11k_dbg(ab, ATH11K_DBG_QMI, c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1897 "qmi fail to get reg from hremote\n"); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1898 return ret; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1899 } c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1900 c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1901 if (res.end - res.start + 1 < ab->qmi.target_mem[i].size) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1902 ath11k_dbg(ab, ATH11K_DBG_QMI, c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1903 "qmi fail to assign memory of sz\n"); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1904 return -EINVAL; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1905 } c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1906 c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1907 ab->qmi.target_mem[idx].paddr = res.start; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1908 ab->qmi.target_mem[idx].iaddr = c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1909 ioremap(ab->qmi.target_mem[idx].paddr, c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1910 ab->qmi.target_mem[i].size); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1911 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1912 host_ddr_sz = ab->qmi.target_mem[i].size; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1913 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1914 idx++; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1915 break; d5c65159f28953 Kalle Valo 2019-11-23 1916 case BDF_MEM_REGION_TYPE: 3b94ae4c62db0b Anilkumar Kolli 2020-06-16 1917 ab->qmi.target_mem[idx].paddr = ab->hw_params.bdf_addr; 6eb6ea51382873 Govind Singh 2020-08-14 1918 ab->qmi.target_mem[idx].vaddr = NULL; d5c65159f28953 Kalle Valo 2019-11-23 1919 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size; d5c65159f28953 Kalle Valo 2019-11-23 1920 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type; d5c65159f28953 Kalle Valo 2019-11-23 1921 idx++; d5c65159f28953 Kalle Valo 2019-11-23 1922 break; d5c65159f28953 Kalle Valo 2019-11-23 1923 case CALDB_MEM_REGION_TYPE: d5c65159f28953 Kalle Valo 2019-11-23 1924 if (ab->qmi.target_mem[i].size > ATH11K_QMI_CALDB_SIZE) { d5c65159f28953 Kalle Valo 2019-11-23 1925 ath11k_warn(ab, "qmi mem size is low to load caldata\n"); d5c65159f28953 Kalle Valo 2019-11-23 1926 return -EINVAL; d5c65159f28953 Kalle Valo 2019-11-23 1927 } 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1928 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1929 if (ath11k_cold_boot_cal && ab->hw_params.cold_boot_calib) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1930 if (hremote_node) { c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1931 ab->qmi.target_mem[idx].paddr = c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1932 res.start + host_ddr_sz; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1933 ab->qmi.target_mem[idx].iaddr = c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1934 ioremap(ab->qmi.target_mem[idx].paddr, c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1935 ab->qmi.target_mem[i].size); c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1936 } else { 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1937 ab->qmi.target_mem[idx].paddr = 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1938 ATH11K_QMI_CALDB_ADDRESS; c18d89a2ec8070 Anilkumar Kolli 2021-12-09 1939 } 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1940 } else { d5c65159f28953 Kalle Valo 2019-11-23 1941 ab->qmi.target_mem[idx].paddr = 0; 6eb6ea51382873 Govind Singh 2020-08-14 1942 ab->qmi.target_mem[idx].vaddr = NULL; 02f9d3c1b918a6 Govindaraj Saminathan 2020-10-16 1943 } d5c65159f28953 Kalle Valo 2019-11-23 1944 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size; d5c65159f28953 Kalle Valo 2019-11-23 1945 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type; d5c65159f28953 Kalle Valo 2019-11-23 1946 idx++; d5c65159f28953 Kalle Valo 2019-11-23 1947 break; d5c65159f28953 Kalle Valo 2019-11-23 1948 default: d5c65159f28953 Kalle Valo 2019-11-23 1949 ath11k_warn(ab, "qmi ignore invalid mem req type %d\n", d5c65159f28953 Kalle Valo 2019-11-23 1950 ab->qmi.target_mem[i].type); d5c65159f28953 Kalle Valo 2019-11-23 1951 break; d5c65159f28953 Kalle Valo 2019-11-23 1952 } d5c65159f28953 Kalle Valo 2019-11-23 1953 } d5c65159f28953 Kalle Valo 2019-11-23 1954 ab->qmi.mem_seg_count = idx; d5c65159f28953 Kalle Valo 2019-11-23 1955 d5c65159f28953 Kalle Valo 2019-11-23 1956 return 0; d5c65159f28953 Kalle Valo 2019-11-23 1957 } d5c65159f28953 Kalle Valo 2019-11-23 1958 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/[email protected] _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
