All of lore.kernel.org
 help / color / mirror / Atom feed
From: Suman Anna <s-anna@ti.com>
To: Kishon Vijay Abraham I <kishon@ti.com>,
	Tony Lindgren <tony@atomide.com>,
	Bjorn Helgaas <bhelgaas@google.com>,
	richardcochran@gmail.com
Cc: Russell King <linux@arm.linux.org.uk>,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, nsekhar@ti.com
Subject: Re: [PATCH 0/3] dra7xx: get pcie working in mainline
Date: Mon, 11 Jan 2016 11:12:35 -0600	[thread overview]
Message-ID: <5693E283.3090000@ti.com> (raw)
In-Reply-To: <1452440488-28398-1-git-send-email-kishon@ti.com>

Hi Kishon,

On 01/10/2016 09:41 AM, Kishon Vijay Abraham I wrote:
> This series adds pdata-quirk mechanism to reset PCIe as a temporary
> fix till reset controller driver is added in mainline.
> 
> Without this series, a stall is observed if pci dra7xx driver
> is enabled.
> 
> Kishon Vijay Abraham I (3):
>   ARM: DRA7: hwmod: Add reset data for PCIe
>   HACK: ARM: DRA7: add pdata-quirks to do reset of PCIe
>   HACK: pci: dra7xx: use pdata callbacks to perform reset

Please drop the HACK: tag from the commit headers, that was an internal
convention/choice made by Sekhar, and we had our differences there as it
is. On the same note, my internal Reviewed-by also mean nothing as they
were done against a different kernel.

regards
Suman

> 
>  arch/arm/mach-omap2/omap_hwmod_7xx_data.c |   15 ++++++++++++++
>  arch/arm/mach-omap2/pdata-quirks.c        |   11 ++++++++++
>  arch/arm/mach-omap2/prm7xx.h              |    1 +
>  drivers/pci/host/pci-dra7xx.c             |   32 +++++++++++++++++++++++++++++
>  include/linux/platform_data/pci-dra7xx.h  |   22 ++++++++++++++++++++
>  5 files changed, 81 insertions(+)
>  create mode 100644 include/linux/platform_data/pci-dra7xx.h
> 

WARNING: multiple messages have this Message-ID (diff)
From: s-anna@ti.com (Suman Anna)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] dra7xx: get pcie working in mainline
Date: Mon, 11 Jan 2016 11:12:35 -0600	[thread overview]
Message-ID: <5693E283.3090000@ti.com> (raw)
In-Reply-To: <1452440488-28398-1-git-send-email-kishon@ti.com>

Hi Kishon,

On 01/10/2016 09:41 AM, Kishon Vijay Abraham I wrote:
> This series adds pdata-quirk mechanism to reset PCIe as a temporary
> fix till reset controller driver is added in mainline.
> 
> Without this series, a stall is observed if pci dra7xx driver
> is enabled.
> 
> Kishon Vijay Abraham I (3):
>   ARM: DRA7: hwmod: Add reset data for PCIe
>   HACK: ARM: DRA7: add pdata-quirks to do reset of PCIe
>   HACK: pci: dra7xx: use pdata callbacks to perform reset

Please drop the HACK: tag from the commit headers, that was an internal
convention/choice made by Sekhar, and we had our differences there as it
is. On the same note, my internal Reviewed-by also mean nothing as they
were done against a different kernel.

regards
Suman

> 
>  arch/arm/mach-omap2/omap_hwmod_7xx_data.c |   15 ++++++++++++++
>  arch/arm/mach-omap2/pdata-quirks.c        |   11 ++++++++++
>  arch/arm/mach-omap2/prm7xx.h              |    1 +
>  drivers/pci/host/pci-dra7xx.c             |   32 +++++++++++++++++++++++++++++
>  include/linux/platform_data/pci-dra7xx.h  |   22 ++++++++++++++++++++
>  5 files changed, 81 insertions(+)
>  create mode 100644 include/linux/platform_data/pci-dra7xx.h
> 

WARNING: multiple messages have this Message-ID (diff)
From: Suman Anna <s-anna@ti.com>
To: Kishon Vijay Abraham I <kishon@ti.com>,
	Tony Lindgren <tony@atomide.com>,
	Bjorn Helgaas <bhelgaas@google.com>, <richardcochran@gmail.com>
Cc: Russell King <linux@arm.linux.org.uk>,
	<linux-omap@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <nsekhar@ti.com>
Subject: Re: [PATCH 0/3] dra7xx: get pcie working in mainline
Date: Mon, 11 Jan 2016 11:12:35 -0600	[thread overview]
Message-ID: <5693E283.3090000@ti.com> (raw)
In-Reply-To: <1452440488-28398-1-git-send-email-kishon@ti.com>

Hi Kishon,

On 01/10/2016 09:41 AM, Kishon Vijay Abraham I wrote:
> This series adds pdata-quirk mechanism to reset PCIe as a temporary
> fix till reset controller driver is added in mainline.
> 
> Without this series, a stall is observed if pci dra7xx driver
> is enabled.
> 
> Kishon Vijay Abraham I (3):
>   ARM: DRA7: hwmod: Add reset data for PCIe
>   HACK: ARM: DRA7: add pdata-quirks to do reset of PCIe
>   HACK: pci: dra7xx: use pdata callbacks to perform reset

Please drop the HACK: tag from the commit headers, that was an internal
convention/choice made by Sekhar, and we had our differences there as it
is. On the same note, my internal Reviewed-by also mean nothing as they
were done against a different kernel.

regards
Suman

> 
>  arch/arm/mach-omap2/omap_hwmod_7xx_data.c |   15 ++++++++++++++
>  arch/arm/mach-omap2/pdata-quirks.c        |   11 ++++++++++
>  arch/arm/mach-omap2/prm7xx.h              |    1 +
>  drivers/pci/host/pci-dra7xx.c             |   32 +++++++++++++++++++++++++++++
>  include/linux/platform_data/pci-dra7xx.h  |   22 ++++++++++++++++++++
>  5 files changed, 81 insertions(+)
>  create mode 100644 include/linux/platform_data/pci-dra7xx.h
> 

  parent reply	other threads:[~2016-01-11 17:12 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-10 15:41 [PATCH 0/3] dra7xx: get pcie working in mainline Kishon Vijay Abraham I
2016-01-10 15:41 ` Kishon Vijay Abraham I
2016-01-10 15:41 ` Kishon Vijay Abraham I
2016-01-10 15:41 ` [PATCH 1/3] ARM: DRA7: hwmod: Add reset data for PCIe Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-10 15:41 ` [PATCH 2/3] HACK: ARM: DRA7: add pdata-quirks to do reset of PCIe Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-10 15:41 ` [PATCH 3/3] HACK: pci: dra7xx: use pdata callbacks to perform reset Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-10 15:41   ` Kishon Vijay Abraham I
2016-01-11 17:12 ` Suman Anna [this message]
2016-01-11 17:12   ` [PATCH 0/3] dra7xx: get pcie working in mainline Suman Anna
2016-01-11 17:12   ` Suman Anna

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=5693E283.3090000@ti.com \
    --to=s-anna@ti.com \
    --cc=bhelgaas@google.com \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nsekhar@ti.com \
    --cc=richardcochran@gmail.com \
    --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.