Re: [PATCH] mtd: part: add generic parsing of linux,part-probe

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

 




On 19 May 2015 at 23:59, Hauke Mehrtens <hauke@xxxxxxxxxx> wrote:
>> So my first Idea would be having something like this:
>>
>> mtdevice@foo {
>>         ...
>>         bootloader: partition@0 {
>>                 reg = <0 0x10000>;
>>                 device_type = "flash-partition";
>>                 read-only;
>>         };
>>
>>         firmware: partition@10000 {
>>                 reg = <0x1000 0x7e0000>;
>>                 device_type = "flash-partition";
>>                 compatible = "brcm,trx";
>>         };
>>
>>         calibration: partition@7f0000 {
>>                 reg = < >;
>>                 device_type = "flash-partition";
>>                 read-only;
>>         };
>> };
>>
>> Then the ofpart parsing could be integrated into the core, and
>> partition parsers get run according to their compatibles on the
>> individual partitions.
>>
>> Of course this also might be total overengineering and completely the
>> wrong place for devicetree as it's essentially configuration data, but
>> since the now described partitions are now more or less "static", IMHO
>> this would be a-okay.
>
> This sounds like the problem Rafał wanted to solve in his patch "mtd:
> add support for typed parsers splitting partitions"
>
> This would also solve my problem and would make the bcm47xx part parser
> a lot simpler and more secure for targets using device tree.
> I still think we should add a alternative for the people that do not
> want to rewrite their partition parser. Specifying the list of available
> partition parser in the flash driver is a bad idea because it is the
> wrong place.

This is what I hoped for all the time. To get both patches accepted
(linux,part-probe & typed parsers). That would give us a really nice
way to handle partitioning on every device.


> I am in favor of both concepts, because there is already one driver
> reading the partition parser list from device tree and it is implemented
> pretty easy. Jonas Approach sounds nice because it really solves two
> more problems.
> Jonas approach would work on bcm47xx and bcm53xx targets, because there
> the header is located after the boot loader and does not contain all
> partitions. This could also be used to divide the partition containing
> the kernel and the read only root fs in OpenWrt. Are there any more
> users which could use this extended approach?

I got a bit lost with above. The patch adding typed parsers I sent
will allow us to write both: TRX parser (splitter) and SquashFS parser
(splitter). This is all we need in OpenWrt and I my patch is highly
based on OpenWrt version obviously too.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux