Message ID | 20230312131318.351173-1-krzysztof.kozlowski@linaro.org |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp712814wrd; Sun, 12 Mar 2023 06:17:45 -0700 (PDT) X-Google-Smtp-Source: AK7set9Ll/U6nZfhw48HwxMCQiSdafagPLC0madDHt88O8vNKe3mX9mtcKVRhQDHsYUiIk8t2jzO X-Received: by 2002:a17:902:ce92:b0:19c:f8c5:d504 with SMTP id f18-20020a170902ce9200b0019cf8c5d504mr13564782plg.59.1678627065019; Sun, 12 Mar 2023 06:17:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1678627065; cv=none; d=google.com; s=arc-20160816; b=zoybQLPKV855rhJH28GUHUMUFoFq00OmO7xAund6/H9ti4nqJOhj1JIY2TIHhAsirR /HCYpFcRon3qDx9qQ4LQVTAummNQJet5u020tFqv6tDqCK/f/E2LQ+CtymjHWnubpOXM lws5at5GIMqD4tYV+of7IauPDUwle4dG7+bUmZo/cUaSpFUILWVPfOtfgzd7Em/bGbH+ K8Tke7GrA2aeCnap+Jn//P3l8DQ6uXJYKMWgTzmb/lJvgpghjyRChs/bmX7RtESNQahX uDkSXH2chVlnFqRUdyQ0VHCX4quXzbOiSntOusAzDnhvcp+z2HEaWMOdzm3gjAVMZVxN PrCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=mjvZGlI9gqxKHJb6Zb5Fkc5S1/fsh/NgaCfi/T9v11Y=; b=CyPnEkWWRIsPAXVuCZVM0y0tpgNHXcMCMOLCVlNgYrqg4ekl02Ip5/vEcboWpyZzds BFXKuue95bPpPRsk+1DUZYOwa41Spk6EjDBc3iMn0JRUriqzkgqC+bCodhztxTJ08rBB bnsQtQGxllB3oYDYimyfY0nIX/qjpsqFhGKmqQ6cDcki+sOoNC7nFc0umByFS7SiVle2 muaCoYtNkcRIh+HWbEeS0BrojuNdD2okOXChtAUoGXeCHUc+TO2B5XcJ6qqmq+VApfmt 5u0eOJ1hWbTLGh9tsqufxoLvDA3lfMMYWyDPyTzs4poyaCsu1M8q2UYIHKd8eDGw5j4C pd/w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=lKlYxxhY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b18-20020a170902d51200b0019b7766afdfsi4633877plg.626.2023.03.12.06.17.30; Sun, 12 Mar 2023 06:17:44 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=lKlYxxhY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229768AbjCLNNb (ORCPT <rfc822;realc9580@gmail.com> + 99 others); Sun, 12 Mar 2023 09:13:31 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41994 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229668AbjCLNN2 (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Sun, 12 Mar 2023 09:13:28 -0400 Received: from mail-ed1-x52e.google.com (mail-ed1-x52e.google.com [IPv6:2a00:1450:4864:20::52e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 424971F93D for <linux-kernel@vger.kernel.org>; Sun, 12 Mar 2023 06:13:27 -0700 (PDT) Received: by mail-ed1-x52e.google.com with SMTP id cy23so38394667edb.12 for <linux-kernel@vger.kernel.org>; Sun, 12 Mar 2023 06:13:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1678626806; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=mjvZGlI9gqxKHJb6Zb5Fkc5S1/fsh/NgaCfi/T9v11Y=; b=lKlYxxhYCKn0JxPwkYKx3vASmrkH+srcYSRHily2tzLUxFl3QklS8Q4NAB9udLBZho dDGPZTBINPga+76lbL33R0Ue0ftkVUUQLxJzi4w9KwBDni2xOXZwaZzbSux02prlPDHP vJ7nGGelocPKxiZAPsodErx2JPF3u6mnEqcDFc2Wm/P0mWtV1EylhbCj0fTAm6z6v6t4 CdJ8Mm8qbbJwTOuhYW9bDTdnFKTImZMFdRSyvSmbjiOQG5UwzZqJrRtqjTlrTWLJdknY zHGpr1crUtQGblfLv3akSTMscNClIduQLP5C6mBhyVCwurXuWWNgdYQ93BE30x0tKpwz Z2KA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678626806; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=mjvZGlI9gqxKHJb6Zb5Fkc5S1/fsh/NgaCfi/T9v11Y=; b=UtmZLR7+w45kwKuMNl8lmyDO4VmDQJ0vFmrumJoxkDnvlItjtBVQj8pdfEt6P5zZ9j L6f2cKog99kPJdgOSm5mMkTmi945wivkHmP99Jc5B/emA5JJJ3+GVMuxleJ0JUfxS0PF oppveVn6YBcRC1iaNWIkXzTi9sKm6I6qNauCmJFWweYjMTya5Xn6GBDGdNMclhrvRy5S ULCvvvF6zIgIVyE3lTQc8HDxk76yQ4xcSeWAYdwZMGFURRB2Q7hMUtJGdrq9hehekzaT AXLqUF6HmpOspeLLBygCz+bhL9zGKLBHlRqtgZxk2/yD55uaWsDYAbj4jwWKqHF2xGhJ OP2Q== X-Gm-Message-State: AO0yUKUe/kejOsFEKDqYVSv/HTfDUjykLLnYIe0uz6xFxNFHsEwIGBLL pgBTDMnktYt/QRObiqpZFjarDA== X-Received: by 2002:a17:907:98e6:b0:929:43b5:8e69 with SMTP id ke6-20020a17090798e600b0092943b58e69mr166365ejc.40.1678626805682; Sun, 12 Mar 2023 06:13:25 -0700 (PDT) Received: from krzk-bin.. ([2a02:810d:15c0:828:d9f6:3e61:beeb:295a]) by smtp.gmail.com with ESMTPSA id r19-20020a1709067fd300b008eddbd46d7esm2213279ejs.31.2023.03.12.06.13.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 12 Mar 2023 06:13:25 -0700 (PDT) From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: Joe Tessler <jrt@google.com>, Hans Verkuil <hverkuil-cisco@xs4all.nl>, Mauro Carvalho Chehab <mchehab@kernel.org>, Neil Armstrong <neil.armstrong@linaro.org>, Kevin Hilman <khilman@baylibre.com>, Jerome Brunet <jbrunet@baylibre.com>, Martin Blumenstingl <martin.blumenstingl@googlemail.com>, Thierry Reding <thierry.reding@gmail.com>, Jonathan Hunter <jonathanh@nvidia.com>, Manivannan Sadhasivam <mani@kernel.org>, Michael Tretter <m.tretter@pengutronix.de>, Pengutronix Kernel Team <kernel@pengutronix.de>, Jacopo Mondi <jacopo+renesas@jmondi.org>, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, =?utf-8?q?Nik?= =?utf-8?q?las_S=C3=B6derlund?= <niklas.soderlund+renesas@ragnatech.se>, Rui Miguel Silva <rmfrfs@gmail.com>, Wenyou Yang <wenyou.yang@microchip.com>, Bin Liu <bin.liu@mediatek.com>, Matthias Brugger <matthias.bgg@gmail.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, Minghsiu Tsai <minghsiu.tsai@mediatek.com>, Houlong Wei <houlong.wei@mediatek.com>, Andrew-CT Chen <andrew-ct.chen@mediatek.com>, Andrzej Pietrasiewicz <andrzejtp2010@gmail.com>, Jacek Anaszewski <jacek.anaszewski@gmail.com>, Sylwester Nawrocki <s.nawrocki@samsung.com>, Patrice Chotard <patrice.chotard@foss.st.com>, Yong Deng <yong.deng@magewell.com>, Paul Kocialkowski <paul.kocialkowski@bootlin.com>, Chen-Yu Tsai <wens@csie.org>, Jernej Skrabec <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org>, "Lad, Prabhakar" <prabhakar.csengg@gmail.com>, Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>, Philipp Zabel <p.zabel@pengutronix.de>, Sean Young <sean@mess.org>, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-rockchip@lists.infradead.org Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Subject: [PATCH 01/28] media: cec: ch7322: drop of_match_ptr for ID table Date: Sun, 12 Mar 2023 14:12:51 +0100 Message-Id: <20230312131318.351173-1-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1760168053270281452?= X-GMAIL-MSGID: =?utf-8?q?1760168053270281452?= |
Series |
[01/28] media: cec: ch7322: drop of_match_ptr for ID table
|
|
Commit Message
Krzysztof Kozlowski
March 12, 2023, 1:12 p.m. UTC
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@linaro.org>
---
drivers/media/cec/i2c/ch7322.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
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@linaro.org> Reviewed-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> I assume you want to take this series, but if you prefer to have us do it, then just let me know. Thanks! Hans > --- > drivers/media/cec/i2c/ch7322.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/cec/i2c/ch7322.c b/drivers/media/cec/i2c/ch7322.c > index 34fad7123704..3c6e6496a001 100644 > --- a/drivers/media/cec/i2c/ch7322.c > +++ b/drivers/media/cec/i2c/ch7322.c > @@ -589,7 +589,7 @@ MODULE_DEVICE_TABLE(of, ch7322_of_match); > static struct i2c_driver ch7322_i2c_driver = { > .driver = { > .name = "ch7322", > - .of_match_table = of_match_ptr(ch7322_of_match), > + .of_match_table = ch7322_of_match, > }, > .probe_new = ch7322_probe, > .remove = ch7322_remove,
On Sun, Mar 12, 2023 at 02:12:51PM +0100, 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@linaro.org> Reviewed-by: Guenter Roeck <linux@roeck-us.net> FWIW: There is also https://patches.linaro.org/project/linux-media/patch/20230215214724.3798917-1-linux@roeck-us.net/ The lack of ACPI support is a real problem and very relevant. Guenter > --- > drivers/media/cec/i2c/ch7322.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/cec/i2c/ch7322.c b/drivers/media/cec/i2c/ch7322.c > index 34fad7123704..3c6e6496a001 100644 > --- a/drivers/media/cec/i2c/ch7322.c > +++ b/drivers/media/cec/i2c/ch7322.c > @@ -589,7 +589,7 @@ MODULE_DEVICE_TABLE(of, ch7322_of_match); > static struct i2c_driver ch7322_i2c_driver = { > .driver = { > .name = "ch7322", > - .of_match_table = of_match_ptr(ch7322_of_match), > + .of_match_table = ch7322_of_match, > }, > .probe_new = ch7322_probe, > .remove = ch7322_remove,
On 13/03/2023 23:01, Guenter Roeck wrote: > On Sun, Mar 12, 2023 at 02:12:51PM +0100, 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@linaro.org> > > Reviewed-by: Guenter Roeck <linux@roeck-us.net> > > FWIW: There is also > https://patches.linaro.org/project/linux-media/patch/20230215214724.3798917-1-linux@roeck-us.net/ I superseded that patch in patchwork in favor of this patch since this is part of a larger series. Krzysztof, please let me know if you will take this series yourself or want me to make a PR for it. Regards, Hans > > The lack of ACPI support is a real problem and very relevant. > > Guenter > >> --- >> drivers/media/cec/i2c/ch7322.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/media/cec/i2c/ch7322.c b/drivers/media/cec/i2c/ch7322.c >> index 34fad7123704..3c6e6496a001 100644 >> --- a/drivers/media/cec/i2c/ch7322.c >> +++ b/drivers/media/cec/i2c/ch7322.c >> @@ -589,7 +589,7 @@ MODULE_DEVICE_TABLE(of, ch7322_of_match); >> static struct i2c_driver ch7322_i2c_driver = { >> .driver = { >> .name = "ch7322", >> - .of_match_table = of_match_ptr(ch7322_of_match), >> + .of_match_table = ch7322_of_match, >> }, >> .probe_new = ch7322_probe, >> .remove = ch7322_remove,
On 13/03/2023 12:46, Hans Verkuil 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@linaro.org> > > Reviewed-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> > > I assume you want to take this series, but if you prefer to have us do it, then > just let me know. I prefer if you to take it. Please grab it! :) Best regards, Krzysztof
On 14/03/2023 09:34, Hans Verkuil wrote: > On 13/03/2023 23:01, Guenter Roeck wrote: >> On Sun, Mar 12, 2023 at 02:12:51PM +0100, 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@linaro.org> >> >> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >> >> FWIW: There is also >> https://patches.linaro.org/project/linux-media/patch/20230215214724.3798917-1-linux@roeck-us.net/ > > I superseded that patch in patchwork in favor of this patch since this is part of > a larger series. I am personally fan of FIFO, so Guenter's patch should go in. > > Krzysztof, please let me know if you will take this series yourself or want me to > make a PR for it. I prefer if you take entire patchset. Best regards, Krzysztof
On 3/14/23 09:32, Krzysztof Kozlowski wrote: > On 14/03/2023 09:34, Hans Verkuil wrote: >> On 13/03/2023 23:01, Guenter Roeck wrote: >>> On Sun, Mar 12, 2023 at 02:12:51PM +0100, 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@linaro.org> >>> >>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>> >>> FWIW: There is also >>> https://patches.linaro.org/project/linux-media/patch/20230215214724.3798917-1-linux@roeck-us.net/ >> >> I superseded that patch in patchwork in favor of this patch since this is part of >> a larger series. > > I am personally fan of FIFO, so Guenter's patch should go in. > FWIW, I do not care either way, I just wanted to point out that this patch _is_ both relevant and needed, and that it (or, rather, the original version) has been reviewed and tested. Thanks, Guenter
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@linaro.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. Best regards, Krzysztof
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@linaro.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? Regards, Hans > > Best regards, > Krzysztof >
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@linaro.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.
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@linaro.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>
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@linaro.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? Best regards, Krzysztof
On Sun, Mar 12, 2023 at 02:12:51PM +0100, 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@linaro.org> What happened wit hthis patch ? I still don't see it applied. FWIW: Reviewed-and-tested-by: Guenter Roeck <linux@roeck-us.net> Thanks, Guenter > --- > drivers/media/cec/i2c/ch7322.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/cec/i2c/ch7322.c b/drivers/media/cec/i2c/ch7322.c > index 34fad7123704..3c6e6496a001 100644 > --- a/drivers/media/cec/i2c/ch7322.c > +++ b/drivers/media/cec/i2c/ch7322.c > @@ -589,7 +589,7 @@ MODULE_DEVICE_TABLE(of, ch7322_of_match); > static struct i2c_driver ch7322_i2c_driver = { > .driver = { > .name = "ch7322", > - .of_match_table = of_match_ptr(ch7322_of_match), > + .of_match_table = ch7322_of_match, > }, > .probe_new = ch7322_probe, > .remove = ch7322_remove, > -- > 2.34.1 >
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@linaro.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... Best regards, Krzysztof
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@linaro.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
Hi Krzysztof, On Thu, Jul 13, 2023 at 08:46:58AM +0200, 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@linaro.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... These are now in the media-stage tree <URL:https://git.linuxtv.org/media_stage.git/log/>. Assuming nothing goes wrong, these should end up in Linus's tree for 6.6. The issues related to media tree maintenance are being addressed as we're changing the process how the tree is maintained. The patches will eventually get in still, also expect this to improve in the future.
On 25/07/2023 12:51, Sakari Ailus wrote: >> >> I don't think this was merged in v6.5-rc1. It's not in linux-next, >> either. Another month passed... > > These are now in the media-stage tree > <URL:https://git.linuxtv.org/media_stage.git/log/>. Assuming nothing goes > wrong, these should end up in Linus's tree for 6.6. Great, thank you! Best regards, Krzysztof
diff --git a/drivers/media/cec/i2c/ch7322.c b/drivers/media/cec/i2c/ch7322.c index 34fad7123704..3c6e6496a001 100644 --- a/drivers/media/cec/i2c/ch7322.c +++ b/drivers/media/cec/i2c/ch7322.c @@ -589,7 +589,7 @@ MODULE_DEVICE_TABLE(of, ch7322_of_match); static struct i2c_driver ch7322_i2c_driver = { .driver = { .name = "ch7322", - .of_match_table = of_match_ptr(ch7322_of_match), + .of_match_table = ch7322_of_match, }, .probe_new = ch7322_probe, .remove = ch7322_remove,