All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>,
	Julia Lawall <Julia.Lawall@lip6.fr>,
	kernel-janitors@vger.kernel.org,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource
Date: Tue, 27 Aug 2013 16:11:07 +0000	[thread overview]
Message-ID: <521CCF9B.9000004@wwwdotorg.org> (raw)
In-Reply-To: <20130827081443.GI8686@ulmo>

On 08/27/2013 02:14 AM, Thierry Reding wrote:
> On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote:
>> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding 
>> <thierry.reding@gmail.com> wrote:
>>> On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote:
>>>> On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding 
>>>> <thierry.reding@gmail.com> wrote:
>>>> 
>>>>> Bjorn, how do you want to handle patches to the Tegra PCIe
>>>>> driver in the future? Do you want me to prepare a branch
>>>>> and pull from that or would you rather just take simple
>>>>> patches?
>>>> 
>>>> I'm in the habit of applying patches from email, so that's
>>>> easy for me.  But a branch would be OK, too.
>>> 
>>> Patches work for me too. Is this cleanup patch something that
>>> you'd be comfortable with applying after 3.12-rc1 or would you
>>> rather defer it to 3.13?
>> 
>> I'm not really sure how we should manage drivers/pci/host/*.
>> Those files are mostly arch code, and I'm not sure it's useful
>> for me to be in the middle of managing them.
>> 
>> I assume Stephen or somebody has a tree with the pci-tegra.c
>> stuff that's in -next right now; it seems like it'd be simplest
>> to just add this patch there and merge in during the v3.12 merge
>> window.
> 
> If Stephen's fine with it I suppose we could take pci-tegra.c
> driver changes through the Tegra tree. But I think it'd be good if
> we could still Cc you on patches so you're aware of what we're
> doing (that is the same for all drivers drivers/pci/host/*). And
> we're going to need your Acked-by on the patches as well.

I can push Tegra PCIe patches through the arm-soc tree before
3.12-rc1, but after that point, I think it's best if they go through
the PCIe tree, unless there's some cross-subsystem dependency for a
specific patch, and with the new PCIe driver, that's less likely.

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren@wwwdotorg.org>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>,
	Julia Lawall <Julia.Lawall@lip6.fr>,
	kernel-janitors@vger.kernel.org,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource
Date: Tue, 27 Aug 2013 10:11:07 -0600	[thread overview]
Message-ID: <521CCF9B.9000004@wwwdotorg.org> (raw)
In-Reply-To: <20130827081443.GI8686@ulmo>

On 08/27/2013 02:14 AM, Thierry Reding wrote:
> On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote:
>> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding 
>> <thierry.reding@gmail.com> wrote:
>>> On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote:
>>>> On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding 
>>>> <thierry.reding@gmail.com> wrote:
>>>> 
>>>>> Bjorn, how do you want to handle patches to the Tegra PCIe
>>>>> driver in the future? Do you want me to prepare a branch
>>>>> and pull from that or would you rather just take simple
>>>>> patches?
>>>> 
>>>> I'm in the habit of applying patches from email, so that's
>>>> easy for me.  But a branch would be OK, too.
>>> 
>>> Patches work for me too. Is this cleanup patch something that
>>> you'd be comfortable with applying after 3.12-rc1 or would you
>>> rather defer it to 3.13?
>> 
>> I'm not really sure how we should manage drivers/pci/host/*.
>> Those files are mostly arch code, and I'm not sure it's useful
>> for me to be in the middle of managing them.
>> 
>> I assume Stephen or somebody has a tree with the pci-tegra.c
>> stuff that's in -next right now; it seems like it'd be simplest
>> to just add this patch there and merge in during the v3.12 merge
>> window.
> 
> If Stephen's fine with it I suppose we could take pci-tegra.c
> driver changes through the Tegra tree. But I think it'd be good if
> we could still Cc you on patches so you're aware of what we're
> doing (that is the same for all drivers drivers/pci/host/*). And
> we're going to need your Acked-by on the patches as well.

I can push Tegra PCIe patches through the arm-soc tree before
3.12-rc1, but after that point, I think it's best if they go through
the PCIe tree, unless there's some cross-subsystem dependency for a
specific patch, and with the new PCIe driver, that's less likely.

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	Julia Lawall <Julia.Lawall-L2FTfq7BK8M@public.gmane.org>,
	kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource
Date: Tue, 27 Aug 2013 10:11:07 -0600	[thread overview]
Message-ID: <521CCF9B.9000004@wwwdotorg.org> (raw)
In-Reply-To: <20130827081443.GI8686@ulmo>

On 08/27/2013 02:14 AM, Thierry Reding wrote:
> On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote:
>> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding 
>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>> On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote:
>>>> On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding 
>>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>> 
>>>>> Bjorn, how do you want to handle patches to the Tegra PCIe
>>>>> driver in the future? Do you want me to prepare a branch
>>>>> and pull from that or would you rather just take simple
>>>>> patches?
>>>> 
>>>> I'm in the habit of applying patches from email, so that's
>>>> easy for me.  But a branch would be OK, too.
>>> 
>>> Patches work for me too. Is this cleanup patch something that
>>> you'd be comfortable with applying after 3.12-rc1 or would you
>>> rather defer it to 3.13?
>> 
>> I'm not really sure how we should manage drivers/pci/host/*.
>> Those files are mostly arch code, and I'm not sure it's useful
>> for me to be in the middle of managing them.
>> 
>> I assume Stephen or somebody has a tree with the pci-tegra.c
>> stuff that's in -next right now; it seems like it'd be simplest
>> to just add this patch there and merge in during the v3.12 merge
>> window.
> 
> If Stephen's fine with it I suppose we could take pci-tegra.c
> driver changes through the Tegra tree. But I think it'd be good if
> we could still Cc you on patches so you're aware of what we're
> doing (that is the same for all drivers drivers/pci/host/*). And
> we're going to need your Acked-by on the patches as well.

I can push Tegra PCIe patches through the arm-soc tree before
3.12-rc1, but after that point, I think it's best if they go through
the PCIe tree, unless there's some cross-subsystem dependency for a
specific patch, and with the new PCIe driver, that's less likely.

  reply	other threads:[~2013-08-27 16:11 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-19 11:20 [PATCH 0/7] replace devm_request_and_ioremap by devm_ioremap_resource Julia Lawall
2013-08-19 11:20 ` Julia Lawall
2013-08-19 11:20 ` Julia Lawall
2013-08-19 11:20 ` [PATCH 1/7] PCI: tegra: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-19 12:02   ` Thierry Reding
2013-08-19 12:02     ` Thierry Reding
2013-08-19 12:07     ` Julia Lawall
2013-08-19 12:07       ` Julia Lawall
2013-08-19 12:07       ` Julia Lawall
2013-08-19 12:12       ` Thierry Reding
2013-08-19 12:12         ` Thierry Reding
2013-08-19 12:15         ` Julia Lawall
2013-08-19 12:15           ` Julia Lawall
2013-08-19 12:15           ` Julia Lawall
2013-08-19 19:33           ` Thierry Reding
2013-08-19 19:33             ` Thierry Reding
2013-08-19 19:33             ` Thierry Reding
2013-08-19 20:56             ` Stephen Warren
2013-08-19 20:56               ` Stephen Warren
2013-08-19 20:56               ` Stephen Warren
2013-08-20  9:13             ` Julia Lawall
2013-08-20  9:13               ` Julia Lawall
2013-08-20  9:13               ` Julia Lawall
2013-08-20 16:26               ` Bjorn Helgaas
2013-08-20 16:26                 ` Bjorn Helgaas
2013-08-20 16:26                 ` Bjorn Helgaas
2013-08-20 18:53                 ` Stephen Warren
2013-08-20 18:53                   ` Stephen Warren
2013-08-20 19:31                   ` Thierry Reding
2013-08-20 19:31                     ` Thierry Reding
2013-08-20 19:31                     ` Thierry Reding
2013-08-19 20:04     ` Bjorn Helgaas
2013-08-19 20:04       ` Bjorn Helgaas
2013-08-19 20:12       ` Thierry Reding
2013-08-19 20:12         ` Thierry Reding
2013-08-19 20:49         ` Bjorn Helgaas
2013-08-19 20:49           ` Bjorn Helgaas
2013-08-19 20:49           ` Bjorn Helgaas
2013-08-27  8:14           ` Thierry Reding
2013-08-27  8:14             ` Thierry Reding
2013-08-27 16:11             ` Stephen Warren [this message]
2013-08-27 16:11               ` Stephen Warren
2013-08-27 16:11               ` Stephen Warren
2013-08-27 18:03               ` Bjorn Helgaas
2013-08-27 18:03                 ` Bjorn Helgaas
2013-08-27 18:03                 ` Bjorn Helgaas
2013-08-28 13:02                 ` Bjorn Helgaas
2013-08-28 13:02                   ` Bjorn Helgaas
2013-08-28 13:27                   ` Thierry Reding
2013-08-28 13:27                     ` Thierry Reding
2013-08-28 13:27                     ` Thierry Reding
2013-08-28 13:31                 ` Thierry Reding
2013-08-28 13:31                   ` Thierry Reding
2013-08-28 13:31                   ` Thierry Reding
2013-08-19 11:20 ` [PATCH 2/7] serial: st-asc: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-19 11:20 ` [PATCH 3/7] iommu/arm: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-20 11:35   ` Will Deacon
2013-08-20 11:35     ` Will Deacon
2013-08-20 11:35     ` Will Deacon
2013-08-20 11:49     ` Julia Lawall
2013-08-20 11:49       ` Julia Lawall
2013-08-20 11:49       ` Julia Lawall
2013-08-19 11:20 ` [PATCH 4/7] dma: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-09-02 12:07   ` Vinod Koul
2013-09-02 12:19     ` Vinod Koul
2013-08-19 11:20 ` [PATCH 5/7] bcm63xx_enet: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-21  6:22   ` David Miller
2013-08-21  6:22     ` David Miller
2013-08-19 11:20 ` [PATCH 6/7] video: xilinxfb: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall
2013-08-30  9:42   ` Tomi Valkeinen
2013-08-30  9:42     ` Tomi Valkeinen
2013-08-19 11:20 ` [PATCH 7/7] mmc: mvsdio: " Julia Lawall
2013-08-19 11:20   ` Julia Lawall

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=521CCF9B.9000004@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=Julia.Lawall@lip6.fr \
    --cc=bhelgaas@google.com \
    --cc=kernel-janitors@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=thierry.reding@gmail.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.