Re: [PATCH 1/4] wl18xx-fw: add recipe for version R8.7_SP3

[Denys Dmytriyenko <denys@ti.com>]

Eyal,

Usually, git history is enough. Multiple recipe versions are only needed when 
they can be used at the same time or switch between them in the same release.

We have some extra time to get this fixed until tomorrow noon before the next 
tag, so feel free to re-submit v2 by then. If not, I'll get to fix it myself 
tomorrow. Thanks.

-- 
Denys


On Mon, Dec 04, 2017 at 02:54:44PM -0500, Reizer, Eyal wrote:
>    Hi Denys,
> 
>    Yes, of course. Thought you were keeping the history.
>    Can you take care of it or do you need another patch set?
> 
>    Best Regards,
>    Eyal
> 
>    -------- Original Message --------
>    From: "Dmytriyenko, Denys" <denys@ti.com>
>    Date: Mon, Dec 4, 2017, 21:52
>    To: "Reizer, Eyal" <eyalr@ti.com>
>    CC: meta-ti@yoctoproject.org
>    Subject: Re: [PATCH 1/4] wl18xx-fw: add recipe for version R8.7_SP3
> 
>    On Sun, Dec 03, 2017 at 05:34:49AM -0500, Reizer, Eyal wrote:
>    > Signed-off-by: Eyal Reizer <eyalr@ti.com>
>    > ---
>    >  recipes-bsp/wl18xx-fw/wl18xx-fw_8.7.3.bb | 26
>    ++++++++++++++++++++++++++
>    >  1 file changed, 26 insertions(+)
>    >  create mode 100644 recipes-bsp/wl18xx-fw/wl18xx-fw_8.7.3.bb
> 
>    Thanks! Can we just update from 8.7.1 to 8.7.3 by replacing the recipe,
>    instead of adding new one?
> 
>    > diff --git a/recipes-bsp/wl18xx-fw/wl18xx-fw_8.7.3.bb
>    b/recipes-bsp/wl18xx-fw/wl18xx-fw_8.7.3.bb
>    > new file mode 100644
>    > index 0000000..a52ec89
>    > --- /dev/null
>    > +++ b/recipes-bsp/wl18xx-fw/wl18xx-fw_8.7.3.bb
>    > @@ -0,0 +1,26 @@
>    > +DESCRIPTION = "Firmware files for use with TI wl18xx"
>    > +LICENSE = "TI-TSPA"
>    > +LIC_FILES_CHKSUM =
>    "file://LICENCE;md5=4977a0fe767ee17765ae63c435a32a9e"
>    > +
>    > +SRC_URI = " \
>    > +    
>    git://git.ti.com/wilink8-wlan/wl18xx_fw.git;protocol=git;branch=${BRANCH}
>    \
>    > +     file://0001-Add-Makefile-to-install-firmware-files.patch \
>    > +"
>    > +
>    > +# Tag: R8.7-SP3 (8.7.3)
>    > +SRCREV = "f659be25473e4bde8dc790bff703ecacde6e21da"
>    > +BRANCH = "master"
>    > +
>    > +S = "${WORKDIR}/git"
>    > +
>    > +CLEANBROKEN = "1"
>    > +
>    > +do_compile() {
>    > +    :
>    > +}
>    > +
>    > +do_install() {
>    > +    oe_runmake 'DEST_DIR=${D}' install
>    > +}
>    > +
>    > +FILES_${PN} = "/lib/firmware/ti-connectivity/*"
>    > --
>    > 2.7.4
>    >