devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ladislav Michl <ladis@linux-mips.org>
To: Andreas Kemnade <andreas@kemnade.info>
Cc: "Discussions about the Letux Kernel"
	<letux-kernel@openphoenux.org>,
	"H. Nikolaus Schaller" <hns@goldelico.com>,
	"Mark Rutland" <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, "Tony Lindgren" <tony@atomide.com>,
	linux-kernel@vger.kernel.org, "Rob Herring" <robh+dt@kernel.org>,
	"Benoît Cousson" <bcousson@baylibre.com>,
	linux-omap@vger.kernel.org
Subject: Re: [Letux-kernel] [PATCH 09/32] ARM: dts: omap3-gta04: make NAND partitions compatible with recent U-Boot
Date: Wed, 25 Jul 2018 22:07:19 +0200	[thread overview]
Message-ID: <20180725200719.GA28936@lenoch> (raw)
In-Reply-To: <20180725182745.6e436abc@aktux>

On Wed, Jul 25, 2018 at 06:27:45PM +0200, Andreas Kemnade wrote:
> On Wed, 25 Jul 2018 10:33:05 +0200
> Ladislav Michl <ladis@linux-mips.org> wrote:
> 
> > On Wed, Jul 25, 2018 at 10:18:28AM +0200, H. Nikolaus Schaller wrote:
> > >   
> > > > Am 25.07.2018 um 10:07 schrieb Ladislav Michl <ladis@linux-mips.org>:
> > > > 
> > > > On Wed, Jul 25, 2018 at 08:58:41AM +0200, H. Nikolaus Schaller wrote:  
> > > >> Vendor defined U-Boot has changed the partition scheme a while ago:
> > > >> 
> > > >> * kernel partition 6MB
> > > >> * file system partition uses the remainder up to end of the NAND
> > > >> * increased size of the environment partition (to get an OneNAND compatible base address)
> > > >> * shrink the U-Boot partition
> > > >> 
> > > >> Let's be compatible (e.g. Debian kernel built from upstream).  
> > > > 
> > > > That, in fact, is breaking compatibility.  
> > > 
> > > With what? Nobody is using the old u-boot partition scheme any more
> > > (it is >5 years old).
> > >   
> > > > So once you are touching this
> > > > what about relying on partitioning provided by bootloader just to prevent
> > > > something like this happening again?  
> > > 
> > > Well, we define what compatible means here (since we are the vendor).
> > > And people complain with us. We simply recommend them to upgrade the
> > > boot-loader.  
> > 
> > Fair enough. Suggestion was to remove partitioning scheme from DTB alltogether
> > and let U-Boot provide one. But you being vendor you decide, of course :)
> > (I'd use only two partitions: MLO and UBI, latter one with BCH8, and store
> > everything in UBI volumes. That's a bit more flexible approach)
> > 
> hmm, so using mtdparts kernel commandline parameter? Somehow it sounds
> to be sane to not have partition tables in kernel. What only is needed
> is to have a nice transition scheme for systems in the wild, can
> commandline mtdparts overwrite dtb? So dtb is a fallback?

That's beginning to be offtopic here...  Anyway, see U-Boot's
CONFIG_FDT_FIXUP_PARTITIONS. Probably better to start a thread on
U-Boot mailing list if needed.

> But I think all that is a future improvement?

Depends on vendor decision, it could be done in a few days :)

Best regards,
	ladis

  reply	other threads:[~2018-07-25 20:07 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-25  6:58 [PATCH 00/32] Device Tree Updates for GTA04 (A3/A4/A5 variants) H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 01/32] ARM: dts: omap3-gta04: fix typo in backlight pins node name H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 02/32] ARM: dts: omap3-gta04: fix some whitespace and tab style errors H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 03/32] ARM: dts: omap3-gta04: give spi_lcd node a label so that we can overwrite in other DTS files H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 04/32] ARM: dts: omap3-gta04: fixes for tvout / venc H. Nikolaus Schaller
2018-07-25 10:37   ` Sebastian Reichel
2018-07-25 11:40     ` H. Nikolaus Schaller
2018-07-25 12:31     ` H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 05/32] ARM: dts: omap3-gta04: add devconf1 setup for correct tvout pins H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 06/32] ARM: dts: omap3-gta04: tvout: enable as display1 alias H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 07/32] ARM: dts: omap3-gta04: fix touchscreen tsc2007 H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 08/32] ARM: dts: omap3-gta04: add pinmux for bmp085 EOC interrupt H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 09/32] ARM: dts: omap3-gta04: make NAND partitions compatible with recent U-Boot H. Nikolaus Schaller
2018-07-25  8:07   ` Ladislav Michl
2018-07-25  8:18     ` H. Nikolaus Schaller
2018-07-25  8:25       ` [Letux-kernel] " H. Nikolaus Schaller
2018-07-25  8:33       ` Ladislav Michl
2018-07-25 12:27         ` H. Nikolaus Schaller
2018-07-25 13:26           ` Ladislav Michl
2018-07-25 16:27         ` [Letux-kernel] " Andreas Kemnade
2018-07-25 20:07           ` Ladislav Michl [this message]
2018-07-25  6:58 ` [PATCH 10/32] ARM: dts: omap3-gta04: update gpmc NAND setup H. Nikolaus Schaller
2018-07-25  8:10   ` Ladislav Michl
2018-07-25  8:16     ` H. Nikolaus Schaller
2018-07-25  8:28       ` Ladislav Michl
2018-07-25  9:08         ` H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 11/32] ARM: dts: omap3-gta04: keep vpll2 always on H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 12/32] ARM: dts: omap3-gta04: add a comment how to reference the tca6507 gpio H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 13/32] ARM: dts: omap3-gta04: add warning comment to vaux2 regulator H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 14/32] ARM: dts: omap3-gta04: replace ti,non-removable with generic non-removable H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 15/32] ARM: dts: omap3-gta04: make use of input event include file and constants H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 16/32] ARM: dts: omap3-gta04: map antenna detect GPIO to SW_LINEIN_INSERT key event H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 17/32] ARM: dts: omap3-gta04: add camera interface parallel port H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 18/32] ARM: dts: omap3-gta04: add camera pinmux H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 19/32] ARM: dts: omap3-gta04: add mcbsp (audio subsystem) pinmux H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 20/32] ARM: dts: omap3-gta04: add devconf0 setup for mcbsp1 clock pins H. Nikolaus Schaller
2018-07-25  8:12   ` Ladislav Michl
2018-07-25  8:19     ` H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 21/32] ARM: dts: omap3-gta04: add mcbsp (audio subsystem) nodes H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 22/32] ARM: dts: omap3-gta04a3: fix model name for A3 variant H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 23/32] ARM: dts: omap3-gta04a4: fix model name for A4 variant H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 24/32] ARM: dts: omap3-gta04a5: fix copyright of A5 variant H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 25/32] ARM: dts: omap3-gta04a5: fix model name H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 26/32] ARM: dts: omap3-gta04a5: fix whitepsace and tab style H. Nikolaus Schaller
2018-07-25  6:58 ` [PATCH 27/32] ARM: dts: omap3-gta04a5: add support for PPS H. Nikolaus Schaller
2018-07-25  6:59 ` [PATCH 28/32] ARM: dts: omap3-gta04a5: disable IrDA receiver to save power H. Nikolaus Schaller
2018-07-25  6:59 ` [PATCH 29/32] ARM: dts: omap3-gta04a5: add support for ti,wl1837 module H. Nikolaus Schaller
2018-07-25  6:59 ` [PATCH 30/32] ARM: dts: omap3-gta04a5: uses different sensors than gta04a4 H. Nikolaus Schaller
2018-07-25  6:59 ` [PATCH 31/32] ARM: dts: omap3-gta04a5: define pinmux for bluetooth enable of ti,wl1837 module H. Nikolaus Schaller
2018-07-25  6:59 ` [PATCH 32/32] ARM: dts: omap3-gta04a5one: define GTA04A5 variant with OneNAND H. Nikolaus Schaller
2018-07-25  8:20   ` Ladislav Michl
2018-07-25  8:29     ` H. Nikolaus Schaller

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=20180725200719.GA28936@lenoch \
    --to=ladis@linux-mips.org \
    --cc=andreas@kemnade.info \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hns@goldelico.com \
    --cc=letux-kernel@openphoenux.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=tony@atomide.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).