[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5232263d-5836-402d-9c5e-e3c601181080@heusel.eu>
Date: Tue, 14 Jan 2025 17:40:12 +0100
From: Christian Heusel <christian@...sel.eu>
To: Niklas Cassel <cassel@...nel.org>
Cc: Mario Limonciello <mario.limonciello@....com>,
Damien Le Moal <dlemoal@...nel.org>, Mika Westerberg <mika.westerberg@...ux.intel.com>,
linux-ide@...r.kernel.org, linux-kernel@...r.kernel.org, regressions@...ts.linux.dev
Subject: Re: [REGRESSION][BISECTED] BU40N Blu-Ray drive broken since
7627a0edef54
On 25/01/14 05:05PM, Niklas Cassel wrote:
> Hello Christian,
Hello Niklas,
> Looking at the Arch Linux thread, it seems that the person reporting the
> problem says that even on a working kernel, you only see the device the
> user presses the open button on the Blu-Ray player.
>
> This suggests to me that it sends a hotplug/PhyRdy event when the user
> presses the open button.
>
> In Linux, if we don't detect anything connected to the port during initial
> boot, and if the port is not marked (by firmware) as either Hotplug Capable,
> or External, then we mask the PhyRdy interrupt.
>
> We could reconsider this, and leave the PhyRdy interrupt enabled even for a
> port that is not Hotplug Capable or External, if there is no devices detected
> on the port.
>
>
> However, a device should really show up after a COMRESET... A device that
> does not do so seems to be non-spec compliant.
> It would be nice if we could understand why this device does not show up
> after a COMRESET. (If it did, we most likely would not need the workaround
> suggested above.)
>
> Could you please run with this debug prints, which might tell us what is
> going on:
Thanks for investigating this further, I was typing my previous mail and
sent it just after yours because I didn't see it x)
> diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c
> <snip-patch>
Building a kernel with the patch applied just now! I will get back to
you once I have heard back from the user.
On a sidenote it seems like your mail client or something converts the
tabs to spaces on these inline patches, which makes them quite hard to
apply for me (I had to manually apply and format-patch).
Could add them as attachment in the future or reply via git send-mail?
> Kind regards,
> Niklas
Cheers,
Chris
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists