All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
To: Tony Lindgren <tony@atomide.com>
Cc: "Nishanth Menon" <nm@ti.com>, "Paul Walmsley" <paul@pwsan.com>,
	"Aaro Koskinen" <aaro.koskinen@iki.fi>,
	"Sebastian Reichel" <sre@kernel.org>,
	pavel@ucw.cz, "Pali Rohár" <pali.rohar@gmail.com>,
	linux-omap@vger.kernel.org,
	"Brian Hutchinson" <b.hutchman@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	"Roger Quadros" <rogerq@ti.com>
Subject: Re: [PATCH 2/2] memory: omap-gpmc: Add Kconfig option for debug
Date: Wed, 6 Jan 2016 20:39:16 +0200	[thread overview]
Message-ID: <568D5F54.7040900@gmail.com> (raw)
In-Reply-To: <20160106182650.GM12777@atomide.com>

Hi,

On  6.01.2016 20:26, Tony Lindgren wrote:

>
> Hmm. Care to verify that your onenand really gets detected at 83 MHz like
> your earlier logs show? Below is a patch that should show it.

before the corruption appeared, I looked a couple of times in syslog and 
the freq there was 83MHz. Including after I disabled CONFIG_OMAP_GPMC_DEBUG.

>
> Also, do things now work reliably for you with CONFIG_OMAP_GPMC_DEBUG
> enabled? Or does that also produce corruption after few reboots?

CONFIG_OMAP_GPMC_DEBUG is disabled, shall I enable it?

> --- a/arch/arm/mach-omap2/gpmc-onenand.c
> +++ b/arch/arm/mach-omap2/gpmc-onenand.c
> @@ -153,6 +153,8 @@ static int omap2_onenand_get_freq(struct omap_onenand_platform_data *cfg,
>   		freq = 0;
>   	}
>
> +	pr_info("onenand rate detected: %i\n", freq);
> +
>   	return freq;
>   }
>

Where am I supposed to get the output from ^^^ if rootfs become 
corrupted? The problem appears after poweroff/restart when it is already 
too late to get the syslog.

BTW, did you compare all the GPMC registers with and without 
HWMOD_INIT_NO_RESET?

Regards,
Ivo

WARNING: multiple messages have this Message-ID (diff)
From: ivo.g.dimitrov.75@gmail.com (Ivaylo Dimitrov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] memory: omap-gpmc: Add Kconfig option for debug
Date: Wed, 6 Jan 2016 20:39:16 +0200	[thread overview]
Message-ID: <568D5F54.7040900@gmail.com> (raw)
In-Reply-To: <20160106182650.GM12777@atomide.com>

Hi,

On  6.01.2016 20:26, Tony Lindgren wrote:

>
> Hmm. Care to verify that your onenand really gets detected at 83 MHz like
> your earlier logs show? Below is a patch that should show it.

before the corruption appeared, I looked a couple of times in syslog and 
the freq there was 83MHz. Including after I disabled CONFIG_OMAP_GPMC_DEBUG.

>
> Also, do things now work reliably for you with CONFIG_OMAP_GPMC_DEBUG
> enabled? Or does that also produce corruption after few reboots?

CONFIG_OMAP_GPMC_DEBUG is disabled, shall I enable it?

> --- a/arch/arm/mach-omap2/gpmc-onenand.c
> +++ b/arch/arm/mach-omap2/gpmc-onenand.c
> @@ -153,6 +153,8 @@ static int omap2_onenand_get_freq(struct omap_onenand_platform_data *cfg,
>   		freq = 0;
>   	}
>
> +	pr_info("onenand rate detected: %i\n", freq);
> +
>   	return freq;
>   }
>

Where am I supposed to get the output from ^^^ if rootfs become 
corrupted? The problem appears after poweroff/restart when it is already 
too late to get the syslog.

BTW, did you compare all the GPMC registers with and without 
HWMOD_INIT_NO_RESET?

Regards,
Ivo

  reply	other threads:[~2016-01-06 18:39 UTC|newest]

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-20 21:21 [PATCH 0/2] omap gpmc changes for parsing devices and working debug Tony Lindgren
2015-05-20 21:21 ` Tony Lindgren
2015-05-20 21:21 ` [PATCH 1/2] memory: omap-gpmc: Fix parsing of devices Tony Lindgren
2015-05-20 21:21   ` Tony Lindgren
2015-05-20 21:21 ` [PATCH 2/2] memory: omap-gpmc: Add Kconfig option for debug Tony Lindgren
2015-05-20 21:21   ` Tony Lindgren
2015-05-20 22:50   ` Paul Walmsley
2015-05-20 22:50     ` Paul Walmsley
2015-05-20 22:56     ` Tony Lindgren
2015-05-20 22:56       ` Tony Lindgren
2015-05-21  1:06       ` Paul Walmsley
2015-05-21  1:06         ` Paul Walmsley
2015-08-27  6:25   ` Hannes Schmelzer
2015-08-27  6:25     ` Hannes Schmelzer
     [not found]   ` <OFCA2F1DCE.C787A961-ONC1257EAE.001D79BC-C1257EAE.00203AFF@br-automation.com>
2015-08-27 16:59     ` Tony Lindgren
2015-08-27 16:59       ` Tony Lindgren
2015-08-28  4:44       ` Hannes Schmelzer
2015-08-28  4:44         ` Hannes Schmelzer
2015-09-01 12:35     ` Roger Quadros
2015-09-01 12:35       ` Roger Quadros
2015-09-01 13:31       ` Antwort: " Hannes Schmelzer
2015-09-01 13:31         ` Hannes Schmelzer
2015-09-02 14:43         ` Roger Quadros
2015-09-02 14:43           ` Roger Quadros
2015-09-01 12:35     ` Roger Quadros
2015-09-01 12:35       ` Roger Quadros
2016-01-01 11:29   ` Ivaylo Dimitrov
2016-01-01 11:29     ` Ivaylo Dimitrov
2016-01-04 17:02     ` Tony Lindgren
2016-01-04 17:02       ` Tony Lindgren
2016-01-04 17:34       ` Pali Rohár
2016-01-04 17:34         ` Pali Rohár
2016-01-04 17:40         ` Tony Lindgren
2016-01-04 17:40           ` Tony Lindgren
2016-01-04 18:59           ` Ivaylo Dimitrov
2016-01-04 18:59             ` Ivaylo Dimitrov
2016-01-05  4:13             ` Tony Lindgren
2016-01-05  4:13               ` Tony Lindgren
2016-01-05  8:49               ` Pali Rohár
2016-01-05  8:49                 ` Pali Rohár
2016-01-05 22:49                 ` Tony Lindgren
2016-01-05 22:49                   ` Tony Lindgren
2016-01-06  8:55                   ` Ivaylo Dimitrov
2016-01-06  8:55                     ` Ivaylo Dimitrov
2016-01-06  9:05                     ` Pali Rohár
2016-01-06  9:05                       ` Pali Rohár
2016-01-06 16:44                       ` Tony Lindgren
2016-01-06 16:44                         ` Tony Lindgren
2016-01-06 17:36                   ` Aaro Koskinen
2016-01-06 17:36                     ` Aaro Koskinen
2016-01-06 17:40                   ` Sebastian Reichel
2016-01-06 17:40                     ` Sebastian Reichel
2016-01-06 17:47                     ` Tony Lindgren
2016-01-06 17:47                       ` Tony Lindgren
2016-01-06 18:01                       ` Ivaylo Dimitrov
2016-01-06 18:01                         ` Ivaylo Dimitrov
2016-01-06 18:26                         ` Tony Lindgren
2016-01-06 18:26                           ` Tony Lindgren
2016-01-06 18:39                           ` Ivaylo Dimitrov [this message]
2016-01-06 18:39                             ` Ivaylo Dimitrov
2016-01-07 18:07                             ` Tony Lindgren
2016-01-07 18:07                               ` Tony Lindgren
2016-01-07 21:45                               ` Ivaylo Dimitrov
2016-01-07 21:45                                 ` Ivaylo Dimitrov
2016-01-08  2:26                                 ` Tony Lindgren
2016-01-08  2:26                                   ` Tony Lindgren
2016-01-08  5:13                                   ` Ivaylo Dimitrov
2016-01-08  5:13                                     ` Ivaylo Dimitrov
2016-01-08  7:59                                     ` Pali Rohár
2016-01-08  7:59                                       ` Pali Rohár
2016-01-09  0:23                                       ` Ivaylo Dimitrov
2016-01-09  0:23                                         ` Ivaylo Dimitrov
2016-01-21  9:14                                         ` Pali Rohár
2016-01-21  9:14                                           ` Pali Rohár
2016-02-02  9:33                                           ` Ivaylo Dimitrov
2016-02-02  9:33                                             ` Ivaylo Dimitrov
2016-02-02 23:39                                             ` Tony Lindgren
2016-02-02 23:39                                               ` Tony Lindgren
2016-02-03  0:00                                               ` Tony Lindgren
2016-02-03  0:00                                                 ` Tony Lindgren
2016-02-03  7:03                                                 ` Ivaylo Dimitrov
2016-02-03  7:03                                                   ` Ivaylo Dimitrov
2016-02-03 16:50                                                   ` Ivaylo Dimitrov
2016-02-03 16:50                                                     ` Ivaylo Dimitrov
2016-02-05  6:10                                                     ` Tony Lindgren
2016-02-05  6:10                                                       ` Tony Lindgren
2016-02-05 14:43                                                       ` Ivaylo Dimitrov
2016-02-05 14:43                                                         ` Ivaylo Dimitrov
2016-01-08 17:10                                     ` Tony Lindgren
2016-01-08 17:10                                       ` Tony Lindgren
2016-01-08  7:56                                   ` Pali Rohár
2016-01-08  7:56                                     ` Pali Rohár
2016-01-08 17:04                                     ` Tony Lindgren
2016-01-08 17:04                                       ` Tony Lindgren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=568D5F54.7040900@gmail.com \
    --to=ivo.g.dimitrov.75@gmail.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=b.hutchman@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=pali.rohar@gmail.com \
    --cc=paul@pwsan.com \
    --cc=pavel@ucw.cz \
    --cc=rogerq@ti.com \
    --cc=sre@kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.