All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
From: Sean Anderson <sean.anderson@seco.com>
To: u-boot@lists.denx.de
Subject: [PATCH 9/9] fastboot: Document alternate partition names
Date: Thu, 31 Dec 2020 20:54:44 -0500	[thread overview]
Message-ID: <2164fab6-92b8-224b-4538-05998abca7c2@seco.com> (raw)
In-Reply-To: <f37c93d1-3d5c-cf0a-de2a-c601fefd61c3@gmx.de>

On 12/31/20 7:36 PM, Heinrich Schuchardt wrote:
> On 12/31/20 11:48 PM, Sean Anderson wrote:
>> This documents the new partition names added in the previous commit.
>>
>> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
>> ---
>>
>>   doc/android/fastboot.rst | 28 ++++++++++++++++++++++++++++
>>   1 file changed, 28 insertions(+)
>>
>> diff --git a/doc/android/fastboot.rst b/doc/android/fastboot.rst
>> index 2877c3cbaa..2ca80ae844 100644
>> --- a/doc/android/fastboot.rst
>> +++ b/doc/android/fastboot.rst
>> @@ -151,6 +151,34 @@ The device index starts from ``a`` and refers to the interface (e.g. USB
>>   controller, SD/MMC controller) or disk index. The partition index starts
>>   from ``1`` and describes the partition number on the particular device.
>>
>> +
>> +Alternate Partition Names
>> +^^^^^^^^^^^^^^^^^^^^^^^^^
>> +
>> +Partitions may also be specified like::
>> +
>> +    devnum.hwpartnum#partname
> 
> Thank you for getting this all documented.
> 
> Don't we need the interface (mmc), too?

No. At the moment only mmc is supported. I suppose I could build in some
forward-compatibility and have something like

     iface,devnum.hwpartnum#partname

It should be relatively easy to extend the MMC code to be block-generic,
but it's out of scope for this series.

> 
>> +
>> +or like::
>> +
>> +    devnum.hwpartnum:partnum
> 
> I think this is the wrong place to document how partitions are addressed
> as it is not Android specific. I would prefer a sub-chapter of doc/usage/.

Sure. Though atm only fastboot and android a/b uses the #partname syntax
as far as I can tell.

--Sean

> 
> Cf.
> https://u-boot.readthedocs.io/en/latest/usage/index.html
> 
> Best regards
> 
> Heinrich
> 
>> +
>> +Where
>> +
>> +  * ``devnum`` is the MMC device number. This defaults to 0.
>> +  * ``hwpartnum`` is the hardware partition number. This defaults to 0 (the user
>> +    partition on eMMC devices).
>> +  * ``partname`` is the partition name on GPT devices. Partitions do not have
>> +    names on MBR devices.
>> +  * ``partnum`` is the partition number, starting from 1. The partition number 0
>> +    is special, and specifies that the whole device is to be used as one
>> +    "partition."
>> +
>> +If neither ``partname`` nor ``partnum`` is specified and there is a partition
>> +table, then partition 1 is used. If there is no partition table, then the whole
>> +device is used as one "partion." Examples of alternate partition names include
>> +``0.1``, ``0#boot``, and ``:3``.
>> +
>>   Writing Partition Table
>>   -----------------------
>>
>>
> 

  reply	other threads:[~2021-01-01  1:54 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-31 22:48 [PATCH 0/9] This series adds support for flashing eMMC boot partitions, and for Sean Anderson
2020-12-31 22:48 ` [PATCH 1/9] mmc: sandbox: Add support for writing Sean Anderson
2021-01-07 12:35   ` Simon Glass
2020-12-31 22:48 ` [PATCH 2/9] test: dm: Add test for fastboot mmc partition naming Sean Anderson
2021-01-07 12:35   ` Simon Glass
2020-12-31 22:48 ` [PATCH 3/9] part: Give several functions more useful return values Sean Anderson
2021-01-07 12:35   ` Simon Glass
2020-12-31 22:48 ` [PATCH 4/9] part: Support getting whole disk from part_get_info_by_dev_and_name_or_num Sean Anderson
2021-01-07 12:35   ` Simon Glass
2021-01-07 14:12     ` Sean Anderson
2020-12-31 22:48 ` [PATCH 5/9] part: Support string block devices in part_get_info_by_dev_and_name Sean Anderson
2021-01-07 12:35   ` Simon Glass
2021-01-07 14:18     ` Sean Anderson
2020-12-31 22:48 ` [PATCH 6/9] fastboot: Remove mmcpart argument from raw_part_get_info_by_name Sean Anderson
2021-01-07 12:35   ` Simon Glass
2020-12-31 22:48 ` [PATCH 7/9] fastboot: Move part_get_info_by_name_or_alias after raw_part_get_info_by_name Sean Anderson
2021-01-07 12:35   ` Simon Glass
2020-12-31 22:48 ` [PATCH 8/9] fastboot: Allow u-boot-style partitions Sean Anderson
2021-01-04 16:53   ` Sean Anderson
2021-01-07 12:36     ` Simon Glass
2020-12-31 22:48 ` [PATCH 9/9] fastboot: Document alternate partition names Sean Anderson
2021-01-01  0:36   ` Heinrich Schuchardt
2021-01-01  1:54     ` Sean Anderson [this message]
2021-01-07 12:36   ` 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=2164fab6-92b8-224b-4538-05998abca7c2@seco.com \
    --to=sean.anderson@seco.com \
    --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.