On 01/04/2016 05:58 PM, Simon Glass wrote:
Hi Stephen,
On 4 January 2016 at 13:02, Stephen Warren <swar...@wwwdotorg.org> wrote:
On 12/29/2015 01:47 AM, Przemyslaw Marczak wrote:
Hello Stephen,
On 12/16/2015 07:53 PM, Stephen Warren wrote:
On 12/15/2015 09:32 AM, Przemyslaw Marczak wrote:
commit: dm: core: Enable optional use of fdt_translate_address()
enables device's bus/child address translation method, depending
on bus 'ranges' property and including child 'reg' property.
This change makes impossible to decode the 'reg' for node with
'#size-cells' equal to 0.
Such case is possible by the specification and is also used in U-Boot,
e.g. by I2C uclass or S5P GPIO - the last one is broken at present.
Can you please explain the problem you're seeing in more detail? Without
any context, my initial reaction is that this is simply a bug somewhere.
That bug should be fixed, rather than introducing new APIs to hide the
problem.
Some time ago I send a patch with such fix:
[1] https://patchwork.ozlabs.org/patch/537372/
Sorry, I didn't add you to the 'CC' list.
However. I checked this in linux, and the code is the same, the
size-cells == 0 is not supported also in Linux.
The discussion there does indicate that removing the check on #size-cells would
be incorrect.
OK, but since we have a breakage and a release in a week, I'm planning
on picking up Przemyslaw's patch. We can revert it immediately
afterwards.
Who is going to work out a proper solution (post-release)?
I would assume the owner of the affected I2C/SPI controllers, or if the
issue is triggered by core/subsystem code, then the owner of that subsystem.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
