All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 0/3] murata-wireless firmware packages
Date: Tue, 7 May 2019 17:41:17 +0200	[thread overview]
Message-ID: <20190507154117.GG6887@scaer> (raw)
In-Reply-To: <20190507083218.24283-1-m.niestroj@grinn-global.com>

Marcin, All,

On 2019-05-07 10:32 +0200, Marcin Niestroj spake thusly:
> This set adds 3 firmware packages for muRata wireless modules (WiFi + BLE).
> I've put all three packages into single murata-wireless directory and Kconfig
> submenu, as they relate to the same hardware and there are plenty of options
> to choose from. Not sure whether this is the correct approach, so looking
> forward to get feedback on this.

I wonder why we would actually need three packages.

As far as I can see from a cursory look, they are all very tightly
coupled; they even have the same choice about the actual variant to use.

So, I would like we look at making that a single package, which
downloads multiple tarballs, something like:

    MURATA_WIRELESS_CYW_FW_VERSION = 8d87950bfad28c65926695b7357bd8995b60016a
    MURATA_WIRELESS_CYW_FW_VERSION_NVRAM = d27f1bf105fa1e5b828e355793b88d4b66188411
    MURATA_WIRELESS_CYW_FW_VERSION_PATCH = 748462f0b02ec4aeb500bedd60780ac51c37be31

    MURATA_WIRELESS_CYW_FW_SITE = $(call github,murata-wireless,cyw-fmac-fw,$(MURATA_WIRELESS_CYW_FW_VERSION)
    MURATA_WIRELESS_CYW_FW_EXTRA_DOWNLOADS = \
        $(call github,murata-wireless,cyw-fmac-nvram,$(MURATA_WIRELESS_CYW_FW_VERSION_NVRAM)/cyw-bt-nvram-$(MURATA_WIRELESS_CYW_FW_VERSION_NVRAM).tar.gz \
        $(call github,murata-wireless,cyw-fmac-patch,$(MURATA_WIRELESS_CYW_FW_VERSION_PATCH)/cyw-bt-patch-$(MURATA_WIRELESS_CYW_FW_VERSION_PATCH).tar.gz

    define MURATA_WIRELESS_CYW_FW_EXTRACT_NVRAM_PATCH
        $(foreach tar, $(notdir $(MURATA_WIRELESS_CYW_FW_EXTRA_DOWNLOADS)), \
            $(call suitable-extractor,$(tar)) $(MURATA_WIRELESS_CYW_FW_DL_DIR)/$(tar) | \
            $(TAR) --strip-components=1 -C $(@D) $(TAR_OPTIONS) -$(sep) \
        )
    endef
    MURATA_WIRELESS_CYW_FW_POST_EXTRACT_HOOKS += MURATA_WIRELESS_CYW_FW_EXTRACT_NVRAM_PATCH

And a similar hook for the installation step (which I leave up to you as
an exercise ;-) ).

Regards,
Yann E. MORIN.

> Marcin Niestroj (3):
>   package/murata-wireless-cyw-fmac-fw: new package
>   package/murata-wireless-cyw-fmac-nvram: new package
>   package/murata-wireless-cyw-bt-patch: new package
> 
>  DEVELOPERS                                    |  1 +
>  package/Config.in                             |  1 +
>  package/murata-wireless/Config.in             |  7 ++
>  .../murata-wireless-cyw-bt-patch/Config.in    | 47 ++++++++++
>  .../murata-wireless-cyw-bt-patch.hash         |  3 +
>  .../murata-wireless-cyw-bt-patch.mk           | 50 ++++++++++
>  .../murata-wireless-cyw-fmac-fw/Config.in     | 91 +++++++++++++++++++
>  .../murata-wireless-cyw-fmac-fw.hash          |  3 +
>  .../murata-wireless-cyw-fmac-fw.mk            | 80 ++++++++++++++++
>  .../murata-wireless-cyw-fmac-nvram/Config.in  | 91 +++++++++++++++++++
>  .../murata-wireless-cyw-fmac-nvram.hash       |  3 +
>  .../murata-wireless-cyw-fmac-nvram.mk         | 75 +++++++++++++++
>  package/murata-wireless/murata-wireless.mk    |  9 ++
>  13 files changed, 461 insertions(+)
>  create mode 100644 package/murata-wireless/Config.in
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-bt-patch/Config.in
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-bt-patch/murata-wireless-cyw-bt-patch.hash
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-bt-patch/murata-wireless-cyw-bt-patch.mk
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-fw/Config.in
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-fw/murata-wireless-cyw-fmac-fw.hash
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-fw/murata-wireless-cyw-fmac-fw.mk
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-nvram/Config.in
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-nvram/murata-wireless-cyw-fmac-nvram.hash
>  create mode 100644 package/murata-wireless/murata-wireless-cyw-fmac-nvram/murata-wireless-cyw-fmac-nvram.mk
>  create mode 100644 package/murata-wireless/murata-wireless.mk
> 
> -- 
> 2.21.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  parent reply	other threads:[~2019-05-07 15:41 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-07  8:32 [Buildroot] [PATCH v2 0/3] murata-wireless firmware packages Marcin Niestroj
2019-05-07  8:32 ` [Buildroot] [PATCH v2 1/3] package/murata-wireless-cyw-fmac-fw: new package Marcin Niestroj
2019-05-07 23:10   ` Arnout Vandecappelle
2019-05-09  7:44     ` Marcin Niestrój
2019-05-07  8:32 ` [Buildroot] [PATCH v2 2/3] package/murata-wireless-cyw-fmac-nvram: " Marcin Niestroj
2019-05-07  8:32 ` [Buildroot] [PATCH v2 3/3] package/murata-wireless-cyw-bt-patch: " Marcin Niestroj
2019-05-07 15:41 ` Yann E. MORIN [this message]
2019-05-07 23:02   ` [Buildroot] [PATCH v2 0/3] murata-wireless firmware packages Arnout Vandecappelle
2019-05-09  7:52   ` Marcin Niestrój
2019-05-09 18:10     ` Yann E. MORIN

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=20190507154117.GG6887@scaer \
    --to=yann.morin.1998@free.fr \
    --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.