diff mbox series

opensbi: bump to 1.5.1

Message ID 20240802121336.1642376-1-thomas.perrot@bootlin.com
State Accepted, archived
Commit f102511f3a7707d2564e17af5dfdb1406177d7bb
Headers show
Series opensbi: bump to 1.5.1 | expand

Commit Message

Thomas Perrot Aug. 2, 2024, 12:13 p.m. UTC
From: Thomas Perrot <thomas.perrot@bootlin.com>

This intermediate release has following fixes:
- Save/restore menvcfg only when it exists
- Adjust Sscofpmf mhpmevent mask for upper 8 bits
- Fix potential NULL pointer dereferences in SBI DBTR
- Fix incorrect size passed to sbi_zalloc() in SBI FWFT
- Check result of pmp_get() in is_pmp_entry_mapped()

Signed-off-by: Thomas Perrot <thomas.perrot@bootlin.com>
---
 meta/recipes-bsp/opensbi/opensbi_1.5.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Kanavin Aug. 2, 2024, 2 p.m. UTC | #1
You need to rename the recipe as well?

Alex

On Fri, 2 Aug 2024 at 14:13, Thomas Perrot via lists.openembedded.org
<thomas.perrot=bootlin.com@lists.openembedded.org> wrote:
>
> From: Thomas Perrot <thomas.perrot@bootlin.com>
>
> This intermediate release has following fixes:
> - Save/restore menvcfg only when it exists
> - Adjust Sscofpmf mhpmevent mask for upper 8 bits
> - Fix potential NULL pointer dereferences in SBI DBTR
> - Fix incorrect size passed to sbi_zalloc() in SBI FWFT
> - Check result of pmp_get() in is_pmp_entry_mapped()
>
> Signed-off-by: Thomas Perrot <thomas.perrot@bootlin.com>
> ---
>  meta/recipes-bsp/opensbi/opensbi_1.5.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-bsp/opensbi/opensbi_1.5.bb b/meta/recipes-bsp/opensbi/opensbi_1.5.bb
> index 6b31d15a051b..652b81182b80 100644
> --- a/meta/recipes-bsp/opensbi/opensbi_1.5.bb
> +++ b/meta/recipes-bsp/opensbi/opensbi_1.5.bb
> @@ -8,8 +8,8 @@ require opensbi-payloads.inc
>
>  inherit autotools-brokensep deploy
>
> -SRCREV = "455de672dd7c2aa1992df54dfb08dc11abbc1b1a"
> -SRC_URI = "git://github.com/riscv/opensbi.git;branch=master;protocol=https"
> +SRCREV = "43cace6c3671e5172d0df0a8963e552bb04b7b20"
> +SRC_URI = "git://github.com/riscv/opensbi.git;branch=release-1.5.x;protocol=https"
>
>  S = "${WORKDIR}/git"
>
> --
> 2.45.2
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#202901): https://lists.openembedded.org/g/openembedded-core/message/202901
> Mute This Topic: https://lists.openembedded.org/mt/107683486/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Thomas Perrot Aug. 2, 2024, 2:57 p.m. UTC | #2
Hello Alexander,

On Fri, 2024-08-02 at 16:00 +0200, Alexander Kanavin wrote:
> You need to rename the recipe as well?

As for some recipes, like the kernel recipes, I only used the first two
numbers of the release version.
I can rename it to use all numbers if you prefer.

Kind regards,
Thomas Perrot
Alexander Kanavin Aug. 2, 2024, 3:03 p.m. UTC | #3
On Fri, 2 Aug 2024 at 16:57, Thomas Perrot <thomas.perrot@bootlin.com> wrote:
> As for some recipes, like the kernel recipes, I only used the first two
> numbers of the release version.
> I can rename it to use all numbers if you prefer.

PV (set via file name) needs to exactly match the version tag in git
repo that matches the git revision in the recipe, it's not a
preference, but standard practice. Otherwise the recipe would be
packaging a random revision and not the actual PV.

Alex
Thomas Perrot Aug. 2, 2024, 3:15 p.m. UTC | #4
Hello Alex,

On Fri, 2024-08-02 at 17:03 +0200, Alexander Kanavin wrote:
> On Fri, 2 Aug 2024 at 16:57, Thomas Perrot
> <thomas.perrot@bootlin.com> wrote:
> > As for some recipes, like the kernel recipes, I only used the first
> > two
> > numbers of the release version.
> > I can rename it to use all numbers if you prefer.
> 
> PV (set via file name) needs to exactly match the version tag in git
> repo that matches the git revision in the recipe, it's not a
> preference, but standard practice. Otherwise the recipe would be
> packaging a random revision and not the actual PV.
> 

Thx, I'm submitting a v2 that renames the recipe.

Kind regards,
Thomas Perrot

> Alex
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#202918):
> https://lists.openembedded.org/g/openembedded-core/message/202918
> Mute This Topic: https://lists.openembedded.org/mt/107683486/5443093
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe:
> https://lists.openembedded.org/g/openembedded-core/unsubĀ [
> thomas.perrot@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta/recipes-bsp/opensbi/opensbi_1.5.bb b/meta/recipes-bsp/opensbi/opensbi_1.5.bb
index 6b31d15a051b..652b81182b80 100644
--- a/meta/recipes-bsp/opensbi/opensbi_1.5.bb
+++ b/meta/recipes-bsp/opensbi/opensbi_1.5.bb
@@ -8,8 +8,8 @@  require opensbi-payloads.inc
 
 inherit autotools-brokensep deploy
 
-SRCREV = "455de672dd7c2aa1992df54dfb08dc11abbc1b1a"
-SRC_URI = "git://github.com/riscv/opensbi.git;branch=master;protocol=https"
+SRCREV = "43cace6c3671e5172d0df0a8963e552bb04b7b20"
+SRC_URI = "git://github.com/riscv/opensbi.git;branch=release-1.5.x;protocol=https"
 
 S = "${WORKDIR}/git"