* Re: [PATCH v7 3/4] phy: Add Sparx5 ethernet serdes PHY driver
[not found] <20201202130438.3330228-4-steen.hegelund@microchip.com>
@ 2020-12-02 15:09 ` kernel test robot
2020-12-02 21:24 ` kernel test robot
1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2020-12-02 15:09 UTC (permalink / raw)
To: Steen Hegelund, Kishon Vijay Abraham I, Vinod Koul, linux-arm-kernel
Cc: Bjarni Jonasson, Alexandre Belloni, kbuild-all, Steen Hegelund,
linux-kernel, Microchip UNG Driver List, Lars Povlsen
[-- Attachment #1: Type: text/plain, Size: 18152 bytes --]
Hi Steen,
I love your patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next]
[also build test WARNING on linux/master linus/master v5.10-rc6 next-20201201]
[cannot apply to phy/next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Steen-Hegelund/Adding-the-Sparx5-Serdes-driver/20201202-211116
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: sh-allmodconfig (attached as .config)
compiler: sh4-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/f9670130c2b0d7ccd38a459e66d8da36b8edb7e0
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Steen-Hegelund/Adding-the-Sparx5-Serdes-driver/20201202-211116
git checkout f9670130c2b0d7ccd38a459e66d8da36b8edb7e0
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sh
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/phy/microchip/sparx5_serdes.c:2236:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046660608' to '276856832' [-Woverflow]
2236 | { TARGET_SD_CMU, 0x610808000 }, /* sd_cmu_0 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2237:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046693376' to '276889600' [-Woverflow]
2237 | { TARGET_SD_CMU + 1, 0x610810000 }, /* sd_cmu_1 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2238:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046726144' to '276922368' [-Woverflow]
2238 | { TARGET_SD_CMU + 2, 0x610818000 }, /* sd_cmu_2 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2239:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046758912' to '276955136' [-Woverflow]
2239 | { TARGET_SD_CMU + 3, 0x610820000 }, /* sd_cmu_3 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2240:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046791680' to '276987904' [-Woverflow]
2240 | { TARGET_SD_CMU + 4, 0x610828000 }, /* sd_cmu_4 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2241:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046824448' to '277020672' [-Woverflow]
2241 | { TARGET_SD_CMU + 5, 0x610830000 }, /* sd_cmu_5 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2242:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046857216' to '277053440' [-Woverflow]
2242 | { TARGET_SD_CMU + 6, 0x610838000 }, /* sd_cmu_6 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2243:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046889984' to '277086208' [-Woverflow]
2243 | { TARGET_SD_CMU + 7, 0x610840000 }, /* sd_cmu_7 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2244:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046922752' to '277118976' [-Woverflow]
2244 | { TARGET_SD_CMU + 8, 0x610848000 }, /* sd_cmu_8 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2245:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046955520' to '277151744' [-Woverflow]
2245 | { TARGET_SD_CMU_CFG, 0x610850000 }, /* sd_cmu_cfg_0 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2246:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26046988288' to '277184512' [-Woverflow]
2246 | { TARGET_SD_CMU_CFG + 1, 0x610858000 }, /* sd_cmu_cfg_1 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2247:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047021056' to '277217280' [-Woverflow]
2247 | { TARGET_SD_CMU_CFG + 2, 0x610860000 }, /* sd_cmu_cfg_2 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2248:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047053824' to '277250048' [-Woverflow]
2248 | { TARGET_SD_CMU_CFG + 3, 0x610868000 }, /* sd_cmu_cfg_3 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2249:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047086592' to '277282816' [-Woverflow]
2249 | { TARGET_SD_CMU_CFG + 4, 0x610870000 }, /* sd_cmu_cfg_4 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2250:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047119360' to '277315584' [-Woverflow]
2250 | { TARGET_SD_CMU_CFG + 5, 0x610878000 }, /* sd_cmu_cfg_5 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2251:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047152128' to '277348352' [-Woverflow]
2251 | { TARGET_SD_CMU_CFG + 6, 0x610880000 }, /* sd_cmu_cfg_6 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2252:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047184896' to '277381120' [-Woverflow]
2252 | { TARGET_SD_CMU_CFG + 7, 0x610888000 }, /* sd_cmu_cfg_7 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2253:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047217664' to '277413888' [-Woverflow]
2253 | { TARGET_SD_CMU_CFG + 8, 0x610890000 }, /* sd_cmu_cfg_8 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2254:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047250432' to '277446656' [-Woverflow]
2254 | { TARGET_SD6G_LANE, 0x610898000 }, /* sd6g_lane_0 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2255:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047283200' to '277479424' [-Woverflow]
2255 | { TARGET_SD6G_LANE + 1, 0x6108a0000 }, /* sd6g_lane_1 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2256:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047315968' to '277512192' [-Woverflow]
2256 | { TARGET_SD6G_LANE + 2, 0x6108a8000 }, /* sd6g_lane_2 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2257:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047348736' to '277544960' [-Woverflow]
2257 | { TARGET_SD6G_LANE + 3, 0x6108b0000 }, /* sd6g_lane_3 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2258:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047381504' to '277577728' [-Woverflow]
2258 | { TARGET_SD6G_LANE + 4, 0x6108b8000 }, /* sd6g_lane_4 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2259:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047414272' to '277610496' [-Woverflow]
2259 | { TARGET_SD6G_LANE + 5, 0x6108c0000 }, /* sd6g_lane_5 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2260:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047447040' to '277643264' [-Woverflow]
2260 | { TARGET_SD6G_LANE + 6, 0x6108c8000 }, /* sd6g_lane_6 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2261:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047479808' to '277676032' [-Woverflow]
2261 | { TARGET_SD6G_LANE + 7, 0x6108d0000 }, /* sd6g_lane_7 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2262:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047512576' to '277708800' [-Woverflow]
2262 | { TARGET_SD6G_LANE + 8, 0x6108d8000 }, /* sd6g_lane_8 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2263:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047545344' to '277741568' [-Woverflow]
2263 | { TARGET_SD6G_LANE + 9, 0x6108e0000 }, /* sd6g_lane_9 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2264:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047578112' to '277774336' [-Woverflow]
2264 | { TARGET_SD6G_LANE + 10, 0x6108e8000 }, /* sd6g_lane_10 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2265:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047610880' to '277807104' [-Woverflow]
2265 | { TARGET_SD6G_LANE + 11, 0x6108f0000 }, /* sd6g_lane_11 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2266:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047643648' to '277839872' [-Woverflow]
2266 | { TARGET_SD6G_LANE + 12, 0x6108f8000 }, /* sd6g_lane_12 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2267:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047676416' to '277872640' [-Woverflow]
2267 | { TARGET_SD10G_LANE, 0x610900000 }, /* sd10g_lane_0 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2268:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047709184' to '277905408' [-Woverflow]
2268 | { TARGET_SD10G_LANE + 1, 0x610908000 }, /* sd10g_lane_1 */
| ^~~~~~~~~~~
drivers/phy/microchip/sparx5_serdes.c:2269:28: warning: unsigned conversion from 'long long int' to 'unsigned int' changes value from '26047741952' to '277938176' [-Woverflow]
2269 | { TARGET_SD10G_LANE + 2, 0x610910000 }, /* sd10g_lane_2 */
vim +2236 drivers/phy/microchip/sparx5_serdes.c
2234
2235 static struct sparx5_serdes_io_resource sparx5_serdes_iomap[] = {
> 2236 { TARGET_SD_CMU, 0x610808000 }, /* sd_cmu_0 */
2237 { TARGET_SD_CMU + 1, 0x610810000 }, /* sd_cmu_1 */
2238 { TARGET_SD_CMU + 2, 0x610818000 }, /* sd_cmu_2 */
2239 { TARGET_SD_CMU + 3, 0x610820000 }, /* sd_cmu_3 */
2240 { TARGET_SD_CMU + 4, 0x610828000 }, /* sd_cmu_4 */
2241 { TARGET_SD_CMU + 5, 0x610830000 }, /* sd_cmu_5 */
2242 { TARGET_SD_CMU + 6, 0x610838000 }, /* sd_cmu_6 */
2243 { TARGET_SD_CMU + 7, 0x610840000 }, /* sd_cmu_7 */
2244 { TARGET_SD_CMU + 8, 0x610848000 }, /* sd_cmu_8 */
2245 { TARGET_SD_CMU_CFG, 0x610850000 }, /* sd_cmu_cfg_0 */
2246 { TARGET_SD_CMU_CFG + 1, 0x610858000 }, /* sd_cmu_cfg_1 */
2247 { TARGET_SD_CMU_CFG + 2, 0x610860000 }, /* sd_cmu_cfg_2 */
2248 { TARGET_SD_CMU_CFG + 3, 0x610868000 }, /* sd_cmu_cfg_3 */
2249 { TARGET_SD_CMU_CFG + 4, 0x610870000 }, /* sd_cmu_cfg_4 */
2250 { TARGET_SD_CMU_CFG + 5, 0x610878000 }, /* sd_cmu_cfg_5 */
2251 { TARGET_SD_CMU_CFG + 6, 0x610880000 }, /* sd_cmu_cfg_6 */
2252 { TARGET_SD_CMU_CFG + 7, 0x610888000 }, /* sd_cmu_cfg_7 */
2253 { TARGET_SD_CMU_CFG + 8, 0x610890000 }, /* sd_cmu_cfg_8 */
2254 { TARGET_SD6G_LANE, 0x610898000 }, /* sd6g_lane_0 */
2255 { TARGET_SD6G_LANE + 1, 0x6108a0000 }, /* sd6g_lane_1 */
2256 { TARGET_SD6G_LANE + 2, 0x6108a8000 }, /* sd6g_lane_2 */
2257 { TARGET_SD6G_LANE + 3, 0x6108b0000 }, /* sd6g_lane_3 */
2258 { TARGET_SD6G_LANE + 4, 0x6108b8000 }, /* sd6g_lane_4 */
2259 { TARGET_SD6G_LANE + 5, 0x6108c0000 }, /* sd6g_lane_5 */
2260 { TARGET_SD6G_LANE + 6, 0x6108c8000 }, /* sd6g_lane_6 */
2261 { TARGET_SD6G_LANE + 7, 0x6108d0000 }, /* sd6g_lane_7 */
2262 { TARGET_SD6G_LANE + 8, 0x6108d8000 }, /* sd6g_lane_8 */
2263 { TARGET_SD6G_LANE + 9, 0x6108e0000 }, /* sd6g_lane_9 */
2264 { TARGET_SD6G_LANE + 10, 0x6108e8000 }, /* sd6g_lane_10 */
2265 { TARGET_SD6G_LANE + 11, 0x6108f0000 }, /* sd6g_lane_11 */
2266 { TARGET_SD6G_LANE + 12, 0x6108f8000 }, /* sd6g_lane_12 */
2267 { TARGET_SD10G_LANE, 0x610900000 }, /* sd10g_lane_0 */
2268 { TARGET_SD10G_LANE + 1, 0x610908000 }, /* sd10g_lane_1 */
2269 { TARGET_SD10G_LANE + 2, 0x610910000 }, /* sd10g_lane_2 */
2270 { TARGET_SD10G_LANE + 3, 0x610918000 }, /* sd10g_lane_3 */
2271 { TARGET_SD_LANE, 0x6109a8000 }, /* sd_lane_0 */
2272 { TARGET_SD_LANE + 1, 0x6109b0000 }, /* sd_lane_1 */
2273 { TARGET_SD_LANE + 2, 0x6109b8000 }, /* sd_lane_2 */
2274 { TARGET_SD_LANE + 3, 0x6109c0000 }, /* sd_lane_3 */
2275 { TARGET_SD_LANE + 4, 0x6109c8000 }, /* sd_lane_4 */
2276 { TARGET_SD_LANE + 5, 0x6109d0000 }, /* sd_lane_5 */
2277 { TARGET_SD_LANE + 6, 0x6109d8000 }, /* sd_lane_6 */
2278 { TARGET_SD_LANE + 7, 0x6109e0000 }, /* sd_lane_7 */
2279 { TARGET_SD_LANE + 8, 0x6109e8000 }, /* sd_lane_8 */
2280 { TARGET_SD_LANE + 9, 0x6109f0000 }, /* sd_lane_9 */
2281 { TARGET_SD_LANE + 10, 0x6109f8000 }, /* sd_lane_10 */
2282 { TARGET_SD_LANE + 11, 0x610a00000 }, /* sd_lane_11 */
2283 { TARGET_SD_LANE + 12, 0x610a08000 }, /* sd_lane_12 */
2284 { TARGET_SD_LANE + 13, 0x610a10000 }, /* sd_lane_13 */
2285 { TARGET_SD_LANE + 14, 0x610a18000 }, /* sd_lane_14 */
2286 { TARGET_SD_LANE + 15, 0x610a20000 }, /* sd_lane_15 */
2287 { TARGET_SD_LANE + 16, 0x610a28000 }, /* sd_lane_16 */
2288 { TARGET_SD_CMU + 9, 0x610c08000 }, /* sd_cmu_9 */
2289 { TARGET_SD_CMU + 10, 0x610c10000 }, /* sd_cmu_10 */
2290 { TARGET_SD_CMU + 11, 0x610c18000 }, /* sd_cmu_11 */
2291 { TARGET_SD_CMU + 12, 0x610c20000 }, /* sd_cmu_12 */
2292 { TARGET_SD_CMU + 13, 0x610c28000 }, /* sd_cmu_13 */
2293 { TARGET_SD_CMU_CFG + 9, 0x610c30000 }, /* sd_cmu_cfg_9 */
2294 { TARGET_SD_CMU_CFG + 10, 0x610c38000 }, /* sd_cmu_cfg_10 */
2295 { TARGET_SD_CMU_CFG + 11, 0x610c40000 }, /* sd_cmu_cfg_11 */
2296 { TARGET_SD_CMU_CFG + 12, 0x610c48000 }, /* sd_cmu_cfg_12 */
2297 { TARGET_SD_CMU_CFG + 13, 0x610c50000 }, /* sd_cmu_cfg_13 */
2298 { TARGET_SD10G_LANE + 4, 0x610c58000 }, /* sd10g_lane_4 */
2299 { TARGET_SD10G_LANE + 5, 0x610c60000 }, /* sd10g_lane_5 */
2300 { TARGET_SD10G_LANE + 6, 0x610c68000 }, /* sd10g_lane_6 */
2301 { TARGET_SD10G_LANE + 7, 0x610c70000 }, /* sd10g_lane_7 */
2302 { TARGET_SD10G_LANE + 8, 0x610c78000 }, /* sd10g_lane_8 */
2303 { TARGET_SD10G_LANE + 9, 0x610c80000 }, /* sd10g_lane_9 */
2304 { TARGET_SD10G_LANE + 10, 0x610c88000 }, /* sd10g_lane_10 */
2305 { TARGET_SD10G_LANE + 11, 0x610c90000 }, /* sd10g_lane_11 */
2306 { TARGET_SD25G_LANE, 0x610c98000 }, /* sd25g_lane_0 */
2307 { TARGET_SD25G_LANE + 1, 0x610ca0000 }, /* sd25g_lane_1 */
2308 { TARGET_SD25G_LANE + 2, 0x610ca8000 }, /* sd25g_lane_2 */
2309 { TARGET_SD25G_LANE + 3, 0x610cb0000 }, /* sd25g_lane_3 */
2310 { TARGET_SD25G_LANE + 4, 0x610cb8000 }, /* sd25g_lane_4 */
2311 { TARGET_SD25G_LANE + 5, 0x610cc0000 }, /* sd25g_lane_5 */
2312 { TARGET_SD25G_LANE + 6, 0x610cc8000 }, /* sd25g_lane_6 */
2313 { TARGET_SD25G_LANE + 7, 0x610cd0000 }, /* sd25g_lane_7 */
2314 { TARGET_SD_LANE + 17, 0x610d58000 }, /* sd_lane_17 */
2315 { TARGET_SD_LANE + 18, 0x610d60000 }, /* sd_lane_18 */
2316 { TARGET_SD_LANE + 19, 0x610d68000 }, /* sd_lane_19 */
2317 { TARGET_SD_LANE + 20, 0x610d70000 }, /* sd_lane_20 */
2318 { TARGET_SD_LANE + 21, 0x610d78000 }, /* sd_lane_21 */
2319 { TARGET_SD_LANE + 22, 0x610d80000 }, /* sd_lane_22 */
2320 { TARGET_SD_LANE + 23, 0x610d88000 }, /* sd_lane_23 */
2321 { TARGET_SD_LANE + 24, 0x610d90000 }, /* sd_lane_24 */
2322 { TARGET_SD_LANE_25G, 0x610d98000 }, /* sd_lane_25g_25 */
2323 { TARGET_SD_LANE_25G + 1, 0x610da0000 }, /* sd_lane_25g_26 */
2324 { TARGET_SD_LANE_25G + 2, 0x610da8000 }, /* sd_lane_25g_27 */
2325 { TARGET_SD_LANE_25G + 3, 0x610db0000 }, /* sd_lane_25g_28 */
2326 { TARGET_SD_LANE_25G + 4, 0x610db8000 }, /* sd_lane_25g_29 */
2327 { TARGET_SD_LANE_25G + 5, 0x610dc0000 }, /* sd_lane_25g_30 */
2328 { TARGET_SD_LANE_25G + 6, 0x610dc8000 }, /* sd_lane_25g_31 */
2329 { TARGET_SD_LANE_25G + 7, 0x610dd0000 }, /* sd_lane_25g_32 */
2330 };
2331
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 53581 bytes --]
[-- Attachment #3: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v7 3/4] phy: Add Sparx5 ethernet serdes PHY driver
[not found] <20201202130438.3330228-4-steen.hegelund@microchip.com>
2020-12-02 15:09 ` [PATCH v7 3/4] phy: Add Sparx5 ethernet serdes PHY driver kernel test robot
@ 2020-12-02 21:24 ` kernel test robot
1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2020-12-02 21:24 UTC (permalink / raw)
To: Steen Hegelund, Kishon Vijay Abraham I, Vinod Koul, linux-arm-kernel
Cc: Bjarni Jonasson, Alexandre Belloni, kbuild-all, Steen Hegelund,
linux-kernel, Microchip UNG Driver List, clang-built-linux,
Lars Povlsen
[-- Attachment #1: Type: text/plain, Size: 4216 bytes --]
Hi Steen,
I love your patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next]
[also build test WARNING on linux/master linus/master v5.10-rc6 next-20201201]
[cannot apply to phy/next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Steen-Hegelund/Adding-the-Sparx5-Serdes-driver/20201202-211116
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: riscv-randconfig-r016-20201202 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 2671fccf0381769276ca8246ec0499adcb9b0355)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/0day-ci/linux/commit/f9670130c2b0d7ccd38a459e66d8da36b8edb7e0
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Steen-Hegelund/Adding-the-Sparx5-Serdes-driver/20201202-211116
git checkout f9670130c2b0d7ccd38a459e66d8da36b8edb7e0
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/phy/microchip/sparx5_serdes.c:2393:54: warning: variable 'idx' is uninitialized when used here [-Wuninitialized]
iores = platform_get_resource(pdev, IORESOURCE_MEM, idx);
^~~
drivers/phy/microchip/sparx5_serdes.c:2367:9: note: initialize the variable 'idx' to silence this warning
int idx;
^
= 0
1 warning generated.
vim +/idx +2393 drivers/phy/microchip/sparx5_serdes.c
2357
2358 static int sparx5_serdes_probe(struct platform_device *pdev)
2359 {
2360 struct device_node *np = pdev->dev.of_node;
2361 struct sparx5_serdes_private *priv;
2362 struct phy_provider *provider;
2363 struct resource *iores;
2364 void __iomem *iomem;
2365 unsigned long clock;
2366 struct clk *clk;
2367 int idx;
2368 int err;
2369
2370 if (!np && !pdev->dev.platform_data)
2371 return -ENODEV;
2372
2373 priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
2374 if (!priv)
2375 return -ENOMEM;
2376
2377 platform_set_drvdata(pdev, priv);
2378 priv->dev = &pdev->dev;
2379
2380 /* Get coreclock */
2381 clk = devm_clk_get(priv->dev, NULL);
2382 if (IS_ERR(clk)) {
2383 dev_err(priv->dev, "Failed to get coreclock\n");
2384 return PTR_ERR(clk);
2385 }
2386 clock = clk_get_rate(clk);
2387 if (clock == 0) {
2388 dev_err(priv->dev, "Invalid coreclock %lu\n", clock);
2389 return -EINVAL;
2390 }
2391 priv->coreclock = clock;
2392
> 2393 iores = platform_get_resource(pdev, IORESOURCE_MEM, idx);
2394 iomem = devm_ioremap(priv->dev, iores->start, iores->end - iores->start + 1);
2395 if (IS_ERR(iomem)) {
2396 dev_err(priv->dev, "Unable to get serdes registers: %s\n",
2397 iores->name);
2398 return PTR_ERR(iomem);
2399 }
2400 for (idx = 0; idx < ARRAY_SIZE(sparx5_serdes_iomap); idx++) {
2401 struct sparx5_serdes_io_resource *iomap = &sparx5_serdes_iomap[idx];
2402
2403 priv->regs[iomap->id] = iomem + iomap->phys - iores->start;
2404 }
2405 for (idx = 0; idx < SPX5_SERDES_MAX; idx++) {
2406 err = sparx5_phy_create(priv, idx, &priv->phys[idx]);
2407 if (err)
2408 return err;
2409 }
2410
2411 provider = devm_of_phy_provider_register(priv->dev, sparx5_serdes_xlate);
2412
2413 return PTR_ERR_OR_ZERO(provider);
2414 }
2415
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28674 bytes --]
[-- Attachment #3: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-02 21:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20201202130438.3330228-4-steen.hegelund@microchip.com>
2020-12-02 15:09 ` [PATCH v7 3/4] phy: Add Sparx5 ethernet serdes PHY driver kernel test robot
2020-12-02 21:24 ` kernel test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).