* [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
@ 2019-05-17 15:01 Jan Kundrát
2019-05-17 19:31 ` Gregory CLEMENT
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-05-17 15:01 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Baruch Siach, Maxime Chevallier, Russell King, Gregory CLEMENT,
David S. Miller
Compared to kernel 5.1, patches merged for 5.2 added support for A38x'
PHY guarded by a config option which was not enabled by default. As a
result, there was no eth1 and eth2 on a Solid Run Clearfog Base.
Ensure that A38x PHY is enabled on mvebu.
Signed-off-by: Jan Kundrát <jan.kundrat@cesnet.cz>
Cc: Baruch Siach <baruch@tkos.co.il>
Cc: Gregory CLEMENT <gregory.clement@bootlin.com>
Cc: Russell King <rmk+kernel@armlinux.org.uk>
Cc: David S. Miller <davem@davemloft.net>
Cc: Maxime Chevallier <maxime.chevallier@bootlin.com>
---
arch/arm/configs/mvebu_v7_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
index 48f7b4277b8d..3ac2e84fdeaa 100644
--- a/arch/arm/configs/mvebu_v7_defconfig
+++ b/arch/arm/configs/mvebu_v7_defconfig
@@ -131,6 +131,7 @@ CONFIG_MV_XOR=y
# CONFIG_IOMMU_SUPPORT is not set
CONFIG_MEMORY=y
CONFIG_PWM=y
+CONFIG_PHY_MVEBU_A38X_COMPHY=y
CONFIG_EXT4_FS=y
CONFIG_ISO9660_FS=y
CONFIG_JOLIET=y
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-17 15:01 [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog Jan Kundrát
@ 2019-05-17 19:31 ` Gregory CLEMENT
2019-05-17 19:34 ` Jan Kundrát
0 siblings, 1 reply; 14+ messages in thread
From: Gregory CLEMENT @ 2019-05-17 19:31 UTC (permalink / raw)
To: Jan Kundrát, linux-arm-kernel
Cc: Baruch Siach, Maxime Chevallier, Russell King, David S. Miller
Hi Jan Kundrát,
> Compared to kernel 5.1, patches merged for 5.2 added support for A38x'
> PHY guarded by a config option which was not enabled by default. As a
> result, there was no eth1 and eth2 on a Solid Run Clearfog Base.
>
> Ensure that A38x PHY is enabled on mvebu.
>
Applied on mvebu/arm-5.3
Thanks,
Gregory
> Signed-off-by: Jan Kundrát <jan.kundrat@cesnet.cz>
> Cc: Baruch Siach <baruch@tkos.co.il>
> Cc: Gregory CLEMENT <gregory.clement@bootlin.com>
> Cc: Russell King <rmk+kernel@armlinux.org.uk>
> Cc: David S. Miller <davem@davemloft.net>
> Cc: Maxime Chevallier <maxime.chevallier@bootlin.com>
> ---
> arch/arm/configs/mvebu_v7_defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/configs/mvebu_v7_defconfig b/arch/arm/configs/mvebu_v7_defconfig
> index 48f7b4277b8d..3ac2e84fdeaa 100644
> --- a/arch/arm/configs/mvebu_v7_defconfig
> +++ b/arch/arm/configs/mvebu_v7_defconfig
> @@ -131,6 +131,7 @@ CONFIG_MV_XOR=y
> # CONFIG_IOMMU_SUPPORT is not set
> CONFIG_MEMORY=y
> CONFIG_PWM=y
> +CONFIG_PHY_MVEBU_A38X_COMPHY=y
> CONFIG_EXT4_FS=y
> CONFIG_ISO9660_FS=y
> CONFIG_JOLIET=y
> --
> 2.21.0
>
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-17 19:31 ` Gregory CLEMENT
@ 2019-05-17 19:34 ` Jan Kundrát
2019-05-17 19:53 ` Gregory CLEMENT
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-05-17 19:34 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
> Applied on mvebu/arm-5.3
To me this looks like a 5.2 material because it fixes a regression that got
introduced within the 5.2 cycle. Or did I get the process wrong and this is
perhaps just a first step?
Cheers,
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-17 19:34 ` Jan Kundrát
@ 2019-05-17 19:53 ` Gregory CLEMENT
2019-05-17 22:50 ` Jan Kundrát
0 siblings, 1 reply; 14+ messages in thread
From: Gregory CLEMENT @ 2019-05-17 19:53 UTC (permalink / raw)
To: Jan Kundrát
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
Jan Kundrát <jan.kundrat@cesnet.cz> writes:
>> Applied on mvebu/arm-5.3
>
> To me this looks like a 5.2 material because it fixes a regression that got
> introduced within the 5.2 cycle. Or did I get the process wrong and this is
> perhaps just a first step?
Well, this is just about configuration, I don't consider this is
something that is a candidate for a fix.
If there is a regression, then, it is maybe located in the Kconfig
dependency.
Of course I can change my mind with good arguments :)
Gregory
>
> Cheers,
> Jan
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-17 19:53 ` Gregory CLEMENT
@ 2019-05-17 22:50 ` Jan Kundrát
2019-05-27 15:57 ` Jan Kundrát
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-05-17 22:50 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
> Well, this is just about configuration, I don't consider this is
> something that is a candidate for a fix.
>
> If there is a regression, then, it is maybe located in the Kconfig
> dependency.
>
> Of course I can change my mind with good arguments :)
Hi Gregory,
I agree that it's just a config bug, but it's also something which can
silently produce broken systems. If this is not fixed, people building
their 5.2 kernels will not have working network on Clearfog unless they
take an extra action. For example, a Buildroot defconfig that's been
available for quite some time (and which uses just `mvebu_v7_defconfig` for
kernel) suddenly becomes broken.
Isn't the whole point of the -rc release to find *and* fix bugs early? This
trivial patch does not introduce any new or untested code. I made a choice
to test a pre-release kernel, I hit a bug -- no big deal. I found the root
cause, I sent a trivial fix upstream, and now I'm told by a maintainer that
they will let the next kernel version, which is about seven -rc releases
away, be released without a fully functioning network, I am surprised by
that. I would have understood this better if we were at the final -rc
stage, but during the merge window? Or is that perhaps a misunderstanding
and you're planning to send this in time after -rc1?
Cheers,
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-17 22:50 ` Jan Kundrát
@ 2019-05-27 15:57 ` Jan Kundrát
2019-06-11 11:28 ` Gregory CLEMENT
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-05-27 15:57 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
On sobota 18. května 2019 0:50:28 CEST, Jan Kundrát wrote:
>> Well, this is just about configuration, I don't consider this is
>> something that is a candidate for a fix.
>>
>> If there is a regression, then, it is maybe located in the Kconfig
>> dependency.
>>
>> Of course I can change my mind with good arguments :)
>
> Hi Gregory,
> I agree that it's just a config bug, but it's also something
> which can silently produce broken systems. If this is not fixed,
> people building their 5.2 kernels will not have working network
> on Clearfog unless they take an extra action. For example, a
> Buildroot defconfig that's been available for quite some time
> (and which uses just `mvebu_v7_defconfig` for kernel) suddenly
> becomes broken.
>
> Isn't the whole point of the -rc release to find *and* fix bugs
> early? This trivial patch does not introduce any new or untested
> code. I made a choice to test a pre-release kernel, I hit a bug
> -- no big deal. I found the root cause, I sent a trivial fix
> upstream, and now I'm told by a maintainer that they will let
> the next kernel version, which is about seven -rc releases away,
> be released without a fully functioning network, I am surprised
> by that. I would have understood this better if we were at the
> final -rc stage, but during the merge window? Or is that perhaps
> a misunderstanding and you're planning to send this in time
> after -rc1?
Hi Gregory,
was I successful in persuading you that this patch should be included in
the 5.2 tree, so that Clearfog Base has all three Ethernet interfaces?
With kind regards,
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-05-27 15:57 ` Jan Kundrát
@ 2019-06-11 11:28 ` Gregory CLEMENT
2019-06-11 14:15 ` Jan Kundrát
2019-06-11 14:32 ` Russell King - ARM Linux admin
0 siblings, 2 replies; 14+ messages in thread
From: Gregory CLEMENT @ 2019-06-11 11:28 UTC (permalink / raw)
To: Jan Kundrát
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
Jan Kundrát <jan.kundrat@cesnet.cz> writes:
> On sobota 18. května 2019 0:50:28 CEST, Jan Kundrát wrote:
>>> Well, this is just about configuration, I don't consider this is
>>> something that is a candidate for a fix.
>>>
>>> If there is a regression, then, it is maybe located in the Kconfig
>>> dependency.
>>>
>>> Of course I can change my mind with good arguments :)
>>
>> Hi Gregory,
>> I agree that it's just a config bug, but it's also something
>> which can silently produce broken systems. If this is not fixed,
>> people building their 5.2 kernels will not have working network
>> on Clearfog unless they take an extra action. For example, a
>> Buildroot defconfig that's been available for quite some time
>> (and which uses just `mvebu_v7_defconfig` for kernel) suddenly
>> becomes broken.
>>
>> Isn't the whole point of the -rc release to find *and* fix bugs
>> early? This trivial patch does not introduce any new or untested
>> code. I made a choice to test a pre-release kernel, I hit a bug
>> -- no big deal. I found the root cause, I sent a trivial fix
>> upstream, and now I'm told by a maintainer that they will let
>> the next kernel version, which is about seven -rc releases away,
>> be released without a fully functioning network, I am surprised
>> by that. I would have understood this better if we were at the
>> final -rc stage, but during the merge window? Or is that perhaps
>> a misunderstanding and you're planning to send this in time
>> after -rc1?
>
> Hi Gregory,
> was I successful in persuading you that this patch should be included in
> the 5.2 tree, so that Clearfog Base has all three Ethernet interfaces?
Finally I moved the commit from mvebu/arm to mvebu/fixes. I still think
the problem is at driver level, but I didn't take enough time to find
where and we didn't have any feedback from the author of the initila
patch.
So let's try to push it to fixes, I will do the pull request for arm-soc
before end of the week.
Gregory
>
> With kind regards,
> Jan
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 11:28 ` Gregory CLEMENT
@ 2019-06-11 14:15 ` Jan Kundrát
2019-06-11 14:32 ` Russell King - ARM Linux admin
1 sibling, 0 replies; 14+ messages in thread
From: Jan Kundrát @ 2019-06-11 14:15 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Baruch Siach, Maxime Chevallier, Russell King, linux-arm-kernel,
David S. Miller
On úterý 11. června 2019 13:28:32 CEST, Gregory CLEMENT wrote:
> Jan Kundrát <jan.kundrat@cesnet.cz> writes:
>
>> On sobota 18. května 2019 0:50:28 CEST, Jan Kundrát wrote: ...
>
> Finally I moved the commit from mvebu/arm to mvebu/fixes. I still think
> the problem is at driver level, but I didn't take enough time to find
> where and we didn't have any feedback from the author of the initila
> patch.
>
> So let's try to push it to fixes, I will do the pull request for arm-soc
> before end of the week.
Thanks!
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 11:28 ` Gregory CLEMENT
2019-06-11 14:15 ` Jan Kundrát
@ 2019-06-11 14:32 ` Russell King - ARM Linux admin
2019-06-11 15:04 ` Gregory CLEMENT
1 sibling, 1 reply; 14+ messages in thread
From: Russell King - ARM Linux admin @ 2019-06-11 14:32 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Jan Kundrát, Maxime Chevallier, Baruch Siach,
linux-arm-kernel, David S. Miller
On Tue, Jun 11, 2019 at 01:28:32PM +0200, Gregory CLEMENT wrote:
> Jan Kundrát <jan.kundrat@cesnet.cz> writes:
>
> > On sobota 18. května 2019 0:50:28 CEST, Jan Kundrát wrote:
> >>> Well, this is just about configuration, I don't consider this is
> >>> something that is a candidate for a fix.
> >>>
> >>> If there is a regression, then, it is maybe located in the Kconfig
> >>> dependency.
> >>>
> >>> Of course I can change my mind with good arguments :)
> >>
> >> Hi Gregory,
> >> I agree that it's just a config bug, but it's also something
> >> which can silently produce broken systems. If this is not fixed,
> >> people building their 5.2 kernels will not have working network
> >> on Clearfog unless they take an extra action. For example, a
> >> Buildroot defconfig that's been available for quite some time
> >> (and which uses just `mvebu_v7_defconfig` for kernel) suddenly
> >> becomes broken.
> >>
> >> Isn't the whole point of the -rc release to find *and* fix bugs
> >> early? This trivial patch does not introduce any new or untested
> >> code. I made a choice to test a pre-release kernel, I hit a bug
> >> -- no big deal. I found the root cause, I sent a trivial fix
> >> upstream, and now I'm told by a maintainer that they will let
> >> the next kernel version, which is about seven -rc releases away,
> >> be released without a fully functioning network, I am surprised
> >> by that. I would have understood this better if we were at the
> >> final -rc stage, but during the merge window? Or is that perhaps
> >> a misunderstanding and you're planning to send this in time
> >> after -rc1?
> >
> > Hi Gregory,
> > was I successful in persuading you that this patch should be included in
> > the 5.2 tree, so that Clearfog Base has all three Ethernet interfaces?
>
> Finally I moved the commit from mvebu/arm to mvebu/fixes. I still think
> the problem is at driver level, but I didn't take enough time to find
> where and we didn't have any feedback from the author of the initila
> patch.
I don't see that there's much that I need to say, and I'm at a total
loss to work out why you think it's a problem at driver level.
Why do you think it's appropriate for mvneta to know whether the a38x
comphy driver is configured for the current kernel or not, given that
mvneta is not exclusively used on Armada 38x systems?
We're already doing the best we can do with ignoring the comphy if
not present; the only case that we defer probe is when
devm_of_phy_get() returns -EPROBE_DEFER, which could mean "the comphy
driver is a module but is not loaded yet" or "the comphy driver has
not been probed yet" - we can't ignore those.
> So let's try to push it to fixes, I will do the pull request for arm-soc
> before end of the week.
That's the correct solution, and it should also have a Fixes: tag on
it. Unfortunately, keeping the defconfigs up to date is quite a hard
problem unless you have lots of computing power to build and boot all
the defconfigs on all platforms (I don't.)
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 14:32 ` Russell King - ARM Linux admin
@ 2019-06-11 15:04 ` Gregory CLEMENT
2019-06-11 15:12 ` Jan Kundrát
0 siblings, 1 reply; 14+ messages in thread
From: Gregory CLEMENT @ 2019-06-11 15:04 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Jan Kundrát, Maxime Chevallier, Baruch Siach,
linux-arm-kernel, David S. Miller
Hello Russell King,
> On Tue, Jun 11, 2019 at 01:28:32PM +0200, Gregory CLEMENT wrote:
>> Jan Kundrát <jan.kundrat@cesnet.cz> writes:
>>
>> > On sobota 18. května 2019 0:50:28 CEST, Jan Kundrát wrote:
>> >>> Well, this is just about configuration, I don't consider this is
>> >>> something that is a candidate for a fix.
>> >>>
>> >>> If there is a regression, then, it is maybe located in the Kconfig
>> >>> dependency.
>> >>>
>> >>> Of course I can change my mind with good arguments :)
>> >>
>> >> Hi Gregory,
>> >> I agree that it's just a config bug, but it's also something
>> >> which can silently produce broken systems. If this is not fixed,
>> >> people building their 5.2 kernels will not have working network
>> >> on Clearfog unless they take an extra action. For example, a
>> >> Buildroot defconfig that's been available for quite some time
>> >> (and which uses just `mvebu_v7_defconfig` for kernel) suddenly
>> >> becomes broken.
>> >>
>> >> Isn't the whole point of the -rc release to find *and* fix bugs
>> >> early? This trivial patch does not introduce any new or untested
>> >> code. I made a choice to test a pre-release kernel, I hit a bug
>> >> -- no big deal. I found the root cause, I sent a trivial fix
>> >> upstream, and now I'm told by a maintainer that they will let
>> >> the next kernel version, which is about seven -rc releases away,
>> >> be released without a fully functioning network, I am surprised
>> >> by that. I would have understood this better if we were at the
>> >> final -rc stage, but during the merge window? Or is that perhaps
>> >> a misunderstanding and you're planning to send this in time
>> >> after -rc1?
>> >
>> > Hi Gregory,
>> > was I successful in persuading you that this patch should be included in
>> > the 5.2 tree, so that Clearfog Base has all three Ethernet interfaces?
>>
>> Finally I moved the commit from mvebu/arm to mvebu/fixes. I still think
>> the problem is at driver level, but I didn't take enough time to find
>> where and we didn't have any feedback from the author of the initila
>> patch.
>
> I don't see that there's much that I need to say, and I'm at a total
> loss to work out why you think it's a problem at driver level.
>
> Why do you think it's appropriate for mvneta to know whether the a38x
> comphy driver is configured for the current kernel or not, given that
> mvneta is not exclusively used on Armada 38x systems?
>
> We're already doing the best we can do with ignoring the comphy if
> not present; the only case that we defer probe is when
> devm_of_phy_get() returns -EPROBE_DEFER, which could mean "the comphy
> driver is a module but is not loaded yet" or "the comphy driver has
> not been probed yet" - we can't ignore those.
>
>> So let's try to push it to fixes, I will do the pull request for arm-soc
>> before end of the week.
>
> That's the correct solution, and it should also have a Fixes: tag on
> it. Unfortunately, keeping the defconfigs up to date is quite a hard
> problem unless you have lots of computing power to build and boot all
> the defconfigs on all platforms (I don't.)
Thanks for the explanation and confirmation. As the issue appeared only
in 5.2-rc1 then it won't be backported and for my point of view the main
interest of Fixes is for the backport, that's I though that not having a
Fixes tag was an issue.
But I don't mind adding a Fixes: tag.
Jan, do you find which exact commit cause this regression?
Thanks,
Gregory
>
> --
> RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
> FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
> According to speedtest.net: 11.9Mbps down 500kbps up
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 15:04 ` Gregory CLEMENT
@ 2019-06-11 15:12 ` Jan Kundrát
2019-06-11 15:31 ` Russell King - ARM Linux admin
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-06-11 15:12 UTC (permalink / raw)
To: Gregory CLEMENT
Cc: Baruch Siach, Maxime Chevallier, Russell King - ARM Linux admin,
linux-arm-kernel, David S. Miller
On úterý 11. června 2019 17:04:38 CEST, Gregory CLEMENT wrote:
> Jan, do you find which exact commit cause this regression?
I have not bisected, but I think that it is due to
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a10c1c8191e04c21769656c2ca8e1c69a6218954
No guarantees there, though.
Cheers,
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 15:12 ` Jan Kundrát
@ 2019-06-11 15:31 ` Russell King - ARM Linux admin
2019-06-11 15:36 ` Jan Kundrát
0 siblings, 1 reply; 14+ messages in thread
From: Russell King - ARM Linux admin @ 2019-06-11 15:31 UTC (permalink / raw)
To: Jan Kundrát
Cc: Gregory CLEMENT, David S. Miller, Baruch Siach, linux-arm-kernel,
Maxime Chevallier
On Tue, Jun 11, 2019 at 05:12:45PM +0200, Jan Kundrát wrote:
> On úterý 11. června 2019 17:04:38 CEST, Gregory CLEMENT wrote:
> > Jan, do you find which exact commit cause this regression?
>
> I have not bisected, but I think that it is due to https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a10c1c8191e04c21769656c2ca8e1c69a6218954
>
> No guarantees there, though.
Correct, which means it does need backporting to 5.1, since that was
introduced in the 5.1 merge window.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 15:31 ` Russell King - ARM Linux admin
@ 2019-06-11 15:36 ` Jan Kundrát
2019-06-12 8:44 ` Gregory CLEMENT
0 siblings, 1 reply; 14+ messages in thread
From: Jan Kundrát @ 2019-06-11 15:36 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Gregory CLEMENT, David S. Miller, Baruch Siach, linux-arm-kernel,
Maxime Chevallier
>> I have not bisected, but I think that it is due to
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a10c1c8191e04c21769656c2ca8e1c69a6218954
>>
>> No guarantees there, though.
>
> Correct, which means it does need backporting to 5.1, since that was
> introduced in the 5.1 merge window.
...which means that I made a mistake during my initial report. If it's in
5.1 already, then 5.1 could not have worked for me. Sorry for confusion,
it's quite possible that I compared 5.2-rcX with 5.0.y.
Jan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog
2019-06-11 15:36 ` Jan Kundrát
@ 2019-06-12 8:44 ` Gregory CLEMENT
0 siblings, 0 replies; 14+ messages in thread
From: Gregory CLEMENT @ 2019-06-12 8:44 UTC (permalink / raw)
To: Jan Kundrát, Russell King - ARM Linux admin
Cc: Baruch Siach, David S. Miller, linux-arm-kernel, Maxime Chevallier
Jan Kundrát <jan.kundrat@cesnet.cz> writes:
>>> I have not bisected, but I think that it is due to
>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a10c1c8191e04c21769656c2ca8e1c69a6218954
>>>
>>> No guarantees there, though.
>>
>> Correct, which means it does need backporting to 5.1, since that was
>> introduced in the 5.1 merge window.
>
> ...which means that I made a mistake during my initial report. If it's in
> 5.1 already, then 5.1 could not have worked for me. Sorry for confusion,
> it's quite possible that I compared 5.2-rcX with 5.0.y.
So I added the Fixes tag and correct the commit log, mentioning 5.1
instead of 5.2.
The commit is in mvebu/fixes and for-next on
git://git.infradead.org/linux-mvebu.git and will be part of the Fixes
pull request I will do at the end of the week.
Gregory
>
> Jan
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-06-12 8:44 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-17 15:01 [PATCH] ARM: mvebu_v7_defconfig: fix Ethernet on Clearfog Jan Kundrát
2019-05-17 19:31 ` Gregory CLEMENT
2019-05-17 19:34 ` Jan Kundrát
2019-05-17 19:53 ` Gregory CLEMENT
2019-05-17 22:50 ` Jan Kundrát
2019-05-27 15:57 ` Jan Kundrát
2019-06-11 11:28 ` Gregory CLEMENT
2019-06-11 14:15 ` Jan Kundrát
2019-06-11 14:32 ` Russell King - ARM Linux admin
2019-06-11 15:04 ` Gregory CLEMENT
2019-06-11 15:12 ` Jan Kundrát
2019-06-11 15:31 ` Russell King - ARM Linux admin
2019-06-11 15:36 ` Jan Kundrát
2019-06-12 8:44 ` Gregory CLEMENT
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).