* [PATCH v1 0/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks
@ 2020-09-09 16:15 Edgar E. Iglesias
2020-09-09 16:15 ` [PATCH v1 1/1] " Edgar E. Iglesias
0 siblings, 1 reply; 5+ messages in thread
From: Edgar E. Iglesias @ 2020-09-09 16:15 UTC (permalink / raw)
To: qemu-devel
Cc: figlesia, peter.maydell, sstabellini, edgar.iglesias,
sai.pavan.boddu, frasse.iglesias, alistair, frederic.konrad,
qemu-arm, philmd, luc.michel
From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
This corrects the Ethernet tx/rx clocks in the generated DTB.
Avoids a Linux kernel warning.
Cheers,
Edgar
Edgar E. Iglesias (1):
hw/arm: versal-virt: Correct the tx/rx GEM clocks
hw/arm/xlnx-versal-virt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v1 1/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks
2020-09-09 16:15 [PATCH v1 0/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks Edgar E. Iglesias
@ 2020-09-09 16:15 ` Edgar E. Iglesias
2020-09-09 16:26 ` Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Edgar E. Iglesias @ 2020-09-09 16:15 UTC (permalink / raw)
To: qemu-devel
Cc: figlesia, peter.maydell, sstabellini, edgar.iglesias,
sai.pavan.boddu, frasse.iglesias, alistair, frederic.konrad,
qemu-arm, philmd, luc.michel
From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
Correct the GEMs tx/rx clocks to use the 125Mhz fixed-clock.
This matches the setup with the fixed-link 100Mbit PHY.
I also avoids the following warnings from the Linux kernel
driver:
eth0: unable to generate target frequency: 125000000 Hz
Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
---
hw/arm/xlnx-versal-virt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
index 4b3152ee77..43d74b850f 100644
--- a/hw/arm/xlnx-versal-virt.c
+++ b/hw/arm/xlnx-versal-virt.c
@@ -212,7 +212,7 @@ static void fdt_add_gem_nodes(VersalVirt *s)
s->phandle.ethernet_phy[i]);
qemu_fdt_setprop_cells(s->fdt, name, "clocks",
s->phandle.clk_25Mhz, s->phandle.clk_25Mhz,
- s->phandle.clk_25Mhz, s->phandle.clk_25Mhz);
+ s->phandle.clk_125Mhz, s->phandle.clk_125Mhz);
qemu_fdt_setprop(s->fdt, name, "clock-names",
clocknames, sizeof(clocknames));
qemu_fdt_setprop_cells(s->fdt, name, "interrupts",
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks
2020-09-09 16:15 ` [PATCH v1 1/1] " Edgar E. Iglesias
@ 2020-09-09 16:26 ` Philippe Mathieu-Daudé
2020-09-09 16:29 ` Luc Michel
2020-09-09 17:21 ` Edgar E. Iglesias
2 siblings, 0 replies; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-09-09 16:26 UTC (permalink / raw)
To: Edgar E. Iglesias, qemu-devel
Cc: figlesia, peter.maydell, sstabellini, edgar.iglesias,
sai.pavan.boddu, frasse.iglesias, alistair, frederic.konrad,
qemu-arm, luc.michel
On 9/9/20 6:15 PM, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>
> Correct the GEMs tx/rx clocks to use the 125Mhz fixed-clock.
> This matches the setup with the fixed-link 100Mbit PHY.
> I also avoids the following warnings from the Linux kernel
> driver:
>
> eth0: unable to generate target frequency: 125000000 Hz
>
> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/arm/xlnx-versal-virt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
> index 4b3152ee77..43d74b850f 100644
> --- a/hw/arm/xlnx-versal-virt.c
> +++ b/hw/arm/xlnx-versal-virt.c
> @@ -212,7 +212,7 @@ static void fdt_add_gem_nodes(VersalVirt *s)
> s->phandle.ethernet_phy[i]);
> qemu_fdt_setprop_cells(s->fdt, name, "clocks",
> s->phandle.clk_25Mhz, s->phandle.clk_25Mhz,
> - s->phandle.clk_25Mhz, s->phandle.clk_25Mhz);
> + s->phandle.clk_125Mhz, s->phandle.clk_125Mhz);
> qemu_fdt_setprop(s->fdt, name, "clock-names",
> clocknames, sizeof(clocknames));
> qemu_fdt_setprop_cells(s->fdt, name, "interrupts",
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks
2020-09-09 16:15 ` [PATCH v1 1/1] " Edgar E. Iglesias
2020-09-09 16:26 ` Philippe Mathieu-Daudé
@ 2020-09-09 16:29 ` Luc Michel
2020-09-09 17:21 ` Edgar E. Iglesias
2 siblings, 0 replies; 5+ messages in thread
From: Luc Michel @ 2020-09-09 16:29 UTC (permalink / raw)
To: Edgar E. Iglesias, qemu-devel
Cc: figlesia, peter.maydell, sstabellini, edgar.iglesias,
sai.pavan.boddu, frasse.iglesias, alistair, frederic.konrad,
qemu-arm, philmd
On 9/9/20 6:15 PM, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>
> Correct the GEMs tx/rx clocks to use the 125Mhz fixed-clock.
> This matches the setup with the fixed-link 100Mbit PHY.
> I also avoids the following warnings from the Linux kernel
> driver:
>
> eth0: unable to generate target frequency: 125000000 Hz
>
> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
> ---
> hw/arm/xlnx-versal-virt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c
> index 4b3152ee77..43d74b850f 100644
> --- a/hw/arm/xlnx-versal-virt.c
> +++ b/hw/arm/xlnx-versal-virt.c
> @@ -212,7 +212,7 @@ static void fdt_add_gem_nodes(VersalVirt *s)
> s->phandle.ethernet_phy[i]);
> qemu_fdt_setprop_cells(s->fdt, name, "clocks",
> s->phandle.clk_25Mhz, s->phandle.clk_25Mhz,
> - s->phandle.clk_25Mhz, s->phandle.clk_25Mhz);
> + s->phandle.clk_125Mhz, s->phandle.clk_125Mhz);
> qemu_fdt_setprop(s->fdt, name, "clock-names",
> clocknames, sizeof(clocknames));
> qemu_fdt_setprop_cells(s->fdt, name, "interrupts",
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks
2020-09-09 16:15 ` [PATCH v1 1/1] " Edgar E. Iglesias
2020-09-09 16:26 ` Philippe Mathieu-Daudé
2020-09-09 16:29 ` Luc Michel
@ 2020-09-09 17:21 ` Edgar E. Iglesias
2 siblings, 0 replies; 5+ messages in thread
From: Edgar E. Iglesias @ 2020-09-09 17:21 UTC (permalink / raw)
To: Edgar E. Iglesias
Cc: figlesia, peter.maydell, sstabellini, sai.pavan.boddu,
frasse.iglesias, alistair, qemu-devel, frederic.konrad, qemu-arm,
philmd, luc.michel
On Wed, Sep 09, 2020 at 06:15:14PM +0200, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>
> Correct the GEMs tx/rx clocks to use the 125Mhz fixed-clock.
> This matches the setup with the fixed-link 100Mbit PHY.
> I also avoids the following warnings from the Linux kernel
^^^
Argh, this should say It, not I.
I'll send a v2 keeping the RB tags.
Cheers,
Edgar
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-09-09 17:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-09 16:15 [PATCH v1 0/1] hw/arm: versal-virt: Correct the tx/rx GEM clocks Edgar E. Iglesias
2020-09-09 16:15 ` [PATCH v1 1/1] " Edgar E. Iglesias
2020-09-09 16:26 ` Philippe Mathieu-Daudé
2020-09-09 16:29 ` Luc Michel
2020-09-09 17:21 ` Edgar E. Iglesias
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).