Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 3/3] orangepi-pc-plus: enable WiFi support
Date: Mon, 4 Dec 2017 08:59:16 +0100	[thread overview]
Message-ID: <20171204085916.6c91c973@windsurf.home> (raw)
In-Reply-To: <CAMty3ZApVQOY6TbyD3-p=teHpB1TKZWzQymB4-B0owVEzwBD+w@mail.gmail.com>

Hello,

On Mon, 4 Dec 2017 09:55:53 +0530, Jagan Teki wrote:

> > +# wireless driver and firmware
> > +BR2_PACKAGE_RTL8189FS=y
> > +BR2_PACKAGE_LINUX_FIRMWARE=y
> > +BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX=y
> > +  
> 
> I'm trying to understand here about minimal defconfig vs custom
> selection of packages, like adding qt5 on this patch[1]. Does wifi
> here not considered as custom config? if ie true if should be
> orangepi_pc_plus_wif_defconfig?

As usual, the boundaries are not 100% strict and fixed. However, we
have considered for a number of boards that have WiFi support that it
makes sense to enable the relevant driver and firmware to enable WiFi.

This is quite different from Qt5: for WiFi, there is basically no other
option than using this firmware and this driver. For graphics, you may
want to use Qt5. But you may also want to use X11, or Wayland/Weston,
or, etc. So Qt5 is a lot less "the obvious thing that everybody will
obviously want to use on this board".

Also, installing this firmware file and building this driver probably
adds 20 seconds to the build time of the defconfig. It's not exactly
the same for Qt5 :-)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

      reply	other threads:[~2017-12-04  7:59 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-08 19:43 [Buildroot] [PATCH 0/3] orangepi_pc_plus: new board and its drivers Sergey Matyukevich
2017-11-08 19:43 ` [Buildroot] [PATCH 1/3] board/orangepi: add support for orangepi-pc-plus board Sergey Matyukevich
2017-12-03 22:25   ` Thomas Petazzoni
2017-12-04  5:58     ` Sergey Matyukevich
2017-11-08 19:43 ` [Buildroot] [PATCH 2/3] rtl8189fs: new package Sergey Matyukevich
2017-12-03 22:28   ` Thomas Petazzoni
2017-11-08 19:43 ` [Buildroot] [PATCH 3/3] orangepi-pc-plus: enable WiFi support Sergey Matyukevich
2017-12-03 22:33   ` Thomas Petazzoni
2017-12-04  4:25   ` Jagan Teki
2017-12-04  7:59     ` Thomas Petazzoni [this message]

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=20171204085916.6c91c973@windsurf.home \
    --to=thomas.petazzoni@free-electrons.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox