Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1659772

Re: [PATCH] arm: aspeed: Add clock-names property to timer node

From Daniel Lezcano <daniel.lezcano@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH] arm: aspeed: Add clock-names property to timer node
Date 2017-06-07 15:00 +0200
Message-ID <tPIJ4-6JB-7@gated-at.bofh.it> (permalink)
References <tOUVX-8qA-3@gated-at.bofh.it> <tOWbo-Zl-15@gated-at.bofh.it> <tPjhF-6Wc-27@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, Jun 06, 2017 at 11:41:11AM +0200, Arnd Bergmann wrote:
> On Mon, Jun 5, 2017 at 10:59 AM, Joel Stanley <joel@jms.id.au> wrote:
> > On Mon, Jun 5, 2017 at 5:18 PM, Andrew Jeffery <andrew@aj.id.au> wrote:
> >> The merging of a number of clocksource drivers into fttmr010 means we
> >> require clock-names to be specified in the Aspeed timer node, else the
> >> clocksource fails to probe and boot hangs.
> >
> > Arnd,
> >
> > Linus' reworked timer driver will go into 4.13.
> >
> > Can we get this patch merged into 4.12 as a fix so we don't end up
> > with a broken boot at any stage?
> 
> Hmm, can't we make the driver backward-compatible and have it fall
> back on the first clock if no clk named "PCLK" is found? Otherwise
> you still have an incompatible change in the DT binding and it will
> break if someone uses an older dtb with a newer kernel.

I would like to avoid to hack the kernel code for backward DT compatible
things.



-- 

 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] arm: aspeed: Add clock-names property to timer node Andrew Jeffery <andrew@aj.id.au> - 2017-06-05 09:50 +0200
  Re: [PATCH] arm: aspeed: Add clock-names property to timer node Joel Stanley <joel@jms.id.au> - 2017-06-05 11:10 +0200
    Re: [PATCH] arm: aspeed: Add clock-names property to timer node Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-06-05 23:20 +0200
      Re: [PATCH] arm: aspeed: Add clock-names property to timer node Joel Stanley <joel@jms.id.au> - 2017-06-06 04:20 +0200
    Re: [PATCH] arm: aspeed: Add clock-names property to timer node Arnd Bergmann <arnd@arndb.de> - 2017-06-06 11:50 +0200
      Re: [PATCH] arm: aspeed: Add clock-names property to timer node Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-06-07 15:00 +0200
        Re: [PATCH] arm: aspeed: Add clock-names property to timer node Arnd Bergmann <arnd@arndb.de> - 2017-06-07 16:40 +0200
          Re: [PATCH] arm: aspeed: Add clock-names property to timer node Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-06-08 15:50 +0200
            Re: [PATCH] arm: aspeed: Add clock-names property to timer node Arnd Bergmann <arnd@arndb.de> - 2017-06-08 16:40 +0200
              [PATCH] arm: aspeed: Add Aspeed board file with clocksource devicetree fixup Andrew Jeffery <andrew@aj.id.au> - 2017-06-09 09:40 +0200
                Re: [PATCH] arm: aspeed: Add Aspeed board file with clocksource  devicetree fixup Arnd Bergmann <arnd@arndb.de> - 2017-06-09 11:20 +0200
                Re: [PATCH] arm: aspeed: Add Aspeed board file with clocksource  devicetree fixup Andrew Jeffery <andrew@aj.id.au> - 2017-06-09 15:00 +0200
      Re: [PATCH] arm: aspeed: Add clock-names property to timer node Linus Walleij <linus.walleij@linaro.org> - 2017-06-09 15:50 +0200

csiph-web