mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Christian Melki <christian.melki@t2data.com>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] kernel: fix and improve DTS help texts
Date: Wed, 8 Sep 2021 16:16:07 +0200	[thread overview]
Message-ID: <0dc89141-2197-7b2e-726c-2a045647595e@t2data.com> (raw)
In-Reply-To: <20210908135851.3694997-1-m.olbrich@pengutronix.de>

Hmm.

You're right. I probably misread the functionality as I tried to 
implement "make dtbs". Which isn't going to align with the current 
functionality IIUC?

I'm fine with the increased verbosity. My main goal was "make dtbs" for 
vendor kernels that are a bit obnoxious.

Regards,
Christian

On 9/8/21 3:58 PM, Michael Olbrich wrote:
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> ---
> 
> The default is correct. The paths are separated by colon and _one_ of the
> paths can be relative. The help text could be better. Maybe something like
> this?
> 
> Michael
> 
>   platforms/kernel.in | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/platforms/kernel.in b/platforms/kernel.in
> index ff3cc8df4f47..06b6e24a6756 100644
> --- a/platforms/kernel.in
> +++ b/platforms/kernel.in
> @@ -185,14 +185,16 @@ config KERNEL_DTS_PATH
>   	default "${PTXDIST_PLATFORMCONFIG_SUBDIR}/dts:${KERNEL_DIR}/arch/${GENERIC_KERNEL_ARCH}/boot/dts"
>   	help
>   	  Define path to the dts source file. Multiple directories can be
> -	  specified separated by ':'.
> +	  specified separated by ':'. A relative path will be expanded relative
> +	  to the workspace and all other layers. Only on of the specified paths
> +	  can be a relative path.
>   
>   config KERNEL_DTS
>   	string "source dts file"
>   	default "<yourboard>.dts"
>   	help
>   	  Select the dts file to use for the device tree binary
> -	  blob generation. For relative file names DTC_OFTREE_DTS_PATH
> +	  blob generation. For relative file names KERNEL_DTS_PATH
>   	  is used as a search path for the device tree files specified
>   	  here. Multiple dts files can be specified, separated by spaces.
>   
> 

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de


      reply	other threads:[~2021-09-08 14:16 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-07 15:58 [ptxdist] Invalid KERNEL_DTS_PATH description and default? Christian Melki
2021-09-08 13:58 ` [ptxdist] [PATCH] kernel: fix and improve DTS help texts Michael Olbrich
2021-09-08 14:16   ` Christian Melki [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0dc89141-2197-7b2e-726c-2a045647595e@t2data.com \
    --to=christian.melki@t2data.com \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox