RE: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-11 Thread Adrian Schmutzler
Hi,

> -Original Message-
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of Paul Spooren
> Sent: Dienstag, 7. September 2021 23:37
> To: Rui Salvaterra ; Daniel Golle
> 
> Cc: openwrt-devel@lists.openwrt.org; pepe.schleho...@gmail.com;
> klaus.kudie...@gmail.com; tmn...@gmail.com; ha...@hauke-m.de
> Subject: Re: [PATCH] mvebu/kernel: promote 5.10 to stable
> 
> 
> On 9/7/21 11:31, Rui Salvaterra wrote:
> > Hi, Daniel,
> >
> > On Tue, 7 Sept 2021 at 22:09, Daniel Golle 
wrote:
> >> I'd remove the KERNEL_TESTING_PATCHVER line then until we add 5.14 or
> >> whatever it's going to be.
> > Sure, I didn't know what the usual procedure was, so I just mimicked
> > aparcar. ;)
> > V2 coming up.
> 
> This seems to be an ongoing thing, some add it and some remove it, I'm
> happy to make this consistent:
> 
> * Remove KERNEL_TESTING_PATCHVER when promoting
> 
> or
> 
> * Set KERNEL_{,TESTING}_PATCHVER to the same value until a new testing
> Kernel is added

Some prefer not having to add it again, some prefer to have it removed.

In any case, lines are directly next to each other, so nobody should have a
problem with finding out that both are the same if it's kept.

Since there have been different opinions expressed by committers (e.g. I
prefer to keep it), we would technically need a vote on this.
However, I think this is definitely way too unimportant to vote on it. So
I'd just leave it to the submitters' and committers' taste.

Best

Adrian


> 
> I'd prefer the former since it removes the feature flag `testing-kernel`,
> therefore I'd update my patch and we should do that for all further
*Kernel
> promotions*. Objections?
> 
> 
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Andre Heider

On 07/09/2021 23:40, Rui Salvaterra wrote:

Hi, Paul,

On Tue, 7 Sept 2021 at 22:37, Paul Spooren  wrote:


This seems to be an ongoing thing, some add it and some remove it, I'm
happy to make this consistent:

* Remove KERNEL_TESTING_PATCHVER when promoting

or

* Set KERNEL_{,TESTING}_PATCHVER to the same value until a new testing
Kernel is added

I'd prefer the former since it removes the feature flag
`testing-kernel`, therefore I'd update my patch and we should do that
for all further *Kernel promotions*. Objections?


Agreed, I also prefer the former. Should cause less confusion when
configuring, at least.


That has been solved fwiw, for the record:

commit cf6f01c0e69a31afc6c5f384405ace51e03fe481
Author: David Bauer 
Date:   Tue Jul 7 10:32:09 2020 +0200

build: conditionally enable testing-kernel feature

Only enable the testing-kernel feature for the target when the testing
kernel version does not match the stable kernel version.

This way, the option for building the testing kernel in the build 
config

menu is only exposed when there's a testing kernel available.

Signed-off-by: David Bauer 
Acked-by: Adrian Schmutzler 

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Rui Salvaterra
Hi, Paul,

On Tue, 7 Sept 2021 at 22:37, Paul Spooren  wrote:
>
> This seems to be an ongoing thing, some add it and some remove it, I'm
> happy to make this consistent:
>
> * Remove KERNEL_TESTING_PATCHVER when promoting
>
> or
>
> * Set KERNEL_{,TESTING}_PATCHVER to the same value until a new testing
> Kernel is added
>
> I'd prefer the former since it removes the feature flag
> `testing-kernel`, therefore I'd update my patch and we should do that
> for all further *Kernel promotions*. Objections?

Agreed, I also prefer the former. Should cause less confusion when
configuring, at least.

Thanks,
Rui

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Paul Spooren



On 9/7/21 11:31, Rui Salvaterra wrote:

Hi, Daniel,

On Tue, 7 Sept 2021 at 22:09, Daniel Golle  wrote:

I'd remove the KERNEL_TESTING_PATCHVER line then until we add 5.14 or
whatever it's going to be.

Sure, I didn't know what the usual procedure was, so I just mimicked aparcar. ;)
V2 coming up.


This seems to be an ongoing thing, some add it and some remove it, I'm 
happy to make this consistent:


* Remove KERNEL_TESTING_PATCHVER when promoting

or

* Set KERNEL_{,TESTING}_PATCHVER to the same value until a new testing 
Kernel is added


I'd prefer the former since it removes the feature flag 
`testing-kernel`, therefore I'd update my patch and we should do that 
for all further *Kernel promotions*. Objections?



___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Rui Salvaterra
Hi, Daniel,

On Tue, 7 Sept 2021 at 22:09, Daniel Golle  wrote:
>
> I'd remove the KERNEL_TESTING_PATCHVER line then until we add 5.14 or
> whatever it's going to be.

Sure, I didn't know what the usual procedure was, so I just mimicked aparcar. ;)
V2 coming up.

Thanks,
Rui

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Daniel Golle
On Tue, Sep 07, 2021 at 09:23:06PM +0100, Rui Salvaterra wrote:
> It's been brewing on my cortexa9 subtarget (Turris Omnia) for months. 
> Perfectly
> stable.
> 
> Signed-off-by: Rui Salvaterra 
> ---
>  target/linux/mvebu/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/target/linux/mvebu/Makefile b/target/linux/mvebu/Makefile
> index 3737a4aa23..6c2280cc79 100644
> --- a/target/linux/mvebu/Makefile
> +++ b/target/linux/mvebu/Makefile
> @@ -9,7 +9,7 @@ BOARDNAME:=Marvell EBU Armada
>  FEATURES:=fpu usb pci pcie gpio nand squashfs ramdisk boot-part rootfs-part 
> legacy-sdcard
>  SUBTARGETS:=cortexa9 cortexa53 cortexa72
>  
> -KERNEL_PATCHVER:=5.4
> +KERNEL_PATCHVER:=5.10
>  KERNEL_TESTING_PATCHVER:=5.10

I'd remove the KERNEL_TESTING_PATCHVER line then until we add 5.14 or
whatever it's going to be.


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Josef Schlehofer
Ășt 7. 9. 2021 v 22:23 odesĂ­latel Rui Salvaterra  napsal:
>
> It's been brewing on my cortexa9 subtarget (Turris Omnia) for months. 
> Perfectly
> stable.
>
> Signed-off-by: Rui Salvaterra 
Tested-by: Josef Schlehofer 
> ---
>  target/linux/mvebu/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/linux/mvebu/Makefile b/target/linux/mvebu/Makefile
> index 3737a4aa23..6c2280cc79 100644
> --- a/target/linux/mvebu/Makefile
> +++ b/target/linux/mvebu/Makefile
> @@ -9,7 +9,7 @@ BOARDNAME:=Marvell EBU Armada
>  FEATURES:=fpu usb pci pcie gpio nand squashfs ramdisk boot-part rootfs-part 
> legacy-sdcard
>  SUBTARGETS:=cortexa9 cortexa53 cortexa72
>
> -KERNEL_PATCHVER:=5.4
> +KERNEL_PATCHVER:=5.10
>  KERNEL_TESTING_PATCHVER:=5.10
>
>  include $(INCLUDE_DIR)/target.mk
> --
> 2.33.0
>

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[PATCH] mvebu/kernel: promote 5.10 to stable

2021-09-07 Thread Rui Salvaterra
It's been brewing on my cortexa9 subtarget (Turris Omnia) for months. Perfectly
stable.

Signed-off-by: Rui Salvaterra 
---
 target/linux/mvebu/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/linux/mvebu/Makefile b/target/linux/mvebu/Makefile
index 3737a4aa23..6c2280cc79 100644
--- a/target/linux/mvebu/Makefile
+++ b/target/linux/mvebu/Makefile
@@ -9,7 +9,7 @@ BOARDNAME:=Marvell EBU Armada
 FEATURES:=fpu usb pci pcie gpio nand squashfs ramdisk boot-part rootfs-part 
legacy-sdcard
 SUBTARGETS:=cortexa9 cortexa53 cortexa72
 
-KERNEL_PATCHVER:=5.4
+KERNEL_PATCHVER:=5.10
 KERNEL_TESTING_PATCHVER:=5.10
 
 include $(INCLUDE_DIR)/target.mk
-- 
2.33.0


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel