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

Groups > linux.kernel > #1400152

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

Show all headers | View raw

> 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.

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