All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <monstr@monstr.eu>
To: u-boot@lists.denx.de
Subject: [U-Boot] u-boot-socfpga repository
Date: Fri, 12 Sep 2014 20:00:48 +0200	[thread overview]
Message-ID: <541334D0.80109@monstr.eu> (raw)
In-Reply-To: <20140911074618.51A09380CA2@gemini.denx.de>

Hi Wolfgang,

On 09/11/2014 09:46 AM, Wolfgang Denk wrote:
> Dear Michal,
> 
> In message <54112B64.5010104@monstr.eu> you wrote:
>>
>> I am not sure if you need to have separate repo to work like this.
>> I am keeping zynq patches in my microblaze repo and sending pull request
>> to Albert
>> (or Tom now) and there is no problem with that.
> 
> Well, technically of course this works, but it is far from perfect.
> It works only for those who actually know about this.  But anybody
> looking at the U-Boot site for any zynq related stuff will have hard
> times to find it.

Please don't get me wrong I am just trying to understand what happened.

We have MAINTAINERS file where we can simply
add person who is responsible for specific SOC. I also believe that
this is exactly reason why to have it.

From that file everybody can find out who should take the patches
and and almost doesn't matter if git is at denx.de or somewhere else.

But still all ARM patches should go through our ARM custodian
not directly to Tom.

get_maintainer.pl should also keep all interested people in the loop.

> I think it is much better to make this knowledge public information -
> and one easy way to do this is to have a separate repository for it,
> which is listed on the custodians page, so everybody looking for it
> will find all relevant information.

Isn't that MAINTAINERS file already publicly available?
Of course if you want to add the same information on wiki, I can't see
any problem there.
But creating separate repository for every SoC in u-boot seems to me
just too much.

>> In socfpga case I think there are guys from Altera who maintain it.
> 
> Well, they maintain the stuff at rocketboards.org ; there are efforts
> on the way to mainline stuff, but the process is not exactly
> satisfactory.  I highly appreciate that Marek volunteers to put
> efforts in this.

Definitely I also really appreciate that Marek volunteers to be responsible
for socfpga. But that means that guys who are responsible for socfpga failed
and they are not responding for patches which others are sending to mailing list for review.
I am not closely following them but did this really happen?

If they don't want to maintain their socfpga in mainline and Marek
wants to do it I will definitely support Marek in this new possition
to get things done properly but also I just want to make sure that
this is really happen and isn't it just the part of misunderstanding
how u-boot community is working.

> As far as I am concerned, I support both Marek's and Masahiro's
> requests.
> 
> @ Marek and Masahiro: if we reach an agreement to create such repos,
> 	please send me your SSH public keys that shall be used for
> 	these.  Also, what should the names be - u-boot-socfpga ?
> 	And u-boot-uniphier ? [But is there not a chance that Pana-
> 	sonic might have other SoCs that might be mainlines?  So
> 	maybe we should use u-boot-panasonic instead?]

If they want to have separate repository because they think that
it is better then their current one then sure why not to create them.

Having arm SOC sub maintainers is definitely good concept
but I am just not sure that creating separate repository fully solve
this.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140912/0b3505b8/attachment.pgp>

  parent reply	other threads:[~2014-09-12 18:00 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-10 23:33 [U-Boot] u-boot-socfpga repository Marek Vasut
2014-09-11  3:09 ` Masahiro Yamada
2014-09-11  4:56   ` Michal Simek
2014-09-11  5:18     ` Masahiro Yamada
2014-09-18  7:27       ` [U-Boot] U-Boot panasonic repo Marek Vasut
2014-09-18  7:58         ` Michal Simek
2014-09-18  8:02           ` Marek Vasut
2014-09-18  8:24             ` Michal Simek
2014-09-18 10:13               ` Masahiro Yamada
2014-09-18 11:44                 ` Albert ARIBAUD
2014-09-18 13:32                   ` Masahiro YAMADA
2014-09-18  8:32         ` Wolfgang Denk
2014-09-18  8:43           ` Masahiro Yamada
2014-09-18  9:03             ` Wolfgang Denk
     [not found]               ` <CAMhH57QtP7H8rx8cq--s8HqeJFq+C4xmGhh1V2RB6CvDqOiiTg@mail.gmail.com>
2014-09-27  9:01                 ` Masahiro YAMADA
2014-09-27 13:00                   ` Wolfgang Denk
2014-09-27 15:02                     ` Masahiro YAMADA
2014-09-27 15:57                       ` Wolfgang Denk
2014-09-28  5:20                         ` Masahiro YAMADA
2014-09-11  7:46     ` [U-Boot] u-boot-socfpga repository Wolfgang Denk
2014-09-11  7:55       ` Pavel Machek
2014-09-11 15:39         ` Marek Vasut
2014-09-11 16:01         ` Dinh Nguyen
2014-09-11 17:21           ` Marek Vasut
2014-09-16  9:56           ` Pavel Machek
2014-09-17 11:54             ` Chin Liang See
2014-09-19  9:54               ` Chin Liang See
2014-09-23 12:54                 ` Tom Rini
2014-09-23 13:43                   ` Wolfgang Denk
2014-09-23 13:46                     ` Tom Rini
2014-09-23 14:30                       ` Wolfgang Denk
2014-09-11 15:45       ` Marek Vasut
2014-09-11 16:14       ` Dinh Nguyen
2014-09-11 16:51         ` Marek Vasut
2014-09-11 23:14           ` Dinh Nguyen
2014-09-11 17:14         ` Tom Rini
2014-09-11 23:19           ` Dinh Nguyen
2014-09-12  5:25             ` Wolfgang Denk
2014-09-12 18:27               ` Dinh Nguyen
2014-09-12 19:46                 ` Wolfgang Denk
2014-09-12 21:15                   ` Dinh Nguyen
2014-09-12 22:14                     ` Wolfgang Denk
2014-09-12 22:29                       ` Dinh Nguyen
2014-09-12 22:51                         ` Wolfgang Denk
2014-09-14 16:02                           ` Dinh Nguyen
2014-09-14 16:46                             ` Tom Rini
2014-09-15  1:07                               ` Chin Liang See
2014-09-15  1:27                                 ` Marek Vasut
2014-09-15  1:43                                   ` Chin Liang See
2014-09-15  1:47                                     ` Marek Vasut
2014-09-14 16:51                             ` Wolfgang Denk
2014-09-12 19:57                 ` Michael Trimarchi
2014-09-12 21:05                 ` David Hawkins
2014-09-12 21:20                   ` Michael Trimarchi
2014-09-12 21:23                     ` David Hawkins
2014-09-12 21:30                       ` Michael Trimarchi
2014-09-12 21:32                   ` Dinh Nguyen
2014-09-12 22:37                     ` Wolfgang Denk
2014-09-15  1:35                       ` Chin Liang See
2014-09-15  8:53                         ` [U-Boot] mainline u-boot on socfpga Pavel Machek
2014-09-13 17:24                     ` [U-Boot] u-boot-socfpga repository David Hawkins
2014-09-14 15:39                       ` Dinh Nguyen
2014-09-14 16:30                         ` Wolfgang Denk
2014-09-14 20:04                         ` David Hawkins
2014-09-13  0:16                   ` David Hawkins
2014-09-12 18:00       ` Michal Simek [this message]
2014-09-12 19:45         ` Wolfgang Denk
2014-09-11 15:46     ` Marek Vasut

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=541334D0.80109@monstr.eu \
    --to=monstr@monstr.eu \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.