linux-kernel - Re: [PATCH 01/28] media: cec: ch7322: drop of_match

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Thu, 13 Jul 2023 09:07:31 +0200
From:   Hans Verkuil <hverkuil-cisco@...all.nl>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Sakari Ailus <sakari.ailus@....fi>
Cc:     Joe Tessler <jrt@...gle.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Neil Armstrong <neil.armstrong@...aro.org>,
        Kevin Hilman <khilman@...libre.com>,
        Jerome Brunet <jbrunet@...libre.com>,
        Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
        Thierry Reding <thierry.reding@...il.com>,
        Jonathan Hunter <jonathanh@...dia.com>,
        Manivannan Sadhasivam <mani@...nel.org>,
        Michael Tretter <m.tretter@...gutronix.de>,
        Pengutronix Kernel Team <kernel@...gutronix.de>,
        Jacopo Mondi <jacopo+renesas@...ndi.org>,
        Kieran Bingham <kieran.bingham+renesas@...asonboard.com>,
        Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>,
        Niklas Söderlund 
        <niklas.soderlund+renesas@...natech.se>,
        Rui Miguel Silva <rmfrfs@...il.com>,
        Wenyou Yang <wenyou.yang@...rochip.com>,
        Bin Liu <bin.liu@...iatek.com>,
        Matthias Brugger <matthias.bgg@...il.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>,
        Minghsiu Tsai <minghsiu.tsai@...iatek.com>,
        Houlong Wei <houlong.wei@...iatek.com>,
        Andrew-CT Chen <andrew-ct.chen@...iatek.com>,
        Andrzej Pietrasiewicz <andrzejtp2010@...il.com>,
        Jacek Anaszewski <jacek.anaszewski@...il.com>,
        Sylwester Nawrocki <s.nawrocki@...sung.com>,
        Patrice Chotard <patrice.chotard@...s.st.com>,
        Yong Deng <yong.deng@...ewell.com>,
        Paul Kocialkowski <paul.kocialkowski@...tlin.com>,
        Chen-Yu Tsai <wens@...e.org>,
        Jernej Skrabec <jernej.skrabec@...il.com>,
        Samuel Holland <samuel@...lland.org>,
        "Lad, Prabhakar" <prabhakar.csengg@...il.com>,
        Ezequiel Garcia <ezequiel@...guardiasur.com.ar>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Sean Young <sean@...s.org>, linux-media@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-amlogic@...ts.infradead.org,
        linux-arm-kernel@...ts.infradead.org, linux-tegra@...r.kernel.org,
        linux-mediatek@...ts.infradead.org, linux-sunxi@...ts.linux.dev,
        linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH 01/28] media: cec: ch7322: drop of_match_ptr for ID table

On 13/07/2023 08:46, Krzysztof Kozlowski wrote:
> On 19/06/2023 16:13, Krzysztof Kozlowski wrote:
>> On 25/05/2023 15:19, Sakari Ailus wrote:
>>> Hi folks,
>>>
>>> On Thu, May 25, 2023 at 03:40:04PM +0300, Sakari Ailus wrote:
>>>> Hi Hans,
>>>>
>>>> On Sat, May 13, 2023 at 11:57:33AM +0200, Hans Verkuil wrote:
>>>>> On 12/05/2023 18:35, Krzysztof Kozlowski wrote:
>>>>>> On 12/03/2023 14:12, Krzysztof Kozlowski wrote:
>>>>>>> The driver can match only via the DT table so the table should be always
>>>>>>> used and the of_match_ptr does not have any sense (this also allows ACPI
>>>>>>> matching via PRP0001, even though it might not be relevant here).
>>>>>>>
>>>>>>>   drivers/media/cec/i2c/ch7322.c:583:34: error: ‘ch7322_of_match’ defined but not used [-Werror=unused-const-variable=]
>>>>>>>
>>>>>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>>>>> ---
>>>>>>
>>>>>> Hans, Sakari,
>>>>>>
>>>>>> Can you pick up the patchset? There was positive feedback:
>>>>>> https://patchwork.linuxtv.org/project/linux-media/patch/20230312131318.351173-1-krzysztof.kozlowski@linaro.org/
>>>>>>
>>>>>> but it seems it was not applied.
>>>>>
>>>>> I see it is delegated to Sakari in patchwork and marked Under Review, but I don't
>>>>> see a corresponding pull request for this series.
>>>>>
>>>>> Sakari, did something go wrong?
>>>>
>>>> I spotted this as Hans notified me in IRC, I wasn't cc'd. Apologies for
>>>> this --- I intended to take these but I think I must have missed something
>>>> important in the process. I'll take them now.
>>>>
>>>> Thanks.
>>>
>>> This no longer applied cleanly. Mostly there was fuzz near .of_match_table
>>> changes as probe_new (and remove_new?) changes have been recently merged.
>>> There were other issues as well, I marked a few patches in the set "not
>>> applicable" as other patches had already done equivalent changes earlier.
>>>
>>> There were substance-changing changes in the 20th patch, replied to the
>>> disuccsion there.
>>>
>>> I've pushed the result here and intend to send PR to Mauro soon if there
>>> are no issues:
>>>
>>> <URL:https://git.linuxtv.org/sailus/media_tree.git/log/?h=of-match-ptr>
>>
>> One month later, I still don't see this set in the linux-next.  What's
>> happening here?
>>
> 
> I don't think this was merged in v6.5-rc1. It's not in linux-next,
> either. Another month passed...

I'm really hopeful it will be merged very soon. As you may have noticed (!) we
have problems with maintaining the media subsystem (too many patches, not enough
maintainers). We had a meeting about that a few weeks ago and are working on
it.

The core problem for this particular series was that the PR containing this series
was posted much too late (as in three months too late!), and it came in too late
for 6.5.

Regards,

	Hans