mirror of https://lore.kernel.org/linux-amlogic/
 help / color / mirror / Atom feed
From: narmstrong@baylibre.com (Neil Armstrong)
To: linus-amlogic@lists.infradead.org
Subject: DT: newbie questions
Date: Tue, 16 May 2017 10:08:31 +0200	[thread overview]
Message-ID: <33e616ca-fb13-343b-f464-4a934c5ed4fe@baylibre.com> (raw)
In-Reply-To: <00d73329-ff75-5041-7400-1f52f18ca46e@free.fr>

Hi Chris,

On 05/16/2017 06:18 AM, Chris Moore wrote:
> Le 15/05/2017 ? 11:28, Chris Moore a ?crit :
>> Hi,
>>
>> Sorry for these newbie questions :(
>>
>> I would like to add support for the cheap Chinese KI PLUS S905 boxes which seem very popular.
>> They are re-branded (or cloned) by several different vendors.
>> In particular the most serious seem to be:-
>> VideoStrong (http://www.videostrong.com/product.asp?sid=257)
>> Acemax (http://www.acemax.net.cn/droid-world/droid-box/hybrid-android-dvb-t2-dvb-s2/kii-pro-ki-plus-s2-t2-combo-149.html#.WRls_GjyiUk)
>>
>> I actually have an Xtreamer mxV (http://www.xtreamer.net/mxv) which seems to be an exact clone.
>>
>> My first question is what to put in the compatible line.
>> I could put:
>> a) compatible = "videostrong,kiplus", "acemax,kiplus", "xtreamer,mxv", "amlogic,meson-gxbb";
>> but there are many other KI PLUS vendors so I could maybe omit the vendor:
>> b) compatible = "kiplus", "xtreamer,mxv", "amlogic,meson-gxbb";
>> but this poses the problem of what to put in amlogic.txt which seems to require a vendor.
> 
> Another solution to the multi-vendor problem could be to create a pseudo-vendor "generic" (or similar) and then have:
> c) compatible = "generic,kiplus", "xtreamer,mxv", "amlogic,meson-gxbb";
> Would this be acceptable?

Sorry, the generic is not really acceptable here since there is always a vendor, and adding a new
vendor is easy and well accepted.

The point with Amlogic based Set-Top-Boxes is that they are 99% based on a reference design given
from Amlogic, and only the SBCs (Odroid-C*, Khadas, NanoPi K2) differs.

Here is the list of Amlogic Reference Designs I am aware of :
- S905 : P200 and P201
- S905X : P212
- S905D : P230 and P231
- S912 : Q200 and Q201

When pushing S905 based boxes, we were not really aware of that, but starting with S905X we started
to base the board DTS on the Reference Design DTSI like the Khadas Vim and the Wetek Hub/Play2.

I'm aware there is a lot of work to clean up all the other S905/S905X and S912 based boxes... but
the majority of boxes should run unchanged with the reference design DTS.

> 
>>
>> My second question is what to base my patch on:
>> Should it be v4.12/dt64 in Kevin's linux-amlogic.git?

You can now use 4.12-rc1.

>>
>> My third question concerns the SDIO WiFi.
>> The WiFi chip seems to vary between boards.
>> My board has a Realtek RTL8189ETV.
>> I note that its (unfortunately out of tree) driver is loaded even if the DT specifies a Broadcom chip.
>> Is it really necessary to specify the chip in the DT?
>> (It seems that the SDIO, like the PCI and USB buses, recognizes connected devices by their IDs.)

Yes, the SDIO id will act like PCI and USB busses.

>>
>> My final question is how to submit my patch.
>> Should I send an RFC to this list for approval before possibly spamming others?

Not really, but if you have doubts like here, you can send question and a RFC if code is better than text to explain !

>>
>> I thank you in advance for your replies and your tolerance.
>>
>> Cheers,
>> Chris Moore

Neil

>>
>> _______________________________________________
>> linux-amlogic mailing list
>> linux-amlogic at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-amlogic
>>
> 
> 
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-amlogic

  reply	other threads:[~2017-05-16  8:08 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-15  9:28 Chris Moore
2017-05-16  4:18 ` Chris Moore
2017-05-16  8:08   ` Neil Armstrong [this message]
2017-05-16 10:51     ` Chris Moore
2017-05-17 15:41       ` Chris Moore
2017-05-17 15:44         ` Neil Armstrong
2017-05-18  8:01           ` GXBB: 4.12-rc1 boot failure (was DT: newbie questions) Chris Moore
2017-05-17 22:38     ` DT: newbie questions Kevin Hilman

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=33e616ca-fb13-343b-f464-4a934c5ed4fe@baylibre.com \
    --to=narmstrong@baylibre.com \
    --cc=linus-amlogic@lists.infradead.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

all inboxes | Powered by JetHome®