All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Osipenko <digetx@gmail.com>
To: Ulf Hansson <ulf.hansson@linaro.org>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Jens Axboe <axboe@kernel.dk>
Cc: "Thierry Reding" <thierry.reding@gmail.com>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Michał Mirosław" <mirq-linux@rere.qmqm.pl>,
	"David Heidelberg" <david@ixit.cz>,
	"Peter Geis" <pgwipeout@gmail.com>,
	"Nicolas Chauvet" <kwizart@gmail.com>,
	"Adrian Hunter" <adrian.hunter@intel.com>,
	"Billy Laws" <blaws05@gmail.com>,
	linux-tegra <linux-tegra@vger.kernel.org>,
	linux-block <linux-block@vger.kernel.org>,
	"Andrey Danin" <danindrey@mail.ru>,
	"Gilles Grandou" <gilles@grandou.net>,
	"Ryan Grachek" <ryan@edited.us>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v1 3/3] partitions: Introduce NVIDIA Tegra Partition Table
Date: Sat, 7 Mar 2020 02:11:46 +0300	[thread overview]
Message-ID: <6047f07b-c0af-08c8-90d1-79a0d880e0a2@gmail.com> (raw)
In-Reply-To: <CAPDyKFric6pZbJ5-2qkwAFoeJ0c0kcha99zHJ12AUrWO6FQmgg@mail.gmail.com>

06.03.2020 16:37, Ulf Hansson пишет:
...
>>
>> Technically, it should be possible to chain-load some custom secondary
>> bootloader instead of a kernel image, but this is not very practical
>> because now:
>>
>> 1. There is a need to make a custom bootloader and it is quite a lot of
>> work.
>>
>> 2. You'll have to tell everybody that a custom booloader may need to be
>> used in order to get a working eMMC.
> 
> Yeah, I get the point. It's not an optimal situation, but I assume
> it's about informing developers. They can cope with this, no?

Perhaps no, it's not only about the informing. The need for a custom
bootloader creates other inconveniences because:

1. It won't be possible to boot a vanilla upstream kernel using
Android's "fastboot boot ..." without applying extra patches to kernel
for the partition table support. Advanced users usually tend to use
fastboot and it's also very useful for a regular development purposes as
well.

2. Somebody (a developer / advanced user) will have to create a custom
bootloader for each device in the first place. This is not what an
average person will be able to do and there are not that many developers
who would want to dedicate theirs time to this.

3. The entry barrier for upstreaming Android devices support to the
kernel is already quite enormous. Adding extra hurdles isn't a step into
the right direction, IMO.

>> 3. NVIDIA's bootloader already passes a command line parameter to kernel
>> for locating GPT entry, but this hack is not acceptable for the upstream
>> kernel.
> 
> Well, I am just worried that we will end up with one partition format
> per vendor/product, that wouldn't scale very well.
> 
> In any case, from mmc point of view I am less concerned, we can find a
> way to support the needed bits. I just need to review the series more
> carefully and provide some comments. :-)
> 
> However, before I do that, I would like to hear Jens opinion about
> adding a new partition format, so I don't waste my time here.

Sure, no problems :) Let's wait for the comments from Jens.

      parent reply	other threads:[~2020-03-06 23:11 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-24 23:18 [PATCH v1 0/3] Introduce NVIDIA Tegra Partition Table Dmitry Osipenko
2020-02-24 23:18 ` Dmitry Osipenko
2020-02-24 23:18 ` [PATCH v1 1/3] mmc: core: Add raw_boot_mult field to mmc_ext_csd Dmitry Osipenko
     [not found]   ` <20200224231841.26550-2-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-03-01 10:50     ` Avri Altman
2020-03-01 10:50       ` Avri Altman
     [not found]       ` <MN2PR04MB699121991FCB80BE39FC106FFCE60-lFdnouXQuT6w2H5UOYkxGlM8qxBPnqtHvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2020-03-01 23:09         ` Dmitry Osipenko
2020-03-01 23:09           ` Dmitry Osipenko
2020-02-24 23:18 ` [PATCH v1 2/3] mmc: block: Add mmc_bdev_to_card() helper Dmitry Osipenko
2020-02-25 14:53   ` Ulf Hansson
2020-02-25 15:46     ` Dmitry Osipenko
     [not found]   ` <20200224231841.26550-3-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-27  0:40     ` kbuild test robot
2020-02-27  0:40       ` kbuild test robot
2020-02-27  0:40       ` kbuild test robot
2020-02-24 23:18 ` [PATCH v1 3/3] partitions: Introduce NVIDIA Tegra Partition Table Dmitry Osipenko
     [not found]   ` <20200224231841.26550-4-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-25  0:20     ` Stephen Warren
2020-02-25  0:20       ` Stephen Warren
2020-02-25  1:35       ` Dmitry Osipenko
     [not found]       ` <44c22925-a14e-96d0-1f93-1979c0c60525-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2020-03-04 16:36         ` Ulf Hansson
2020-03-04 16:36           ` Ulf Hansson
2020-03-04 17:09           ` Dmitry Osipenko
     [not found]             ` <824a4d5f-8280-8860-3e80-68188a13aa3d-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-03-06 13:37               ` Ulf Hansson
2020-03-06 13:37                 ` Ulf Hansson
     [not found]                 ` <CAPDyKFric6pZbJ5-2qkwAFoeJ0c0kcha99zHJ12AUrWO6FQmgg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2020-03-06 16:52                   ` Stephen Warren
2020-03-06 16:52                     ` Stephen Warren
2020-03-06 21:56                     ` Dmitry Osipenko
2020-03-06 23:11                 ` Dmitry Osipenko [this message]

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=6047f07b-c0af-08c8-90d1-79a0d880e0a2@gmail.com \
    --to=digetx@gmail.com \
    --cc=adrian.hunter@intel.com \
    --cc=axboe@kernel.dk \
    --cc=blaws05@gmail.com \
    --cc=danindrey@mail.ru \
    --cc=david@ixit.cz \
    --cc=gilles@grandou.net \
    --cc=jonathanh@nvidia.com \
    --cc=kwizart@gmail.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=mirq-linux@rere.qmqm.pl \
    --cc=pgwipeout@gmail.com \
    --cc=ryan@edited.us \
    --cc=swarren@wwwdotorg.org \
    --cc=thierry.reding@gmail.com \
    --cc=ulf.hansson@linaro.org \
    /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.