* [OE-core] [kirkstone] [PATCH 0/1] Fix a compilation issue of mmc-utils
@ 2022-05-15 12:43 liu.ming50
2022-05-15 12:43 ` [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision liu.ming50
0 siblings, 1 reply; 7+ messages in thread
From: liu.ming50 @ 2022-05-15 12:43 UTC (permalink / raw)
To: openembedded-core; +Cc: Ming Liu
From: Ming Liu <liu.ming50@gmail.com>
Update mmc-utils to the latest git hash to fix a compilation issue,
which was observed on kirkstone when the kernel 5.4 is being used.
Alexander Kanavin (1):
mmc-utils: upgrade to latest revision
meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
2.25.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
2022-05-15 12:43 [OE-core] [kirkstone] [PATCH 0/1] Fix a compilation issue of mmc-utils liu.ming50
@ 2022-05-15 12:43 ` liu.ming50
2022-05-15 21:42 ` richard.purdie
[not found] ` <16EF654890391C3F.10172@lists.openembedded.org>
0 siblings, 2 replies; 7+ messages in thread
From: liu.ming50 @ 2022-05-15 12:43 UTC (permalink / raw)
To: openembedded-core
Cc: Alexander Kanavin, Alexander Kanavin, Luca Ceresoli,
Richard Purdie, Ming Liu
From: Alexander Kanavin <alex.kanavin@gmail.com>
Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
Signed-off-by: Ming Liu <liu.ming50@gmail.com>
---
meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/mmc/mmc-utils_git.bb b/meta/recipes-devtools/mmc/mmc-utils_git.bb
index addefe9abf..3e611d1306 100644
--- a/meta/recipes-devtools/mmc/mmc-utils_git.bb
+++ b/meta/recipes-devtools/mmc/mmc-utils_git.bb
@@ -5,7 +5,7 @@ LICENSE = "GPL-2.0-only"
LIC_FILES_CHKSUM = "file://mmc.c;beginline=1;endline=20;md5=fae32792e20f4d27ade1c5a762d16b7d"
SRCBRANCH ?= "master"
-SRCREV = "4303889c8bd9a2357587eb6ebacecb70098a264d"
+SRCREV = "b7e4d5a6ae9942d26a11de9b05ae7d52c0802802"
PV = "0.1+git${SRCPV}"
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
2022-05-15 12:43 ` [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision liu.ming50
@ 2022-05-15 21:42 ` richard.purdie
2022-05-16 7:17 ` Ming Liu
[not found] ` <16EF654890391C3F.10172@lists.openembedded.org>
1 sibling, 1 reply; 7+ messages in thread
From: richard.purdie @ 2022-05-15 21:42 UTC (permalink / raw)
To: liu.ming50, openembedded-core
Cc: Alexander Kanavin, Alexander Kanavin, Luca Ceresoli
On Sun, 2022-05-15 at 14:43 +0200, liu.ming50@gmail.com wrote:
> From: Alexander Kanavin <alex.kanavin@gmail.com>
>
> Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> (cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
> Signed-off-by: Ming Liu <liu.ming50@gmail.com>
> ---
> meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Why does kirkstone need this though?
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
[not found] ` <16EF654890391C3F.10172@lists.openembedded.org>
@ 2022-05-15 21:46 ` richard.purdie
0 siblings, 0 replies; 7+ messages in thread
From: richard.purdie @ 2022-05-15 21:46 UTC (permalink / raw)
To: liu.ming50, openembedded-core
Cc: Alexander Kanavin, Alexander Kanavin, Luca Ceresoli
On Sun, 2022-05-15 at 22:42 +0100, Richard Purdie via
lists.openembedded.org wrote:
> On Sun, 2022-05-15 at 14:43 +0200, liu.ming50@gmail.com wrote:
> > From: Alexander Kanavin <alex.kanavin@gmail.com>
> >
> > Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> > Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > (cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
> > Signed-off-by: Ming Liu <liu.ming50@gmail.com>
> > ---
> > meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Why does kirkstone need this though?
Sorry, you cc'd me on the patch but not the cover letter which meant
this sorted badly in my mail folders but I noticed the cover letter
now.
You mention a compile failure in the cover letter, it would probably be
useful to give a little more info about the kind of compile failure
given we've not seen this anywhere else?
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
2022-05-15 21:42 ` richard.purdie
@ 2022-05-16 7:17 ` Ming Liu
2022-05-16 14:29 ` Steve Sakoman
0 siblings, 1 reply; 7+ messages in thread
From: Ming Liu @ 2022-05-16 7:17 UTC (permalink / raw)
To: Richard Purdie
Cc: OE-core, Alexander Kanavin, Alexander Kanavin, Luca Ceresoli
[-- Attachment #1: Type: text/plain, Size: 1382 bytes --]
Hi, Richard:
Our products are based on LTS kirkstone, we found a build issue of
mmc-utils, which has been fixed by
https://kernel.googlesource.com/pub/scm/utils/mmc/mmc-utils/+/0d493fb091b592bde6004a618c28f1c3619af1a4
this uplift only involves in two commits including the above fix:
```
b7e4d5a mmc-utils: Add General command CMD56 read support
0d493fb mmc-utils: Fix build error MMC_BLOCK_MAJOR undeclared
```
considering kirkstone mmc-utils recipe fetches mmc-utils master branch and
does not stick to any release tag, so I think we can uplift mmc-utils to
the latest rather than backport that fix to kirkstone.
//Ming Liu
<richard.purdie@linuxfoundation.org> 於 2022年5月15日 週日 下午11:42寫道:
> On Sun, 2022-05-15 at 14:43 +0200, liu.ming50@gmail.com wrote:
> > From: Alexander Kanavin <alex.kanavin@gmail.com>
> >
> > Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> > Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > (cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
> > Signed-off-by: Ming Liu <liu.ming50@gmail.com>
> > ---
> > meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Why does kirkstone need this though?
>
> Cheers,
>
> Richard
>
[-- Attachment #2: Type: text/html, Size: 2484 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
2022-05-16 7:17 ` Ming Liu
@ 2022-05-16 14:29 ` Steve Sakoman
2022-05-17 7:37 ` Ming Liu
0 siblings, 1 reply; 7+ messages in thread
From: Steve Sakoman @ 2022-05-16 14:29 UTC (permalink / raw)
To: Ming Liu
Cc: Richard Purdie, OE-core, Alexander Kanavin, Alexander Kanavin,
Luca Ceresoli
On Sun, May 15, 2022 at 9:17 PM Ming Liu <liu.ming50@gmail.com> wrote:
> Our products are based on LTS kirkstone, we found a build issue of mmc-utils, which has been fixed by https://kernel.googlesource.com/pub/scm/utils/mmc/mmc-utils/+/0d493fb091b592bde6004a618c28f1c3619af1a4
So this error only occurs if you are using a BSP with a 5.4 kernel?
That would explain why I don't see this in my testing, since it uses
only 5.10 and 5.15
BTW, for future kirkstone patches you should cc me instead of Richard
since I am the kirkstone maintainer.
> this uplift only involves in two commits including the above fix:
>
> ```
> b7e4d5a mmc-utils: Add General command CMD56 read support
> 0d493fb mmc-utils: Fix build error MMC_BLOCK_MAJOR undeclared
> ```
>
> considering kirkstone mmc-utils recipe fetches mmc-utils master branch and does not stick to any release tag, so I think we can uplift mmc-utils to the latest rather than backport that fix to kirkstone.
This seems reasonable to me. I will add this patch to my test queue
and edit the commit message to include the above information.
Steve
> <richard.purdie@linuxfoundation.org> 於 2022年5月15日 週日 下午11:42寫道:
>>
>> On Sun, 2022-05-15 at 14:43 +0200, liu.ming50@gmail.com wrote:
>> > From: Alexander Kanavin <alex.kanavin@gmail.com>
>> >
>> > Signed-off-by: Alexander Kanavin <alex@linutronix.de>
>> > Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
>> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> > (cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
>> > Signed-off-by: Ming Liu <liu.ming50@gmail.com>
>> > ---
>> > meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>>
>> Why does kirkstone need this though?
>>
>> Cheers,
>>
>> Richard
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#165665): https://lists.openembedded.org/g/openembedded-core/message/165665
> Mute This Topic: https://lists.openembedded.org/mt/91117508/3620601
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [steve@sakoman.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision
2022-05-16 14:29 ` Steve Sakoman
@ 2022-05-17 7:37 ` Ming Liu
0 siblings, 0 replies; 7+ messages in thread
From: Ming Liu @ 2022-05-17 7:37 UTC (permalink / raw)
To: Steve Sakoman
Cc: Richard Purdie, OE-core, Alexander Kanavin, Alexander Kanavin,
Luca Ceresoli
[-- Attachment #1: Type: text/plain, Size: 2626 bytes --]
Yes, this only occurs on 5.4 kernel, will CC to you next time for kirkstone
patches.
//Ming Liu
Steve Sakoman <steve@sakoman.com> 於 2022年5月16日 週一 下午4:29寫道:
> On Sun, May 15, 2022 at 9:17 PM Ming Liu <liu.ming50@gmail.com> wrote:
>
> > Our products are based on LTS kirkstone, we found a build issue of
> mmc-utils, which has been fixed by
> https://kernel.googlesource.com/pub/scm/utils/mmc/mmc-utils/+/0d493fb091b592bde6004a618c28f1c3619af1a4
>
> So this error only occurs if you are using a BSP with a 5.4 kernel?
> That would explain why I don't see this in my testing, since it uses
> only 5.10 and 5.15
>
> BTW, for future kirkstone patches you should cc me instead of Richard
> since I am the kirkstone maintainer.
>
> > this uplift only involves in two commits including the above fix:
> >
> > ```
> > b7e4d5a mmc-utils: Add General command CMD56 read support
> > 0d493fb mmc-utils: Fix build error MMC_BLOCK_MAJOR undeclared
> > ```
> >
> > considering kirkstone mmc-utils recipe fetches mmc-utils master branch
> and does not stick to any release tag, so I think we can uplift mmc-utils
> to the latest rather than backport that fix to kirkstone.
>
> This seems reasonable to me. I will add this patch to my test queue
> and edit the commit message to include the above information.
>
> Steve
>
> > <richard.purdie@linuxfoundation.org> 於 2022年5月15日 週日 下午11:42寫道:
> >>
> >> On Sun, 2022-05-15 at 14:43 +0200, liu.ming50@gmail.com wrote:
> >> > From: Alexander Kanavin <alex.kanavin@gmail.com>
> >> >
> >> > Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> >> > Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> >> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >> > (cherry picked from commit d98b06c9c6f480de1e5167bfe8392e39300fc02c)
> >> > Signed-off-by: Ming Liu <liu.ming50@gmail.com>
> >> > ---
> >> > meta/recipes-devtools/mmc/mmc-utils_git.bb | 2 +-
> >> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >> >
> >>
> >> Why does kirkstone need this though?
> >>
> >> Cheers,
> >>
> >> Richard
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#165665):
> https://lists.openembedded.org/g/openembedded-core/message/165665
> > Mute This Topic: https://lists.openembedded.org/mt/91117508/3620601
> > Group Owner: openembedded-core+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> steve@sakoman.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
>
[-- Attachment #2: Type: text/html, Size: 4459 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-05-17 7:38 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-15 12:43 [OE-core] [kirkstone] [PATCH 0/1] Fix a compilation issue of mmc-utils liu.ming50
2022-05-15 12:43 ` [OE-core] [kirkstone] [PATCH 1/1] mmc-utils: upgrade to latest revision liu.ming50
2022-05-15 21:42 ` richard.purdie
2022-05-16 7:17 ` Ming Liu
2022-05-16 14:29 ` Steve Sakoman
2022-05-17 7:37 ` Ming Liu
[not found] ` <16EF654890391C3F.10172@lists.openembedded.org>
2022-05-15 21:46 ` richard.purdie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.