public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH/review] Blackfin: add support for BF538/BF539
Date: Sun, 1 Jun 2008 19:03:41 -0400	[thread overview]
Message-ID: <200806011903.42442.vapier@gentoo.org> (raw)
In-Reply-To: <20080601220555.04F222430A@gemini.denx.de>

On Sunday 01 June 2008, Wolfgang Denk wrote:
> In message <200806011718.32290.vapier@gentoo.org> you wrote:
> > > What is the licensing of this file, and who is the Copyright holder?
> >
> > it's all ADI written and owned.  we dont particularly care about the
> > license,
>
> Ummm... Mike, you are a well-know contributor to Free Software. Such a
> statement from someone like you is kind of a shock to me.

the code in question is autogenerated.  people sniping it is no sweat off our 
back.  in general, my willingness to freely give all my code away doesnt mean 
i subscribe to the FSF mentality.

> > but i guess i can tag it GPL-2.
>
> Your guess is not good enough. We need to know for sure. And if it's
> owned by ADI *you* cannot do that.

i most certainly can.  seeing as how i'm an ADI employee and they've given me 
carte blanche for managing this code and seeing as how i'm the one who 
actually wrote all of it in the first place ...

> > > I do not think that we should allow such code in U-Boot.
> >
> > it is the designed programming style for all low level Blackfin systems. 
> > it is unified across Linux, U-Boot, bare metal code, and the official ADI
> > propriety compiler.  i thought your point was to keep U-Boot and Linux
> > the same at the API level so code sharing is very easy between it ?
>
> Yes, that's what we normally do.
>
> However here I'm really uncertain. Actually I am deeply disappointed
> that such code made it into the Linux kernel. This should have never
> happened.

there was already a debate on lkml on the topic about obvious pros/cons, but 
the code that's here is the result.  we take the stance of putting more logic 
into the headers/arch code so that end developers (like drivers) get a much 
easier time.  after all, the core stuff (like this) rarely changes whereas 
end/drivers constantly churn.  we've already seen substantially easier to 
manage drivers as a result in the kernel.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080601/afdf123a/attachment.pgp 

  reply	other threads:[~2008-06-01 23:03 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-01  5:38 [U-Boot-Users] [PATCH/review] Blackfin: dont call i2c_init() from board_init_r() Mike Frysinger
2008-06-01  5:38 ` [U-Boot-Users] [PATCH/review] Blackfin: protect default flash according to CFG_MONITOR_LEN Mike Frysinger
2008-06-01  5:38   ` [U-Boot-Users] [PATCH/review] Blackfin: update proc headers from toolchain Mike Frysinger
2008-06-01  5:38     ` [U-Boot-Users] [PATCH/review] Blackfin: unify cache handling code Mike Frysinger
2008-06-01  5:38       ` [U-Boot-Users] [PATCH/review] Blackfin: use common memcpy routine during init Mike Frysinger
2008-06-01  5:38         ` [U-Boot-Users] [PATCH/review] Blackfin: enable support for nested interrupts Mike Frysinger
2008-06-01  5:38           ` [U-Boot-Users] [PATCH/review] Blackfin: respect CONFIG_CLKIN_HALF Mike Frysinger
2008-06-01  5:38             ` [U-Boot-Users] [PATCH/review] Blackfin: setup a sane default EBIU_SDBCTL for SDRAM controllers Mike Frysinger
2008-06-01  5:38               ` [U-Boot-Users] [PATCH/review] Blackfin: use on-chip syscontrol() rom function when available Mike Frysinger
2008-06-01  5:38                 ` [U-Boot-Users] [PATCH/review] Blackfin: use on-chip syscontrol() to reset Mike Frysinger
2008-06-01  5:38                   ` [U-Boot-Users] [PATCH/review] Blackfin: add support for BF538/BF539 Mike Frysinger
2008-06-01 20:14                     ` Wolfgang Denk
2008-06-01 21:18                       ` Mike Frysinger
2008-06-01 22:05                         ` Wolfgang Denk
2008-06-01 23:03                           ` Mike Frysinger [this message]
2008-06-04  8:56                             ` Detlev Zundel
2008-06-04 10:18                               ` Mike Frysinger
2008-06-04 17:36                                 ` Detlev Zundel
2008-06-04 20:16                                   ` Mike Frysinger
2008-06-05 13:13                                     ` Detlev Zundel
2008-06-05 13:24                                     ` Wolfgang Denk
2008-06-01 19:56                   ` [U-Boot-Users] [PATCH/review] Blackfin: use on-chip syscontrol() to reset Wolfgang Denk
2008-06-01 21:20                     ` Mike Frysinger
2008-06-01 22:07                       ` Wolfgang Denk
2008-06-04  9:05                       ` Detlev Zundel
2008-06-04 10:10                         ` Mike Frysinger
2008-06-01 19:54               ` [U-Boot-Users] [PATCH/review] Blackfin: setup a sane default EBIU_SDBCTL for SDRAM controllers Wolfgang Denk
2008-06-01 21:12                 ` Mike Frysinger
2008-06-01 21:59                   ` Wolfgang Denk
2008-06-01 22:49                     ` Mike Frysinger
2008-06-01 19:47   ` [U-Boot-Users] [PATCH/review] Blackfin: protect default flash according to CFG_MONITOR_LEN Wolfgang Denk

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=200806011903.42442.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --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