Hi, Your series seems to have some coding style problems. See output below for more information:
Type: series Message-id: [email protected] Subject: [Qemu-devel] [PULL 00/32] target-arm queue === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 # Useful git options git config --local diff.renamelimit 0 git config --local diff.renames True commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git show --no-patch --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 Switched to a new branch 'test' a4bdaba i2c: Add asserts for second smbus i2c_start_transfer() 28b6053 target-arm: Implement new HLT trap for semihosting 09d4ec5 hw/display: QOM'ify pl110.c b3e97d0 hw/arm: QOM'ify strongarm.c 4b8eae3 hw/arm: QOM'ify pxa2xx_gpio.c 4e06f2b hw/arm: QOM'ify musicpal.c 92e91ad timer: stm32f2xx_timer: add check for prescaler value 967e6fa i2c: Fix SMBus read transactions to avoid double events 43ae1e2 timer: a9gtimer: remove loop to auto-increment comparator b95b19e ARM: Virt: ACPI: Build an IORT table with RC and ITS nodes 34a7836 ACPI: Add IORT Structure definition cf7ab3a tests: Add tests for the ARM MPTimer 8de02d8 arm_mptimer: Convert to use ptimer fd8b3a0 tests: ptimer: Replace 10000 with 1 d032b39 tests: ptimer: Change the copyright comment 2baf14c tests: ptimer: Add tests for "no counter round down" policy 6f60e0f hw/ptimer: Add "no counter round down" policy 4bf3a94 tests: ptimer: Add tests for "no immediate reload" policy 289b047 hw/ptimer: Add "no immediate reload" policy ce78ba6 tests: ptimer: Add tests for "no immediate trigger" policy e3f51a9 hw/ptimer: Add "no immediate trigger" policy 53ee1b7 tests: ptimer: Add tests for "continuous trigger" policy 6212b41 hw/ptimer: Add "continuous trigger" policy b9f1014 tests: ptimer: Add tests for "wraparound after one period" policy 5552280 hw/ptimer: Add "wraparound after one period" policy 499ec72 hw/arm/virt: Set minimum_page_bits to 12 9dea37c target-arm: Make page size a runtime setting 1f435b7 migration/savevm.c: migrate non-default page size 5eae29a cpu: Support a target CPU having a variable page size d48bab7 translate-all.c: Compute L1 page table properties at runtime 49f5d09 exec.c: Remove static allocation of sub_section of sub_page fc33d3c migration: Remove static allocation of xzblre cache buffer === OUTPUT BEGIN === Checking PATCH 1/32: migration: Remove static allocation of xzblre cache buffer... Checking PATCH 2/32: exec.c: Remove static allocation of sub_section of sub_page... Checking PATCH 3/32: translate-all.c: Compute L1 page table properties at runtime... Checking PATCH 4/32: cpu: Support a target CPU having a variable page size... Checking PATCH 5/32: migration/savevm.c: migrate non-default page size... ERROR: spaces required around that '*' (ctx:VxV) #96: FILE: migration/savevm.c:359: + .subsections = (const VMStateDescription*[]) { ^ total: 1 errors, 0 warnings, 79 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Checking PATCH 6/32: target-arm: Make page size a runtime setting... Checking PATCH 7/32: hw/arm/virt: Set minimum_page_bits to 12... Checking PATCH 8/32: hw/ptimer: Add "wraparound after one period" policy... Checking PATCH 9/32: tests: ptimer: Add tests for "wraparound after one period" policy... Checking PATCH 10/32: hw/ptimer: Add "continuous trigger" policy... Checking PATCH 11/32: tests: ptimer: Add tests for "continuous trigger" policy... Checking PATCH 12/32: hw/ptimer: Add "no immediate trigger" policy... Checking PATCH 13/32: tests: ptimer: Add tests for "no immediate trigger" policy... Checking PATCH 14/32: hw/ptimer: Add "no immediate reload" policy... Checking PATCH 15/32: tests: ptimer: Add tests for "no immediate reload" policy... Checking PATCH 16/32: hw/ptimer: Add "no counter round down" policy... Checking PATCH 17/32: tests: ptimer: Add tests for "no counter round down" policy... Checking PATCH 18/32: tests: ptimer: Change the copyright comment... Checking PATCH 19/32: tests: ptimer: Replace 10000 with 1... Checking PATCH 20/32: arm_mptimer: Convert to use ptimer... Checking PATCH 21/32: tests: Add tests for the ARM MPTimer... WARNING: line over 80 characters #1044: FILE: tests/test-arm-mptimer.c:1003: + qtest_add_func("mptimer/set_counter_disabled", test_timer_set_counter_disabled); WARNING: line over 80 characters #1046: FILE: tests/test-arm-mptimer.c:1005: + test_timer_zero_load_prescaled_periodic_to_nonscaled_oneshot); WARNING: line over 80 characters #1048: FILE: tests/test-arm-mptimer.c:1007: + test_timer_zero_load_prescaled_oneshot_to_nonscaled_periodic); WARNING: line over 80 characters #1050: FILE: tests/test-arm-mptimer.c:1009: + test_timer_zero_load_nonscaled_oneshot_to_prescaled_periodic); WARNING: line over 80 characters #1052: FILE: tests/test-arm-mptimer.c:1011: + test_timer_zero_load_nonscaled_periodic_to_prescaled_oneshot); WARNING: line over 80 characters #1054: FILE: tests/test-arm-mptimer.c:1013: + qtest_add_func("mptimer/prescaler_on_the_fly", test_timer_prescaler_on_the_fly); WARNING: line over 80 characters #1094: FILE: tests/test-arm-mptimer.c:1053: + g_strdup_printf("mptimer/zero_load_oneshot_to_nonzero scaler=%d", *scaler), WARNING: line over 80 characters #1097: FILE: tests/test-arm-mptimer.c:1056: + g_strdup_printf("mptimer/zero_load_periodic_to_nonzero scaler=%d", *scaler), WARNING: line over 80 characters #1100: FILE: tests/test-arm-mptimer.c:1059: + g_strdup_printf("mptimer/nonzero_load_oneshot_to_zero scaler=%d", *scaler), WARNING: line over 80 characters #1103: FILE: tests/test-arm-mptimer.c:1062: + g_strdup_printf("mptimer/nonzero_load_periodic_to_zero scaler=%d", *scaler), WARNING: line over 80 characters #1106: FILE: tests/test-arm-mptimer.c:1065: + g_strdup_printf("mptimer/set_periodic_counter_on_the_fly scaler=%d", *scaler), WARNING: line over 80 characters #1115: FILE: tests/test-arm-mptimer.c:1074: + g_strdup_printf("mptimer/oneshot_with_counter_0_on_start scaler=%d", *scaler), WARNING: line over 80 characters #1118: FILE: tests/test-arm-mptimer.c:1077: + g_strdup_printf("mptimer/periodic_with_counter_0_on_start scaler=%d", *scaler), WARNING: line over 80 characters #1124: FILE: tests/test-arm-mptimer.c:1083: + g_strdup_printf("mptimer/set_counter_periodic_with_zero_load scaler=%d", *scaler), total: 0 errors, 14 warnings, 1120 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Checking PATCH 22/32: ACPI: Add IORT Structure definition... Checking PATCH 23/32: ARM: Virt: ACPI: Build an IORT table with RC and ITS nodes... Checking PATCH 24/32: timer: a9gtimer: remove loop to auto-increment comparator... Checking PATCH 25/32: i2c: Fix SMBus read transactions to avoid double events... Checking PATCH 26/32: timer: stm32f2xx_timer: add check for prescaler value... Checking PATCH 27/32: hw/arm: QOM'ify musicpal.c... Checking PATCH 28/32: hw/arm: QOM'ify pxa2xx_gpio.c... Checking PATCH 29/32: hw/arm: QOM'ify strongarm.c... Checking PATCH 30/32: hw/display: QOM'ify pl110.c... Checking PATCH 31/32: target-arm: Implement new HLT trap for semihosting... Checking PATCH 32/32: i2c: Add asserts for second smbus i2c_start_transfer()... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to [email protected]
