From: Stephen Hemminger <shemminger@vyatta.com>
To: Francois Romieu <romieu@fr.zoreil.com>
Cc: hayeswang <hayeswang@realtek.com>,
"'Ben Hutchings'" <benh@debian.org>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
"'David Miller'" <davem@davemloft.net>,
564628@bugs.debian.org
Subject: Re: [PATCH] net/r8169: Correct the ram code for RTL8111D(L)
Date: Mon, 29 Nov 2010 16:13:44 -0800 [thread overview]
Message-ID: <20101129161344.059239d6@nehalam> (raw)
In-Reply-To: <20101129233334.GA2888@electric-eye.fr.zoreil.com>
On Tue, 30 Nov 2010 00:33:34 +0100
Francois Romieu <romieu@fr.zoreil.com> wrote:
> hayeswang <hayeswang@realtek.com> :
> > Excuse me, I have some questions about the firmware patch.
> >
> > 1. I should convert the data into the binary files (.bin). Is it right?
>
> You may do it.
>
> Fwiw I have cooked something for it in the attached patch #9 this WE. Feel
> free to take bits from it. I will not do more changes while you work on it.
>
> > 2. Where should I update the firmware files? Is the path the
> > linux-2.6/firmware?
>
> ! This directory is only here to contain firmware images extracted from old
> ! device drivers which predate the common use of request_firmware().
>
> It is fine for the existing code.
>
> > However, according to linux-2.6/firmeware/README.AddingFirmware, I
> > should update they to another repository:
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/dwmw2/linux-firmware.git
>
> /me scratches head.
>
> Assuming Realtek does not intend to stand this code / data as GPLable, yes.
> It will help people staying clear from non-free code, it will help packaging
> something useful and it will remove some cruft from the code.
>
> So everybody will end happy. Especially after an aspirin.
The plan is to do away with linux-2.6/firmware entirely.
Distributions are shipping the firmware from linux-firmware, not the
kernel files.
next prev parent reply other threads:[~2010-11-30 0:13 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-26 11:54 [PATCH] net/r8169: Correct the ram code for RTL8111D(L) Hayes Wang
2010-11-26 13:50 ` Ben Hutchings
2010-11-26 22:49 ` Francois Romieu
2010-11-26 23:12 ` Ben Hutchings
2010-11-29 3:47 ` hayeswang
2010-11-29 23:33 ` Francois Romieu
2010-11-30 0:13 ` Stephen Hemminger [this message]
2010-11-30 6:37 ` hayeswang
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=20101129161344.059239d6@nehalam \
--to=shemminger@vyatta.com \
--cc=564628@bugs.debian.org \
--cc=benh@debian.org \
--cc=davem@davemloft.net \
--cc=hayeswang@realtek.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=romieu@fr.zoreil.com \
/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.