All of lore.kernel.org
 help / color / mirror / Atom feed
From: b32955@freescale.com (Huang Shijie)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 5/6] ARM: dts: imx6dl: add pinctrls for WEIM NOR
Date: Tue, 28 May 2013 13:51:49 +0800	[thread overview]
Message-ID: <51A445F5.4050206@freescale.com> (raw)
In-Reply-To: <60BA5429A0E1584BA3633194F6F993B5025754D1@NA-MBX-03.mgc.mentorg.com>

? 2013?05?25? 01:51, Chaiken, Alison ??:
> n your patch, the address lines are set to 0xb0b1, which is correct, as hysteresis is disabled and SRE is fast for those lines.     However, for data lines, hysteresis is enabled and SRE is slow, so the 0x1b0b0 setting is more appropriate.
0xb0b1 is the legacy value from the BSP code. But i agree that 0x1b0b0 
is better.
I will use it in next version.

thanks
Huang Shijie

WARNING: multiple messages have this Message-ID (diff)
From: Huang Shijie <b32955-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
To: "Chaiken,
	Alison" <Alison_Chaiken-nmGgyN9QBj3QT0dZR+AlfA@public.gmane.org>
Cc: "shc_work-JGs/UdohzUI@public.gmane.org"
	<shc_work-JGs/UdohzUI@public.gmane.org>,
	"s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org"
	<s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	"rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org"
	<rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	"grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org"
	<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org"
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH v3 5/6] ARM: dts: imx6dl: add pinctrls for WEIM NOR
Date: Tue, 28 May 2013 13:51:49 +0800	[thread overview]
Message-ID: <51A445F5.4050206@freescale.com> (raw)
In-Reply-To: <60BA5429A0E1584BA3633194F6F993B5025754D1-0dz9ie/QGrnnlEkxMdpx1dQH9K4/4qFeAL8bYrjMMd8@public.gmane.org>

于 2013年05月25日 01:51, Chaiken, Alison 写道:
> n your patch, the address lines are set to 0xb0b1, which is correct, as hysteresis is disabled and SRE is fast for those lines.     However, for data lines, hysteresis is enabled and SRE is slow, so the 0x1b0b0 setting is more appropriate.
0xb0b1 is the legacy value from the BSP code. But i agree that 0x1b0b0 
is better.
I will use it in next version.

thanks
Huang Shijie



_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

  reply	other threads:[~2013-05-28  5:51 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-24  9:59 [PATCH v3 0/6] ARM: imx6q{dl}: add the WEIM driver Huang Shijie
2013-05-24  9:59 ` Huang Shijie
2013-05-24  9:59 ` [PATCH v3 1/6] drivers: bus: add a new driver for WEIM Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-27  6:34   ` Shawn Guo
2013-05-27  6:34     ` Shawn Guo
2013-05-27  7:44   ` Sascha Hauer
2013-05-27  7:44     ` Sascha Hauer
2013-05-28  6:02     ` Huang Shijie
2013-05-28  6:02       ` Huang Shijie
2013-05-24  9:59 ` [PATCH v3 2/6] ARM: dts: imx6q{dl}: fix the pin conflict between SPI and WEIM Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-24  9:59 ` [PATCH v3 3/6] ARM: dts: imx6qdl: add more information for WEIM Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-24  9:59 ` [PATCH v3 4/6] ARM: dts: imx6q: add pinctrls for WEIM NOR Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-27  6:48   ` Shawn Guo
2013-05-27  6:48     ` Shawn Guo
2013-05-24  9:59 ` [PATCH v3 5/6] ARM: dts: imx6dl: " Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-24 17:51   ` Chaiken, Alison
2013-05-24 17:51     ` Chaiken, Alison
2013-05-28  5:51     ` Huang Shijie [this message]
2013-05-28  5:51       ` Huang Shijie
2013-05-24  9:59 ` [PATCH v3 6/6] ARM: dts: imx6qdl: enable the " Huang Shijie
2013-05-24  9:59   ` Huang Shijie
2013-05-27  7:44   ` Sascha Hauer
2013-05-27  7:44     ` Sascha Hauer

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=51A445F5.4050206@freescale.com \
    --to=b32955@freescale.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.