From: Jerry Huang <jerry.huang@nxp.com>
To: buildroot@busybox.net
Subject: [Buildroot] [EXT] Re: [PATCH v2 07/10] package/freescale-qoriq/fsl-qoriq-cadence-dp-fw: new package
Date: Mon, 25 Nov 2019 02:40:10 +0000 [thread overview]
Message-ID: <VI1PR04MB5136E63116B6AFD4EBD9344FFE4A0@VI1PR04MB5136.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <be3bd51212032b8fad3e3741433a7b4f@walle.cc>
Best Regards
Jerry Huang
> -----Original Message-----
> From: Michael Walle <michael@walle.cc>
> Sent: Friday, November 22, 2019 7:00 PM
> To: Jerry Huang <jerry.huang@nxp.com>
> Cc: buildroot at busybox.net; geomatsi at gmail.com;
> matthew.weber at collins.com; thomas.petazzoni at bootlin.com
> Subject: Re: [EXT] Re: [PATCH v2 07/10]
> package/freescale-qoriq/fsl-qoriq-cadence-dp-fw: new package
>
> Caution: EXT Email
>
> Hi,
> >> > +define FSL_QORIQ_CADENCE_DP_FW_EXTRACT_CMDS
> >> > + cd $(@D); \
> >> > + sh
> >> >
> >>
> $(FSL_QORIQ_CADENCE_DP_FW_DL_DIR)/$(FSL_QORIQ_CADENCE_DP_FW_SO
> >> URCE)
> >> > --auto-accept;
> >> > +endef
> >>
> >> There is already an FREESCALE_IMX_EXTRACT_HELPER macro. I guess that
> >> should be renamed and used here.
> >
> > I changed this codes accordance with freescale-imx, create new option
> > FREESCALE_QORIQ_EXTRACT_HELPER
>
> why FREESCALE_QORIQ_EXTRACT_HELPER? are there two different ones now?
> Shouldn't it be just FREESCALE_EXTRACT_HELPER? and the definition of the
> macro should be moved out of the freescale-imx.mk?
>
> or of course, there might be another subdirectory structure like:
> packages/freescale/imx
> packages/freescale/qoriq
> pacakges/freescale/helper.mk
I think it makes sense to create another subdirectory like this.
> Or..
>
> packages/freescale/fsl-imx-<package>
> packages/freescale/fsl-qoriq-<package>
> pacakges/freescale/helper.mk
>
> And the macro might end up in the pacakges/freescale/helper.mk file.
>
> TBH. I guess there have to be a cleanup before adding the qoriq stuff.
> Thomas? Matthew?
>
> -michael
next prev parent reply other threads:[~2019-11-25 2:40 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-21 10:23 [Buildroot] [PATCH v2 0/9] new board ls1028ardb introduced Changming Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 01/10] package/freescale-qoriq: new package directory Changming Huang
2019-11-25 21:13 ` Thomas Petazzoni
2019-11-26 2:49 ` [Buildroot] [EXT] " Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 02/10] package/freescale-qoriq/fsl-qoriq-rcw: move rcw into freescale-qoriq and rename it Changming Huang
2019-11-21 10:34 ` Michael Walle
2019-11-21 10:42 ` [Buildroot] [EXT] " Jerry Huang
2019-11-25 21:15 ` [Buildroot] " Thomas Petazzoni
2019-11-26 3:25 ` [Buildroot] [EXT] " Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 03/10] package/freescale-qoriq/fsl-qoriq-rcw: upgrade the rcw version to LSDK-19.09 Changming Huang
2019-11-25 21:17 ` Thomas Petazzoni
2019-11-26 3:51 ` [Buildroot] [EXT] " Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 04/10] package/freescale-qoriq/fsl-qoriq-rcw: add target rcw binary support Changming Huang
2019-11-25 21:23 ` Thomas Petazzoni
2019-11-26 4:33 ` [Buildroot] [EXT] " Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 05/10] boot/arm-trusted-firmware: Add option BR2_TARGET_ARM_TRUSTED_FIRMWARE_UBOOT_BL33_IMAGE Changming Huang
2019-11-22 9:11 ` Sergey Matyukevich
2019-11-22 9:24 ` [Buildroot] [EXT] " Jerry Huang
2019-11-25 21:59 ` [Buildroot] " Thomas Petazzoni
2019-11-21 10:23 ` [Buildroot] [PATCH v2 06/10] boot/arm-trusted-firmware: Add RCW support Changming Huang
2019-11-22 9:19 ` Sergey Matyukevich
2019-11-22 9:31 ` [Buildroot] [EXT] " Jerry Huang
2019-11-25 22:09 ` [Buildroot] " Thomas Petazzoni
2019-11-26 7:06 ` [Buildroot] [EXT] " Jerry Huang
2019-11-26 7:34 ` Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 07/10] package/freescale-qoriq/fsl-qoriq-cadence-dp-fw: new package Changming Huang
2019-11-21 10:42 ` Michael Walle
2019-11-21 11:04 ` [Buildroot] [EXT] " Jerry Huang
2019-11-22 10:41 ` Jerry Huang
2019-11-21 10:51 ` [Buildroot] " Michael Walle
2019-11-21 11:09 ` [Buildroot] [EXT] " Jerry Huang
2019-11-22 10:40 ` Jerry Huang
2019-11-22 11:00 ` Michael Walle
2019-11-25 2:40 ` Jerry Huang [this message]
2019-11-22 11:05 ` Thomas Petazzoni
2019-11-25 6:25 ` Jerry Huang
2019-11-25 22:12 ` [Buildroot] " Thomas Petazzoni
2019-11-26 8:09 ` [Buildroot] [EXT] " Jerry Huang
2019-11-26 9:21 ` Michael Walle
2019-11-26 10:13 ` Jerry Huang
2019-11-26 9:29 ` Michael Walle
2019-11-26 10:12 ` Thomas Petazzoni
2019-11-26 10:31 ` Michael Walle
2019-11-26 12:10 ` Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 08/10] package/freescale-qoriq/fsl-qoriq-rcw:Enable IIC5_PMUX and CLK_OUT_PMUX for GPIO Changming Huang
2019-11-25 22:13 ` Thomas Petazzoni
2019-11-26 8:15 ` [Buildroot] [EXT] " Jerry Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 09/10] board/freescale/common/ls: Add standard post image script for Layerscape processors Changming Huang
2019-11-21 10:23 ` [Buildroot] [PATCH v2 10/10] configs/freescale_ls1028ardb*: new board Changming Huang
2019-11-25 22:17 ` Thomas Petazzoni
2019-11-26 12:02 ` [Buildroot] [EXT] " Jerry Huang
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=VI1PR04MB5136E63116B6AFD4EBD9344FFE4A0@VI1PR04MB5136.eurprd04.prod.outlook.com \
--to=jerry.huang@nxp.com \
--cc=buildroot@busybox.net \
/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.