Linux Archive

Linux Archive (http://www.linux-archive.org/)
-   Ubuntu Kernel Team (http://www.linux-archive.org/ubuntu-kernel-team/)
-   -   d-i: make all firmware optional too (http://www.linux-archive.org/ubuntu-kernel-team/380906-d-i-make-all-firmware-optional-too.html)

Andy Whitcroft 06-03-2010 09:30 AM

d-i: make all firmware optional too
 
BugLink: http://bugs.launchpad.net/bugs/588805

Signed-off-by: Andy Whitcroft <apw@canonical.com>
---
debian.master/d-i/firmware/nic-modules | 6 +++---
debian.master/d-i/firmware/scsi-modules | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/debian.master/d-i/firmware/nic-modules b/debian.master/d-i/firmware/nic-modules
index bf601e5..1683452 100644
--- a/debian.master/d-i/firmware/nic-modules
+++ b/debian.master/d-i/firmware/nic-modules
@@ -5,6 +5,6 @@ bnx2/bnx2-rv2p-09ax-5.0.0.j3.fw ?
bnx2/bnx2-rv2p-09-5.0.0.j3.fw ?
bnx2x-e1-5.0.21.0.fw ?
bnx2x-e1h-5.0.21.0.fw ?
-e100/d101m_ucode.bin
-e100/d101s_ucode.bin
-e100/d102e_ucode.bin
+e100/d101m_ucode.bin ?
+e100/d101s_ucode.bin ?
+e100/d102e_ucode.bin ?
diff --git a/debian.master/d-i/firmware/scsi-modules b/debian.master/d-i/firmware/scsi-modules
index b089c7d..56bf990 100644
--- a/debian.master/d-i/firmware/scsi-modules
+++ b/debian.master/d-i/firmware/scsi-modules
@@ -1,3 +1,3 @@
-qlogic/1040.bin
-qlogic/12160.bin
-qlogic/1280.bin
+qlogic/1040.bin ?
+qlogic/12160.bin ?
+qlogic/1280.bin ?
--
1.7.0.4


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team

Leann Ogasawara 06-07-2010 06:43 PM

d-i: make all firmware optional too
 
Hi Tim and Andy,

I wanted to get come clarification in regards to firmware files being
marked as optional. In a completely separate thread to this one, Tim
had submitted a patch which updates all the bnx2 udeb firmware files to
*not* be marked as optional:

https://lists.ubuntu.com/archives/kernel-team/2010-June/010944.html

Tim, you had mentioned that "Its worth noting that firmware files,
unlike modules, are not generated and should never be marked
optional..."

So Andy my question here is do we actually need the e100 and qlogic
firmware files marked as optional for the armel-omap udebs? Or can I
drop this patch as part of the series?

Thanks,
Leann

On Thu, 2010-06-03 at 10:30 +0100, Andy Whitcroft wrote:
> BugLink: http://bugs.launchpad.net/bugs/588805
>
> Signed-off-by: Andy Whitcroft <apw@canonical.com>
> ---
> debian.master/d-i/firmware/nic-modules | 6 +++---
> debian.master/d-i/firmware/scsi-modules | 6 +++---
> 2 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/debian.master/d-i/firmware/nic-modules b/debian.master/d-i/firmware/nic-modules
> index bf601e5..1683452 100644
> --- a/debian.master/d-i/firmware/nic-modules
> +++ b/debian.master/d-i/firmware/nic-modules
> @@ -5,6 +5,6 @@ bnx2/bnx2-rv2p-09ax-5.0.0.j3.fw ?
> bnx2/bnx2-rv2p-09-5.0.0.j3.fw ?
> bnx2x-e1-5.0.21.0.fw ?
> bnx2x-e1h-5.0.21.0.fw ?
> -e100/d101m_ucode.bin
> -e100/d101s_ucode.bin
> -e100/d102e_ucode.bin
> +e100/d101m_ucode.bin ?
> +e100/d101s_ucode.bin ?
> +e100/d102e_ucode.bin ?
> diff --git a/debian.master/d-i/firmware/scsi-modules b/debian.master/d-i/firmware/scsi-modules
> index b089c7d..56bf990 100644
> --- a/debian.master/d-i/firmware/scsi-modules
> +++ b/debian.master/d-i/firmware/scsi-modules
> @@ -1,3 +1,3 @@
> -qlogic/1040.bin
> -qlogic/12160.bin
> -qlogic/1280.bin
> +qlogic/1040.bin ?
> +qlogic/12160.bin ?
> +qlogic/1280.bin ?
> --
> 1.7.0.4
>
>



--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team

Tim Gardner 06-07-2010 06:56 PM

d-i: make all firmware optional too
 
The primary reason I believe firmware files should not be marked
optional is because they are not generated, as opposed to modules which
may or may not be built depending on the flavour config. If a firmware
file changes names or locations, then its better to detect that at build
time rather then after the package has been released.

rtg

On 06/07/2010 12:43 PM, Leann Ogasawara wrote:
> Hi Tim and Andy,
>
> I wanted to get come clarification in regards to firmware files being
> marked as optional. In a completely separate thread to this one, Tim
> had submitted a patch which updates all the bnx2 udeb firmware files to
> *not* be marked as optional:
>
> https://lists.ubuntu.com/archives/kernel-team/2010-June/010944.html
>
> Tim, you had mentioned that "Its worth noting that firmware files,
> unlike modules, are not generated and should never be marked
> optional..."
>
> So Andy my question here is do we actually need the e100 and qlogic
> firmware files marked as optional for the armel-omap udebs? Or can I
> drop this patch as part of the series?
>
> Thanks,
> Leann
>
> On Thu, 2010-06-03 at 10:30 +0100, Andy Whitcroft wrote:
>> BugLink: http://bugs.launchpad.net/bugs/588805
>>
>> Signed-off-by: Andy Whitcroft<apw@canonical.com>
>> ---
>> debian.master/d-i/firmware/nic-modules | 6 +++---
>> debian.master/d-i/firmware/scsi-modules | 6 +++---
>> 2 files changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/debian.master/d-i/firmware/nic-modules b/debian.master/d-i/firmware/nic-modules
>> index bf601e5..1683452 100644
>> --- a/debian.master/d-i/firmware/nic-modules
>> +++ b/debian.master/d-i/firmware/nic-modules
>> @@ -5,6 +5,6 @@ bnx2/bnx2-rv2p-09ax-5.0.0.j3.fw ?
>> bnx2/bnx2-rv2p-09-5.0.0.j3.fw ?
>> bnx2x-e1-5.0.21.0.fw ?
>> bnx2x-e1h-5.0.21.0.fw ?
>> -e100/d101m_ucode.bin
>> -e100/d101s_ucode.bin
>> -e100/d102e_ucode.bin
>> +e100/d101m_ucode.bin ?
>> +e100/d101s_ucode.bin ?
>> +e100/d102e_ucode.bin ?
>> diff --git a/debian.master/d-i/firmware/scsi-modules b/debian.master/d-i/firmware/scsi-modules
>> index b089c7d..56bf990 100644
>> --- a/debian.master/d-i/firmware/scsi-modules
>> +++ b/debian.master/d-i/firmware/scsi-modules
>> @@ -1,3 +1,3 @@
>> -qlogic/1040.bin
>> -qlogic/12160.bin
>> -qlogic/1280.bin
>> +qlogic/1040.bin ?
>> +qlogic/12160.bin ?
>> +qlogic/1280.bin ?
>> --
>> 1.7.0.4
>>
>>
>
>


--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team


All times are GMT. The time now is 08:47 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.