From: Fabio Estevam <festevam@gmail.com>
To: Peter Senna Tschudin <peter.senna@collabora.com>
Cc: Kishon Vijay Abraham I <kishon@ti.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Jingoo Han <jingoohan1@gmail.com>,
Richard Zhu <hongxing.zhu@nxp.com>,
Lucas Stach <l.stach@pengutronix.de>,
Murali Karicheri <m-karicheri2@ti.com>,
Minghuan Lian <minghuan.Lian@freescale.com>,
Mingkai Hu <mingkai.hu@freescale.com>,
Roy Zang <tie-fei.zang@freescale.com>,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Niklas Cassel <niklas.cassel@axis.com>,
Jesper Nilsson <jesper.nilsson@axis.com>,
Joao Pinto <Joao.Pinto@synopsys.com>,
Zhou Wang <wangzhou1@hisilicon.com>,
Gabriele Paoloni <gabriele.paoloni@huawei.com>,
Stanimir Varbanov <svarbanov@mm-sol.com>,
Pratyush Anand <pratyush.anand@gmail.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Tim Harvey <tharvey@gateworks.com>
Subject: Re: Regression: 442ec4c04d1: PCI: dwc: all: Split struct pcie_port into host-only and core structures
Date: Tue, 9 May 2017 07:21:24 -0300 [thread overview]
Message-ID: <CAOMZO5CXjb7LVT5ByP1RqgBPkkfRn7PcRPmW3hos=UNXGnok=A@mail.gmail.com> (raw)
In-Reply-To: <20170509063439.GC9791@collabora.com>
Hi Peter,
On Tue, May 9, 2017 at 3:34 AM, Peter Senna Tschudin
<peter.senna@collabora.com> wrote:
> Something that ocurred to me is that u-boot is initializing the PCI, and
> the PCI networkd cards. Ideally this should not affect anything, but
> can this be related to the issue?
Yes, in order to narrow things down: please boot 4.11 without PCI
support in U-Boot.
The problem is that mx6q does not have a way to properly reset the PCI block.
On the board I tested there is no PCI support in U-Boot.
Maybe we need the following approach in U-Boot as Lucas did for Barebox:
https://git.pengutronix.de/cgit/barebox/commit/?id=f1da98da2760c21487bbba8f7fb957c843a22896
According to this patch that went into 4.11 the bootloader must put
PCI back in a safe state:
https://patchwork.kernel.org/patch/9528911
WARNING: multiple messages have this Message-ID (diff)
From: Fabio Estevam <festevam@gmail.com>
To: Peter Senna Tschudin <peter.senna@collabora.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Jesper Nilsson <jesper.nilsson@axis.com>,
Joao Pinto <Joao.Pinto@synopsys.com>,
Richard Zhu <hongxing.zhu@nxp.com>,
Gabriele Paoloni <gabriele.paoloni@huawei.com>,
Jingoo Han <jingoohan1@gmail.com>,
Tim Harvey <tharvey@gateworks.com>,
Pratyush Anand <pratyush.anand@gmail.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Kishon Vijay Abraham I <kishon@ti.com>,
Stanimir Varbanov <svarbanov@mm-sol.com>,
Minghuan Lian <minghuan.Lian@freescale.com>,
Zhou Wang <wangzhou1@hisilicon.com>,
Murali Karicheri <m-karicheri2@ti.com>,
Niklas Cassel <niklas.cassel@axis.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Mingkai Hu <mingkai.hu@freescale.com>,
Roy Zang <tie-fei.zang@freescale.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Lucas Stach <l.stach@pengutronix.de>
Subject: Re: Regression: 442ec4c04d1: PCI: dwc: all: Split struct pcie_port into host-only and core structures
Date: Tue, 9 May 2017 07:21:24 -0300 [thread overview]
Message-ID: <CAOMZO5CXjb7LVT5ByP1RqgBPkkfRn7PcRPmW3hos=UNXGnok=A@mail.gmail.com> (raw)
In-Reply-To: <20170509063439.GC9791@collabora.com>
Hi Peter,
On Tue, May 9, 2017 at 3:34 AM, Peter Senna Tschudin
<peter.senna@collabora.com> wrote:
> Something that ocurred to me is that u-boot is initializing the PCI, and
> the PCI networkd cards. Ideally this should not affect anything, but
> can this be related to the issue?
Yes, in order to narrow things down: please boot 4.11 without PCI
support in U-Boot.
The problem is that mx6q does not have a way to properly reset the PCI block.
On the board I tested there is no PCI support in U-Boot.
Maybe we need the following approach in U-Boot as Lucas did for Barebox:
https://git.pengutronix.de/cgit/barebox/commit/?id=f1da98da2760c21487bbba8f7fb957c843a22896
According to this patch that went into 4.11 the bootloader must put
PCI back in a safe state:
https://patchwork.kernel.org/patch/9528911
_______________________________________________
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: festevam@gmail.com (Fabio Estevam)
To: linux-arm-kernel@lists.infradead.org
Subject: Regression: 442ec4c04d1: PCI: dwc: all: Split struct pcie_port into host-only and core structures
Date: Tue, 9 May 2017 07:21:24 -0300 [thread overview]
Message-ID: <CAOMZO5CXjb7LVT5ByP1RqgBPkkfRn7PcRPmW3hos=UNXGnok=A@mail.gmail.com> (raw)
In-Reply-To: <20170509063439.GC9791@collabora.com>
Hi Peter,
On Tue, May 9, 2017 at 3:34 AM, Peter Senna Tschudin
<peter.senna@collabora.com> wrote:
> Something that ocurred to me is that u-boot is initializing the PCI, and
> the PCI networkd cards. Ideally this should not affect anything, but
> can this be related to the issue?
Yes, in order to narrow things down: please boot 4.11 without PCI
support in U-Boot.
The problem is that mx6q does not have a way to properly reset the PCI block.
On the board I tested there is no PCI support in U-Boot.
Maybe we need the following approach in U-Boot as Lucas did for Barebox:
https://git.pengutronix.de/cgit/barebox/commit/?id=f1da98da2760c21487bbba8f7fb957c843a22896
According to this patch that went into 4.11 the bootloader must put
PCI back in a safe state:
https://patchwork.kernel.org/patch/9528911
next prev parent reply other threads:[~2017-05-09 10:21 UTC|newest]
Thread overview: 71+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-08 15:02 Regression: 442ec4c04d1: PCI: dwc: all: Split struct pcie_port into host-only and core structures Peter Senna Tschudin
2017-05-08 15:02 ` Peter Senna Tschudin
2017-05-08 15:02 ` Peter Senna Tschudin
2017-05-08 15:13 ` Joao Pinto
2017-05-08 15:13 ` Joao Pinto
2017-05-08 15:13 ` Joao Pinto
2017-05-08 15:20 ` Kishon Vijay Abraham I
2017-05-08 15:20 ` Kishon Vijay Abraham I
2017-05-08 15:20 ` Kishon Vijay Abraham I
2017-05-08 15:31 ` Joao Pinto
2017-05-08 15:31 ` Joao Pinto
2017-05-08 15:31 ` Joao Pinto
2017-05-08 15:46 ` Kishon Vijay Abraham I
2017-05-08 15:46 ` Kishon Vijay Abraham I
2017-05-08 15:46 ` Kishon Vijay Abraham I
2017-05-08 15:39 ` Kishon Vijay Abraham I
2017-05-08 15:39 ` Kishon Vijay Abraham I
2017-05-08 15:39 ` Kishon Vijay Abraham I
2017-05-08 15:50 ` Kishon Vijay Abraham I
2017-05-08 15:50 ` Kishon Vijay Abraham I
2017-05-08 15:50 ` Kishon Vijay Abraham I
2017-05-08 17:15 ` Peter Senna Tschudin
2017-05-08 17:15 ` Peter Senna Tschudin
2017-05-08 17:15 ` Peter Senna Tschudin
2017-05-08 16:13 ` Peter Senna Tschudin
2017-05-08 16:13 ` Peter Senna Tschudin
2017-05-08 16:13 ` Peter Senna Tschudin
2017-05-08 16:34 ` Lucas Stach
2017-05-08 16:34 ` Lucas Stach
2017-05-08 16:34 ` Lucas Stach
2017-05-08 16:44 ` Peter Senna Tschudin
2017-05-08 16:44 ` Peter Senna Tschudin
2017-05-08 16:48 ` Lucas Stach
2017-05-08 16:48 ` Lucas Stach
2017-05-08 16:48 ` Lucas Stach
2017-05-08 16:58 ` Peter Senna Tschudin
2017-05-08 16:58 ` Peter Senna Tschudin
2017-05-08 16:58 ` Peter Senna Tschudin
2017-05-08 17:55 ` Lorenzo Pieralisi
2017-05-08 17:55 ` Lorenzo Pieralisi
2017-05-08 17:55 ` Lorenzo Pieralisi
2017-05-08 20:33 ` Peter Senna Tschudin
2017-05-08 20:33 ` Peter Senna Tschudin
2017-05-08 20:33 ` Peter Senna Tschudin
2017-05-09 16:27 ` Lorenzo Pieralisi
2017-05-09 16:27 ` Lorenzo Pieralisi
2017-05-09 16:27 ` Lorenzo Pieralisi
2017-05-08 18:23 ` Lorenzo Pieralisi
2017-05-08 18:23 ` Lorenzo Pieralisi
2017-05-08 18:23 ` Lorenzo Pieralisi
2017-05-09 1:30 ` Fabio Estevam
2017-05-09 1:30 ` Fabio Estevam
2017-05-09 1:30 ` Fabio Estevam
2017-05-09 6:34 ` Peter Senna Tschudin
2017-05-09 6:34 ` Peter Senna Tschudin
2017-05-09 6:34 ` Peter Senna Tschudin
2017-05-09 10:21 ` Fabio Estevam [this message]
2017-05-09 10:21 ` Fabio Estevam
2017-05-09 10:21 ` Fabio Estevam
2017-05-10 13:17 ` Peter Senna Tschudin
2017-05-10 13:17 ` Peter Senna Tschudin
2017-05-10 13:17 ` Peter Senna Tschudin
2017-05-10 13:27 ` Fabio Estevam
2017-05-10 13:27 ` Fabio Estevam
2017-05-10 13:27 ` Fabio Estevam
2017-05-10 13:32 ` Lucas Stach
2017-05-10 13:32 ` Lucas Stach
2017-05-10 13:32 ` Lucas Stach
2017-05-10 14:14 ` Peter Senna Tschudin
2017-05-10 14:14 ` Peter Senna Tschudin
2017-05-10 14:14 ` Peter Senna Tschudin
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='CAOMZO5CXjb7LVT5ByP1RqgBPkkfRn7PcRPmW3hos=UNXGnok=A@mail.gmail.com' \
--to=festevam@gmail.com \
--cc=Joao.Pinto@synopsys.com \
--cc=bhelgaas@google.com \
--cc=gabriele.paoloni@huawei.com \
--cc=hongxing.zhu@nxp.com \
--cc=jesper.nilsson@axis.com \
--cc=jingoohan1@gmail.com \
--cc=kishon@ti.com \
--cc=l.stach@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=m-karicheri2@ti.com \
--cc=minghuan.Lian@freescale.com \
--cc=mingkai.hu@freescale.com \
--cc=niklas.cassel@axis.com \
--cc=peter.senna@collabora.com \
--cc=pratyush.anand@gmail.com \
--cc=svarbanov@mm-sol.com \
--cc=tharvey@gateworks.com \
--cc=thomas.petazzoni@free-electrons.com \
--cc=tie-fei.zang@freescale.com \
--cc=wangzhou1@hisilicon.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.