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

Groups > linux.kernel > #1400198

Re: [PATCH 0/4] add minimal bcm2835-sdram driver

Show all headers | View raw

Hi,

> Martin Sperl <kernel@martin.sperl.org> hat am 12. Mai 2016 um 17:28
> geschrieben:
> 
> 
> 
> > On 12.05.2016, at 16:50, Stefan Wahren <stefan.wahren@i2se.com> wrote:
> > 
> > Hi Martin,
> > 
> >> kernel@martin.sperl.org hat am 12. Mai 2016 um 14:38 geschrieben:
> >> 
> >> 
> >> From: Martin Sperl <kernel@martin.sperl.org>
> >> 
> >> As the sdram clock is a critical clock to the system
> >> the minimal bcm2835-sdram driver claims (and enables)
> >> this clock and also exposes the corresponding sdram
> >> registers via debugfs.
> > 
> > sounds like this driver should fix an clock handling issue. Unfortunately
> > this
> > isn't a solution in case the driver is disabled.
> Unfortunately there is no way around this - the driver has
> to be enabled so that the sdram clock or the parent pll,
> which typically is plld_core, never gets disabled.
> 
> The only other option would be marking the clock as critical
> for those legacy drivers.

i would prefer this option. Since this would be more a fix we could get this
faster in, it's more clear why we are doing that and less to review.

Did i miss a drawback?

Stefan

> 
> See also the discussions around the clock register for the
> sdram clock, where we have the “normal” clock and some
> pll as well - even if the “normal” clock is disabled, 
> then clearing the sdram register (or the parent) freezes
> the system.
> 
> > 
> > Does the GPU firmware handle the SDRAM controller or is it initialized by
> > bootcode?
> > 
> 
> AFAIK it is enabled in bootcode.bin and - as of now - the firmware
> updates refresh cycles when SOC temperatures change.
> FW checks every 30 seconds unless there is a key set in config.txt,
> which - supposedly - produces a slight impact every 30 seconds to
> the system.
> 
> Martin

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

Thread

[PATCH 0/4] add minimal bcm2835-sdram driver kernel@martin.sperl.org - 2016-05-12 14:40 +0200
  [PATCH 3/4] ARM: dts: bcm2835: add the bcm2835-sdram-controller to the dt kernel@martin.sperl.org - 2016-05-12 14:40 +0200
    Re: [PATCH 3/4] ARM: dts: bcm2835: add the bcm2835-sdram-controller  to the dt Stefan Wahren <stefan.wahren@i2se.com> - 2016-05-12 17:00 +0200
      Re: [PATCH 3/4] ARM: dts: bcm2835: add the bcm2835-sdram-controller to the dt Martin Sperl <kernel@martin.sperl.org> - 2016-05-12 18:10 +0200
  [PATCH 1/4] dt: bindings: add bcm2835-memory-controller documentation kernel@martin.sperl.org - 2016-05-12 14:40 +0200
    Re: [PATCH 1/4] dt: bindings: add bcm2835-memory-controller  documentation Rob Herring <robh@kernel.org> - 2016-05-16 18:10 +0200
  [PATCH 2/4] memory: bcm2835: add bcm2835-memory controller kernel@martin.sperl.org - 2016-05-12 14:40 +0200
  [PATCH 4/4] ARM: bcm2835_defconfig: add bcm2835-sdram controller kernel@martin.sperl.org - 2016-05-12 14:40 +0200
  Re: [PATCH 0/4] add minimal bcm2835-sdram driver Stefan Wahren <stefan.wahren@i2se.com> - 2016-05-12 17:00 +0200
    Re: [PATCH 0/4] add minimal bcm2835-sdram driver Martin Sperl <kernel@martin.sperl.org> - 2016-05-12 17:30 +0200
      Re: [PATCH 0/4] add minimal bcm2835-sdram driver Stefan Wahren <stefan.wahren@i2se.com> - 2016-05-12 18:00 +0200
        Re: [PATCH 0/4] add minimal bcm2835-sdram driver Martin Sperl <kernel@martin.sperl.org> - 2016-05-12 18:10 +0200
  Re: [PATCH 0/4] add minimal bcm2835-sdram driver Eric Anholt <eric@anholt.net> - 2016-05-12 20:20 +0200
    Re: [PATCH 0/4] add minimal bcm2835-sdram driver Martin Sperl <kernel@martin.sperl.org> - 2016-05-12 21:50 +0200

csiph-web