On Mon, 15 Jun 2015 10:42:13 +0300 Antony Pavlov <antonynpavlov@xxxxxxxxx> wrote: > On Wed, 10 Jun 2015 23:58:11 +0200 > Alban <albeu@xxxxxxx> wrote: > > > On Mon, 8 Jun 2015 13:17:58 +0300 > > Antony Pavlov <antonynpavlov@xxxxxxxxx> wrote: > > > > > IMHO AR9132 SoC can't work without external oscilator. > > > > > > Can we just move basic extosc declaration to SoC dt file > > > (ar9132.dtsi)? So board dt file ar9132_tl_wr1043nd_v1.dts will > > > contain only oscilator clock frequency value. > > > > I would prefer to keep the split between the files in sync with the > > hardware. I understand that most simple board designs use a fixed > > oscillator, but that might not always be the case. > > > > The AR9132 SoC __always__ use one external oscilator. Yes, but what I don't like is to impose the clock source being a fixed-oscillator. What if the board use a clock from another component that need to be represented in the DT as something else than a fixed-oscillator? > So it's reasonable to have the first mention of extosc in > ar9132.dtsi not in a board file. This description style is always > sync with hardware. In your proposal it wouldn't as the AR9132 doesn't have a fixed-oscillator on chip. So boards using another type of clock would still have that fixed-oscillator hanging around. > On the other hand pll-controller is always part > of the SoC not a part of a board. So pll-controller on extosc > dependency have to go to SoC dts file not to a board file. In your dts > description pll-controller is a part of a dts board file. The PLL controller shows up in the board DTS as the connection between the SoC and the other components on boards has to be represented. We could use a label and reference it in the board file but that is the same in the end. But that's not really the point, more important is the fact that DTS don't allow to delete nodes. This mean that DTSI should really not start to define more than is strictly needed. Alban
