From: Eric Nelson <eric.nelson@boundarydevices.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-boot] fastboot command question
Date: Thu, 22 Aug 2013 07:30:31 -0700 [thread overview]
Message-ID: <52162087.60909@boundarydevices.com> (raw)
In-Reply-To: <20130822135930.GK2644@bill-the-cat>
Thanks Tom,
On 08/22/2013 06:59 AM, Tom Rini wrote:
> On Wed, Aug 21, 2013 at 05:30:52PM -0700, Eric Nelson wrote:
>> On 08/21/2013 04:30 AM, TigerLiu at viatech.com.cn wrote:
>>> Hi, experts:
>>>
>>> Why not include cmd_fastboot.c in common directory in 2013.07 release
>>> package?
>>>
>>> Because of code license?
>>>
>>> Or other reason?
>>>
>>
>> <snip>
>>
>> Do we need a clean-room implementation in order to comply with?
>>
>> Is anybody working either the licensing or implementation of this?
>
> So, putting my TI hat on, I've poked one of our legal teams about one of
> the fastboot versions we've done (the shove everything into
> cmd_fastboot.c one, which I don't like as much as the split up ones,
> but, that's a technical thing not a license thing) to see if there's any
> problems or not.
>
> Historically, Wolfgang objected, roughly, on the grounds of "great, Yet
> Another Standard by Bigcompany forcing things on us". Which I can
> understand, but frankly, it's more of an ABI than some of the "funny"
> things Android did within the kernel, so I'm willing to live with it,
> so long as the implementation is done well, and there's no legal
> hurdles.
Customers using Android ask for fastboot support, and this isn't
a lot of code, so it seems worth pursuing IMHO.
> In fact, http://patchwork.ozlabs.org/patch/126797/ was "looks
> legally fine, needs technical changes".
>
Hmmm. I missed this.
The only remaining objection seems to have been the commit
message and attribution of the original source.
I'm CC'ing Aneesh to see if he can provide any guidance.
Regards,
Eric
next prev parent reply other threads:[~2013-08-22 14:30 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-21 11:30 [U-Boot] [U-boot] fastboot command question TigerLiu at viatech.com.cn
2013-08-22 0:30 ` Eric Nelson
2013-08-22 5:08 ` Marek Vasut
2013-08-22 9:52 ` TigerLiu at viatech.com.cn
2013-08-22 14:01 ` [U-Boot] " Eric Nelson
2013-08-22 13:56 ` [U-Boot] [U-boot] " Eric Nelson
2013-08-22 13:59 ` Tom Rini
2013-08-22 14:30 ` Eric Nelson [this message]
2013-08-22 15:02 ` Tom Rini
2013-08-22 15:26 ` Tom Rini
-- strict thread matches above, loose matches on Subject: below --
2013-08-21 10:15 TigerLiu at viatech.com.cn
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=52162087.60909@boundarydevices.com \
--to=eric.nelson@boundarydevices.com \
--cc=u-boot@lists.denx.de \
/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