All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rasmus Villemoes <linux@rasmusvillemoes.dk>
To: Christian Marangi <ansuelsmth@gmail.com>
Cc: Jens Axboe <axboe@kernel.dk>,  Jonathan Corbet <corbet@lwn.net>,
	 Ulf Hansson <ulf.hansson@linaro.org>,
	 Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	 Conor Dooley <conor+dt@kernel.org>,  Kees Cook <kees@kernel.org>,
	 Andy Shevchenko <andy@kernel.org>,
	 Daniel Golle <daniel@makrotopia.org>,
	 INAGAKI Hiroshi <musashino.open@gmail.com>,
	 Christian Brauner <brauner@kernel.org>,
	 Al Viro <viro@zeniv.linux.org.uk>,
	 Li Lingfeng <lilingfeng3@huawei.com>,
	Ming Lei <ming.lei@redhat.com>,
	 Christian Heusel <christian@heusel.eu>,
	linux-block@vger.kernel.org,  linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org,  linux-mmc@vger.kernel.org,
	devicetree@vger.kernel.org,  linux-hardening@vger.kernel.org,
	 Miquel Raynal <miquel.raynal@bootlin.com>,
	 Lorenzo Bianconi <lorenzo@kernel.org>,
	 upstream@airoha.com
Subject: Re: [PATCH v4 5/5] dt-bindings: mmc: Document support for partition table in mmc-card
Date: Mon, 30 Sep 2024 14:18:14 +0200	[thread overview]
Message-ID: <87y139jpx5.fsf@prevas.dk> (raw)
In-Reply-To: <20240930113045.28616-6-ansuelsmth@gmail.com> (Christian Marangi's message of "Mon, 30 Sep 2024 13:30:12 +0200")

Christian Marangi <ansuelsmth@gmail.com> writes:

> Document support for defining a partition table in the mmc-card node.
>
> This is needed if the eMMC doesn't have a partition table written and
> the bootloader of the device load data by using absolute offset of the
> block device. This is common on embedded device that have eMMC installed
> to save space and have non removable block devices.
>
> If an OF partition table is detected, any partition table written in the
> eMMC will be ignored and won't be parsed.
>
> eMMC provide a generic disk for user data and if supported (JEDEC 4.4+)
> also provide two additional disk ("boot0" and "boot1") for special usage
> of boot operation where normally is stored the bootloader or boot info.
>

This looks quite useful.

Could this be extended to also be applicable to the four "general
purpose" hardware partitions, i.e. what is exposed as /dev/mmcblkXgpY ?
These would often also contain some fundamental boot data at various
offsets but also, as for the boot partitions, often without a regular
partition table.

The eMMC spec consistently refers to the boot partitions as "boot
partition 1" and "boot partition 2"; the boot0/boot1 naming is kind of a
linux'ism. Similarly, the general purpose partitions are _almost_
exclusively referred to as 1 through 4, except (at least in my copy),
the heading for 7.4.89 says GP_SIZE_MULT_GP0 - GP_SIZE_MULT_GP3, but
then goes on to describe GP_SIZE_MULT_1_y through GP_SIZE_MULT_4_y. So I
wonder if on the binding level one should use partitions-{boot1,boot2}
and, if implemented, partitions-{gp1,gp2,gp3,gp4} ?

Rasmus

  reply	other threads:[~2024-09-30 12:18 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-30 11:30 [PATCH v4 0/5] block: partition table OF support Christian Marangi
2024-09-30 11:30 ` [PATCH v4 1/5] block: add support for defining read-only partitions Christian Marangi
2024-09-30 11:30 ` [PATCH v4 2/5] docs: block: Document support for read-only partition in cmdline part Christian Marangi
2024-09-30 11:30 ` [PATCH v4 3/5] string: add strends() helper to check if a string ends with a suffix Christian Marangi
2024-09-30 12:45   ` Andy Shevchenko
2024-10-01 14:33   ` kernel test robot
2024-10-01 21:47   ` kernel test robot
2024-09-30 11:30 ` [PATCH v4 4/5] block: add support for partition table defined in OF Christian Marangi
2024-09-30 12:49   ` Andy Shevchenko
2024-09-30 11:30 ` [PATCH v4 5/5] dt-bindings: mmc: Document support for partition table in mmc-card Christian Marangi
2024-09-30 12:18   ` Rasmus Villemoes [this message]
2024-09-30 12:22     ` Christian Marangi
2024-09-30 14:00       ` Rasmus Villemoes
2024-09-30 12:54 ` [PATCH v4 0/5] block: partition table OF support Andy Shevchenko
2024-10-02  9:20   ` Rasmus Villemoes
2024-10-03  9:59     ` Andy Shevchenko

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=87y139jpx5.fsf@prevas.dk \
    --to=linux@rasmusvillemoes.dk \
    --cc=andy@kernel.org \
    --cc=ansuelsmth@gmail.com \
    --cc=axboe@kernel.dk \
    --cc=brauner@kernel.org \
    --cc=christian@heusel.eu \
    --cc=conor+dt@kernel.org \
    --cc=corbet@lwn.net \
    --cc=daniel@makrotopia.org \
    --cc=devicetree@vger.kernel.org \
    --cc=kees@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=lilingfeng3@huawei.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-hardening@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=lorenzo@kernel.org \
    --cc=ming.lei@redhat.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=musashino.open@gmail.com \
    --cc=robh@kernel.org \
    --cc=ulf.hansson@linaro.org \
    --cc=upstream@airoha.com \
    --cc=viro@zeniv.linux.org.uk \
    /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.