From: kernel test robot <lkp@intel.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
Abhinav Kumar <quic_abhinavk@quicinc.com>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
devicetree@vger.kernel.org, David Airlie <airlied@linux.ie>,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
Stephen Boyd <swboyd@chromium.org>,
freedreno@lists.freedesktop.org
Subject: Re: [PATCH v2 06/12] drm/msm/hdmi: drop unused GPIO support
Date: Wed, 8 Jun 2022 23:57:36 +0800 [thread overview]
Message-ID: <202206082312.XB745jWy-lkp@intel.com> (raw)
In-Reply-To: <20220608120723.2987843-7-dmitry.baryshkov@linaro.org>
Hi Dmitry,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next linus/master 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/Dmitry-Baryshkov/drm-msm-hdmi-YAML-ify-schema-and-cleanup-some-platform-properties/20220608-200925
base: git://anongit.freedesktop.org/drm/drm drm-next
config: hexagon-randconfig-r045-20220608 (https://download.01.org/0day-ci/archive/20220608/202206082312.XB745jWy-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project b92436efcb7813fc481b30f2593a4907568d917a)
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/c3e263fe0a077b382c2a76911c8ace385bd59a4c
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Dmitry-Baryshkov/drm-msm-hdmi-YAML-ify-schema-and-cleanup-some-platform-properties/20220608-200925
git checkout c3e263fe0a077b382c2a76911c8ace385bd59a4c
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/msm/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/msm/hdmi/hdmi.c:532:2: warning: variable 'hdmi' is uninitialized when used here [-Wuninitialized]
hdmi->hpd_gpiod = devm_gpiod_get_optional(dev, "hpd", GPIOD_IN);
^~~~
drivers/gpu/drm/msm/hdmi/hdmi.c:518:19: note: initialize the variable 'hdmi' to silence this warning
struct hdmi *hdmi;
^
= NULL
1 warning generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for DRM_DP_AUX_BUS
Depends on HAS_IOMEM && DRM && OF
Selected by
- DRM_MSM && HAS_IOMEM && DRM && (ARCH_QCOM || SOC_IMX5 || COMPILE_TEST && COMMON_CLK && IOMMU_SUPPORT && (QCOM_OCMEM || QCOM_OCMEM && (QCOM_LLCC || QCOM_LLCC && (QCOM_COMMAND_DB || QCOM_COMMAND_DB
vim +/hdmi +532 drivers/gpu/drm/msm/hdmi/hdmi.c
513
514 static int msm_hdmi_bind(struct device *dev, struct device *master, void *data)
515 {
516 struct msm_drm_private *priv = dev_get_drvdata(master);
517 struct hdmi_platform_config *hdmi_cfg;
518 struct hdmi *hdmi;
519 struct device_node *of_node = dev->of_node;
520 int err;
521
522 hdmi_cfg = (struct hdmi_platform_config *)
523 of_device_get_match_data(dev);
524 if (!hdmi_cfg) {
525 DRM_DEV_ERROR(dev, "unknown hdmi_cfg: %pOFn\n", of_node);
526 return -ENXIO;
527 }
528
529 hdmi_cfg->mmio_name = "core_physical";
530 hdmi_cfg->qfprom_mmio_name = "qfprom_physical";
531
> 532 hdmi->hpd_gpiod = devm_gpiod_get_optional(dev, "hpd", GPIOD_IN);
533 /* This will catch e.g. -PROBE_DEFER */
534 if (IS_ERR(hdmi->hpd_gpiod))
535 return PTR_ERR(hdmi->hpd_gpiod);
536
537 if (!hdmi->hpd_gpiod)
538 DBG("failed to get HPD gpio");
539
540 if (hdmi->hpd_gpiod)
541 gpiod_set_consumer_name(hdmi->hpd_gpiod, "HDMI_HPD");
542
543 dev->platform_data = hdmi_cfg;
544
545 hdmi = msm_hdmi_init(to_platform_device(dev));
546 if (IS_ERR(hdmi))
547 return PTR_ERR(hdmi);
548 priv->hdmi = hdmi;
549
550 err = msm_hdmi_register_audio_driver(hdmi, dev);
551 if (err) {
552 DRM_ERROR("Failed to attach an audio codec %d\n", err);
553 hdmi->audio_pdev = NULL;
554 }
555
556 return 0;
557 }
558
--
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: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
Abhinav Kumar <quic_abhinavk@quicinc.com>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>
Cc: devicetree@vger.kernel.org, kbuild-all@lists.01.org,
David Airlie <airlied@linux.ie>,
linux-arm-msm@vger.kernel.org, llvm@lists.linux.dev,
dri-devel@lists.freedesktop.org,
Stephen Boyd <swboyd@chromium.org>,
freedreno@lists.freedesktop.org
Subject: Re: [PATCH v2 06/12] drm/msm/hdmi: drop unused GPIO support
Date: Wed, 8 Jun 2022 23:57:36 +0800 [thread overview]
Message-ID: <202206082312.XB745jWy-lkp@intel.com> (raw)
In-Reply-To: <20220608120723.2987843-7-dmitry.baryshkov@linaro.org>
Hi Dmitry,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next linus/master 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/Dmitry-Baryshkov/drm-msm-hdmi-YAML-ify-schema-and-cleanup-some-platform-properties/20220608-200925
base: git://anongit.freedesktop.org/drm/drm drm-next
config: hexagon-randconfig-r045-20220608 (https://download.01.org/0day-ci/archive/20220608/202206082312.XB745jWy-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project b92436efcb7813fc481b30f2593a4907568d917a)
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/c3e263fe0a077b382c2a76911c8ace385bd59a4c
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Dmitry-Baryshkov/drm-msm-hdmi-YAML-ify-schema-and-cleanup-some-platform-properties/20220608-200925
git checkout c3e263fe0a077b382c2a76911c8ace385bd59a4c
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/msm/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/msm/hdmi/hdmi.c:532:2: warning: variable 'hdmi' is uninitialized when used here [-Wuninitialized]
hdmi->hpd_gpiod = devm_gpiod_get_optional(dev, "hpd", GPIOD_IN);
^~~~
drivers/gpu/drm/msm/hdmi/hdmi.c:518:19: note: initialize the variable 'hdmi' to silence this warning
struct hdmi *hdmi;
^
= NULL
1 warning generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for DRM_DP_AUX_BUS
Depends on HAS_IOMEM && DRM && OF
Selected by
- DRM_MSM && HAS_IOMEM && DRM && (ARCH_QCOM || SOC_IMX5 || COMPILE_TEST && COMMON_CLK && IOMMU_SUPPORT && (QCOM_OCMEM || QCOM_OCMEM && (QCOM_LLCC || QCOM_LLCC && (QCOM_COMMAND_DB || QCOM_COMMAND_DB
vim +/hdmi +532 drivers/gpu/drm/msm/hdmi/hdmi.c
513
514 static int msm_hdmi_bind(struct device *dev, struct device *master, void *data)
515 {
516 struct msm_drm_private *priv = dev_get_drvdata(master);
517 struct hdmi_platform_config *hdmi_cfg;
518 struct hdmi *hdmi;
519 struct device_node *of_node = dev->of_node;
520 int err;
521
522 hdmi_cfg = (struct hdmi_platform_config *)
523 of_device_get_match_data(dev);
524 if (!hdmi_cfg) {
525 DRM_DEV_ERROR(dev, "unknown hdmi_cfg: %pOFn\n", of_node);
526 return -ENXIO;
527 }
528
529 hdmi_cfg->mmio_name = "core_physical";
530 hdmi_cfg->qfprom_mmio_name = "qfprom_physical";
531
> 532 hdmi->hpd_gpiod = devm_gpiod_get_optional(dev, "hpd", GPIOD_IN);
533 /* This will catch e.g. -PROBE_DEFER */
534 if (IS_ERR(hdmi->hpd_gpiod))
535 return PTR_ERR(hdmi->hpd_gpiod);
536
537 if (!hdmi->hpd_gpiod)
538 DBG("failed to get HPD gpio");
539
540 if (hdmi->hpd_gpiod)
541 gpiod_set_consumer_name(hdmi->hpd_gpiod, "HDMI_HPD");
542
543 dev->platform_data = hdmi_cfg;
544
545 hdmi = msm_hdmi_init(to_platform_device(dev));
546 if (IS_ERR(hdmi))
547 return PTR_ERR(hdmi);
548 priv->hdmi = hdmi;
549
550 err = msm_hdmi_register_audio_driver(hdmi, dev);
551 if (err) {
552 DRM_ERROR("Failed to attach an audio codec %d\n", err);
553 hdmi->audio_pdev = NULL;
554 }
555
556 return 0;
557 }
558
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-06-08 15:57 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-08 12:07 [PATCH v2 00/12] drm/msm/hdmi: YAML-ify schema and cleanup some platform properties Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 12:07 ` [PATCH v2 01/12] dt-bindings: display/msm: hdmi: split and convert to yaml Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 12:37 ` Krzysztof Kozlowski
2022-06-08 12:37 ` Krzysztof Kozlowski
2022-06-08 22:45 ` David Heidelberg
2022-06-08 22:45 ` David Heidelberg
2022-06-09 12:09 ` Dmitry Baryshkov
2022-06-09 12:09 ` Dmitry Baryshkov
2022-06-09 12:10 ` Dmitry Baryshkov
2022-06-09 12:10 ` Dmitry Baryshkov
2022-06-08 12:07 ` [PATCH v2 02/12] dt-bindings: display/msm: hdmi: mark old GPIO properties as deprecated Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 12:37 ` Krzysztof Kozlowski
2022-06-08 12:37 ` Krzysztof Kozlowski
2022-06-08 20:56 ` Stephen Boyd
2022-06-08 20:56 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 03/12] dt-bindings: display/msm: hdmi: mark hdmi-mux-supply " Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 12:38 ` Krzysztof Kozlowski
2022-06-08 12:38 ` Krzysztof Kozlowski
2022-06-08 20:56 ` Stephen Boyd
2022-06-08 20:56 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 04/12] arm: dts: qcom: apq8064-ifc6410: drop hdmi-mux-supply Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 20:57 ` Stephen Boyd
2022-06-08 20:57 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 05/12] drm/msm/hdmi: drop the hdmi-mux support Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 20:57 ` Stephen Boyd
2022-06-08 20:57 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 06/12] drm/msm/hdmi: drop unused GPIO support Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 15:57 ` kernel test robot [this message]
2022-06-08 15:57 ` kernel test robot
2022-06-08 20:59 ` Stephen Boyd
2022-06-08 20:59 ` Stephen Boyd
2022-06-09 6:27 ` Dmitry Baryshkov
2022-06-09 6:27 ` Dmitry Baryshkov
2022-06-08 12:07 ` [PATCH v2 07/12] drm/msm/hdmi: enable core-vcc/core-vdda-supply for 8996 platform Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:02 ` Stephen Boyd
2022-06-08 21:02 ` Stephen Boyd
2022-06-08 22:58 ` Dmitry Baryshkov
2022-06-08 22:58 ` Dmitry Baryshkov
2022-06-08 12:07 ` [PATCH v2 08/12] drm/msm/hdmi: drop empty 'none' regulator lists Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:00 ` Stephen Boyd
2022-06-08 21:00 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 09/12] drm/msm/hdmi: drop hpd_regs usage on 8x74/8084 Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:03 ` Stephen Boyd
2022-06-08 21:03 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 10/12] drm/msm/hdmi: merge platform config for 8974/8084/8994/8996 Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:04 ` Stephen Boyd
2022-06-08 21:04 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 11/12] drm/msm/hdmi: reuse MSM8960's config for MSM8660 Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:04 ` Stephen Boyd
2022-06-08 21:04 ` Stephen Boyd
2022-06-08 12:07 ` [PATCH v2 12/12] drm/msm/hdmi-phy: populate 8x60 HDMI PHY requirements Dmitry Baryshkov
2022-06-08 12:07 ` Dmitry Baryshkov
2022-06-08 21:05 ` Stephen Boyd
2022-06-08 21:05 ` Stephen Boyd
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=202206082312.XB745jWy-lkp@intel.com \
--to=lkp@intel.com \
--cc=agross@kernel.org \
--cc=airlied@linux.ie \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=kbuild-all@lists.01.org \
--cc=krzk@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=quic_abhinavk@quicinc.com \
--cc=robdclark@gmail.com \
--cc=robh+dt@kernel.org \
--cc=sean@poorly.run \
--cc=swboyd@chromium.org \
/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: link
Be 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.