On Thu, May 3, 2018 at 8:58 PM, Mathieu Malaterre <malat@xxxxxxxxxx> wrote: > On Thu, May 3, 2018 at 8:53 PM, Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: >> On Thu, May 03, 2018 at 08:44:53PM +0200, Mathieu Malaterre wrote: >>> Improve the DTS files by removing all the leading "0x" and zeros to fix the >>> following dtc warnings: >>> >>> Warning (unit_address_format): Node /XXX unit name should not have leading "0x" >>> >>> and >>> >>> Warning (unit_address_format): Node /XXX unit name should not have leading 0s >>> >>> Converted using the following command: >>> >>> find . -type f \( -iname *.dts -o -iname *.dtsi \) -exec sed -i -e "s/@\([0-9a-fA-FxX\.;:#]\+\)\s*{/@\L\1 {/g" -e "s/@0x\(.*\) {/@\1 {/g" -e "s/@0\+\(.*\) {/@\1 {/g" {} + >> >> The firmware node is fixed but in my case the command is doing much >> more: > > Looks like I cannot copy/paste correctly, meant to say: > > find . -type f \( -iname "*.dts" -o -iname "*.dtsi" \) -exec sed -i > -e "s/@\([0-9a-fA-FxX\.;:#]\+\)\s*{/@\L\1 {/g" -e "s/@0x\(.*\) {/@\1 > {/g" -e "s/@0\+\(.\+\) {/@\1 {/g" {} + I split it with backslash and this worked fine. I'll update the commit message, thanks! BR, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html