> +/ {
> + chosen {
> + stdout-path = &blsp1_uart2;
> + };
It would be good if we had the configuration too (see
Documentation/devicetree/bindings/chosen.txt), as that avoids any
reliance on kernel defaults.
You can refer to an alias, so this could be:
aliases {
serial0 = &blsp1_uart2;
};
chosen {
stdout-path = "serial0:115200n8";
};
...assuming that 115200n8 is correct for your UART, of course.
[...]
> +#include "skeleton.dtsi"
I'd like to get rid of skeleton.dtsi; it causes more problems than it
solves (the address/size cells mismatch is confusing and hidden, people
forget to fill in memory nodes appropriately, etc).
Please remove this include and place appropriate #address-cells and #size-cells
here.
I'd strongly recommend going with /#size-cells = <2>; it' will save on a
lot of pain if you need to add PCIe or something with large ranges
later. If things all fall in 4GB within the SoC then have
/soc/#size=cells = <1> and an appropriate /soc/ranges property.
I note this DT doesn't have any memory nodes. Is that an accident or
does the loader fill that in?
If the latter, have an empty node with a comment to that effect.
Mark.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
