From: Grant Likely <grant.likely@secretlab.ca>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org, Timur Tabi <timur@freescale.com>,
lrg@slimlogic.co.uk
Subject: Re: [PATCH] asoc/multi-component: fsl: add support for variable SSI FIFO depth
Date: Thu, 5 Aug 2010 17:19:44 -0600 [thread overview]
Message-ID: <AANLkTimSBKvrPy72TeQAMq-0rKv0ZpUOE95ZX9oRnudL@mail.gmail.com> (raw)
In-Reply-To: <D99680EC-76E2-4D4D-AC7E-DFFFCBE4E666@opensource.wolfsonmicro.com>
On Thu, Aug 5, 2010 at 4:34 PM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On 5 Aug 2010, at 23:04, Grant Likely wrote:
>> On Thu, Aug 5, 2010 at 3:42 PM, Mark Brown
>> <broonie@opensource.wolfsonmicro.com> wrote:
>>> On 5 Aug 2010, at 22:19, Grant Likely wrote:
>>>
>>>> You are; but the lack of dts factorization must be solved first before
>>>> looking at whether or not .dtb overlays make sense. Otherwise we
>>>> don't have a source for the factorized data. I personally don't think
>>>> .dtb overlays are needed, but I'm not closed to the idea either.
>>>
>>> I think all the issues from the recent discussion about bootloaders also make it strongly desirable to get the SoC generic stuff out of the DTS that's shipped separately to the kernel...
>>
>> The board specific data has the same issues as the soc specific data
>> in this regard, so I don't think it helps much to split it up at the
>> .dtb level. However....
>
> This is purely on the basis that the error rate and the volume of data are correlated.
>
>> Absolutely correct. All of this discussion and lessons learned has
>> distilled the fact the .dtb file must not be linked into the boot
>> firmware. Any design that requires a boot firmware upgrade to
>> update/replace the .dtb is broken. There will still be broken cases,
>> but fortunately for the broken cases the .dtb can always be linked
>> into the kernel image. So, for "good" firmware, the .dtb use case
>> works really well. In the "broken" firmware case, the .dtb(s) can be
>> linked into the wrapper image and the correct one can be chosen based
>> on machine-id or some other discernible board data. I'm being very
>> careful to make sure that using a .dtb doesn't paint anybody into a
>> corner.
>
> On the other hand if you do have to link the device tree into the wrapper image you run into fun and games with any per system data like MAC addresses which might be embedded in the device tree.
>
> Also, it's not just good and bad firmware that's the issue - like I say, there are also coordination issues within large development teams to consider.
I'm starting a working group to prepare a set of requirements,
recommendations and sample implementation for an ARM boot
architecture. Particularly useful for more-or-less general purpose
OSes like Ubuntu on ARM. We'll be talking about these kinds of
issues. Would you like me to cc you when I kick it off?
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
next prev parent reply other threads:[~2010-08-05 23:20 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-04 20:04 [PATCH] asoc/multi-component: fsl: add support for variable SSI FIFO depth Timur Tabi
2010-08-05 11:10 ` Mark Brown
2010-08-05 13:14 ` Timur Tabi
2010-08-05 13:51 ` Mark Brown
2010-08-05 14:10 ` Tabi Timur-B04825
2010-08-05 14:54 ` Mark Brown
2010-08-05 15:14 ` Tabi Timur-B04825
2010-08-05 16:03 ` Mark Brown
2010-08-05 16:21 ` Tabi Timur-B04825
2010-08-05 18:18 ` Mark Brown
2010-08-05 19:43 ` Grant Likely
2010-08-05 21:13 ` Timur Tabi
2010-08-05 21:19 ` Grant Likely
2010-08-05 21:42 ` Mark Brown
2010-08-05 22:04 ` Grant Likely
2010-08-05 22:34 ` Mark Brown
2010-08-05 23:19 ` Grant Likely [this message]
2010-08-05 23:23 ` Mark Brown
2010-08-05 23:22 ` Mark Brown
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=AANLkTimSBKvrPy72TeQAMq-0rKv0ZpUOE95ZX9oRnudL@mail.gmail.com \
--to=grant.likely@secretlab.ca \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=lrg@slimlogic.co.uk \
--cc=timur@freescale.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).