From: kernel test robot <lkp@intel.com> To: Prashant Malani <pmalani@chromium.org>, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Cc: kbuild-all@lists.01.org, heikki.krogerus@linux.intel.com, "Andrzej Hajda" <andrzej.hajda@intel.com>, "Neil Armstrong" <narmstrong@baylibre.com>, "David Airlie" <airlied@linux.ie>, "open list:DRM DRIVERS" <dri-devel@lists.freedesktop.org>, "Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>, "Krzysztof Kozlowski" <krzk@kernel.org>, "Sam Ravnborg" <sam@ravnborg.org>, "Jernej Skrabec" <jernej.skrabec@gmail.com>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, "Thomas Zimmermann" <tzimmermann@suse.de>, "Nícolas F. R. A. Prado" <nfraprado@collabora.com>, "Jonas Karlman" <jonas@kwiboo.se>, swboyd@chromium.org, "Pin-Yen Lin" <treapking@chromium.org>, "Rob Herring" <robh+dt@kernel.org>, "Maxime Ripard" <maxime@cerno.tech>, "Hsin-Yi Wang" <hsinyi@chromium.org>, "Xin Ji" <xji@analogixsemi.com>, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, "Robert Foss" <robert.foss@linaro.org>, "Prashant Malani" <pmalani@chromium.org>, "José Expósito" <jose.exposito89@gmail.com> Subject: Re: [PATCH 7/7] drm/bridge: anx7625: Add typec_mux_set callback function Date: Thu, 9 Jun 2022 08:14:49 +0800 [thread overview] Message-ID: <202206090842.UVDwwfxx-lkp@intel.com> (raw) In-Reply-To: <20220607190131.1647511-8-pmalani@chromium.org> Hi Prashant, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm/drm-next] [also build test ERROR on usb/usb-testing v5.19-rc1 next-20220608] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/intel-lab-lkp/linux/commits/Prashant-Malani/usb-typec-Introduce-typec-switch-binding/20220608-042545 base: git://anongit.freedesktop.org/drm/drm drm-next config: nios2-buildonly-randconfig-r006-20220608 (https://download.01.org/0day-ci/archive/20220609/202206090842.UVDwwfxx-lkp@intel.com/config) compiler: nios2-linux-gcc (GCC) 11.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/intel-lab-lkp/linux/commit/49c4c89ed5b50cbd623e611c8f4eba4b5ca9dd02 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Prashant-Malani/usb-typec-Introduce-typec-switch-binding/20220608-042545 git checkout 49c4c89ed5b50cbd623e611c8f4eba4b5ca9dd02 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=nios2 SHELL=/bin/bash drivers/gpu/drm/bridge/analogix/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:83:19: warning: no previous prototype for 'fwnode_typec_mux_get' [-Wmissing-prototypes] 83 | struct typec_mux *fwnode_typec_mux_get(struct fwnode_handle *fwnode, | ^~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:89:6: warning: no previous prototype for 'typec_mux_put' [-Wmissing-prototypes] 89 | void typec_mux_put(struct typec_mux *mux) {} | ^~~~~~~~~~~~~ include/linux/usb/typec_mux.h:91:5: warning: no previous prototype for 'typec_mux_set' [-Wmissing-prototypes] 91 | int typec_mux_set(struct typec_mux *mux, struct typec_mux_state *state) | ^~~~~~~~~~~~~ include/linux/usb/typec_mux.h:103:1: warning: no previous prototype for 'typec_mux_register' [-Wmissing-prototypes] 103 | typec_mux_register(struct device *parent, const struct typec_mux_desc *desc) | ^~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:107:6: warning: no previous prototype for 'typec_mux_unregister' [-Wmissing-prototypes] 107 | void typec_mux_unregister(struct typec_mux *mux) {} | ^~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:109:6: warning: no previous prototype for 'typec_mux_set_drvdata' [-Wmissing-prototypes] 109 | void typec_mux_set_drvdata(struct typec_mux *mux, void *data) {} | ^~~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:110:7: warning: no previous prototype for 'typec_mux_get_drvdata' [-Wmissing-prototypes] 110 | void *typec_mux_get_drvdata(struct typec_mux *mux) | ^~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_typec_mux_set': >> drivers/gpu/drm/bridge/analogix/anx7625.c:2616:64: error: passing argument 1 of 'typec_mux_get_drvdata' from incompatible pointer type [-Werror=incompatible-pointer-types] 2616 | struct anx7625_port_data *data = typec_mux_get_drvdata(mux); | ^~~ | | | struct typec_mux_dev * In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:110:47: note: expected 'struct typec_mux *' but argument is of type 'struct typec_mux_dev *' 110 | void *typec_mux_get_drvdata(struct typec_mux *mux) | ~~~~~~~~~~~~~~~~~~^~~ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_register_mode_switch': drivers/gpu/drm/bridge/analogix/anx7625.c:2675:30: error: assignment to 'struct typec_mux_dev *' from incompatible pointer type 'struct typec_mux *' [-Werror=incompatible-pointer-types] 2675 | port_data->typec_mux = typec_mux_register(dev, &mux_desc); | ^ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_unregister_typec_switches': drivers/gpu/drm/bridge/analogix/anx7625.c:2689:57: error: passing argument 1 of 'typec_mux_unregister' from incompatible pointer type [-Werror=incompatible-pointer-types] 2689 | typec_mux_unregister(ctx->typec_ports[i].typec_mux); | ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~ | | | struct typec_mux_dev * In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:107:45: note: expected 'struct typec_mux *' but argument is of type 'struct typec_mux_dev *' 107 | void typec_mux_unregister(struct typec_mux *mux) {} | ~~~~~~~~~~~~~~~~~~^~~ cc1: some warnings being treated as errors vim +/typec_mux_get_drvdata +2616 drivers/gpu/drm/bridge/analogix/anx7625.c 2612 2613 static int anx7625_typec_mux_set(struct typec_mux_dev *mux, 2614 struct typec_mux_state *state) 2615 { > 2616 struct anx7625_port_data *data = typec_mux_get_drvdata(mux); 2617 struct anx7625_data *ctx = data->ctx; 2618 struct device *dev = &ctx->client->dev; 2619 2620 bool old_dp_connected = (ctx->typec_ports[0].dp_connected || 2621 ctx->typec_ports[1].dp_connected); 2622 bool new_dp_connected; 2623 2624 if (ctx->num_typec_switches == 1) 2625 return 0; 2626 2627 dev_dbg(dev, "mux_set dp_connected: c0=%d, c1=%d\n", 2628 ctx->typec_ports[0].dp_connected, ctx->typec_ports[1].dp_connected); 2629 2630 data->dp_connected = (state->alt && state->alt->svid == USB_TYPEC_DP_SID && 2631 state->alt->mode == USB_TYPEC_DP_MODE); 2632 2633 new_dp_connected = (ctx->typec_ports[0].dp_connected || 2634 ctx->typec_ports[1].dp_connected); 2635 2636 /* dp on, power on first */ 2637 if (!old_dp_connected && new_dp_connected) 2638 pm_runtime_get_sync(dev); 2639 2640 anx7625_typec_two_ports_update(ctx); 2641 2642 /* dp off, power off last */ 2643 if (old_dp_connected && !new_dp_connected) 2644 pm_runtime_put_sync(dev); 2645 2646 return 0; 2647 } 2648 -- 0-DAY CI Kernel Test Service https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: Prashant Malani <pmalani@chromium.org>, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Cc: heikki.krogerus@linux.intel.com, "Neil Armstrong" <narmstrong@baylibre.com>, "David Airlie" <airlied@linux.ie>, "open list:DRM DRIVERS" <dri-devel@lists.freedesktop.org>, "Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>, "Andrzej Hajda" <andrzej.hajda@intel.com>, "Sam Ravnborg" <sam@ravnborg.org>, "Krzysztof Kozlowski" <krzk@kernel.org>, "Jernej Skrabec" <jernej.skrabec@gmail.com>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, "Nícolas F. R. A. Prado" <nfraprado@collabora.com>, "Jonas Karlman" <jonas@kwiboo.se>, swboyd@chromium.org, "Pin-Yen Lin" <treapking@chromium.org>, "Rob Herring" <robh+dt@kernel.org>, "Maxime Ripard" <maxime@cerno.tech>, "Hsin-Yi Wang" <hsinyi@chromium.org>, "Xin Ji" <xji@analogixsemi.com>, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, kbuild-all@lists.01.org, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, "Robert Foss" <robert.foss@linaro.org>, "Prashant Malani" <pmalani@chromium.org>, "Thomas Zimmermann" <tzimmermann@suse.de>, "José Expósito" <jose.exposito89@gmail.com> Subject: Re: [PATCH 7/7] drm/bridge: anx7625: Add typec_mux_set callback function Date: Thu, 9 Jun 2022 08:14:49 +0800 [thread overview] Message-ID: <202206090842.UVDwwfxx-lkp@intel.com> (raw) In-Reply-To: <20220607190131.1647511-8-pmalani@chromium.org> Hi Prashant, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm/drm-next] [also build test ERROR on usb/usb-testing v5.19-rc1 next-20220608] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/intel-lab-lkp/linux/commits/Prashant-Malani/usb-typec-Introduce-typec-switch-binding/20220608-042545 base: git://anongit.freedesktop.org/drm/drm drm-next config: nios2-buildonly-randconfig-r006-20220608 (https://download.01.org/0day-ci/archive/20220609/202206090842.UVDwwfxx-lkp@intel.com/config) compiler: nios2-linux-gcc (GCC) 11.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/intel-lab-lkp/linux/commit/49c4c89ed5b50cbd623e611c8f4eba4b5ca9dd02 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Prashant-Malani/usb-typec-Introduce-typec-switch-binding/20220608-042545 git checkout 49c4c89ed5b50cbd623e611c8f4eba4b5ca9dd02 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=nios2 SHELL=/bin/bash drivers/gpu/drm/bridge/analogix/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:83:19: warning: no previous prototype for 'fwnode_typec_mux_get' [-Wmissing-prototypes] 83 | struct typec_mux *fwnode_typec_mux_get(struct fwnode_handle *fwnode, | ^~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:89:6: warning: no previous prototype for 'typec_mux_put' [-Wmissing-prototypes] 89 | void typec_mux_put(struct typec_mux *mux) {} | ^~~~~~~~~~~~~ include/linux/usb/typec_mux.h:91:5: warning: no previous prototype for 'typec_mux_set' [-Wmissing-prototypes] 91 | int typec_mux_set(struct typec_mux *mux, struct typec_mux_state *state) | ^~~~~~~~~~~~~ include/linux/usb/typec_mux.h:103:1: warning: no previous prototype for 'typec_mux_register' [-Wmissing-prototypes] 103 | typec_mux_register(struct device *parent, const struct typec_mux_desc *desc) | ^~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:107:6: warning: no previous prototype for 'typec_mux_unregister' [-Wmissing-prototypes] 107 | void typec_mux_unregister(struct typec_mux *mux) {} | ^~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:109:6: warning: no previous prototype for 'typec_mux_set_drvdata' [-Wmissing-prototypes] 109 | void typec_mux_set_drvdata(struct typec_mux *mux, void *data) {} | ^~~~~~~~~~~~~~~~~~~~~ include/linux/usb/typec_mux.h:110:7: warning: no previous prototype for 'typec_mux_get_drvdata' [-Wmissing-prototypes] 110 | void *typec_mux_get_drvdata(struct typec_mux *mux) | ^~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_typec_mux_set': >> drivers/gpu/drm/bridge/analogix/anx7625.c:2616:64: error: passing argument 1 of 'typec_mux_get_drvdata' from incompatible pointer type [-Werror=incompatible-pointer-types] 2616 | struct anx7625_port_data *data = typec_mux_get_drvdata(mux); | ^~~ | | | struct typec_mux_dev * In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:110:47: note: expected 'struct typec_mux *' but argument is of type 'struct typec_mux_dev *' 110 | void *typec_mux_get_drvdata(struct typec_mux *mux) | ~~~~~~~~~~~~~~~~~~^~~ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_register_mode_switch': drivers/gpu/drm/bridge/analogix/anx7625.c:2675:30: error: assignment to 'struct typec_mux_dev *' from incompatible pointer type 'struct typec_mux *' [-Werror=incompatible-pointer-types] 2675 | port_data->typec_mux = typec_mux_register(dev, &mux_desc); | ^ drivers/gpu/drm/bridge/analogix/anx7625.c: In function 'anx7625_unregister_typec_switches': drivers/gpu/drm/bridge/analogix/anx7625.c:2689:57: error: passing argument 1 of 'typec_mux_unregister' from incompatible pointer type [-Werror=incompatible-pointer-types] 2689 | typec_mux_unregister(ctx->typec_ports[i].typec_mux); | ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~ | | | struct typec_mux_dev * In file included from drivers/gpu/drm/bridge/analogix/anx7625.c:19: include/linux/usb/typec_mux.h:107:45: note: expected 'struct typec_mux *' but argument is of type 'struct typec_mux_dev *' 107 | void typec_mux_unregister(struct typec_mux *mux) {} | ~~~~~~~~~~~~~~~~~~^~~ cc1: some warnings being treated as errors vim +/typec_mux_get_drvdata +2616 drivers/gpu/drm/bridge/analogix/anx7625.c 2612 2613 static int anx7625_typec_mux_set(struct typec_mux_dev *mux, 2614 struct typec_mux_state *state) 2615 { > 2616 struct anx7625_port_data *data = typec_mux_get_drvdata(mux); 2617 struct anx7625_data *ctx = data->ctx; 2618 struct device *dev = &ctx->client->dev; 2619 2620 bool old_dp_connected = (ctx->typec_ports[0].dp_connected || 2621 ctx->typec_ports[1].dp_connected); 2622 bool new_dp_connected; 2623 2624 if (ctx->num_typec_switches == 1) 2625 return 0; 2626 2627 dev_dbg(dev, "mux_set dp_connected: c0=%d, c1=%d\n", 2628 ctx->typec_ports[0].dp_connected, ctx->typec_ports[1].dp_connected); 2629 2630 data->dp_connected = (state->alt && state->alt->svid == USB_TYPEC_DP_SID && 2631 state->alt->mode == USB_TYPEC_DP_MODE); 2632 2633 new_dp_connected = (ctx->typec_ports[0].dp_connected || 2634 ctx->typec_ports[1].dp_connected); 2635 2636 /* dp on, power on first */ 2637 if (!old_dp_connected && new_dp_connected) 2638 pm_runtime_get_sync(dev); 2639 2640 anx7625_typec_two_ports_update(ctx); 2641 2642 /* dp off, power off last */ 2643 if (old_dp_connected && !new_dp_connected) 2644 pm_runtime_put_sync(dev); 2645 2646 return 0; 2647 } 2648 -- 0-DAY CI Kernel Test Service https://01.org/lkp
next prev parent reply other threads:[~2022-06-09 0:15 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-06-07 19:00 [PATCH 0/7] usb: typec: Introduce typec-switch binding Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-07 19:00 ` [PATCH 1/7] usb: typec: mux: Allow muxes to specify mode-switch Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-07 19:00 ` [PATCH 2/7] usb: typec: mux: Add CONFIG guards for functions Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-08 20:13 ` Prashant Malani 2022-06-08 20:13 ` Prashant Malani 2022-06-07 19:00 ` [PATCH 3/7] dt-bindings: usb: Add Type-C switch binding Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-08 9:18 ` Krzysztof Kozlowski 2022-06-08 9:18 ` Krzysztof Kozlowski 2022-06-08 9:21 ` Krzysztof Kozlowski 2022-06-08 9:21 ` Krzysztof Kozlowski 2022-06-08 17:15 ` Prashant Malani 2022-06-08 17:15 ` Prashant Malani 2022-06-09 6:43 ` Krzysztof Kozlowski 2022-06-09 6:43 ` Krzysztof Kozlowski 2022-06-07 19:00 ` [PATCH 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-08 9:24 ` Krzysztof Kozlowski 2022-06-08 9:24 ` Krzysztof Kozlowski 2022-06-08 17:08 ` Prashant Malani 2022-06-08 17:08 ` Prashant Malani 2022-06-08 21:56 ` Prashant Malani 2022-06-08 21:56 ` Prashant Malani 2022-06-09 6:41 ` Krzysztof Kozlowski 2022-06-09 6:41 ` Krzysztof Kozlowski 2022-06-09 18:24 ` Prashant Malani 2022-06-09 18:24 ` Prashant Malani 2022-06-07 19:00 ` [PATCH 5/7] drm/bridge: anx7625: Register number of Type C switches Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-07 19:00 ` [PATCH 6/7] drm/bridge: anx7625: Register Type-C mode switches Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-08 18:14 ` kernel test robot 2022-06-08 18:14 ` kernel test robot 2022-06-08 18:35 ` kernel test robot 2022-06-08 18:35 ` kernel test robot 2022-06-08 22:31 ` kernel test robot 2022-06-08 22:31 ` kernel test robot 2022-06-07 19:00 ` [PATCH 7/7] drm/bridge: anx7625: Add typec_mux_set callback function Prashant Malani 2022-06-07 19:00 ` Prashant Malani 2022-06-09 0:14 ` kernel test robot [this message] 2022-06-09 0:14 ` kernel test robot
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=202206090842.UVDwwfxx-lkp@intel.com \ --to=lkp@intel.com \ --cc=Laurent.pinchart@ideasonboard.com \ --cc=airlied@linux.ie \ --cc=andrzej.hajda@intel.com \ --cc=angelogioacchino.delregno@collabora.com \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=gregkh@linuxfoundation.org \ --cc=heikki.krogerus@linux.intel.com \ --cc=hsinyi@chromium.org \ --cc=jernej.skrabec@gmail.com \ --cc=jonas@kwiboo.se \ --cc=jose.exposito89@gmail.com \ --cc=kbuild-all@lists.01.org \ --cc=krzk@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=maxime@cerno.tech \ --cc=narmstrong@baylibre.com \ --cc=nfraprado@collabora.com \ --cc=pmalani@chromium.org \ --cc=robert.foss@linaro.org \ --cc=robh+dt@kernel.org \ --cc=sam@ravnborg.org \ --cc=swboyd@chromium.org \ --cc=treapking@chromium.org \ --cc=tzimmermann@suse.de \ --cc=xji@analogixsemi.com \ /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: linkBe 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.