[PATCH 0/3] Add ability to choose MDIO phy number to i.MX processors

Jean-Christophe Dubois posted 3 patches 3 years, 9 months ago
Test FreeBSD passed
Test docker-quick@centos7 failed
Test checkpatch passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1593806826.git.jcd@tribudubois.net
Maintainers: Peter Maydell <peter.maydell@linaro.org>, Jean-Christophe Dubois <jcd@tribudubois.net>, Andrey Smirnov <andrew.smirnov@gmail.com>
hw/arm/fsl-imx25.c         | 7 +++++++
hw/arm/fsl-imx6.c          | 7 +++++++
hw/arm/fsl-imx7.c          | 9 +++++++++
include/hw/arm/fsl-imx25.h | 1 +
include/hw/arm/fsl-imx6.h  | 1 +
include/hw/arm/fsl-imx7.h  | 1 +
6 files changed, 26 insertions(+)
[PATCH 0/3] Add ability to choose MDIO phy number to i.MX processors
Posted by Jean-Christophe Dubois 3 years, 9 months ago
This patch set adds the ability to select the Ethernet PHY device number
for the i.MX25 processor, the i.MX6 processor and the i.MX7 processor.

Note: the i.MX6UL processor was addressed in a previous patch.

Jean-Christophe Dubois (3):
  Add the ability to change the FEC PHY MDIO device number on i.MX25
    processor
  Add the ability to change the FEC PHY MDIO device number on i.MX6
    processor
  Add the ability to change the FEC PHY MDIO devices numbers on i.MX7
    processor

 hw/arm/fsl-imx25.c         | 7 +++++++
 hw/arm/fsl-imx6.c          | 7 +++++++
 hw/arm/fsl-imx7.c          | 9 +++++++++
 include/hw/arm/fsl-imx25.h | 1 +
 include/hw/arm/fsl-imx6.h  | 1 +
 include/hw/arm/fsl-imx7.h  | 1 +
 6 files changed, 26 insertions(+)

-- 
2.25.1


Re: [PATCH 0/3] Add ability to choose MDIO phy number to i.MX processors
Posted by no-reply@patchew.org 3 years, 9 months ago
Patchew URL: https://patchew.org/QEMU/cover.1593806826.git.jcd@tribudubois.net/



Hi,

This series failed the docker-quick@centos7 build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
make docker-image-centos7 V=1 NETWORK=1
time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
=== TEST SCRIPT END ===

qemu-system-aarch64: Property '.phy-num' not found
Broken pipe
/tmp/qemu-test/src/tests/qtest/libqtest.c:175: kill_qemu() detected QEMU death from signal 6 (Aborted) (core dumped)
ERROR - too few tests run (expected 66, got 0)
make: *** [check-qtest-aarch64] Error 1
make: *** Waiting for unfinished jobs....
  TEST    iotest-qcow2: 154
Could not access KVM kernel module: No such file or directory
---
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=4a2335c3dedc4400bae7b327958dfa5a', '-u', '1001', '--security-opt', 'seccomp=unconfined', '--rm', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/home/patchew/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-9e58kfzn/src/docker-src.2020-07-03-16.29.20.28169:/var/tmp/qemu:z,ro', 'qemu:centos7', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2.
filter=--filter=label=com.qemu.instance.uuid=4a2335c3dedc4400bae7b327958dfa5a
make[1]: *** [docker-run] Error 1
make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-9e58kfzn/src'
make: *** [docker-run-test-quick@centos7] Error 2

real    14m58.666s
user    0m9.169s


The full log is available at
http://patchew.org/logs/cover.1593806826.git.jcd@tribudubois.net/testing.docker-quick@centos7/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Re: [PATCH 0/3] Add ability to choose MDIO phy number to i.MX processors
Posted by Jean-Christophe DUBOIS 3 years, 9 months ago
This patch set is to be applied on top of the patch serie recently 
accepted by Peter on his tree.

I guess I will have to wait a bit that his pull request is accepted in 
mainline before resubmitting my patch.

JC

Le 03/07/2020 à 22:44, no-reply@patchew.org a écrit :
> Patchew URL: https://patchew.org/QEMU/cover.1593806826.git.jcd@tribudubois.net/
>
>
>
> Hi,
>
> This series failed the docker-quick@centos7 build test. Please find the testing commands and
> their output below. If you have Docker installed, you can probably reproduce it
> locally.
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> make docker-image-centos7 V=1 NETWORK=1
> time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
> === TEST SCRIPT END ===
>
> qemu-system-aarch64: Property '.phy-num' not found
> Broken pipe
> /tmp/qemu-test/src/tests/qtest/libqtest.c:175: kill_qemu() detected QEMU death from signal 6 (Aborted) (core dumped)
> ERROR - too few tests run (expected 66, got 0)
> make: *** [check-qtest-aarch64] Error 1
> make: *** Waiting for unfinished jobs....
>    TEST    iotest-qcow2: 154
> Could not access KVM kernel module: No such file or directory
> ---
>      raise CalledProcessError(retcode, cmd)
> subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=4a2335c3dedc4400bae7b327958dfa5a', '-u', '1001', '--security-opt', 'seccomp=unconfined', '--rm', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/home/patchew/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-9e58kfzn/src/docker-src.2020-07-03-16.29.20.28169:/var/tmp/qemu:z,ro', 'qemu:centos7', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2.
> filter=--filter=label=com.qemu.instance.uuid=4a2335c3dedc4400bae7b327958dfa5a
> make[1]: *** [docker-run] Error 1
> make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-9e58kfzn/src'
> make: *** [docker-run-test-quick@centos7] Error 2
>
> real    14m58.666s
> user    0m9.169s
>
>
> The full log is available at
> http://patchew.org/logs/cover.1593806826.git.jcd@tribudubois.net/testing.docker-quick@centos7/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com



Re: [PATCH v2 0/3] Add ability to choose MDIO phy number to i.MX processors
Posted by Peter Maydell 3 years, 9 months ago
On Sat, 4 Jul 2020 at 20:12, Jean-Christophe Dubois <jcd@tribudubois.net> wrote:
>
> This patch set adds the ability to select the Ethernet PHY device number
> for the i.MX25 processor, the i.MX6 processor and the i.MX7 processor.
>
> Note: the i.MX6UL processor was addressed in a previous patch.
>
> Jean-Christophe Dubois (3):
>   Add the ability to change the FEC PHY MDIO device number on i.MX25
>     processor
>   Add the ability to change the FEC PHY MDIO device number on i.MX6
>     processor
>   Add the ability to change the FEC PHY MDIO devices numbers on i.MX7
>     processor



Applied to target-arm.next, thanks.

-- PMM