Re: [PATCH] firewire: core: mask previous entry's type bits when looking for leaf
Brought to you by:
aeb,
bencollins
From: Takashi S. <o-t...@sa...> - 2024-01-30 19:10:57
|
Hi, On Fri, Jan 26, 2024 at 08:53:42PM -0800, Adam Goldman wrote: > On Fri, Jan 26, 2024 at 09:19:17PM +0900, Takashi Sakamoto wrote: > > I think we can handle the quirk of configuration ROM without changing > > the kernel API. Would you test the following patch? (not tested in my > > side). > > > > ======== 8< -------- > > > > >From 83bf1e04d308ea89c76c64e3168b9701f9d9191b Mon Sep 17 00:00:00 2001 > > From: Takashi Sakamoto <o-t...@sa...> > > Date: Fri, 26 Jan 2024 20:37:21 +0900 > > Subject: [PATCH] firewire: search descriptor leaf just after vendor directory > > entry in root directory > > Hi Takashi, > > I tested your patch with the DVMC-DA1. I also tested it with another > device with normal placement of the leaf entry. In both cases, it works. Thanks for your test. I reposted the patch in the series of changes for v6.8-rc3[1]. The behaviour change of kernel API is not preferable within the same version of kernel once the release candidates is public, while we need to handle it as the series of changes to support the legacy layout of configuration ROM. So I'll apply my version to for-linus branch and send it to him. Anyway thanks for your work and suggestion. [1] https://lore.kernel.org/lkml/202...@sa.../ Thanks Takashi Sakamoto |