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

Groups > linux.kernel > #1537514

Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate()

From Sekhar Nori <nsekhar@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate()
Date 2016-12-07 07:40 +0100
Message-ID <sLE3w-4PD-5@gated-at.bofh.it> (permalink)
References <sKYnF-3bM-63@gated-at.bofh.it> <sKYxj-3gw-5@gated-at.bofh.it> <sLzGy-1ry-13@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

On Wednesday 07 December 2016 07:24 AM, David Lechner wrote:
> On 12/05/2016 04:09 AM, Bartosz Golaszewski wrote:
>> The aemif clock is added twice to the lookup table in da850.c. This
>> breaks the children list of pll0_sysclk3 as we're using the same list
>> links in struct clk. When calling clk_set_rate(), we get stuck in
>> propagate_rate().
> 
> &emac_clk is used twice in this list as well. Shouldn't we fix it too? I
> would expect that it causes the same problem.

Yes, indeed.

Thanks,
Sekhar

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

Thread

[PATCH v3 0/3] ARM: da850: fix pll0 rate setting Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-05 11:10 +0100
  [PATCH v3 3/3] ARM: da850: fix da850_set_pll0rate() Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-05 11:10 +0100
    Re: [PATCH v3 3/3] ARM: da850: fix da850_set_pll0rate() Sekhar Nori <nsekhar@ti.com> - 2016-12-05 11:50 +0100
    Re: [PATCH v3 3/3] ARM: da850: fix da850_set_pll0rate() David Lechner <david@lechnology.com> - 2016-12-07 03:10 +0100
  Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Sekhar Nori <nsekhar@ti.com> - 2016-12-05 11:20 +0100
    Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-05 11:40 +0100
      Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Sekhar Nori <nsekhar@ti.com> - 2016-12-05 11:50 +0100
    Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-06 13:00 +0100
      Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Sekhar Nori <nsekhar@ti.com> - 2016-12-06 13:30 +0100
  [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-05 11:20 +0100
    Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() David Lechner <david@lechnology.com> - 2016-12-07 03:00 +0100
      Re: [PATCH v3 1/3] ARM: da850: fix infinite loop in clk_set_rate() Sekhar Nori <nsekhar@ti.com> - 2016-12-07 07:40 +0100
  [PATCH v3 2/3] ARM: da850: coding style fix Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-05 11:20 +0100

csiph-web