All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marc Zyngier <maz@kernel.org>
To: Christian Zigotzky <chzigotzky@xenosoft.de>
Cc: Bjorn Helgaas <helgaas@kernel.org>,
	"bhelgaas@google.com >> Bjorn Helgaas" <bhelgaas@google.com>,
	Alyssa Rosenzweig <alyssa@rosenzweig.io>,
	lorenzo.pieralisi@arm.com, Rob Herring <robh@kernel.org>,
	Matthew Leaman <matthew@a-eon.biz>,
	Darren Stevens <darren@stevens-zone.net>,
	mad skateman <madskateman@gmail.com>,
	"R.T.Dickinson" <rtd2@xtra.co.nz>,
	Christian Zigotzky <info@xenosoft.de>,
	axboe@kernel.dk, damien.lemoal@opensource.wdc.com, kw@linux.com,
	Arnd Bergmann <arnd@arndb.de>,
	robert@swiecki.net, Olof Johansson <olof@lixom.net>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>
Subject: Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates
Date: Thu, 11 Nov 2021 10:20:46 +0000	[thread overview]
Message-ID: <87pmr7803l.wl-maz@kernel.org> (raw)
In-Reply-To: <f40294c6-a088-af53-eeea-4dfbd255c5c9@xenosoft.de>

On Thu, 11 Nov 2021 07:47:08 +0000,
Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
> 
> On 11 November 2021 at 08:13 am, Marc Zyngier wrote:
> > On Thu, 11 Nov 2021 05:24:52 +0000,
> > Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
> >> Hello Marc,
> >> 
> >> Here you are:
> >> https://forum.hyperion-entertainment.com/viewtopic.php?p=54406#p54406
> > This is not what I asked. I need the actual source file, or at the
> > very least the compiled object (the /sys/firmware/fdt file, for
> > example). Not an interpretation that I can't feed to the kernel.
> > 
> > Without this, I can't debug your problem.
> > 
> >> We are very happy to have the patch for reverting the bad commit
> >> because we want to test the new PASEMI i2c driver with support for the
> >> Apple M1 [1] on our Nemo boards.
> > You can revert the patch on your own. At this stage, we're not blindly
> > reverting things in the tree, but instead trying to understand what
> > is happening on your particular system.
> > 
> > Thanks,
> > 
> > 	M.
> > 
> I added a download link for the fdt file to the post [1]. Please read
> also Darren's comments in this post.

Thanks for that. The DT looks absolutely bonkers, no wonder that
things break with something like that.

But Darren's comments made me jump a bit, and I quote them here for
everyone to see:

<quote>
[...]

The dtb passed by the CFE firmware has a number of issues, which up till
now have been fixed by use of patches applied to the mainline kernel.
This occasionally causes problems with changes made to mainline.

[...]
</quote>

Am I right in understanding that the upstream kernel does not support
the machine out of the box, and that you actually have to apply out of
tree patches to make it work? That these patches have to do with the
IRQ routing?

If so, I wonder why upstream should revert a patch to work on a system
that isn't supported upstream the first place. I will still try and
come up with a solution for you. But asking for the revert of a patch
on these grounds is not, IMHO, acceptable. Also, please provide these
patches on the list so that I can help you to some extend (and I mean
*on the list*, not on a random forum that collects my information).

Thanks,

	M.

-- 
Without deviation from the norm, progress is not possible.

WARNING: multiple messages have this Message-ID (diff)
From: Marc Zyngier <maz@kernel.org>
To: Christian Zigotzky <chzigotzky@xenosoft.de>
Cc: axboe@kernel.dk, Rob Herring <robh@kernel.org>,
	lorenzo.pieralisi@arm.com, "R.T.Dickinson" <rtd2@xtra.co.nz>,
	Arnd Bergmann <arnd@arndb.de>,
	kw@linux.com,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	damien.lemoal@opensource.wdc.com, Olof Johansson <olof@lixom.net>,
	Darren Stevens <darren@stevens-zone.net>,
	Bjorn Helgaas <helgaas@kernel.org>,
	mad skateman <madskateman@gmail.com>,
	"bhelgaas@google.com >> Bjorn Helgaas" <bhelgaas@google.com>,
	robert@swiecki.net, Matthew Leaman <matthew@a-eon.biz>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	Alyssa Rosenzweig <alyssa@rosenzweig.io>,
	Christian Zigotzky <info@xenosoft.de>
Subject: Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates
Date: Thu, 11 Nov 2021 10:20:46 +0000	[thread overview]
Message-ID: <87pmr7803l.wl-maz@kernel.org> (raw)
In-Reply-To: <f40294c6-a088-af53-eeea-4dfbd255c5c9@xenosoft.de>

On Thu, 11 Nov 2021 07:47:08 +0000,
Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
> 
> On 11 November 2021 at 08:13 am, Marc Zyngier wrote:
> > On Thu, 11 Nov 2021 05:24:52 +0000,
> > Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
> >> Hello Marc,
> >> 
> >> Here you are:
> >> https://forum.hyperion-entertainment.com/viewtopic.php?p=54406#p54406
> > This is not what I asked. I need the actual source file, or at the
> > very least the compiled object (the /sys/firmware/fdt file, for
> > example). Not an interpretation that I can't feed to the kernel.
> > 
> > Without this, I can't debug your problem.
> > 
> >> We are very happy to have the patch for reverting the bad commit
> >> because we want to test the new PASEMI i2c driver with support for the
> >> Apple M1 [1] on our Nemo boards.
> > You can revert the patch on your own. At this stage, we're not blindly
> > reverting things in the tree, but instead trying to understand what
> > is happening on your particular system.
> > 
> > Thanks,
> > 
> > 	M.
> > 
> I added a download link for the fdt file to the post [1]. Please read
> also Darren's comments in this post.

Thanks for that. The DT looks absolutely bonkers, no wonder that
things break with something like that.

But Darren's comments made me jump a bit, and I quote them here for
everyone to see:

<quote>
[...]

The dtb passed by the CFE firmware has a number of issues, which up till
now have been fixed by use of patches applied to the mainline kernel.
This occasionally causes problems with changes made to mainline.

[...]
</quote>

Am I right in understanding that the upstream kernel does not support
the machine out of the box, and that you actually have to apply out of
tree patches to make it work? That these patches have to do with the
IRQ routing?

If so, I wonder why upstream should revert a patch to work on a system
that isn't supported upstream the first place. I will still try and
come up with a solution for you. But asking for the revert of a patch
on these grounds is not, IMHO, acceptable. Also, please provide these
patches on the list so that I can help you to some extend (and I mean
*on the list*, not on a random forum that collects my information).

Thanks,

	M.

-- 
Without deviation from the norm, progress is not possible.

  reply	other threads:[~2021-11-11 10:20 UTC|newest]

Thread overview: 137+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-30 11:59 Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Christian Zigotzky
2021-04-30 11:59 ` Christian Zigotzky
2021-04-30 15:26 ` Deucher, Alexander
2021-04-30 15:26   ` Deucher, Alexander
2021-04-30 15:26   ` Deucher, Alexander
2021-04-30 15:56   ` Gustavo A. R. Silva
2021-04-30 15:56     ` Gustavo A. R. Silva
2021-04-30 15:56     ` Gustavo A. R. Silva
2021-05-03 22:25 ` [FSL P50x0] Xorg always restarts again and again after the the PowerPC updates 5.13-1 Christian Zigotzky
2021-05-04  4:56   ` Christophe Leroy
2021-05-04  7:21     ` Christian Zigotzky
2021-05-04  7:47       ` Christophe Leroy
2021-05-04  8:29         ` Christian Zigotzky
2021-05-04  8:58           ` Christophe Leroy
2021-05-04  9:09             ` Christian Zigotzky
2021-05-04  9:11               ` Christophe Leroy
2021-05-04  9:46                 ` Christian Zigotzky
2021-05-04  9:49                   ` Christophe Leroy
2021-05-04 10:07                     ` Christian Zigotzky
2021-05-04 11:02                       ` Christian Zigotzky
2021-05-04 13:48                         ` Christian Zigotzky
2021-05-04 14:48                           ` Christophe Leroy
2021-05-04 14:52                             ` Christian Zigotzky
2021-05-04 14:41                         ` Christophe Leroy
2021-05-04 14:59                           ` Christian Zigotzky
2021-05-04 15:17                             ` Christophe Leroy
2021-05-05 12:43                               ` Christian Zigotzky
2021-05-06  6:12                                 ` Christophe Leroy
2021-05-06  7:56                                   ` Christian Zigotzky
2021-05-06  8:09                                     ` Christophe Leroy
2021-05-06 13:58                                       ` Christian Zigotzky
2021-05-08 16:39                                         ` Christian Zigotzky
2021-05-09 12:27                                           ` Christian Zigotzky
2021-05-09 17:43                                             ` Christophe Leroy
2021-05-09 21:46                                               ` Christian Zigotzky
2021-05-09 17:36                                           ` Christophe Leroy
2021-05-13  6:47                                             ` Christian Zigotzky
2021-05-13 10:01                                               ` Christophe Leroy
2021-05-13 15:19                                                 ` Christian Zigotzky
2021-05-13 15:51                                                   ` Christophe Leroy
2021-05-13 16:35                                                     ` Christian Zigotzky
2021-05-13 17:00                                                       ` Christophe Leroy
2021-05-13 20:20                                                         ` Christian Zigotzky
2021-05-13 22:58                                                           ` Nicholas Piggin
2021-05-14  0:19                                                             ` Christian Zigotzky
2021-05-04 14:44                   ` Christophe Leroy
2021-05-06 23:55 ` Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Gustavo A. R. Silva
2021-05-06 23:55   ` Gustavo A. R. Silva
2021-05-07  6:43   ` Christian Zigotzky
2021-05-07  6:43     ` Christian Zigotzky
2021-05-08 11:33     ` Christian Zigotzky
2021-05-08 11:33       ` Christian Zigotzky
2021-05-09 23:04       ` Gustavo A. R. Silva
2021-05-09 23:04         ` Gustavo A. R. Silva
2023-11-02 14:45       ` Fbdev issue after the drm updates 'drm-next-2023-10-31-1' Christian Zigotzky
2023-11-07  8:36         ` Christian Zigotzky
2023-11-12 14:23           ` Christian Zigotzky
2023-11-13 12:48             ` Geert Uytterhoeven
2023-11-13 12:48               ` Geert Uytterhoeven
2023-11-13 13:12               ` Christian Zigotzky
2023-11-13 13:12                 ` Christian Zigotzky
2023-11-14  9:45               ` Christian Zigotzky
2023-11-14  9:45                 ` Christian Zigotzky
2023-11-15  8:33                 ` Geert Uytterhoeven
2023-11-15  8:33                   ` Geert Uytterhoeven
2023-11-15  8:33                   ` Geert Uytterhoeven
2023-11-15  9:22                   ` Gerd Hoffmann
2023-11-15  9:22                     ` Gerd Hoffmann
2023-11-15  9:22                     ` Gerd Hoffmann
2023-11-16 13:13                     ` Geert Uytterhoeven
2023-11-16 13:13                       ` Geert Uytterhoeven
2023-11-16 13:13                       ` Geert Uytterhoeven
2021-11-04 10:45 ` [VirtIO GPU] Xorg doesn't start with the DRM updates 'drm-next-2021-11-03' in a virtual e5500 QEMU KVM-HV machine on a Freescale P5040 board Christian Zigotzky
2021-11-04 21:42   ` [PATCH] drm/virtio: Fix NULL dereference error in virtio_gpu_poll Vivek Kasireddy
2021-11-05 18:05     ` Christian Zigotzky
2021-11-05 18:05       ` Christian Zigotzky
2021-11-08 12:56     ` Gerd Hoffmann
2021-11-08 14:48   ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christian Zigotzky
2021-11-09 14:45     ` [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates Christian Zigotzky
2021-11-09 15:10       ` Christian Zigotzky
2021-11-09 15:10         ` Christian Zigotzky
2021-11-09 16:58         ` Bjorn Helgaas
2021-11-09 16:58           ` Bjorn Helgaas
2021-11-09 22:40           ` Krzysztof Wilczyński
2021-11-09 22:40             ` Krzysztof Wilczyński
2021-11-09 23:05             ` Arnd Bergmann
2021-11-09 23:05               ` Arnd Bergmann
2021-11-09 23:18               ` Krzysztof Wilczyński
2021-11-09 23:18                 ` Krzysztof Wilczyński
2021-11-10 16:42                 ` Robert Święcki
2021-11-10 16:42                   ` Robert Święcki
2021-11-10  3:52             ` Damien Le Moal
2021-11-10  3:52               ` Damien Le Moal
2021-11-10 18:07       ` Christian Zigotzky
2021-11-10 18:07         ` Christian Zigotzky
2021-11-10 18:41         ` Bjorn Helgaas
2021-11-10 18:41           ` Bjorn Helgaas
2021-11-10 19:09           ` Marc Zyngier
2021-11-10 19:09             ` Marc Zyngier
2021-11-11  5:24             ` Christian Zigotzky
2021-11-11  5:24               ` Christian Zigotzky
2021-11-11  7:13               ` Marc Zyngier
2021-11-11  7:13                 ` Marc Zyngier
2021-11-11  7:47                 ` Christian Zigotzky
2021-11-11  7:47                   ` Christian Zigotzky
2021-11-11 10:20                   ` Marc Zyngier [this message]
2021-11-11 10:20                     ` Marc Zyngier
2021-11-11 10:44                     ` Christian Zigotzky
2021-11-11 10:44                       ` Christian Zigotzky
2021-11-11 11:24                       ` Marc Zyngier
2021-11-11 11:24                         ` Marc Zyngier
2021-11-11 11:54                         ` Christian Zigotzky
2021-11-11 11:54                           ` Christian Zigotzky
2021-11-11 22:21                     ` Olof Johansson
2021-11-11 22:21                       ` Olof Johansson
2021-11-12 11:49                       ` Segher Boessenkool
2021-11-12 11:49                         ` Segher Boessenkool
2021-11-11 17:39         ` Marc Zyngier
2021-11-11 17:39           ` Marc Zyngier
2021-11-12  9:40           ` Christian Zigotzky
2021-11-12  9:40             ` Christian Zigotzky
2021-11-12 10:11             ` Christian Zigotzky
2021-11-12 10:11               ` Christian Zigotzky
2021-11-12 11:00               ` Christian Zigotzky
2021-11-12 11:00                 ` Christian Zigotzky
2021-11-12 13:41             ` Marc Zyngier
2021-11-12 13:41               ` Marc Zyngier
2021-11-12 14:15               ` Christian Zigotzky
2021-11-12 14:15                 ` Christian Zigotzky
2021-11-12 14:46                 ` Marc Zyngier
2021-11-12 14:46                   ` Marc Zyngier
2021-11-12 15:01                   ` Christian Zigotzky
2021-11-12 15:01                     ` Christian Zigotzky
2021-11-12 15:05                     ` Christian Zigotzky
2021-11-12 15:05                       ` Christian Zigotzky
2021-11-12 16:14         ` [irqchip: irq/irqchip-fixes] of/irq: Don't ignore interrupt-controller when interrupt-map failed irqchip-bot for Marc Zyngier
2021-11-09 16:13     ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christophe Leroy

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=87pmr7803l.wl-maz@kernel.org \
    --to=maz@kernel.org \
    --cc=alyssa@rosenzweig.io \
    --cc=arnd@arndb.de \
    --cc=axboe@kernel.dk \
    --cc=bhelgaas@google.com \
    --cc=chzigotzky@xenosoft.de \
    --cc=damien.lemoal@opensource.wdc.com \
    --cc=darren@stevens-zone.net \
    --cc=helgaas@kernel.org \
    --cc=info@xenosoft.de \
    --cc=kw@linux.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=madskateman@gmail.com \
    --cc=matthew@a-eon.biz \
    --cc=olof@lixom.net \
    --cc=robert@swiecki.net \
    --cc=robh@kernel.org \
    --cc=rtd2@xtra.co.nz \
    /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.