All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: a-bhatia1@ti.com
Cc: Ryan Eatmon <reatmon@ti.com>,
	Denys Dmytriyenko <denys@konsulko.com>,
	Meta-TI List <meta-ti@lists.yoctoproject.org>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Paresh Bhagat <p-bhagat@ti.com>,
	Sai Sree Kartheek Adivi <s-adivi@ti.com>,
	Gyan Gupta <g-gupta@ti.com>
Subject: Re: [meta-ti] [kirkstone-next][PATCH] meta-ti: conf: am62xx-lp-evm: Fix DT and DT prefixes
Date: Fri, 5 May 2023 21:40:21 -0400	[thread overview]
Message-ID: <20230506014021.GI9226@denix.org> (raw)
In-Reply-To: <20230505083824.27085-1-a-bhatia1@ti.com>

Couple minor comments regarding the subject tags:

1. You want to specify the target release branch, such as "kirkstone", not the 
intermediate branches, like "*-wip" or "*-next".

2. We want to keep master up-to-date and in most cases the change is quite 
straightforward and applies to both master and kirkstone, so it's best to 
specify that in the tag as "master/kirkstone". And testing against those 
branches is highly recommended! :)


On Fri, May 05, 2023 at 02:08:24PM +0530, Aradhya Bhatia via lists.yoctoproject.org wrote:
> The DT name k3-am62-lp-sk.dts was adopted instead of k3-am62x-lp-sk.dts
> because the community prefers avoiding the wildcard letter 'x'.[1]
> 
> Drop the letter 'x' from the strings in KERNEL_DEVICETREE_PREFIX and
> KERNEL_DEVICETREE to accurately reflect the DT names.
> 
> Signed-off-by: Aradhya Bhatia <a-bhatia1@ti.com>
> 
> [1]:
> https://lore.kernel.org/all/06cbcd7d-bc83-bfeb-0821-72c7caf9a5e7@linaro.org/
> ---
>  meta-ti-bsp/conf/machine/am62xx-lp-evm.conf | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> index ba1a12a7fdb7..ce306a2a0e17 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> @@ -5,12 +5,12 @@
>  require conf/machine/include/am62xx-lp.inc
>  
>  KERNEL_DEVICETREE_PREFIX = " \
> -    ti/k3-am62x-lp \
> +    ti/k3-am62-lp \
>      ti/k3-am625-sk \
>  "
>  
>  KERNEL_DEVICETREE = " \
> -    ti/k3-am62x-lp-sk.dtb \
> +    ti/k3-am62-lp-sk.dtb \
>  "
>  
>  SPL_BINARY = "tispl.bin_HS"
> -- 
> 2.40.1


  reply	other threads:[~2023-05-06  1:41 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-05  8:38 [kirkstone-next][PATCH] meta-ti: conf: am62xx-lp-evm: Fix DT and DT prefixes Aradhya Bhatia
2023-05-06  1:40 ` Denys Dmytriyenko [this message]
2023-05-11  4:32   ` [meta-ti] " Aradhya Bhatia
2023-05-12  3:00     ` Denys Dmytriyenko
2023-05-15  9:07       ` Aradhya Bhatia
2023-05-15 13:21         ` Ryan Eatmon

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=20230506014021.GI9226@denix.org \
    --to=denis@denix.org \
    --cc=a-bhatia1@ti.com \
    --cc=denys@konsulko.com \
    --cc=g-gupta@ti.com \
    --cc=meta-ti@lists.yoctoproject.org \
    --cc=p-bhagat@ti.com \
    --cc=reatmon@ti.com \
    --cc=s-adivi@ti.com \
    --cc=vigneshr@ti.com \
    /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 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.