Patchew URL: https://patchew.org/QEMU/20200505152926.18877-1-arm...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200505152926.18877-1-arm...@redhat.com Subject: [PATCH v2 00/18] qom: Spring cleaning Type: series === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/20200505174520.cd62a3a0...@moya.office.hostfission.com -> patchew/20200505174520.cd62a3a0...@moya.office.hostfission.com Switched to a new branch 'test' 4cba1a0 qom: Drop @errp parameter of object_property_del() d8b7296 spapr_pci: Drop some dead error handling f3700d4 qdev: Unrealize must not fail f39f651 Drop more @errp parameters after previous commit 93d8e99 qom: Drop parameter @errp of object_property_add() & friends 6a392b4 qdev: Clean up qdev_connect_gpio_out_named() 6f7f8c4 hw/arm/bcm2835: Drop futile attempts at QOM-adopting memory da916a5 e1000: Don't run e1000_instance_init() twice f2cc352 hw/isa/superio: Make the components QOM children 98ac27d s390x/cpumodel: Fix UI to CPU features pcc-cmac-{aes, eaes}-256 b88e1ea tests/check-qom-proplist: Improve iterator coverage 065d2f5 qom: Drop object_property_set_description() parameter @errp eb48860 qom: Make all the object_property_add_FOO() return the property 9647d4a qom: Drop convenience method object_property_get_uint16List() 4f21ae5 qom: Simplify object_property_get_enum() 8bf7167 qom: Drop object_property_del_child()'s unused parameter @errp c86c469 qom: Clean up inconsistent use of gchar * vs. char * 24a691d qom: Clearer reference counting in object_initialize_childv() === OUTPUT BEGIN === 1/18 Checking commit 24a691def353 (qom: Clearer reference counting in object_initialize_childv()) 2/18 Checking commit c86c4696bff7 (qom: Clean up inconsistent use of gchar * vs. char *) 3/18 Checking commit 8bf71679bce1 (qom: Drop object_property_del_child()'s unused parameter @errp) 4/18 Checking commit 4f21ae5ae5ed (qom: Simplify object_property_get_enum()) 5/18 Checking commit 9647d4ae16b7 (qom: Drop convenience method object_property_get_uint16List()) 6/18 Checking commit eb48860a3c26 (qom: Make all the object_property_add_FOO() return the property) 7/18 Checking commit 065d2f55d1d4 (qom: Drop object_property_set_description() parameter @errp) 8/18 Checking commit b88e1ea4bc39 (tests/check-qom-proplist: Improve iterator coverage) 9/18 Checking commit 98ac27d47088 (s390x/cpumodel: Fix UI to CPU features pcc-cmac-{aes, eaes}-256) ERROR: line over 90 characters #53: FILE: target/s390x/cpu_features_def.inc.h:314: +DEF_FEAT(PCC_CMAC_AES_256, "pcc-cmac-aes-256", PCC, 20, "PCC Compute-Last-Block-CMAC-Using-AES-256") total: 1 errors, 0 warnings, 8 lines checked Patch 9/18 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 10/18 Checking commit f2cc352a601b (hw/isa/superio: Make the components QOM children) 11/18 Checking commit da916a5c390b (e1000: Don't run e1000_instance_init() twice) 12/18 Checking commit 6f7f8c446ca6 (hw/arm/bcm2835: Drop futile attempts at QOM-adopting memory) 13/18 Checking commit 6a392b4df970 (qdev: Clean up qdev_connect_gpio_out_named()) 14/18 Checking commit 93d8e9905c38 (qom: Drop parameter @errp of object_property_add() & friends) WARNING: line over 80 characters #209: FILE: backends/hostmem-file.c:187: + file_memory_backend_get_discard_data, file_memory_backend_set_discard_data); WARNING: line over 80 characters #1080: FILE: hw/arm/raspi.c:287: + object_property_add_const_link(OBJECT(&s->soc), "ram", OBJECT(machine->ram)); WARNING: line over 80 characters #3095: FILE: hw/ppc/spapr.c:3346: + &spapr->kernel_addr, OBJ_PROP_FLAG_READWRITE); total: 0 errors, 3 warnings, 4471 lines checked Patch 14/18 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 15/18 Checking commit f39f6513b7a2 (Drop more @errp parameters after previous commit) 16/18 Checking commit f3700d44eab1 (qdev: Unrealize must not fail) 17/18 Checking commit d8b7296cf934 (spapr_pci: Drop some dead error handling) 18/18 Checking commit 4cba1a0de99a (qom: Drop @errp parameter of object_property_del()) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200505152926.18877-1-arm...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
