diff mbox series

manuals: kernel-dev: Fix BSP in recipe space

Message ID 20240221072612.79402-1-uvv.mail@gmail.com
State New
Headers show
Series manuals: kernel-dev: Fix BSP in recipe space | expand

Commit Message

Vyacheslav Yurkov Feb. 21, 2024, 7:26 a.m. UTC
File fetcher doesn't have destsuffix parameter.

Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
---
 documentation/kernel-dev/advanced.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Opdenacker Feb. 21, 2024, 8:59 a.m. UTC | #1
Hi Slava

On 2/21/24 at 08:26, Vyacheslav Yurkov wrote:
> File fetcher doesn't have destsuffix parameter.
>
> Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
> ---
>   documentation/kernel-dev/advanced.rst | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/documentation/kernel-dev/advanced.rst b/documentation/kernel-dev/advanced.rst
> index bcaaaeed1..bddff1158 100644
> --- a/documentation/kernel-dev/advanced.rst
> +++ b/documentation/kernel-dev/advanced.rst
> @@ -718,7 +718,7 @@ If the BSP description is in recipe space, you cannot simply list the
>   form from your kernel append file::
>   
>      SRC_URI:append:myplatform = " \
> -       file://myplatform;type=kmeta;destsuffix=myplatform \
> +       file://myplatform;type=kmeta;subdir=myplatfrom \


Many thanks for this fix.
Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
Merged into "master-next".

Cheers
Michael.
Bruce Ashfield Feb. 23, 2024, 1:44 p.m. UTC | #2
On Wed, Feb 21, 2024 at 2:26 AM Vyacheslav Yurkov <uvv.mail@gmail.com> wrote:
>
> File fetcher doesn't have destsuffix parameter.
>

While the file fetcher doesn't have that option, it was the fact that
it didn't have it that kept things working.

If you use subdir, then in fact, a recipe space BSP doesn't work at
all as the tools can't find the BSP definition.

The kernel-yocto bbclass is what is using this to set up search paths
for kernel features when it sees the type=kmeta.

The option could have had a less confusing name (since it is
effectively passthrough like "type=kmeta", but that's not something
that I'm willing to change right now (I'll do it post release)

But the documents are correct, you do need destsuffix or the
recipe-space BSP will  not work.

Bruce




> Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
> ---
>  documentation/kernel-dev/advanced.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/documentation/kernel-dev/advanced.rst b/documentation/kernel-dev/advanced.rst
> index bcaaaeed1..bddff1158 100644
> --- a/documentation/kernel-dev/advanced.rst
> +++ b/documentation/kernel-dev/advanced.rst
> @@ -718,7 +718,7 @@ If the BSP description is in recipe space, you cannot simply list the
>  form from your kernel append file::
>
>     SRC_URI:append:myplatform = " \
> -       file://myplatform;type=kmeta;destsuffix=myplatform \
> +       file://myplatform;type=kmeta;subdir=myplatfrom \
>         "
>
>  Metadata Outside the Recipe-Space
> --
> 2.35.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#4852): https://lists.yoctoproject.org/g/docs/message/4852
> Mute This Topic: https://lists.yoctoproject.org/mt/104484337/1050810
> Group Owner: docs+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/docs/unsub [bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


--
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
Vyacheslav Yurkov Feb. 26, 2024, 9:30 a.m. UTC | #3
On 23.02.2024 14:44, Bruce Ashfield wrote:
> On Wed, Feb 21, 2024 at 2:26 AM Vyacheslav Yurkov <uvv.mail@gmail.com> wrote:
>> File fetcher doesn't have destsuffix parameter.
>>
> While the file fetcher doesn't have that option, it was the fact that
> it didn't have it that kept things working.
>
> If you use subdir, then in fact, a recipe space BSP doesn't work at
> all as the tools can't find the BSP definition.
>
> The kernel-yocto bbclass is what is using this to set up search paths
> for kernel features when it sees the type=kmeta.
>
> The option could have had a less confusing name (since it is
> effectively passthrough like "type=kmeta", but that's not something
> that I'm willing to change right now (I'll do it post release)
>
> But the documents are correct, you do need destsuffix or the
> recipe-space BSP will  not work.
>
> Bruce
>

Oh, interesting. Perhaps the recipe-space BSP works this way, but the 
recipe-space additions for other parts of 'kmeta' didn't work for me. 
Whenever I wanted to extend kernel features that yocto-kernel-cache 
didn't have, I had to use subdir.

Slava
diff mbox series

Patch

diff --git a/documentation/kernel-dev/advanced.rst b/documentation/kernel-dev/advanced.rst
index bcaaaeed1..bddff1158 100644
--- a/documentation/kernel-dev/advanced.rst
+++ b/documentation/kernel-dev/advanced.rst
@@ -718,7 +718,7 @@  If the BSP description is in recipe space, you cannot simply list the
 form from your kernel append file::
 
    SRC_URI:append:myplatform = " \
-       file://myplatform;type=kmeta;destsuffix=myplatform \
+       file://myplatform;type=kmeta;subdir=myplatfrom \
        "
 
 Metadata Outside the Recipe-Space