Hello Javier, On 26 June 2014 20:51, Javier Martinez Canillas <javier@xxxxxxxxxxxx> wrote: > Hello Naveen, > > On Thu, Jun 26, 2014 at 2:19 PM, Naveen Krishna Chatradhi > <ch.naveen@xxxxxxxxxxx> wrote: >> This patchset does the following >> 1. Create a common dtsi file cros-exynos-peach.dtsi for >> exynos5420-peach-pit.dts and exynos5800-peach-pi.dts > > There was some previous discussion in this list about what's the best > approach to handle common DTS chunks, please take a look to > http://patchwork.ozlabs.org/patch/362633/. > > In summary, there is a common .dtsi file for Daisy based boards > (arch/arm/boot/dts/exynos5250-cros-common.dtsi) but it seems it turned > out to do more harm than good since having a single .dtsi for all the > common DTS chunks is not quite flexible. As more boards gets > introduced you have to start moving stuff from the common .dtsi file > to the board .dts. > > The same was tried to do for OMAP2+ boards and it turned out that you > need to either a) create a hierarchy of .dtsi files to model all the > different board combinations that have similar fragments or b) split > out isolated DTS fragments on an .dtsi and include a set of this > fragments. > > For an example of a) you can take a look to > arch/arm/boot/dts/omap3-overo* and for b) to > arch/arm/boot/dts/omap-gpmc-smsc9*.dtsi. > > For a Peach Pit/Pi specific example of b) you can look at the recent > arch/arm/boot/dts/cros-ec-keyboard.dtsi file that Doug added for Peach > Pit/Pi keyboard. Right. It makes sense to have multiple smaller chunks that can be included in more boards than having one big chunk which can only be used for 2 or 3 boards. But, then i was unsure. If Device Tree communities likes having more smaller fragment files under one "arch/arm/boot/dts/ directory". > > Personally I think that b) is a more flexible and reusable approach. > So, maybe instead of creating a cros-exynos-peach.dts to contain all > the common DT nodes you can add a cros-ec-thermistor.dsti file that > only includes the ADC based Thermistor nodes? I've started with making a fragment dts naming it "cros-exynos-adc.dts". Then moved on with an implementation similar to exynos5250-cros-common.dtsi > > Thanks a lot and best regards, > Javier Will wait for few more opinions and make a fragment instead of common dtsi. Thank you a lot for the information and the references. > >> 2. Adds the ADC based Thermistor nodes and enables them in peach_pit.dts >> and peach_pi.dts >> 3. Adds the ADC based Thermistor nodes for Exynos5250 based Snow >> 4. Corrects the vendor prefix for thermistors in exynos4412-trats2.dts >> >> Naveen Krishna Chatradhi (3): >> ARM: DTS: use new compatible string for thermistors in trats2 >> ARM: DTS: Add NTC thermistor nodes to Exynos5250 based Snow >> ARM: DTS: Add common dts file for Peach PIT and PI along with ADC >> nodes >> >> arch/arm/boot/dts/cros-exynos-peach.dtsi | 41 ++++++++++++++++++++++++++++ >> arch/arm/boot/dts/exynos4412-trats2.dts | 4 +-- >> arch/arm/boot/dts/exynos5250-snow.dts | 34 +++++++++++++++++++++++ >> arch/arm/boot/dts/exynos5420-peach-pit.dts | 6 ++++ >> arch/arm/boot/dts/exynos5800-peach-pi.dts | 6 ++++ >> 5 files changed, 89 insertions(+), 2 deletions(-) >> create mode 100644 arch/arm/boot/dts/cros-exynos-peach.dtsi >> >> -- >> 1.7.9.5 >> >> -- >> 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 -- Shine bright, (: Nav :) -- 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