All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Stewart <christian@paral.in>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v6 1/1] rtl8821au: new package
Date: Fri, 24 Jul 2015 22:29:39 +0000	[thread overview]
Message-ID: <CA+h8R2o7sa9ChHJxZOrJeUFAX7qUzCtp4Vnv_kdD9K4sVtMPvw@mail.gmail.com> (raw)
In-Reply-To: <alpine.LNX.2.00.1507241546220.30794@localhost>

Hey Hans,

It's not unusual in Buildroot to have long commit messages when a package
is first added.

I've addressed your concerns and respun the patch.

Christian

On Fri, Jul 24, 2015 at 6:50 AM Hans Ulli Kroll <ulli.kroll@googlemail.com>
wrote:

> Hi all,
>
> Note : I'm new on the list, so I had insert this tread by hand !
>
> I'm the one working on the RTL8821AU driver.
>
> On Thu, 23 Jul 2015, Christian Stewart wrote:
>
> > Adding a kernel module based USB wifi driver. There is no official
> > version of this driver that works properly on ARM and with newer USB
> > WiFi cards. This driver version builds module 8821au and is based on a
> > kernel module release from ASUS with various fixes integrated in.
>
> This message will be commited please change.
>
> The I think the difference between the version from ASUS and mine based
> the one Edimax are in the handling of the LED.
> Maybe there are more ...
>
> > I've switched to use ulli-kroll's branch of rtl8821au. While it's based
> > on an older vendor version, it has binary firmware and around ~1800
> > commits of work to improve the driver. I think this work is probably
> > more valuable than the vendor's updates.
> >
>
> -> git send-email --compose
> Will add some message an intreductory message
>
> > --- /dev/null
> > +++ b/package/rtl8821au/0001-disable-default-debug.patch
>
> I do add some kind of "disable debug message"
> This will take some time, I'm currently working in the code.
>
> > +config BR2_PACKAGE_RTL8821AU
> > +     bool "rtl8821au"
> > +     depends on BR2_LINUX_KERNEL
> > +     help
> > +       rtl8821au wifi driver
> > +
> > +       https://github.com/paralin/rtl8821au
>
> Please fix this URL !
>
> Maybe add some line "out of tree driver"
> And the USB3 issue I found inthe HW as you can read in
> Readme.md
>
> Greetings from Cologne
> Hans Ulli Kroll
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20150724/be17f1a8/attachment.html>

  reply	other threads:[~2015-07-24 22:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-23 22:41 [Buildroot] [PATCH v6 1/1] rtl8821au: new package Christian Stewart
2015-07-24 13:50 ` Hans Ulli Kroll
2015-07-24 22:29   ` Christian Stewart [this message]
2015-07-25 16:13   ` Yann E. MORIN
2015-07-26 19:59     ` Hans Ulli Kroll
2015-07-26 22:45       ` Christian Stewart
2015-07-27  7:15         ` Hans Ulli Kroll

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=CA+h8R2o7sa9ChHJxZOrJeUFAX7qUzCtp4Vnv_kdD9K4sVtMPvw@mail.gmail.com \
    --to=christian@paral.in \
    --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.