On 5/25/22 14:41, Martin Jansa wrote:

buildroot removed it in 2019 and recommends to use busybox version instead:
https://github.com/buildroot/buildroot/commit/2b2579afebfc7a9b8a458af1f2d206101fbfa19c <https://github.com/buildroot/buildroot/commit/2b2579afebfc7a9b8a458af1f2d206101fbfa19c>
"Use the the Busybox devmem utility, instead, which provides the same functionality."

I want to say there is an issue with the busybox devmem. As I recall (and this is an ancient memory), if you use busybox devmem to write, it does a readback. This is a bad thing if the address you write to is a hardware device that uses the read to reset state.

Philip

On Wed, May 25, 2022 at 8:32 PM Khem Raj <raj.khem@... <mailto:raj.khem@...>> wrote:
On Wed, May 25, 2022 at 11:08 AM Denys Dmytriyenko <denis@...
<mailto:denis@...>> wrote:
>
> Sure, I can do that and put it in the repo, since I've made few
patches on top
> of it... Before I do that, are there any objections? Bootlin?
I would suggest to cross post with buildroot and see if there is
interest in using it from
a separate repository.

>
>
> On Wed, May 25, 2022 at 08:03:58PM +0200, Martin Jansa wrote:
> > Anyone willing to create a repo for this on github instead of
patching this
> > in metadata?
> >
> > This is another reproducer for do_patch issue described here
> > https://lists.yoctoproject.org/g/yocto/message/56602
<https://lists.yoctoproject.org/g/yocto/message/56602>
> >
> > On Fri, May 13, 2022 at 1:43 AM Denys Dmytriyenko
<denis@... <mailto:denis@...>> wrote:
> >
> > > From: Michael Opdenacker <michael.opdenacker@...
<mailto:michael.opdenacker@...>>
> > >
> > > http://www.free-electrons.com <http://www.free-electrons.com>
now redirects to https://bootlin.com <https://bootlin.com>
> > >
> > > Signed-off-by: Michael Opdenacker
<michael.opdenacker@... <mailto:michael.opdenacker@...>>
> > > Signed-off-by: Khem Raj <raj.khem@...
<mailto:raj.khem@...>>
> > > Signed-off-by: Denys Dmytriyenko <denis@...
<mailto:denis@...>>
> > > ---
> > >  meta-oe/recipes-support/devmem2/devmem2.bb
<http://devmem2.bb> | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta-oe/recipes-support/devmem2/devmem2.bb
<http://devmem2.bb>
> > > b/meta-oe/recipes-support/devmem2/devmem2.bb <http://devmem2.bb>
> > > index ba7aad8ab..70d413521 100644
> > > --- a/meta-oe/recipes-support/devmem2/devmem2.bb
<http://devmem2.bb>
> > > +++ b/meta-oe/recipes-support/devmem2/devmem2.bb
<http://devmem2.bb>
> > > @@ -3,7 +3,7 @@ LICENSE = "GPL-2.0-or-later"
> > >  LIC_FILES_CHKSUM =
> > >
"file://devmem2.c;endline=38;md5=a9eb9f3890384519f435aedf986297cf"
> > >  PR = "r7"
> > >
> > > -SRC_URI = "
> > >
http://www.free-electrons.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c
<http://www.free-electrons.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c>
> > > \
> > > +SRC_URI = "
> > >
https://bootlin.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c
<https://bootlin.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c>
\
> > >
 file://devmem2-fixups-2.patch;apply=yes;striplevel=0 \
> > >
> > >
file://0001-devmem.c-ensure-word-is-32-bit-and-add-support-for-6.patch \
> > >
> > >
file://0001-devmem2-support-different-page-sizes-at-run-time.patch \
> > > --
> > > 2.25.1
>
>
>

On Wed, May 25, 2022 at 11:08 AM Denys Dmytriyenko <denis@...> wrote:

Sure, I can do that and put it in the repo, since I've made few patches on top
of it... Before I do that, are there any objections? Bootlin?

I would suggest to cross post with buildroot and see if there is
interest in using it from
a separate repository.

On Wed, May 25, 2022 at 08:03:58PM +0200, Martin Jansa wrote:

Anyone willing to create a repo for this on github instead of patching this
in metadata?

This is another reproducer for do_patch issue described here
https://lists.yoctoproject.org/g/yocto/message/56602

On Fri, May 13, 2022 at 1:43 AM Denys Dmytriyenko <denis@...> wrote:

From: Michael Opdenacker <michael.opdenacker@...>

http://www.free-electrons.com now redirects to https://bootlin.com

Signed-off-by: Michael Opdenacker <michael.opdenacker@...>
Signed-off-by: Khem Raj <raj.khem@...>
Signed-off-by: Denys Dmytriyenko <denis@...>
---
meta-oe/recipes-support/devmem2/devmem2.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-support/devmem2/devmem2.bb
b/meta-oe/recipes-support/devmem2/devmem2.bb
index ba7aad8ab..70d413521 100644
--- a/meta-oe/recipes-support/devmem2/devmem2.bb
+++ b/meta-oe/recipes-support/devmem2/devmem2.bb
@@ -3,7 +3,7 @@ LICENSE = "GPL-2.0-or-later"
LIC_FILES_CHKSUM =
"file://devmem2.c;endline=38;md5=a9eb9f3890384519f435aedf986297cf"
PR = "r7"

-SRC_URI = "
http://www.free-electrons.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c
\
+SRC_URI = "
https://bootlin.com/pub/mirror/devmem2.c;downloadfilename=devmem2-new.c \
file://devmem2-fixups-2.patch;apply=yes;striplevel=0 \

file://0001-devmem.c-ensure-word-is-32-bit-and-add-support-for-6.patch \

file://0001-devmem2-support-different-page-sizes-at-run-time.patch \
--
2.25.1

Sure, I can do that and put it in the repo, since I've made few patches on top
of it... Before I do that, are there any objections? Bootlin?

From: Michael Opdenacker <michael.opdenacker@...>

http://www.free-electrons.com now redirects to https://bootlin.com

Signed-off-by: Michael Opdenacker <michael.opdenacker@...>
Signed-off-by: Khem Raj <raj.khem@...>
Signed-off-by: Denys Dmytriyenko <denis@...>
---
meta-oe/recipes-support/devmem2/devmem2.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-support/devmem2/devmem2.bb b/meta-oe/recipes=
-support/devmem2/devmem2.bb
index ba7aad8ab..70d413521 100644
--- a/meta-oe/recipes-support/devmem2/devmem2.bb
+++ b/meta-oe/recipes-support/devmem2/devmem2.bb
@@ -3,7 +3,7 @@ LICENSE =3D "GPL-2.0-or-later"
LIC_FILES_CHKSUM =3D "file://devmem2.c;endline=3D38;md5=3Da9eb9f38903845=
19f435aedf986297cf"
PR =3D "r7"
=20
-SRC_URI =3D "http://www.free-electrons.com/pub/mirror/devmem2.c;download=
filename=3Ddevmem2-new.c \
+SRC_URI =3D "https://bootlin.com/pub/mirror/devmem2.c;downloadfilename=3D=
devmem2-new.c \
file://devmem2-fixups-2.patch;apply=3Dyes;striplevel=3D0 \
file://0001-devmem.c-ensure-word-is-32-bit-and-add-support-fo=
r-6.patch \
file://0001-devmem2-support-different-page-sizes-at-run-time.=
patch \
--=20
2.25.1