| Message ID | 1381960030-1640-2-git-send-email-tim.kryger@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
On 10/16/2013 11:47 PM, Tim Kryger wrote: > Declare clocks that are enabled and configured by bootloaders as fixed > rate clocks in the DTS such that device drivers may use standard clock > function calls. > > Signed-off-by: Tim Kryger <tim.kryger@linaro.org> > Reviewed-by: Markus Mayer <markus.mayer@linaro.org> > Reviewed-by: Matt Porter <matt.porter@linaro.org> > --- Hi Tim, I was wondering what is the status of this patchset ? The patchset touches different areas maintained by different people. Through what tree do you expect this patchset to be merged ?? Thanks -- Daniel > arch/arm/boot/dts/bcm11351.dtsi | 97 +++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 97 insertions(+) > > diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi > index 98e6bc0..c6464fb 100644 > --- a/arch/arm/boot/dts/bcm11351.dtsi > +++ b/arch/arm/boot/dts/bcm11351.dtsi > @@ -126,4 +126,101 @@ > status = "disabled"; > }; > > + clocks { > + bsc1_clk: bsc1 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + bsc2_clk: bsc2 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + bsc3_clk: bsc3 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + pmu_bsc_clk: pmu_bsc { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + hub_timer_clk: hub_timer { > + compatible = "fixed-clock"; > + clock-frequency = <32768>; > + #clock-cells = <0>; > + }; > + > + pwm_clk: pwm { > + compatible = "fixed-clock"; > + clock-frequency = <26000000>; > + #clock-cells = <0>; > + }; > + > + sdio1_clk: sdio1 { > + compatible = "fixed-clock"; > + clock-frequency = <48000000>; > + #clock-cells = <0>; > + }; > + > + sdio2_clk: sdio2 { > + compatible = "fixed-clock"; > + clock-frequency = <48000000>; > + #clock-cells = <0>; > + }; > + > + sdio3_clk: sdio3 { > + compatible = "fixed-clock"; > + clock-frequency = <48000000>; > + #clock-cells = <0>; > + }; > + > + sdio4_clk: sdio4 { > + compatible = "fixed-clock"; > + clock-frequency = <48000000>; > + #clock-cells = <0>; > + }; > + > + tmon_1m_clk: tmon_1m { > + compatible = "fixed-clock"; > + clock-frequency = <1000000>; > + #clock-cells = <0>; > + }; > + > + uartb_clk: uartb { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + uartb2_clk: uartb2 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + uartb3_clk: uartb3 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + uartb4_clk: uartb4 { > + compatible = "fixed-clock"; > + clock-frequency = <13000000>; > + #clock-cells = <0>; > + }; > + > + usb_otg_ahb_clk: usb_otg_ahb { > + compatible = "fixed-clock"; > + clock-frequency = <52000000>; > + #clock-cells = <0>; > + }; > + }; > }; >
On Fri, Nov 8, 2013 at 2:53 AM, Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > I was wondering what is the status of this patchset ? > > The patchset touches different areas maintained by different people. Through > what tree do you expect this patchset to be merged ?? Hi Daniel, I plan to send an updated version soon to address some concerns raised by Mark. Once all the maintainers are satisfied and provide their acks, I would like Christian to take the series into his tree. I expect Christian will have a few other Broadcom DTS file changes in his tree so sending the series through him should make it easier to avoid/manage conflicts. Christian, would that work for you? Thanks, Tim Kryger
I can pick up the dt related changes as long as the other code works as-is both with and without the change (i.e. no backwards/forwards compatibility issues). If this must be introduced alongside the other changes, then the patches will have to be reworked to break them up by subtree so they can go in atomically. csd On Fri, Nov 8, 2013 at 11:42 AM, Tim Kryger <tim.kryger@linaro.org> wrote: > On Fri, Nov 8, 2013 at 2:53 AM, Daniel Lezcano > <daniel.lezcano@linaro.org> wrote: > >> I was wondering what is the status of this patchset ? >> >> The patchset touches different areas maintained by different people. Through >> what tree do you expect this patchset to be merged ?? > > Hi Daniel, > > I plan to send an updated version soon to address some concerns raised by Mark. > > Once all the maintainers are satisfied and provide their acks, I would > like Christian to take the series into his tree. > > I expect Christian will have a few other Broadcom DTS file changes in > his tree so sending the series through him should make it easier to > avoid/manage conflicts. > > Christian, would that work for you? > > Thanks, > Tim Kryger
diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi index 98e6bc0..c6464fb 100644 --- a/arch/arm/boot/dts/bcm11351.dtsi +++ b/arch/arm/boot/dts/bcm11351.dtsi @@ -126,4 +126,101 @@ status = "disabled"; }; + clocks { + bsc1_clk: bsc1 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + bsc2_clk: bsc2 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + bsc3_clk: bsc3 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + pmu_bsc_clk: pmu_bsc { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + hub_timer_clk: hub_timer { + compatible = "fixed-clock"; + clock-frequency = <32768>; + #clock-cells = <0>; + }; + + pwm_clk: pwm { + compatible = "fixed-clock"; + clock-frequency = <26000000>; + #clock-cells = <0>; + }; + + sdio1_clk: sdio1 { + compatible = "fixed-clock"; + clock-frequency = <48000000>; + #clock-cells = <0>; + }; + + sdio2_clk: sdio2 { + compatible = "fixed-clock"; + clock-frequency = <48000000>; + #clock-cells = <0>; + }; + + sdio3_clk: sdio3 { + compatible = "fixed-clock"; + clock-frequency = <48000000>; + #clock-cells = <0>; + }; + + sdio4_clk: sdio4 { + compatible = "fixed-clock"; + clock-frequency = <48000000>; + #clock-cells = <0>; + }; + + tmon_1m_clk: tmon_1m { + compatible = "fixed-clock"; + clock-frequency = <1000000>; + #clock-cells = <0>; + }; + + uartb_clk: uartb { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + uartb2_clk: uartb2 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + uartb3_clk: uartb3 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + uartb4_clk: uartb4 { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + usb_otg_ahb_clk: usb_otg_ahb { + compatible = "fixed-clock"; + clock-frequency = <52000000>; + #clock-cells = <0>; + }; + }; };