Re: [PATCH] mmc: card: Do not scan boot partitions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 20, 2012 at 7:38 PM, Chris Ball <cjb@xxxxxxxxxx> wrote:
> On Wed, Jun 20 2012, Linus Walleij wrote:
>>
>> Like we add a Kconfig bool option "support partition scanning on boot
>> partitions" and then we default that to "n", so systems that really
>> want to do that can turn it on the day they need it.
>
> I'm not so happy with adding Kconfig options for configuring driver
> behavior -- it's usually a failure to just do the right thing.

This is not for drivers but for the core. But I get the idea...

> If we think that people might want to use a partition table on their
> boot partitions then we should just support doing that, as we do now.

OK in that case I think it's better to disallow partition tables
on the boot partitions, then the day someone has a valid usecase
we can add it back.

So then the patch is good as it is to me:
Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux