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

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

 



Hi,

On Wed, Jun 20 2012, Linus Walleij wrote:
> On Wed, Jun 20, 2012 at 9:27 AM, Johan Rudholm <jrudholm@xxxxxxxxx> wrote:
>> 2012/6/20 Chris Ball <cjb@xxxxxxxxxx>:
>>>
>>> Would it be impossible to add a partition table to mmcblkXbootY,
>>> or just unlikely?
>>
>> I believe this is just an unlikely scenario since the boot partitions
>> are supposed to contain boot code and also because of their relatively
>> small size (a few MBs), but there is no technical problem. Perhaps it
>> should be left up to the user?
>
> 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.

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.

In fact, the original patch submission for supporting boot partitions
used a Kconfig entry for turning on boot partition support, but we
decided to ditch the Kconfig entry and made it just happen by default.
I think we should continue that precedent and avoid a new entry here.

Thanks,

- Chris.
-- 
Chris Ball   <cjb@xxxxxxxxxx>   <http://printf.net/>
One Laptop Per Child
--
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