Re: [microblaze-linux] BRAM Kernel Usage in recent kernels

Steven J. Magnani wrote:
> On Fri, 2011-04-29 at 15:45 -0400, Alejandro Cabrera wrote:
>> Hi
>>
>> Do anyone know if kernel code/data are placed into BRAMs after booting ? 
>> I want to know what is the kernel minimum BRAM memory requirement.
>>
>> Kernel 2.6.20 (petalinux-0.4-final) has config parameters that allow the 
>> kernel BRAM usage, but in recent kernels, >= 2.6.31 I dont see it. Even 
>> I found a patch at mainline that allow system without BRAM at all....

Some kernel parts were moved to the BRAM to speedup networking and maybe others 
but it can't go to the mainline because it is just hack.

The second mainline patch is for people who want to completely remove bram. Main 
memory is place from address 0x0 and there is necessary do some things to 
support it. That's why there is this option.

>>
>> What are the reason to remove the config parameters ??

just hack which is not acceptable for mainline.

>>
>> best regards
>> Alejandro
>>
> 
> Hi Alejandro -
> 

[microblaze-linux] Watchdog device driver

Hi
I had been developing a watchdog device driver for the 
"xps_timebase_wdt" core.
How is the process to submit the path ?
Best regards
Alejandro

Participe en Universidad 2012, del 13 al 17 de febrero de 2012.
Habana, Cuba: http://www.congresouniversidad.cu
Consulte la enciclopedia colaborativa cubana. http://www.ecured.cu

Participe en el Segundo Congreso Medio Ambiente Construido y 
Desarrollo Sustentable (MACDES 2011) del 6 al 9 de diciembre de 2011, 
Hotel Nacional, Habana, Cuba: http://macdes.cujae.edu.cu
_______________________________________________
microblaze-linux mailing list
microblaze-linux@...
https://lists.eait.uq.edu.au/mailman/listinfo/microblaze-linux

Re: [microblaze-linux] Watchdog device driver

Hi,

first of all you should send this patch to the mailing list for review.

Please consider if you want to add your driver to the mainline kernel
because you will have to maintain it for the next couple of years.
If you want to try it then copy appropriate Linux subsystem maintainer.

Regards,
Michal

Alejandro Cabrera wrote:
> Hi
> I had been developing a watchdog device driver for the 
> "xps_timebase_wdt" core.
> How is the process to submit the path ?
> Best regards
> Alejandro
> 
> 
> Participe en Universidad 2012, del 13 al 17 de febrero de 2012.
> Habana, Cuba: http://www.congresouniversidad.cu
> Consulte la enciclopedia colaborativa cubana. http://www.ecured.cu
> 
> Participe en el Segundo Congreso Medio Ambiente Construido y Desarrollo 
> Sustentable (MACDES 2011) del 6 al 9 de diciembre de 2011, Hotel 
> Nacional, Habana, Cuba: http://macdes.cujae.edu.cu

--

-- 
Michal Simek, Ing. (M.Eng)

Re: [microblaze-linux] Watchdog device driver

Hi
I have a doubt about the driver itself.

It is about obtaining the "xps_timebase_wdt" clock frequency (to 
calculate the watchdog timeout), I was using the cpuinfo structure but 
when I build the driver as a module, it cannot see the cpuinfo :(. I 
have a proposal about this, add the "clock-frequency" property to the 
"xps_timebase_wdt" parent, allowing it to get the frequency and in 
addition to all cores that need that value (in a future) can obtain it 
through its parent. Is there another way to obtain that value ??

best regards
Alejandro

On 5/23/2011 1:44 AM, Michal Simek wrote:
> Hi,
>
> first of all you should send this patch to the mailing list for review.
>
> Please consider if you want to add your driver to the mainline kernel
> because you will have to maintain it for the next couple of years.
> If you want to try it then copy appropriate Linux subsystem maintainer.
>
> Regards,
> Michal
>
>
> Alejandro Cabrera wrote:
>> Hi
>> I had been developing a watchdog device driver for the 

Re: [microblaze-linux] Watchdog device driver

Hi,

Alejandro Cabrera wrote:
> Hi
> I have a doubt about the driver itself.
> 
> It is about obtaining the "xps_timebase_wdt" clock frequency (to 
> calculate the watchdog timeout), I was using the cpuinfo structure but 
> when I build the driver as a module, it cannot see the cpuinfo :(. I 
> have a proposal about this, add the "clock-frequency" property to the 
> "xps_timebase_wdt" parent, allowing it to get the frequency and in 
> addition to all cores that need that value (in a future) can obtain it 
> through its parent. Is there another way to obtain that value ??

You can add it to dts. It is the easiest way which you can use.

Regards,
Michal

> 
> best regards
> Alejandro
> 
> 
> 
> On 5/23/2011 1:44 AM, Michal Simek wrote:
>> Hi,
>>
>> first of all you should send this patch to the mailing list for review.
>>

Re: [microblaze-linux] Watchdog device driver

Hi
>
> You can add it to dts. It is the easiest way which you can use.
>

Yes, I added it by hand and even I modified the "device_tree.tcl" to 
automatically generate the property....
to be honest I was trying that the property only be added to the 
"xps_timebase_wdt" but it was added to the PLB.... ;) and I think that 
the result of this "mistake" is better that add only that property to 
the wdt due to that it can be used for any slave that need it.
My proposal is add that functionality to the "device_tree" bsp, could be 
it accepted ?

I don't know very well the tcl language...maybe when the watchdog 
related work finish I would start work around device_tree patch...

regards
Alejandro

Participe en Universidad 2012, del 13 al 17 de febrero de 2012.
Habana, Cuba: http://www.congresouniversidad.cu
Consulte la enciclopedia colaborativa cubana. http://www.ecured.cu

Participe en el Segundo Congreso Medio Ambiente Construido y 
Desarrollo Sustentable (MACDES 2011) del 6 al 9 de diciembre de 2011, 
Hotel Nacional, Habana, Cuba: http://macdes.cujae.edu.cu
_______________________________________________
microblaze-linux mailing list
microblaze-linux@...

[microblaze-linux] [PATCH 1/1] Add watchdog device driver

diff -uprN a/CREDITS b/CREDITS
--- a/CREDITS    2011-05-23 18:29:49.000000000 +0100
+++ b/CREDITS    2011-05-23 19:31:51.015097696 +0100
 <at>  <at>  -561,6 +561,12  <at>  <at>  D: misc kernel hacking
  S: Mandriva
  S: Brazil

+N: Alejandro Cabrera
+E: aldaya@...
+D: Xilinx watchdog (xps_timebase_wdt) driver support
+S: CUJAE
+S: Cuba
+
  N: Remy Card
  E: Remy.Card@...
  E: Remy.Card@...
diff -uprN a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
--- a/drivers/watchdog/Kconfig    2011-05-23 17:31:49.000000000 +0100
+++ b/drivers/watchdog/Kconfig    2011-05-23 20:55:30.390818134 +0100
 <at>  <at>  -881,6 +881,20  <at>  <at>  config M54xx_WATCHDOG
        To compile this driver as a module, choose M here: the
        module will be called m54xx_wdt.

+# MicroBlaze Architecture
+
+config XILINX_WATCHDOG
+    tristate "Xilinx XPS_TimeBase_WDT"
+    depends on MICROBLAZE
+    ---help---
+      Watchdog driver for the xps_timebase_wdt ip core.

Re: [microblaze-linux] [PATCH 1/1] Add watchdog device driver

Hi,

Alejandro Cabrera wrote:
> diff -uprN a/CREDITS b/CREDITS
> --- a/CREDITS    2011-05-23 18:29:49.000000000 +0100
> +++ b/CREDITS    2011-05-23 19:31:51.015097696 +0100
>  <at>  <at>  -561,6 +561,12  <at>  <at>  D: misc kernel hacking
>  S: Mandriva
>  S: Brazil
> 
> +N: Alejandro Cabrera
> +E: aldaya@...
> +D: Xilinx watchdog (xps_timebase_wdt) driver support
> +S: CUJAE
> +S: Cuba
> +
>  N: Remy Card
>  E: Remy.Card@...
>  E: Remy.Card@...
> diff -uprN a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> --- a/drivers/watchdog/Kconfig    2011-05-23 17:31:49.000000000 +0100
> +++ b/drivers/watchdog/Kconfig    2011-05-23 20:55:30.390818134 +0100
>  <at>  <at>  -881,6 +881,20  <at>  <at>  config M54xx_WATCHDOG
>        To compile this driver as a module, choose M here: the
>        module will be called m54xx_wdt.
> 
> +# MicroBlaze Architecture
> +
> +config XILINX_WATCHDOG
> +    tristate "Xilinx XPS_TimeBase_WDT"

[microblaze-linux] [PATCH] microblaze: Add Xilinx watchdog timer driver

Participe en Universidad 2012, del 13 al 17 de febrero de 2012.
Habana, Cuba: http://www.congresouniversidad.cu
Consulte la enciclopedia colaborativa cubana. http://www.ecured.cu

Participe en el Segundo Congreso Medio Ambiente Construido y 
Desarrollo Sustentable (MACDES 2011) del 6 al 9 de diciembre de 2011, 
Hotel Nacional, Habana, Cuba: http://macdes.cujae.edu.cu

>From 4996f0b705080ff81d6a7a14652578ac8b098824 Mon Sep 17 00:00:00 2001
From: Alejandro Cabrera <aldaya@...>
Date: Mon, 30 May 2011 21:14:46 +0100
Subject: [PATCH] microblaze: Add Xilinx watchdog timer driver

Watchdog timer device driver for Xilinx xps_timebase_wdt compatible ip cores.
It takes watchdog timer configuration from device tree and it needs that its
parent has defined the property "clock-frecuency".
It is compatible with watchdog timer kernel API, so user apps like watchdogd
may talk with it.

Signed-off-by: Alejandro Cabrera <aldaya@...>
---
 CREDITS                          |    6 +
 MAINTAINERS                      |    6 +
 drivers/watchdog/Kconfig         |   14 ++
 drivers/watchdog/Makefile        |    3 +
 drivers/watchdog/of_xilinx_wdt.c |  433 ++++++++++++++++++++++++++++++++++++++
 5 files changed, 462 insertions(+), 0 deletions(-)
 create mode 100644 drivers/watchdog/of_xilinx_wdt.c