All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
@ 2022-05-01 18:47 Khem Raj
  2022-05-03 13:06 ` Jon Mason
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2022-05-01 18:47 UTC (permalink / raw)
  To: meta-arm; +Cc: Khem Raj

The module can not be compiled without patching kernel 5.10
This helps meta-ti to live in multi-BSP envs

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
index 4051c34..a4dd38c 100644
--- a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
+++ b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
@@ -27,6 +27,7 @@ do_install:append() {
 }
 
 COMPATIBLE_HOST = "(arm|aarch64).*-linux"
+COMPATIBLE_HOST:ti-soc = "null"
 
 # Kernel modules currently RDEPEND on the kernel, which is troublesome when you want to put a
 # kernel module into a initramfs without pulling the kernel into the initramfs, which would be
-- 
2.36.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
  2022-05-01 18:47 [PATCH] ffa-debugfs-mod: Do not build for TI SOCs Khem Raj
@ 2022-05-03 13:06 ` Jon Mason
  2022-05-03 14:48   ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Jon Mason @ 2022-05-03 13:06 UTC (permalink / raw)
  To: Khem Raj; +Cc: meta-arm

On Sun, May 01, 2022 at 11:47:31AM -0700, Khem Raj wrote:
> The module can not be compiled without patching kernel 5.10
> This helps meta-ti to live in multi-BSP envs
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> index 4051c34..a4dd38c 100644
> --- a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> +++ b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> @@ -27,6 +27,7 @@ do_install:append() {
>  }
>  
>  COMPATIBLE_HOST = "(arm|aarch64).*-linux"
> +COMPATIBLE_HOST:ti-soc = "null"

Shouldn't this live in meta-ti?

>  
>  # Kernel modules currently RDEPEND on the kernel, which is troublesome when you want to put a
>  # kernel module into a initramfs without pulling the kernel into the initramfs, which would be
> -- 
> 2.36.0
> 
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
  2022-05-03 13:06 ` Jon Mason
@ 2022-05-03 14:48   ` Khem Raj
  2022-05-03 15:21     ` [meta-arm] " Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2022-05-03 14:48 UTC (permalink / raw)
  To: Jon Mason; +Cc: meta-arm

On Tue, May 3, 2022 at 6:06 AM Jon Mason <jdmason@kudzu.us> wrote:
>
> On Sun, May 01, 2022 at 11:47:31AM -0700, Khem Raj wrote:
> > The module can not be compiled without patching kernel 5.10
> > This helps meta-ti to live in multi-BSP envs
> >
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > ---
> >  meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > index 4051c34..a4dd38c 100644
> > --- a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > +++ b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > @@ -27,6 +27,7 @@ do_install:append() {
> >  }
> >
> >  COMPATIBLE_HOST = "(arm|aarch64).*-linux"
> > +COMPATIBLE_HOST:ti-soc = "null"
>
> Shouldn't this live in meta-ti?

yeah perhaps better there.

>
> >
> >  # Kernel modules currently RDEPEND on the kernel, which is troublesome when you want to put a
> >  # kernel module into a initramfs without pulling the kernel into the initramfs, which would be
> > --
> > 2.36.0
> >
> >


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arm] [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
  2022-05-03 14:48   ` Khem Raj
@ 2022-05-03 15:21     ` Denys Dmytriyenko
  2022-05-03 15:54       ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2022-05-03 15:21 UTC (permalink / raw)
  To: Khem Raj; +Cc: Jon Mason, meta-arm

On Tue, May 03, 2022 at 07:48:20AM -0700, Khem Raj wrote:
> On Tue, May 3, 2022 at 6:06 AM Jon Mason <jdmason@kudzu.us> wrote:
> >
> > On Sun, May 01, 2022 at 11:47:31AM -0700, Khem Raj wrote:
> > > The module can not be compiled without patching kernel 5.10
> > > This helps meta-ti to live in multi-BSP envs
> > >
> > > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > > ---
> > >  meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > index 4051c34..a4dd38c 100644
> > > --- a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > +++ b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > @@ -27,6 +27,7 @@ do_install:append() {
> > >  }
> > >
> > >  COMPATIBLE_HOST = "(arm|aarch64).*-linux"
> > > +COMPATIBLE_HOST:ti-soc = "null"
> >
> > Shouldn't this live in meta-ti?
> 
> yeah perhaps better there.

If the kernel requires patching to use this module from meta-arm, shouldn't 
meta-arm set a bit stricter COMPATIBLE_HOST? How are other BSPs supposed to 
work along with meta-arm?


> > >  # Kernel modules currently RDEPEND on the kernel, which is troublesome when you want to put a
> > >  # kernel module into a initramfs without pulling the kernel into the initramfs, which would be
> > > --
> > > 2.36.0
> > >
> > >

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arm] [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
  2022-05-03 15:21     ` [meta-arm] " Denys Dmytriyenko
@ 2022-05-03 15:54       ` Khem Raj
  0 siblings, 0 replies; 6+ messages in thread
From: Khem Raj @ 2022-05-03 15:54 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: Jon Mason, meta-arm

On Tue, May 3, 2022 at 8:21 AM Denys Dmytriyenko <denis@denix.org> wrote:
>
> On Tue, May 03, 2022 at 07:48:20AM -0700, Khem Raj wrote:
> > On Tue, May 3, 2022 at 6:06 AM Jon Mason <jdmason@kudzu.us> wrote:
> > >
> > > On Sun, May 01, 2022 at 11:47:31AM -0700, Khem Raj wrote:
> > > > The module can not be compiled without patching kernel 5.10
> > > > This helps meta-ti to live in multi-BSP envs
> > > >
> > > > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > > > ---
> > > >  meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > > index 4051c34..a4dd38c 100644
> > > > --- a/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > > +++ b/meta-arm/recipes-kernel/ffa-debugfs/ffa-debugfs-mod_2.1.0.bb
> > > > @@ -27,6 +27,7 @@ do_install:append() {
> > > >  }
> > > >
> > > >  COMPATIBLE_HOST = "(arm|aarch64).*-linux"
> > > > +COMPATIBLE_HOST:ti-soc = "null"
> > >
> > > Shouldn't this live in meta-ti?
> >
> > yeah perhaps better there.
>
> If the kernel requires patching to use this module from meta-arm, shouldn't
> meta-arm set a bit stricter COMPATIBLE_HOST? How are other BSPs supposed to
> work along with meta-arm?

I think meta-arm is a base dependency of meta-ti not other way around,
so in this case I think its better to fix
in meta-ti, since meta-ti is not using the kernel that meta-arm tests
so this layer has no control over things downstream
layers do.

>
>
> > > >  # Kernel modules currently RDEPEND on the kernel, which is troublesome when you want to put a
> > > >  # kernel module into a initramfs without pulling the kernel into the initramfs, which would be
> > > > --
> > > > 2.36.0
> > > >
> > > >
>
> --
> Regards,
> Denys Dmytriyenko <denis@denix.org>
> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
> Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
@ 2022-05-03 15:09 Khem Raj
  0 siblings, 0 replies; 6+ messages in thread
From: Khem Raj @ 2022-05-03 15:09 UTC (permalink / raw)
  To: meta-ti; +Cc: Khem Raj

The module can not be compiled without patching kernel 5.10
This helps meta-ti to live in multi-BSP envs

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend           | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend

diff --git a/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend b/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend
new file mode 100644
index 00000000..78b509c1
--- /dev/null
+++ b/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend
@@ -0,0 +1,7 @@
+FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
+
+# Added my meta-arm, this needs ceetain backports in 5.10
+# Disable until kernel upgrades to newer than 5.10 where it
+# is already upstreamed
+# Revisit when kernel is upgraded to 5.15 or newer e.g.
+COMPATIBLE_HOST:ti-soc = "null"
-- 
2.36.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-05-03 15:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-01 18:47 [PATCH] ffa-debugfs-mod: Do not build for TI SOCs Khem Raj
2022-05-03 13:06 ` Jon Mason
2022-05-03 14:48   ` Khem Raj
2022-05-03 15:21     ` [meta-arm] " Denys Dmytriyenko
2022-05-03 15:54       ` Khem Raj
2022-05-03 15:09 Khem Raj

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.