Github messages for voidlinux
 help / color / mirror / Atom feed
From: cinerea0 <cinerea0@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: sof-firmware: link firmwares for Elkhart Lake and Tiger Lake
Date: Thu, 21 Jan 2021 01:29:12 +0100	[thread overview]
Message-ID: <20210121002912.c8cxzDjpS_RuNazqymPGjpDnP-xmB-1gRQn0ZBq7tsw@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-27917@inbox.vuxu.org>

[-- Attachment #1: Type: text/plain, Size: 762 bytes --]

New comment by cinerea0 on void-packages repository

https://github.com/void-linux/void-packages/pull/27917#issuecomment-764071602

Comment:
After a discussion with someone from the SOF project, and based on what I've read here, I think we should leave the tools out of this package because they're really only going to be used by firmware developers. If it becomes necessary in the future, I'll package the tools by building them from source.

As for the patch I posted earlier, I realized I don't actually need it. The following `do_install` function takes care of installation without the patch:
```shell
do_install() {
	vlicense LICENCE.NXP
	vmkdir usr/lib/firmware/intel
	export ROOT=${DESTDIR}/usr
	export SOF_VERSION=v${version}
	./go.sh
}
```

  parent reply	other threads:[~2021-01-21  0:29 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-14  9:26 [PR PATCH] sof-firmware: link firmwares for Jasper " adriangoransson
2021-01-14 10:26 ` [PR PATCH] [Updated] " adriangoransson
2021-01-16  3:15 ` [PR REVIEW] " cinerea0
2021-01-16  4:24 ` cinerea0
2021-01-16  9:17 ` [PR PATCH] [Updated] " adriangoransson
2021-01-16  9:17 ` [PR REVIEW] " adriangoransson
2021-01-20  0:06 ` sof-firmware: link firmwares for Elkhart " ericonr
2021-01-20  5:50 ` cinerea0
2021-01-20  9:07 ` adriangoransson
2021-01-20 15:27 ` ericonr
2021-01-21  0:29 ` cinerea0 [this message]
2021-01-21 20:45 ` adriangoransson
2021-01-21 20:45 ` adriangoransson
2021-01-21 22:37 ` cinerea0
2021-01-23 10:51 ` [PR PATCH] [Updated] " adriangoransson
2021-01-23 10:55 ` sof-firmware: use provided install script adriangoransson
2021-01-26  3:12 ` ericonr
2021-01-26  3:12 ` [PR PATCH] [Merged]: " ericonr

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=20210121002912.c8cxzDjpS_RuNazqymPGjpDnP-xmB-1gRQn0ZBq7tsw@z \
    --to=cinerea0@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /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;
as well as URLs for NNTP newsgroup(s).