Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
Hi, This series failed docker-build@min-glib build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180218144021.11641-1-vinc...@bernat.im Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT === TEST SCRIPT BEGIN === #!/bin/bash set -e git submodule update --init dtc # Let docker tests dump environment info export SHOW_ENV=1 export J=8 time make docker-test-build@min-glib === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 Switched to a new branch 'test' c83e23c953 hw/audio/opl2lpt: add support for OPL2LPT === OUTPUT BEGIN === Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Cloning into '/var/tmp/patchew-tester-tmp-_kso1q75/src/dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' BUILD min-glib make[1]: Entering directory '/var/tmp/patchew-tester-tmp-_kso1q75/src' GEN /var/tmp/patchew-tester-tmp-_kso1q75/src/docker-src.2018-02-24-05.22.36.20222/qemu.tar Cloning into '/var/tmp/patchew-tester-tmp-_kso1q75/src/docker-src.2018-02-24-05.22.36.20222/qemu.tar.vroot'... done. Checking out files: 46% (2748/5855) Checking out files: 47% (2752/5855) Checking out files: 48% (2811/5855) Checking out files: 49% (2869/5855) Checking out files: 50% (2928/5855) Checking out files: 51% (2987/5855) Checking out files: 52% (3045/5855) Checking out files: 53% (3104/5855) Checking out files: 54% (3162/5855) Checking out files: 55% (3221/5855) Checking out files: 56% (3279/5855) Checking out files: 57% (3338/5855) Checking out files: 58% (3396/5855) Checking out files: 59% (3455/5855) Checking out files: 60% (3513/5855) Checking out files: 61% (3572/5855) Checking out files: 62% (3631/5855) Checking out files: 63% (3689/5855) Checking out files: 64% (3748/5855) Checking out files: 65% (3806/5855) Checking out files: 66% (3865/5855) Checking out files: 67% (3923/5855) Checking out files: 68% (3982/5855) Checking out files: 69% (4040/5855) Checking out files: 70% (4099/5855) Checking out files: 71% (4158/5855) Checking out files: 72% (4216/5855) Checking out files: 73% (4275/5855) Checking out files: 74% (4333/5855) Checking out files: 75% (4392/5855) Checking out files: 76% (4450/5855) Checking out files: 77% (4509/5855) Checking out files: 78% (4567/5855) Checking out files: 79% (4626/5855) Checking out files: 80% (4684/5855) Checking out files: 81% (4743/5855) Checking out files: 82% (4802/5855) Checking out files: 83% (4860/5855) Checking out files: 84% (4919/5855) Checking out files: 85% (4977/5855) Checking out files: 86% (5036/5855) Checking out files: 87% (5094/5855) Checking out files: 88% (5153/5855) Checking out files: 89% (5211/5855) Checking out files: 90% (5270/5855) Checking out files: 91% (5329/5855) Checking out files: 92% (5387/5855) Checking out files: 93% (5446/5855) Checking out files: 94% (5504/5855) Checking out files: 95% (5563/5855) Checking out files: 96% (5621/5855) Checking out files: 97% (5680/5855) Checking out files: 98% (5738/5855) Checking out files: 99% (5797/5855) Checking out files: 100% (5855/5855) Checking out files: 100% (5855/5855), done. Your branch is up-to-date with 'origin/test'. Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Cloning into '/var/tmp/patchew-tester-tmp-_kso1q75/src/docker-src.2018-02-24-05.22.36.20222/qemu.tar.vroot/dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' Submodule 'ui/keycodemapdb' (git://git.qemu.org/keycodemapdb.git) registered for path 'ui/keycodemapdb' Cloning into '/var/tmp/patchew-tester-tmp-_kso1q75/src/docker-src.2018-02-24-05.22.36.20222/qemu.tar.vroot/ui/keycodemapdb'... Submodule path 'ui/keycodemapdb': checked out '6b3d716e2b6472eb7189d3220552280ef3d832ce' COPYRUNNER RUN test-build in qemu:min-glib Environment variables: HOSTNAME=2a4ff165842d MAKEFLAGS= -j8 J=8 CCACHE_DIR=/var/tmp/ccache EXTRA_CONFIGURE_OPTS= V= SHOW_ENV=1 PATH=/usr/lib/ccache:/usr/lib64/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin PWD=/ TARGET_LIST= SHLVL=1 HOME=/root TEST_DIR=/tmp/qemu-test FEATURES= dtc DEBUG= _=/usr/bin/env Configure options: --enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --prefix=/tmp/qemu-test/install No C++ compiler available; disabling C++ specific optional code Install prefix/tmp/qemu-test/install BIOS directory/tmp/qemu-test/install/share/qemu firmware path /tmp/qemu-test/install/share/qemu-firmware binary directory /tmp/qemu-test/install/bin library directory /tmp/qemu-test/install/lib module directory /tmp/qemu-test/install/lib/qemu libexec directory /tmp/qemu-test
Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
Hi, This series failed build test on ppcbe host. Please find the details below. Type: series Message-id: 20180218144021.11641-1-vinc...@bernat.im Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be invoked under the git checkout with # HEAD pointing to a commit that has the patches applied on top of "base" # branch set -e echo "=== ENV ===" env echo "=== PACKAGES ===" rpm -qa echo "=== TEST BEGIN ===" INSTALL=$PWD/install BUILD=$PWD/build mkdir -p $BUILD $INSTALL SRC=$PWD cd $BUILD $SRC/configure --prefix=$INSTALL make -j100 # XXX: we need reliable clean up # make check -j100 V=1 make install === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/1519439425-27883-1-git-send-email-babu.mo...@amd.com -> patchew/1519439425-27883-1-git-send-email-babu.mo...@amd.com - [tag update] patchew/20180223145959.18761-1-laur...@vivier.eu -> patchew/20180223145959.18761-1-laur...@vivier.eu Submodule 'capstone' (git://git.qemu.org/capstone.git) registered for path 'capstone' Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Submodule 'roms/QemuMacDrivers' (git://git.qemu.org/QemuMacDrivers.git) registered for path 'roms/QemuMacDrivers' Submodule 'roms/SLOF' (git://git.qemu-project.org/SLOF.git) registered for path 'roms/SLOF' Submodule 'roms/ipxe' (git://git.qemu-project.org/ipxe.git) registered for path 'roms/ipxe' Submodule 'roms/openbios' (git://git.qemu-project.org/openbios.git) registered for path 'roms/openbios' Submodule 'roms/openhackware' (git://git.qemu-project.org/openhackware.git) registered for path 'roms/openhackware' Submodule 'roms/qemu-palcode' (git://github.com/rth7680/qemu-palcode.git) registered for path 'roms/qemu-palcode' Submodule 'roms/seabios' (git://git.qemu-project.org/seabios.git/) registered for path 'roms/seabios' Submodule 'roms/seabios-hppa' (git://github.com/hdeller/seabios-hppa.git) registered for path 'roms/seabios-hppa' Submodule 'roms/sgabios' (git://git.qemu-project.org/sgabios.git) registered for path 'roms/sgabios' Submodule 'roms/skiboot' (git://git.qemu.org/skiboot.git) registered for path 'roms/skiboot' Submodule 'roms/u-boot' (git://git.qemu-project.org/u-boot.git) registered for path 'roms/u-boot' Submodule 'roms/vgabios' (git://git.qemu-project.org/vgabios.git/) registered for path 'roms/vgabios' Submodule 'ui/keycodemapdb' (git://git.qemu.org/keycodemapdb.git) registered for path 'ui/keycodemapdb' Cloning into 'capstone'... Submodule path 'capstone': checked out '22ead3e0bfdb87516656453336160e0a37b066bf' Cloning into 'dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' Cloning into 'roms/QemuMacDrivers'... Submodule path 'roms/QemuMacDrivers': checked out 'd4e7d7ac663fcb55f1b93575445fcbca372f17a7' Cloning into 'roms/SLOF'... Submodule path 'roms/SLOF': checked out 'fa981320a1e0968d6fc1b8de319723ff8212b337' Cloning into 'roms/ipxe'... Submodule path 'roms/ipxe': checked out '0600d3ae94f93efd10fc6b3c7420a9557a3a1670' Cloning into 'roms/openbios'... Submodule path 'roms/openbios': checked out '54d959d97fb331708767b2fd4a878efd2bbc41bb' Cloning into 'roms/openhackware'... Submodule path 'roms/openhackware': checked out 'c559da7c8eec5e45ef1f67978827af6f0b9546f5' Cloning into 'roms/qemu-palcode'... Submodule path 'roms/qemu-palcode': checked out 'f3c7e44c70254975df2a00af39701eafbac4d471' Cloning into 'roms/seabios'... Submodule path 'roms/seabios': checked out '63451fca13c75870e1703eb3e20584d91179aebc' Cloning into 'roms/seabios-hppa'... Submodule path 'roms/seabios-hppa': checked out '649e6202b8d65d46c69f542b1380f840fbe8ab13' Cloning into 'roms/sgabios'... Submodule path 'roms/sgabios': checked out 'cbaee52287e5f32373181cff50a00b6c4ac9015a' Cloning into 'roms/skiboot'... Submodule path 'roms/skiboot': checked out 'e0ee24c27a172bcf482f6f2bc905e6211c134bcc' Cloning into 'roms/u-boot'... Submodule path 'roms/u-boot': checked out 'd85ca029f257b53a96da6c2fb421e78a003a9943' Cloning into 'roms/vgabios'... Submodule path 'roms/vgabios': checked out '19ea12c230ded95928ecaef0db47a82231c2e485' Cloning into 'ui/keycodemapdb'... Submodule path 'ui/keycodemapdb': checked out '6b3d716e2b6472eb7189d3220552280ef3d832ce' Switched to a new branch 'test' M roms/openbios c83e23c hw/audio/opl2lpt: add support for OPL2LPT === OUTPUT BEGIN === === ENV === XDG_SESSION_ID=29961 SHELL=/bin/sh USER=patchew PATCHEW=./patchew-cli -s https://patchew.org PATH=/usr/bin:/bin PWD=/var/tmp/patchew-tester-tmp-pbz7m8xl/src LANG=en_US.UTF-8 HOME=/home/patchew SHLVL=2 LOGNAME=patchew XDG_RUNTIME_DIR=/run/user/1000 _=/usr/bin/env === PACKAGES === telepathy-filesystem-0.0.2-6.el7.noarch ipa-common-4.5.0-20.el7.centos.noarch ipa-client-common-4.5.0-20.el7.centos.noarch nhn-nanum-fonts-
Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
Hi, This series failed docker-quick@centos6 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180218144021.11641-1-vinc...@bernat.im Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT === TEST SCRIPT BEGIN === #!/bin/bash set -e git submodule update --init dtc # Let docker tests dump environment info export SHOW_ENV=1 export J=8 time make docker-test-quick@centos6 === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 Switched to a new branch 'test' c83e23c953 hw/audio/opl2lpt: add support for OPL2LPT === OUTPUT BEGIN === Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Cloning into '/var/tmp/patchew-tester-tmp-jcz_ndf1/src/dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' BUILD centos6 make[1]: Entering directory '/var/tmp/patchew-tester-tmp-jcz_ndf1/src' GEN /var/tmp/patchew-tester-tmp-jcz_ndf1/src/docker-src.2018-02-23-01.53.17.7076/qemu.tar Cloning into '/var/tmp/patchew-tester-tmp-jcz_ndf1/src/docker-src.2018-02-23-01.53.17.7076/qemu.tar.vroot'... done. Checking out files: 47% (2773/5855) Checking out files: 48% (2811/5855) Checking out files: 49% (2869/5855) Checking out files: 50% (2928/5855) Checking out files: 51% (2987/5855) Checking out files: 52% (3045/5855) Checking out files: 53% (3104/5855) Checking out files: 54% (3162/5855) Checking out files: 55% (3221/5855) Checking out files: 56% (3279/5855) Checking out files: 57% (3338/5855) Checking out files: 58% (3396/5855) Checking out files: 59% (3455/5855) Checking out files: 60% (3513/5855) Checking out files: 61% (3572/5855) Checking out files: 62% (3631/5855) Checking out files: 63% (3689/5855) Checking out files: 64% (3748/5855) Checking out files: 65% (3806/5855) Checking out files: 66% (3865/5855) Checking out files: 67% (3923/5855) Checking out files: 68% (3982/5855) Checking out files: 69% (4040/5855) Checking out files: 70% (4099/5855) Checking out files: 71% (4158/5855) Checking out files: 72% (4216/5855) Checking out files: 73% (4275/5855) Checking out files: 74% (4333/5855) Checking out files: 75% (4392/5855) Checking out files: 76% (4450/5855) Checking out files: 77% (4509/5855) Checking out files: 78% (4567/5855) Checking out files: 79% (4626/5855) Checking out files: 80% (4684/5855) Checking out files: 81% (4743/5855) Checking out files: 82% (4802/5855) Checking out files: 83% (4860/5855) Checking out files: 84% (4919/5855) Checking out files: 85% (4977/5855) Checking out files: 86% (5036/5855) Checking out files: 87% (5094/5855) Checking out files: 88% (5153/5855) Checking out files: 89% (5211/5855) Checking out files: 90% (5270/5855) Checking out files: 91% (5329/5855) Checking out files: 92% (5387/5855) Checking out files: 93% (5446/5855) Checking out files: 94% (5504/5855) Checking out files: 95% (5563/5855) Checking out files: 96% (5621/5855) Checking out files: 97% (5680/5855) Checking out files: 98% (5738/5855) Checking out files: 99% (5797/5855) Checking out files: 100% (5855/5855) Checking out files: 100% (5855/5855), done. Your branch is up-to-date with 'origin/test'. Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Cloning into '/var/tmp/patchew-tester-tmp-jcz_ndf1/src/docker-src.2018-02-23-01.53.17.7076/qemu.tar.vroot/dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' Submodule 'ui/keycodemapdb' (git://git.qemu.org/keycodemapdb.git) registered for path 'ui/keycodemapdb' Cloning into '/var/tmp/patchew-tester-tmp-jcz_ndf1/src/docker-src.2018-02-23-01.53.17.7076/qemu.tar.vroot/ui/keycodemapdb'... Submodule path 'ui/keycodemapdb': checked out '6b3d716e2b6472eb7189d3220552280ef3d832ce' COPYRUNNER RUN test-quick in qemu:centos6 Packages installed: SDL-devel-1.2.14-7.el6_7.1.x86_64 bison-2.4.1-5.el6.x86_64 bzip2-devel-1.0.5-7.el6_0.x86_64 ccache-3.1.6-2.el6.x86_64 csnappy-devel-0-6.20150729gitd7bc683.el6.x86_64 flex-2.5.35-9.el6.x86_64 gcc-4.4.7-18.el6.x86_64 gettext-0.17-18.el6.x86_64 git-1.7.1-9.el6_9.x86_64 glib2-devel-2.28.8-9.el6.x86_64 libepoxy-devel-1.2-3.el6.x86_64 libfdt-devel-1.4.0-1.el6.x86_64 librdmacm-devel-1.0.21-0.el6.x86_64 lzo-devel-2.03-3.1.el6_5.1.x86_64 make-3.81-23.el6.x86_64 mesa-libEGL-devel-11.0.7-4.el6.x86_64 mesa-libgbm-devel-11.0.7-4.el6.x86_64 package g++ is not installed pixman-devel-0.32.8-1.el6.x86_64 spice-glib-devel-0.26-8.el6.x86_64 spice-server-devel-0.12.4-16.el6.x86_64 tar-1.23-15.el6_8.x86_64 vte-devel-0.25.1-9.el6.x86_64 xen-devel-4.6.6-2.el6.x86_64 zlib-devel-1.2.3-29.el6.x86_64 Environment variables: PACKAGES=bison bzip2-devel ccache csnappy
Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
Hi, This series failed build test on ppcle host. Please find the details below. Type: series Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT Message-id: 20180218144021.11641-1-vinc...@bernat.im === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be invoked under the git checkout with # HEAD pointing to a commit that has the patches applied on top of "base" # branch set -e echo "=== ENV ===" env echo "=== PACKAGES ===" rpm -qa echo "=== TEST BEGIN ===" INSTALL=$PWD/install BUILD=$PWD/build mkdir -p $BUILD $INSTALL SRC=$PWD cd $BUILD $SRC/configure --prefix=$INSTALL make -j100 # XXX: we need reliable clean up # make check -j100 V=1 make install === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu ff86896..0ce9cb9 master -> master - [tag update] patchew/1519053606-9817-1-git-send-email-miny...@acm.org -> patchew/1519053606-9817-1-git-send-email-miny...@acm.org - [tag update] patchew/1519241752-28083-1-git-send-email-wall...@linux.vnet.ibm.com -> patchew/1519241752-28083-1-git-send-email-wall...@linux.vnet.ibm.com * [new tag] patchew/20180222152307.7499-1-peter.mayd...@linaro.org -> patchew/20180222152307.7499-1-peter.mayd...@linaro.org Submodule 'capstone' (git://git.qemu.org/capstone.git) registered for path 'capstone' Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Submodule 'roms/QemuMacDrivers' (git://git.qemu.org/QemuMacDrivers.git) registered for path 'roms/QemuMacDrivers' Submodule 'roms/SLOF' (git://git.qemu-project.org/SLOF.git) registered for path 'roms/SLOF' Submodule 'roms/ipxe' (git://git.qemu-project.org/ipxe.git) registered for path 'roms/ipxe' Submodule 'roms/openbios' (git://git.qemu-project.org/openbios.git) registered for path 'roms/openbios' Submodule 'roms/openhackware' (git://git.qemu-project.org/openhackware.git) registered for path 'roms/openhackware' Submodule 'roms/qemu-palcode' (git://github.com/rth7680/qemu-palcode.git) registered for path 'roms/qemu-palcode' Submodule 'roms/seabios' (git://git.qemu-project.org/seabios.git/) registered for path 'roms/seabios' Submodule 'roms/seabios-hppa' (git://github.com/hdeller/seabios-hppa.git) registered for path 'roms/seabios-hppa' Submodule 'roms/sgabios' (git://git.qemu-project.org/sgabios.git) registered for path 'roms/sgabios' Submodule 'roms/skiboot' (git://git.qemu.org/skiboot.git) registered for path 'roms/skiboot' Submodule 'roms/u-boot' (git://git.qemu-project.org/u-boot.git) registered for path 'roms/u-boot' Submodule 'roms/vgabios' (git://git.qemu-project.org/vgabios.git/) registered for path 'roms/vgabios' Submodule 'ui/keycodemapdb' (git://git.qemu.org/keycodemapdb.git) registered for path 'ui/keycodemapdb' Cloning into 'capstone'... Submodule path 'capstone': checked out '22ead3e0bfdb87516656453336160e0a37b066bf' Cloning into 'dtc'... Submodule path 'dtc': checked out 'e54388015af1fb4bf04d0bca99caba1074d9cc42' Cloning into 'roms/QemuMacDrivers'... Submodule path 'roms/QemuMacDrivers': checked out 'd4e7d7ac663fcb55f1b93575445fcbca372f17a7' Cloning into 'roms/SLOF'... Submodule path 'roms/SLOF': checked out 'fa981320a1e0968d6fc1b8de319723ff8212b337' Cloning into 'roms/ipxe'... Submodule path 'roms/ipxe': checked out '0600d3ae94f93efd10fc6b3c7420a9557a3a1670' Cloning into 'roms/openbios'... Submodule path 'roms/openbios': checked out '54d959d97fb331708767b2fd4a878efd2bbc41bb' Cloning into 'roms/openhackware'... Submodule path 'roms/openhackware': checked out 'c559da7c8eec5e45ef1f67978827af6f0b9546f5' Cloning into 'roms/qemu-palcode'... Submodule path 'roms/qemu-palcode': checked out 'f3c7e44c70254975df2a00af39701eafbac4d471' Cloning into 'roms/seabios'... Submodule path 'roms/seabios': checked out '63451fca13c75870e1703eb3e20584d91179aebc' Cloning into 'roms/seabios-hppa'... Submodule path 'roms/seabios-hppa': checked out '649e6202b8d65d46c69f542b1380f840fbe8ab13' Cloning into 'roms/sgabios'... Submodule path 'roms/sgabios': checked out 'cbaee52287e5f32373181cff50a00b6c4ac9015a' Cloning into 'roms/skiboot'... Submodule path 'roms/skiboot': checked out 'e0ee24c27a172bcf482f6f2bc905e6211c134bcc' Cloning into 'roms/u-boot'... Submodule path 'roms/u-boot': checked out 'd85ca029f257b53a96da6c2fb421e78a003a9943' Cloning into 'roms/vgabios'... Submodule path 'roms/vgabios': checked out '19ea12c230ded95928ecaef0db47a82231c2e485' Cloning into 'ui/keycodemapdb'... Submodule path 'ui/keycodemapdb': checked out '6b3d716e2b6472eb7189d3220552280ef3d832ce' Switched to a new branch 'test' M roms/openbios c83e23c hw/audio/opl2lpt: add support for OPL2LPT === OUTPUT BEGIN === === ENV === XDG_SESSION_ID=202664 SHELL=/bin/sh USER=patchew PATCHEW=/home/patchew/patchew/patchew-cli -s http://patchew.org --nodebug PATH=/usr/bin:/bin PWD=/var/tmp/patchew-tester-tmp-d94yhpy9/src LANG=en
Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180218144021.11641-1-vinc...@bernat.im Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --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' c83e23c953 hw/audio/opl2lpt: add support for OPL2LPT === OUTPUT BEGIN === Checking PATCH 1/1: hw/audio/opl2lpt: add support for OPL2LPT... ERROR: spaces required around that '/' (ctx:VxV) #154: FILE: hw/audio/opl2lpt.c:91: + diff/1000, nport, v); ^ ERROR: spaces required around that '/' (ctx:VxV) #167: FILE: hw/audio/opl2lpt.c:104: + diff/1000, nport, v); ^ ERROR: initializer for struct MemoryRegionPortio should normally be const #206: FILE: hw/audio/opl2lpt.c:143: +static MemoryRegionPortio opl2lpt_portio_list[] = { ERROR: space prohibited before that '++' (ctx:WxB) #216: FILE: hw/audio/opl2lpt.c:153: +for (int i = 0; i < 256; i ++) { ^ WARNING: line over 80 characters #235: FILE: hw/audio/opl2lpt.c:172: +portio_list_init(>port_list, OBJECT(s), opl2lpt_portio_list, s, "opl2lpt"); total: 4 errors, 1 warnings, 231 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. === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@freelists.org
[Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT
OPL2LPT is an OPL2 chip on the parallel port. It can be programmed mostly like an OPL2 chip, except it is write-only. Timers are therefore not usable and we emulate a simplified version of them: they expire immediately (in the future, this could be updated to properly emulate them). Timers are usually only used to detect the card. The driver needs a parallel port to be usable: -chardev parport,id=parport0,path=/dev/parport0 -device opl2lpt,chardev=parport0 >From a timing perspective, usleep() is used to ensure to give enough time to OPL2 to react properly. Ideally, the timing should be handled by the game, but testing with Indy3, this didn't work. I have also tried to be smarter by sleeping less depending on the current clock but a few glitches were still present. With the current approach, Indy3 is able to use the sound card without a glitch. The DOS driver doesn't add delays (but it doesn't work on QEMU either due to similar issues, so it may be a clue). On shutdown, the OPL2 chip should be reset to not continue sustaining a note. I was unable to find the appropriate function to register (unrealize callback is not called on shutdown and qemu_register() isn't always called on shutdown either). The user has to reset the VM, then stop it if they don't want to reset the OPL2LPT manually. Signed-off-by: Vincent Bernat--- default-configs/sound.mak | 1 + hw/audio/Makefile.objs| 1 + hw/audio/opl2lpt.c| 211 ++ hw/audio/soundhw.c| 2 +- 4 files changed, 214 insertions(+), 1 deletion(-) create mode 100644 hw/audio/opl2lpt.c diff --git a/default-configs/sound.mak b/default-configs/sound.mak index 4f22c34b5dac..d18725a1ea30 100644 --- a/default-configs/sound.mak +++ b/default-configs/sound.mak @@ -1,4 +1,5 @@ CONFIG_SB16=y CONFIG_ADLIB=y +CONFIG_OPL2LPT=y CONFIG_GUS=y CONFIG_CS4231A=y diff --git a/hw/audio/Makefile.objs b/hw/audio/Makefile.objs index 63db383709a1..274d5faa504a 100644 --- a/hw/audio/Makefile.objs +++ b/hw/audio/Makefile.objs @@ -3,6 +3,7 @@ common-obj-$(CONFIG_SB16) += sb16.o common-obj-$(CONFIG_ES1370) += es1370.o common-obj-$(CONFIG_AC97) += ac97.o common-obj-$(CONFIG_ADLIB) += fmopl.o adlib.o +common-obj-$(CONFIG_OPL2LPT) += opl2lpt.o common-obj-$(CONFIG_GUS) += gus.o gusemu_hal.o gusemu_mixer.o common-obj-$(CONFIG_CS4231A) += cs4231a.o common-obj-$(CONFIG_HDA) += intel-hda.o hda-codec.o diff --git a/hw/audio/opl2lpt.c b/hw/audio/opl2lpt.c new file mode 100644 index ..53286504c8cd --- /dev/null +++ b/hw/audio/opl2lpt.c @@ -0,0 +1,211 @@ +/* + * QEMU Proxy for OPL2LPT + * + * Copyright (c) 2018 Vincent Bernat + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + */ + +/* TODO: emulate timers */ + +#include "qemu/osdep.h" +#include "qapi/error.h" +#include "hw/hw.h" +#include "hw/audio/soundhw.h" +#include "audio/audio.h" +#include "hw/isa/isa.h" +#include "chardev/char-parallel.h" +#include "chardev/char-fe.h" + +#define DEBUG + +#define OPL2LPT_DESC "OPL2LPT (Yamaha YM3812 over parallel port)" + +#define dolog(...) AUD_log("opl2lpt", __VA_ARGS__) +#ifdef DEBUG +#define ldebug(...) dolog(__VA_ARGS__) +#else +#define ldebug(...) +#endif + +#define TYPE_OPL2LPT "opl2lpt" +#define OPL2LPT(obj) OBJECT_CHECK(Opl2lptState, (obj), TYPE_OPL2LPT) + +#define PP_NOT_STROBE 0x1 +#define PP_NOT_AUTOFD 0x2 +#define PP_INIT0x4 +#define PP_NOT_SELECT 0x8 + +typedef struct { +ISADevice parent_obj; + +uint8_t address; +uint8_t timer_reg; +int64_t last_clock; +PortioList port_list; +CharBackend chr; +} Opl2lptState; + +static void opl2lpt_lpt_write(Opl2lptState *s, uint8_t d, uint8_t c) +{ +qemu_chr_fe_ioctl(>chr, CHR_IOCTL_PP_WRITE_DATA, ); +qemu_chr_fe_ioctl(>chr, CHR_IOCTL_PP_WRITE_CONTROL, ); +c ^= PP_INIT; +qemu_chr_fe_ioctl(>chr, CHR_IOCTL_PP_WRITE_CONTROL, ); +c ^= PP_INIT; +qemu_chr_fe_ioctl(>chr,