On Wed, Jun 7, 2017 at 11:35 AM, Alexandre Torgue <alexandre.torgue@xxxxxx> wrote: > Hi Rob, > > Gentle ping. > > On 04/12/2017 03:31 PM, Alexandre Torgue wrote: >> >> Hi Rob, >> >> On 04/10/2017 10:27 PM, Rob Herring wrote: >>> >>> On Fri, Apr 07, 2017 at 02:43:00PM +0200, Alexandre TORGUE wrote: >>>> >>>> STM32F4 MCU series is composed of several SOC (STM32F429, STM32F469, >>>> ...). >>>> Most of muxing definition are identical. So to avoid to duplicate >>>> bindings >>>> definition, this patch create common definitions. >>> >>> >>> This is a lot of churn. Some confirmation that the resultant dtb is the >>> same before and after would be nice. Perhaps the script you used to >>> convert this as well. >> >> >> I tried to use fdtdump but it seems bugged. So I used directly dtc >> binary to (re)generate dts files (before and after apply the series) and >> I compared "pinmux" field in both case. >> >> Example on stm32f469-disco: >> >> ./scripts/dtc/dtc -I dtb -O dts -o stm32f469-disco-after.dts >> stm32f469-disco-after.dtb >> >> ./scripts/dtc/dtc -I dtb -O dts -o stm32f469-disco-before.dts >> stm32f469-disco-before.dtb >> >> cat stm32f469-disco-after.dts | grep pinmux >> cat stm32f469-disco-before.dts | grep pinmux >> > > Do you agree with verifications ? I would expect you do "diff -u stm32f469-disco-before.dts stm32f469-disco-after.dts". Rob -- 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
