This is reimplimentation of D13648
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 27968 Build 26134: arc lint + arc unit
Event Timeline
this is dts setting.
firmware: partition@50000 { compatible = "denx,uimage"; label = "firmware"; reg = <0x00050000 0x00390000>; read-only; };
This change is unacceptable. Child nodes of a flash device which represent partitions must not have a compatible property, it is forbidden by the bindings document. I believe the correct way to do this is that there must be a child node with a compatible of "fixed-partitions", then within that node, there can be child nodes describing partitions which have compatible properties on them. Right now, fdt_slicer doesn't handle that kind of node structure.
Adding the checks for denx,uimage into the slicer in a way that requires it to do IO doesn't seem correct either. The fdt_slicer is for creating partitions that require out-of-band metadata (fdt data) to describe. Finding partitions by reading the content of the media seems like it should be handled the same way as other partitioning schemes that read media and create partitions from what they find (gpt, mbr, etc).