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 V2 4/6] ARM: dts: imx6q: add pinctrl for WEIM NOR
Date: Thu, 23 May 2013 17:27:37 +0800	[thread overview]
Message-ID: <519DE109.1030903@freescale.com> (raw)
In-Reply-To: <1369300440.725629549@f101.mail.ru>

? 2013?05?23? 17:14, Alexander Shiyan ??:
> I suggest remove "nor" label here since it can be used for other devices later.
this pinctrl is just for NOR.

If we attach other device to WEIM, it maybe needs other PAD, such as

MX6Q_PAD_EIM_LBA__EIM_LBA_B.  We need to add another pinctrl then.



>> >  +						fsl,pins =<
>> >  +							MX6Q_PAD_EIM_CS0__EIM_CS0_B   0xb0b1
> Since driver can support multiple chipselects it would be nice to
> define EIM_CSX in other place for possible using other chipselects.
>
Ditto.

thanks
Huang Shijie

WARNING: multiple messages have this Message-ID (diff)
From: Huang Shijie <b32955-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
To: Alexander Shiyan <shc_work-JGs/UdohzUI@public.gmane.org>
Cc: s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org,
	grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	Alison_Chaiken-nmGgyN9QBj3QT0dZR+AlfA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH V2 4/6] ARM: dts: imx6q: add pinctrl for WEIM NOR
Date: Thu, 23 May 2013 17:27:37 +0800	[thread overview]
Message-ID: <519DE109.1030903@freescale.com> (raw)
In-Reply-To: <1369300440.725629549-LP/JDf0MyD8edp2WBT/QOw@public.gmane.org>

于 2013年05月23日 17:14, Alexander Shiyan 写道:
> I suggest remove "nor" label here since it can be used for other devices later.
this pinctrl is just for NOR.

If we attach other device to WEIM, it maybe needs other PAD, such as

MX6Q_PAD_EIM_LBA__EIM_LBA_B.  We need to add another pinctrl then.



>> >  +						fsl,pins =<
>> >  +							MX6Q_PAD_EIM_CS0__EIM_CS0_B   0xb0b1
> Since driver can support multiple chipselects it would be nice to
> define EIM_CSX in other place for possible using other chipselects.
>
Ditto.

thanks
Huang Shijie


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

  reply	other threads:[~2013-05-23  9:27 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-23  8:16 [PATCH V2 0/6] ARM: imx6q{dl}: add the WEIM driver Huang Shijie
2013-05-23  8:16 ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 1/6] drivers: bus: add a new driver for WEIM Huang Shijie
2013-05-23  8:16   ` Huang Shijie
2013-05-23  8:46   ` Alexander Shiyan
2013-05-23  8:46     ` Alexander Shiyan
2013-05-23  9:02     ` Huang Shijie
2013-05-23  9:02       ` Huang Shijie
2013-05-23  9:23   ` Sascha Hauer
2013-05-23  9:23     ` Sascha Hauer
2013-05-23  9:35     ` Huang Shijie
2013-05-23  9:35       ` Huang Shijie
2013-05-23  9:53       ` Arnd Bergmann
2013-05-23  9:53         ` Arnd Bergmann
2013-05-23 11:55         ` Sascha Hauer
2013-05-23 11:55           ` Sascha Hauer
2013-05-23 13:09           ` Arnd Bergmann
2013-05-23 13:09             ` Arnd Bergmann
2013-05-24  7:16             ` Huang Shijie
2013-05-24  7:16               ` Huang Shijie
2013-05-24  7:19               ` Sascha Hauer
2013-05-24  7:19                 ` Sascha Hauer
2013-05-23 10:19   ` Russell King - ARM Linux
2013-05-23 10:19     ` Russell King - ARM Linux
2013-05-24  2:42     ` Huang Shijie
2013-05-24  2:42       ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 2/6] ARM: dts: imx6q{dl}: fix the pin conflict between SPI and WEIM Huang Shijie
2013-05-23  8:16   ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 3/6] ARM: dts: imx6qdl: add more information for WEIM Huang Shijie
2013-05-23  8:16   ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 4/6] ARM: dts: imx6q: add pinctrl for WEIM NOR Huang Shijie
2013-05-23  8:16   ` Huang Shijie
2013-05-23  9:14   ` Alexander Shiyan
2013-05-23  9:14     ` Alexander Shiyan
2013-05-23  9:27     ` Huang Shijie [this message]
2013-05-23  9:27       ` Huang Shijie
2013-05-23  9:39       ` Re[2]: " Alexander Shiyan
2013-05-23  9:39         ` Alexander Shiyan
2013-05-23  9:52         ` Huang Shijie
2013-05-23  9:52           ` Huang Shijie
2013-05-23  9:55           ` Re[2]: " Alexander Shiyan
2013-05-23  9:55             ` Alexander Shiyan
2013-05-24  2:41             ` Huang Shijie
2013-05-24  2:41               ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 5/6] ARM: dts: imx6dl: add a " Huang Shijie
2013-05-23  8:16   ` Huang Shijie
2013-05-23  8:16 ` [PATCH V2 6/6] ARM: dts: imx6qdl: enable the " Huang Shijie
2013-05-23  8:16   ` Huang Shijie

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=519DE109.1030903@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.