public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ian Campbell <ijc@hellion.org.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 29/30] RFC: Use binman for a sunxi board
Date: Tue, 26 Jul 2016 14:35:01 +0100	[thread overview]
Message-ID: <1469540101.8003.9.camel@hellion.org.uk> (raw)
In-Reply-To: <1469494766-26601-30-git-send-email-sjg@chromium.org>

(adding Hans)

On Mon, 2016-07-25 at 18:59 -0600, Simon Glass wrote:
> Add an example usage of binman for a sunxi board. This involves adding the
> > image definition to the device tree and using it in the Makefile.

My concern with this approach is that the *.dts(i) files used for sunxi
are sync'd unmodified from the upstream repo. I think this is a
valuable property to retain in terms of maintenance burden for us
(resync is trivial only if there are no local changes).

The conflict is then whether the device tree people would consider this
new block to be "describing the hardware" or not and therefore whether
these changes would be welcomed/allowed upstream. I'm not sure which
way it would go TBH.

There is also an issue with duplication, since this stanza is going to
be the same for all the dozens of sunxi platforms. I suppose this can
be addressed by putting it in a suitable .dtsi file, but that does sort
of start to suggest that this information belongs outside the per-board 
.dts in a per-platform file (at which point I would have have to ask if
.dts is the best/most convenient syntax?).

Ian.

  reply	other threads:[~2016-07-26 13:35 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-26  0:58 [U-Boot] [PATCH 00/30] binman: A tool for creating firmware images Simon Glass
2016-07-26  0:58 ` [U-Boot] [PATCH 01/30] x86: Add debugging when a microcode update fails Simon Glass
2016-07-27  4:22   ` Heiko Schocher
2016-07-28  6:53   ` Bin Meng
2016-08-29  1:45     ` Bin Meng
2016-07-26  0:58 ` [U-Boot] [PATCH 02/30] x86: ivybridge: Allow microcode to be collated Simon Glass
2016-07-28  6:53   ` Bin Meng
2016-08-29  1:45     ` Bin Meng
2016-07-26  0:58 ` [U-Boot] [PATCH 03/30] x86: Add debugging when cpu_common_init() fails Simon Glass
2016-07-28  6:53   ` Bin Meng
2016-08-29  1:45     ` Bin Meng
2016-07-26  0:59 ` [U-Boot] [PATCH 04/30] patman: Adjust command.Output() to raise an error by default Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 05/30] dtoc: Move the struct import into the correct order Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 06/30] dtoc: Move the fdt library selection into fdt_select Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 07/30] dtoc: Rename fdt.py to fdt_normal.py Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 08/30] dtoc: Create a base class for Fdt Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 09/30] dtoc: Move BytesToValue() and GetEmpty() into PropBase Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 10/30] dtoc: Move Widen() and GetPhandle() into the base class Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 11/30] dtoc: Move a few more common functions into fdt.py Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 12/30] patman: Add a tools library for using temporary files Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 13/30] patman: Add a library to handle logging and progress Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 14/30] dtoc: Allow the device tree to be compiled from source Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 15/30] dtoc: Drop the convert_dash parameter to GetProps() Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 16/30] dtoc: Prepare for supporting changing of device trees Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 17/30] dtoc: Move to using bytearray Simon Glass
2016-08-27 16:05   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 18/30] dtoc: Support deleting device tree properties Simon Glass
2016-08-27 16:06   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 19/30] dtoc: Support packing the device tree Simon Glass
2016-08-27 16:06   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 20/30] dtoc: Support finding the offset of a property Simon Glass
2016-08-27 16:06   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 21/30] dtoc: Correct quotes in fdt_util Simon Glass
2016-08-27 16:06   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 22/30] dtoc: Add methods for reading data from properties Simon Glass
2016-08-27 16:06   ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 23/30] binman: Introduce binman, a tool for building binary images Simon Glass
2016-07-27  4:59   ` Heiko Schocher
2016-07-26  0:59 ` [U-Boot] [PATCH 24/30] binman: Add basic entry types for U-Boot Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 25/30] binman: Add support for building x86 ROMs Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 26/30] binman: Add support for u-boot.img as an input binary Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 27/30] binman: Add a build rule for binman Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 28/30] binman: Allow configuration options to be used in .dts files Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 29/30] RFC: Use binman for a sunxi board Simon Glass
2016-07-26 13:35   ` Ian Campbell [this message]
2016-07-26 23:49     ` Simon Glass
2016-07-26  0:59 ` [U-Boot] [PATCH 30/30] RFC: Use binman for an x86 board Simon Glass
2016-08-27 15:28 ` [U-Boot] [PATCH 00/30] binman: A tool for creating firmware images Simon Glass

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=1469540101.8003.9.camel@hellion.org.uk \
    --to=ijc@hellion.org.uk \
    --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