Add miscellaneous patches required for mediated device drivers support. "Work in progress" branch with mdev common code and i40e driver can be found here: https://github.com/MykytaI/odp/tree/caterpillar_mdev_i40e_demo Last two patches aren't yet ready for upstreaming (i40e works, but code needs more documentation & cleanup) and cxgb4+e1000e drivers aren't merged on the top of the branch either.
----------------github------------------------ /** Email created from pull request 359 (MykytaI:caterpillar_mdev_auxiliary) ** https://github.com/Linaro/odp/pull/359 ** Patch: https://github.com/Linaro/odp/pull/359.patch ** Base sha: 63fd88635cc10caaa02fdccd3f52c9494487bdd2 ** Merge commit sha: 0e9885d54ac547d0bdf599d65b6fe32de445adb7 **/ ----------------/github------------------------ ----------------checkpatch.pl------------------------ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #57: FILE: include/odp/drv/spec/mmio.h:30: +void odpdrv_mmio_u8le_write(uint8_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #64: FILE: include/odp/drv/spec/mmio.h:37: +void odpdrv_mmio_u8be_write(uint8_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #71: FILE: include/odp/drv/spec/mmio.h:44: +void odpdrv_mmio_u16le_write(uint16_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #78: FILE: include/odp/drv/spec/mmio.h:51: +void odpdrv_mmio_u16be_write(uint16_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #85: FILE: include/odp/drv/spec/mmio.h:58: +void odpdrv_mmio_u32le_write(uint32_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #92: FILE: include/odp/drv/spec/mmio.h:65: +void odpdrv_mmio_u32be_write(uint32_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #99: FILE: include/odp/drv/spec/mmio.h:72: +void odpdrv_mmio_u64le_write(uint64_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #106: FILE: include/odp/drv/spec/mmio.h:79: +void odpdrv_mmio_u64be_write(uint64_t value, volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #113: FILE: include/odp/drv/spec/mmio.h:86: +uint8_t odpdrv_mmio_u8le_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #120: FILE: include/odp/drv/spec/mmio.h:93: +uint8_t odpdrv_mmio_u8be_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #127: FILE: include/odp/drv/spec/mmio.h:100: +uint16_t odpdrv_mmio_u16le_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #134: FILE: include/odp/drv/spec/mmio.h:107: +uint16_t odpdrv_mmio_u16be_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #141: FILE: include/odp/drv/spec/mmio.h:114: +uint32_t odpdrv_mmio_u32le_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #148: FILE: include/odp/drv/spec/mmio.h:121: +uint32_t odpdrv_mmio_u32be_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #155: FILE: include/odp/drv/spec/mmio.h:128: +uint64_t odpdrv_mmio_u64le_read(volatile void *addr); WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #162: FILE: include/odp/drv/spec/mmio.h:135: +uint64_t odpdrv_mmio_u64be_read(volatile void *addr); WARNING: Single statement macros should not use a do {} while (0) loop #213: FILE: platform/linux-generic/include/odp/drv/mmio.h:35: +#define odpdrv_io_mb() do { __asm__ __volatile__("" ::: "memory"); } while (0) WARNING: Single statement macros should not use a do {} while (0) loop #214: FILE: platform/linux-generic/include/odp/drv/mmio.h:36: +#define odpdrv_io_rmb() do { __asm__ __volatile__("" ::: "memory"); } while (0) WARNING: Single statement macros should not use a do {} while (0) loop #215: FILE: platform/linux-generic/include/odp/drv/mmio.h:37: +#define odpdrv_io_wmb() do { __asm__ __volatile__("" ::: "memory"); } while (0) CHECK: spaces preferred around that '*' (ctx:WxV) #218: FILE: platform/linux-generic/include/odp/drv/mmio.h:40: +odpdrv_mmio_u8le_write(uint8_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #218: FILE: platform/linux-generic/include/odp/drv/mmio.h:40: +odpdrv_mmio_u8le_write(uint8_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #221: FILE: platform/linux-generic/include/odp/drv/mmio.h:43: + *(__odp_force volatile uint8_t *)addr = value; CHECK: spaces preferred around that '*' (ctx:WxV) #225: FILE: platform/linux-generic/include/odp/drv/mmio.h:47: +odpdrv_mmio_u8be_write(uint8_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #225: FILE: platform/linux-generic/include/odp/drv/mmio.h:47: +odpdrv_mmio_u8be_write(uint8_t value, volatile void __odpdrv_mmio *addr) CHECK: spaces preferred around that '*' (ctx:WxV) #231: FILE: platform/linux-generic/include/odp/drv/mmio.h:53: +odpdrv_mmio_u16le_write(uint16_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #231: FILE: platform/linux-generic/include/odp/drv/mmio.h:53: +odpdrv_mmio_u16le_write(uint16_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #234: FILE: platform/linux-generic/include/odp/drv/mmio.h:56: + *(__odp_force volatile odp_u16le_t *)addr = odp_cpu_to_le_16(value); CHECK: spaces preferred around that '*' (ctx:WxV) #238: FILE: platform/linux-generic/include/odp/drv/mmio.h:60: +odpdrv_mmio_u16be_write(uint16_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #238: FILE: platform/linux-generic/include/odp/drv/mmio.h:60: +odpdrv_mmio_u16be_write(uint16_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #241: FILE: platform/linux-generic/include/odp/drv/mmio.h:63: + *(__odp_force volatile odp_u16be_t *)addr = odp_cpu_to_be_16(value); CHECK: spaces preferred around that '*' (ctx:WxV) #245: FILE: platform/linux-generic/include/odp/drv/mmio.h:67: +odpdrv_mmio_u32le_write(uint32_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #245: FILE: platform/linux-generic/include/odp/drv/mmio.h:67: +odpdrv_mmio_u32le_write(uint32_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #248: FILE: platform/linux-generic/include/odp/drv/mmio.h:70: + *(__odp_force volatile odp_u32le_t *)addr = odp_cpu_to_le_32(value); CHECK: spaces preferred around that '*' (ctx:WxV) #252: FILE: platform/linux-generic/include/odp/drv/mmio.h:74: +odpdrv_mmio_u32be_write(uint32_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #252: FILE: platform/linux-generic/include/odp/drv/mmio.h:74: +odpdrv_mmio_u32be_write(uint32_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #255: FILE: platform/linux-generic/include/odp/drv/mmio.h:77: + *(__odp_force volatile odp_u32be_t *)addr = odp_cpu_to_be_32(value); CHECK: spaces preferred around that '*' (ctx:WxV) #259: FILE: platform/linux-generic/include/odp/drv/mmio.h:81: +odpdrv_mmio_u64le_write(uint64_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #259: FILE: platform/linux-generic/include/odp/drv/mmio.h:81: +odpdrv_mmio_u64le_write(uint64_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #262: FILE: platform/linux-generic/include/odp/drv/mmio.h:84: + *(__odp_force volatile odp_u64le_t *)addr = odp_cpu_to_le_64(value); CHECK: spaces preferred around that '*' (ctx:WxV) #266: FILE: platform/linux-generic/include/odp/drv/mmio.h:88: +odpdrv_mmio_u64be_write(uint64_t value, volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #266: FILE: platform/linux-generic/include/odp/drv/mmio.h:88: +odpdrv_mmio_u64be_write(uint64_t value, volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #269: FILE: platform/linux-generic/include/odp/drv/mmio.h:91: + *(__odp_force volatile odp_u64be_t *)addr = odp_cpu_to_be_64(value); CHECK: spaces preferred around that '*' (ctx:WxV) #273: FILE: platform/linux-generic/include/odp/drv/mmio.h:95: +odpdrv_mmio_u8le_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #273: FILE: platform/linux-generic/include/odp/drv/mmio.h:95: +odpdrv_mmio_u8le_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #275: FILE: platform/linux-generic/include/odp/drv/mmio.h:97: + uint8_t value = *(__odp_force volatile uint8_t *)addr; CHECK: spaces preferred around that '*' (ctx:WxV) #282: FILE: platform/linux-generic/include/odp/drv/mmio.h:104: +odpdrv_mmio_u8be_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #282: FILE: platform/linux-generic/include/odp/drv/mmio.h:104: +odpdrv_mmio_u8be_read(volatile void __odpdrv_mmio *addr) CHECK: spaces preferred around that '*' (ctx:WxV) #288: FILE: platform/linux-generic/include/odp/drv/mmio.h:110: +odpdrv_mmio_u16le_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #288: FILE: platform/linux-generic/include/odp/drv/mmio.h:110: +odpdrv_mmio_u16le_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #291: FILE: platform/linux-generic/include/odp/drv/mmio.h:113: + odp_le_to_cpu_16(*(__odp_force volatile odp_u16le_t *)addr); CHECK: spaces preferred around that '*' (ctx:WxV) #298: FILE: platform/linux-generic/include/odp/drv/mmio.h:120: +odpdrv_mmio_u16be_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #298: FILE: platform/linux-generic/include/odp/drv/mmio.h:120: +odpdrv_mmio_u16be_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #301: FILE: platform/linux-generic/include/odp/drv/mmio.h:123: + odp_be_to_cpu_16(*(__odp_force volatile odp_u16be_t *)addr); CHECK: spaces preferred around that '*' (ctx:WxV) #308: FILE: platform/linux-generic/include/odp/drv/mmio.h:130: +odpdrv_mmio_u32le_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #308: FILE: platform/linux-generic/include/odp/drv/mmio.h:130: +odpdrv_mmio_u32le_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #311: FILE: platform/linux-generic/include/odp/drv/mmio.h:133: + odp_le_to_cpu_32(*(__odp_force volatile odp_u32le_t *)addr); CHECK: spaces preferred around that '*' (ctx:WxV) #318: FILE: platform/linux-generic/include/odp/drv/mmio.h:140: +odpdrv_mmio_u32be_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #318: FILE: platform/linux-generic/include/odp/drv/mmio.h:140: +odpdrv_mmio_u32be_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #321: FILE: platform/linux-generic/include/odp/drv/mmio.h:143: + odp_be_to_cpu_32(*(__odp_force volatile odp_u32be_t *)addr); CHECK: spaces preferred around that '*' (ctx:WxV) #328: FILE: platform/linux-generic/include/odp/drv/mmio.h:150: +odpdrv_mmio_u64le_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #328: FILE: platform/linux-generic/include/odp/drv/mmio.h:150: +odpdrv_mmio_u64le_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #331: FILE: platform/linux-generic/include/odp/drv/mmio.h:153: + odp_le_to_cpu_64(*(__odp_force volatile odp_u64le_t *)addr); CHECK: spaces preferred around that '*' (ctx:WxV) #338: FILE: platform/linux-generic/include/odp/drv/mmio.h:160: +odpdrv_mmio_u64be_read(volatile void __odpdrv_mmio *addr) ^ WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #338: FILE: platform/linux-generic/include/odp/drv/mmio.h:160: +odpdrv_mmio_u64be_read(volatile void __odpdrv_mmio *addr) WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt #341: FILE: platform/linux-generic/include/odp/drv/mmio.h:163: + odp_be_to_cpu_64(*(__odp_force volatile odp_u64be_t *)addr); total: 0 errors, 49 warnings, 16 checks, 331 lines checked to_send-p-000.patch has style problems, please review. If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 0 warnings, 0 checks, 63 lines checked to_send-p-001.patch has no obvious style problems and is ready for submission. total: 0 errors, 0 warnings, 0 checks, 272 lines checked to_send-p-002.patch has no obvious style problems and is ready for submission. total: 0 errors, 0 warnings, 0 checks, 201 lines checked to_send-p-003.patch has no obvious style problems and is ready for submission. ----------------/checkpatch.pl------------------------