All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bjorn Helgaas <helgaas@kernel.org>
To: honghui.zhang@mediatek.com
Cc: bhelgaas@google.com, robh@kerenl.org, robh+dt@kernel.org,
	matthias.bgg@gmail.com, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org, linux-pci@vger.kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	yingjoe.chen@mediatek.com, eddie.huang@mediatek.com,
	ryder.lee@mediatek.com, hongkun.cao@mediatek.com,
	youlin.pei@mediatek.com, yong.wu@mediatek.com,
	yt.shen@mediatek.com, sean.wang@mediatek.com,
	xinping.qian@mediatek.com
Subject: Re: [PATCH v2 1/5] PCI: mediatek: Add a structure to abstract the controller generations
Date: Thu, 3 Aug 2017 17:15:46 -0500	[thread overview]
Message-ID: <20170803221546.GL20308@bhelgaas-glaptop.roam.corp.google.com> (raw)
In-Reply-To: <948f9bd0881402d13e6913ad425e7dff50f3fcfc.1501122135.git.honghui.zhang@mediatek.com>

On Thu, Jul 27, 2017 at 10:58:35AM +0800, honghui.zhang@mediatek.com wrote:
> From: Ryder Lee <ryder.lee@mediatek.com>
> 
> Introduce a structure "mtk_pcie_soc" to abstract the differences between
> controller generations, and the .startup() hook is used to encapsulate
> some SoC-dependent related setting. In doing so, the common code which
> will be reused by future chips.
> 
> In addition, we change the approaches to waiting Gen2 training by using
> readl_poll_timeout() calls.

Please split the Gen2 training change to a separate patch, since
that's not related to the mtk_pcie_soc changes.

WARNING: multiple messages have this Message-ID (diff)
From: Bjorn Helgaas <helgaas@kernel.org>
To: honghui.zhang@mediatek.com
Cc: youlin.pei@mediatek.com, devicetree@vger.kernel.org,
	hongkun.cao@mediatek.com, ryder.lee@mediatek.com,
	linux-pci@vger.kernel.org, sean.wang@mediatek.com,
	xinping.qian@mediatek.com, linux-kernel@vger.kernel.org,
	robh+dt@kernel.org, yt.shen@mediatek.com, matthias.bgg@gmail.com,
	robh@kerenl.org, linux-mediatek@lists.infradead.org,
	yong.wu@mediatek.com, bhelgaas@google.com,
	yingjoe.chen@mediatek.com, eddie.huang@mediatek.com,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 1/5] PCI: mediatek: Add a structure to abstract the controller generations
Date: Thu, 3 Aug 2017 17:15:46 -0500	[thread overview]
Message-ID: <20170803221546.GL20308@bhelgaas-glaptop.roam.corp.google.com> (raw)
In-Reply-To: <948f9bd0881402d13e6913ad425e7dff50f3fcfc.1501122135.git.honghui.zhang@mediatek.com>

On Thu, Jul 27, 2017 at 10:58:35AM +0800, honghui.zhang@mediatek.com wrote:
> From: Ryder Lee <ryder.lee@mediatek.com>
> 
> Introduce a structure "mtk_pcie_soc" to abstract the differences between
> controller generations, and the .startup() hook is used to encapsulate
> some SoC-dependent related setting. In doing so, the common code which
> will be reused by future chips.
> 
> In addition, we change the approaches to waiting Gen2 training by using
> readl_poll_timeout() calls.

Please split the Gen2 training change to a separate patch, since
that's not related to the mtk_pcie_soc changes.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: helgaas@kernel.org (Bjorn Helgaas)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 1/5] PCI: mediatek: Add a structure to abstract the controller generations
Date: Thu, 3 Aug 2017 17:15:46 -0500	[thread overview]
Message-ID: <20170803221546.GL20308@bhelgaas-glaptop.roam.corp.google.com> (raw)
In-Reply-To: <948f9bd0881402d13e6913ad425e7dff50f3fcfc.1501122135.git.honghui.zhang@mediatek.com>

On Thu, Jul 27, 2017 at 10:58:35AM +0800, honghui.zhang at mediatek.com wrote:
> From: Ryder Lee <ryder.lee@mediatek.com>
> 
> Introduce a structure "mtk_pcie_soc" to abstract the differences between
> controller generations, and the .startup() hook is used to encapsulate
> some SoC-dependent related setting. In doing so, the common code which
> will be reused by future chips.
> 
> In addition, we change the approaches to waiting Gen2 training by using
> readl_poll_timeout() calls.

Please split the Gen2 training change to a separate patch, since
that's not related to the mtk_pcie_soc changes.

  parent reply	other threads:[~2017-08-03 22:15 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-27  2:58 [PATCH v2 0/5] PCI: MediaTek: Add support for new generation host controller honghui.zhang
2017-07-27  2:58 ` honghui.zhang at mediatek.com
2017-07-27  2:58 ` honghui.zhang
2017-07-27  2:58 ` honghui.zhang
2017-07-27  2:58 ` [PATCH v2 1/5] PCI: mediatek: Add a structure to abstract the controller generations honghui.zhang
2017-07-27  2:58   ` honghui.zhang at mediatek.com
2017-07-27  2:58   ` honghui.zhang
2017-07-27  3:19   ` Honghui Zhang
2017-07-27  3:19     ` Honghui Zhang
2017-07-27  3:19     ` Honghui Zhang
2017-07-27  3:19     ` Honghui Zhang
2017-07-27  3:28     ` Honghui Zhang
2017-07-27  3:28       ` Honghui Zhang
2017-07-27  3:28       ` Honghui Zhang
2017-07-27  3:28       ` Honghui Zhang
2017-08-03 22:15   ` Bjorn Helgaas [this message]
2017-08-03 22:15     ` Bjorn Helgaas
2017-08-03 22:15     ` Bjorn Helgaas
2017-07-27  2:58 ` [PATCH v2 2/5] PCI: mediatek: switch to use platform_get_resource_byname() honghui.zhang
2017-07-27  2:58   ` honghui.zhang at mediatek.com
2017-07-27  2:58   ` honghui.zhang
2017-07-27  2:58 ` [PATCH v2 3/5] dt-bindings: PCI: rename and cleanup MediaTek binding text honghui.zhang
2017-07-27  2:58   ` honghui.zhang at mediatek.com
2017-07-27  2:58   ` honghui.zhang
2017-07-27  2:58   ` honghui.zhang
2017-08-03 19:10   ` Rob Herring
2017-08-03 19:10     ` Rob Herring
2017-08-03 19:10     ` Rob Herring
2017-08-03 22:17   ` Bjorn Helgaas
2017-08-03 22:17     ` Bjorn Helgaas
2017-08-03 22:17     ` Bjorn Helgaas
2017-07-27  2:58 ` [PATCH v2 4/5] PCI: mediatek: Add new generation controller support honghui.zhang
2017-07-27  2:58   ` honghui.zhang at mediatek.com
2017-07-27  2:58   ` honghui.zhang
2017-08-03 22:42   ` Bjorn Helgaas
2017-08-03 22:42     ` Bjorn Helgaas
2017-08-03 22:42     ` Bjorn Helgaas
2017-08-04  8:39     ` Honghui Zhang
2017-08-04  8:39       ` Honghui Zhang
2017-08-04  8:39       ` Honghui Zhang
2017-08-04 13:18       ` Bjorn Helgaas
2017-08-04 13:18         ` Bjorn Helgaas
2017-08-04 13:18         ` Bjorn Helgaas
2017-08-05  4:52         ` Ryder Lee
2017-08-05  4:52           ` Ryder Lee
2017-08-05  4:52           ` Ryder Lee
2017-08-05  6:16           ` Ryder Lee
2017-08-05  6:16             ` Ryder Lee
2017-08-05  6:16             ` Ryder Lee
2017-08-07  3:40             ` Honghui Zhang
2017-08-07  3:40               ` Honghui Zhang
2017-08-07  3:40               ` Honghui Zhang
2017-08-08 20:16           ` Bjorn Helgaas
2017-08-08 20:16             ` Bjorn Helgaas
2017-08-08 20:16             ` Bjorn Helgaas
2017-08-08 20:16             ` Bjorn Helgaas
2017-08-08 20:19         ` Bjorn Helgaas
2017-08-08 20:19           ` Bjorn Helgaas
2017-08-08 20:19           ` Bjorn Helgaas
2017-08-09  6:49           ` Honghui Zhang
2017-08-09  6:49             ` Honghui Zhang
2017-08-09  6:49             ` Honghui Zhang
2017-08-09 16:43             ` Paul Burton
2017-08-09 16:43               ` Paul Burton
2017-08-09 16:43               ` Paul Burton
2017-07-27  2:58 ` [PATCH v2 5/5] dt-bindings: PCI: add support for new generation controller honghui.zhang
2017-07-27  2:58   ` honghui.zhang at mediatek.com
2017-07-27  2:58   ` honghui.zhang
2017-07-27  2:58   ` honghui.zhang
2017-07-27  3:31   ` Honghui Zhang
2017-07-27  3:31     ` Honghui Zhang
2017-07-27  3:31     ` Honghui Zhang
2017-07-27  3:31     ` Honghui Zhang
2017-07-27  3:38   ` Honghui Zhang
2017-07-27  3:38     ` Honghui Zhang
2017-07-27  3:38     ` Honghui Zhang
2017-07-27  3:38     ` Honghui Zhang
2017-08-03 22:45   ` Bjorn Helgaas
2017-08-03 22:45     ` Bjorn Helgaas
2017-08-03 22:45     ` Bjorn Helgaas

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=20170803221546.GL20308@bhelgaas-glaptop.roam.corp.google.com \
    --to=helgaas@kernel.org \
    --cc=bhelgaas@google.com \
    --cc=devicetree@vger.kernel.org \
    --cc=eddie.huang@mediatek.com \
    --cc=honghui.zhang@mediatek.com \
    --cc=hongkun.cao@mediatek.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=robh@kerenl.org \
    --cc=ryder.lee@mediatek.com \
    --cc=sean.wang@mediatek.com \
    --cc=xinping.qian@mediatek.com \
    --cc=yingjoe.chen@mediatek.com \
    --cc=yong.wu@mediatek.com \
    --cc=youlin.pei@mediatek.com \
    --cc=yt.shen@mediatek.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.