[intel-lts:5.15/linux 44/62] drivers/media/i2c/kmb-imx327.c:1962: warning: expecting prototype for kmb_imx327_probe(). Prototype was for kmb_imx327_pdev_probe() instead
by kernel test robot
tree: https://github.com/intel/linux-intel-lts.git 5.15/linux
head: 719d8431f0cc54f9e54ee62dfd2f5354a6f879f7
commit: 49fa3e4bf70e3787f53ec225a1f9962598606bb9 [44/62] media: keembay-camera: sync with API change
config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20220121/202201212142.jo6p4UtN-lk...)
compiler: alpha-linux-gcc (GCC) 11.2.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/linux-intel-lts/commit/49fa3e4bf70e3787f53ec225a...
git remote add intel-lts https://github.com/intel/linux-intel-lts.git
git fetch --no-tags intel-lts 5.15/linux
git checkout 49fa3e4bf70e3787f53ec225a1f9962598606bb9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=alpha SHELL=/bin/bash drivers/media/i2c/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/media/i2c/kmb-imx327.c:134: warning: Function parameter or member 'fps' not described in 'kmb_imx327_mode'
drivers/media/i2c/kmb-imx327.c:982: warning: Function parameter or member 'code' not described in 'kmb_imx327_enum_mbus_code'
drivers/media/i2c/kmb-imx327.c:982: warning: Excess function parameter 'fsize' description in 'kmb_imx327_enum_mbus_code'
drivers/media/i2c/kmb-imx327.c:1041: warning: Function parameter or member 'fie' not described in 'kmb_imx327_enum_frame_interval'
drivers/media/i2c/kmb-imx327.c:1041: warning: Excess function parameter 'fsize' description in 'kmb_imx327_enum_frame_interval'
drivers/media/i2c/kmb-imx327.c:1764: warning: Function parameter or member 'client' not described in 'kmb_imx327_i2c_probe'
drivers/media/i2c/kmb-imx327.c:1764: warning: Function parameter or member 'id' not described in 'kmb_imx327_i2c_probe'
drivers/media/i2c/kmb-imx327.c:1764: warning: Excess function parameter 'pdev' description in 'kmb_imx327_i2c_probe'
>> drivers/media/i2c/kmb-imx327.c:1962: warning: expecting prototype for kmb_imx327_probe(). Prototype was for kmb_imx327_pdev_probe() instead
>> drivers/media/i2c/kmb-imx327.c:2086: warning: expecting prototype for kmb_imx327_remove(). Prototype was for kmb_imx327_pdev_remove() instead
--
drivers/media/i2c/kmb-imx412.c:209: warning: Function parameter or member 'fps' not described in 'kmb_imx412_mode'
>> drivers/media/i2c/kmb-imx412.c:2275: warning: expecting prototype for kmb_imx412_open(). Prototype was for kmb_imx412_sync_start() instead
drivers/media/i2c/kmb-imx412.c:2887: warning: Function parameter or member 'code' not described in 'kmb_imx412_enum_mbus_code'
drivers/media/i2c/kmb-imx412.c:2887: warning: Excess function parameter 'fsize' description in 'kmb_imx412_enum_mbus_code'
drivers/media/i2c/kmb-imx412.c:2980: warning: Function parameter or member 'fie' not described in 'kmb_imx412_enum_frame_interval'
drivers/media/i2c/kmb-imx412.c:2980: warning: Excess function parameter 'fsize' description in 'kmb_imx412_enum_frame_interval'
vim +1962 drivers/media/i2c/kmb-imx327.c
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1954
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1955 /**
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1956 * kmb_imx327_probe - Platform device binding
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1957 * @pdev: pointer to platform device
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1958 *
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1959 * Return: 0 if successful
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1960 */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1961 static int kmb_imx327_pdev_probe(struct platform_device *pdev)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 @1962 {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1963 struct kmb_imx327 *kmb_imx327;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1964 struct gpio_descs *detect_gpios;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1965 int ret;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1966
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1967 kmb_imx327 = devm_kzalloc(&pdev->dev, sizeof(*kmb_imx327), GFP_KERNEL);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1968 if (!kmb_imx327)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1969 return -ENOMEM;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1970
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1971 platform_set_drvdata(pdev, kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1972
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1973 mutex_init(&kmb_imx327->mutex);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1974
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1975 kmb_imx327->dev = &pdev->dev;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1976
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1977 /* Initialize subdev */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1978 v4l2_subdev_init(&kmb_imx327->sd, &kmb_imx327_subdev_ops);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1979 kmb_imx327->sd.owner = pdev->dev.driver->owner;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1980 kmb_imx327->sd.dev = &pdev->dev;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1981
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1982 /* request optional detect pins */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1983 detect_gpios =
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1984 gpiod_get_array_optional(&pdev->dev, "detect", GPIOD_OUT_LOW);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1985 if (IS_ERR(detect_gpios)) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1986 ret = PTR_ERR(detect_gpios);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1987 dev_info(&pdev->dev, "failed to get detect gpios %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1988 /* Defer the probe if detect gpios are busy */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1989 ret = (ret == -EBUSY) ? -EPROBE_DEFER : ret;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1990 goto error_mutex_destroy;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1991 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1992
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1993 ret = kmb_imx327_power_on(kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1994 if (ret) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1995 dev_dbg(&pdev->dev, "failed to power-on the sensor %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1996 /* Defer the probe if resourcess are busy */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1997 ret = (ret == -EBUSY) ? -EPROBE_DEFER : ret;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1998 goto error_put_detect_gpios;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 1999 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2000
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2001 ret = kmb_imx327_get_i2c_client(kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2002 if (ret) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2003 dev_dbg(&pdev->dev, "failed to get i2c %d\n", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2004 goto error_sensor_power_off;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2005 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2006
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2007 v4l2_set_subdevdata(&kmb_imx327->sd, kmb_imx327->client);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2008 i2c_set_clientdata(kmb_imx327->client, &kmb_imx327->sd);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2009 v4l2_i2c_subdev_set_name(&kmb_imx327->sd, kmb_imx327->client,
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2010 KMB_IMX327_DRV_NAME, pdev->name);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2011
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2012 /* Check module identity */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2013 ret = kmb_imx327_detect(kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2014 if (ret) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2015 dev_err(&pdev->dev, "failed to find sensor: %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2016 goto error_unregister_i2c_dev;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2017 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2018
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2019 /* Set default mode to max resolution */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2020 kmb_imx327->cur_mode = &supported_modes[0];
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2021 kmb_imx327->fps = kmb_imx327->cur_mode->fps.def;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2022 kmb_imx327->lpfr = kmb_imx327->cur_mode->lpfr[0];
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2023
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2024 ret = kmb_imx327_init_controls(kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2025 if (ret) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2026 dev_err(&pdev->dev, "failed to init controls: %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2027 goto error_unregister_i2c_dev;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2028 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2029
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2030 /* Initialize subdev */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2031 kmb_imx327->sd.internal_ops = &kmb_imx327_internal_ops;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2032 kmb_imx327->sd.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2033 kmb_imx327->sd.entity.ops = &kmb_imx327_subdev_entity_ops;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2034 kmb_imx327->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2035
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2036 /* Initialize source pad */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2037 kmb_imx327->pad.flags = MEDIA_PAD_FL_SOURCE;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2038 ret = media_entity_pads_init(&kmb_imx327->sd.entity, 1,
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2039 &kmb_imx327->pad);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2040 if (ret) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2041 dev_err(&pdev->dev, "failed to init entity pads: %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2042 goto error_handler_free;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2043 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2044
49fa3e4bf70e378 Zhou Furong 2021-10-20 2045 ret = v4l2_async_register_subdev_sensor(&kmb_imx327->sd);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2046 if (ret < 0) {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2047 dev_err(&pdev->dev, "failed to register async subdev: %d", ret);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2048 goto error_media_entity;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2049 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2050
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2051 if (detect_gpios)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2052 gpiod_put_array(detect_gpios);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2053
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2054 pm_runtime_set_active(&pdev->dev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2055 pm_runtime_enable(&pdev->dev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2056 pm_runtime_idle(&pdev->dev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2057
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2058 dev_info(&pdev->dev, "Probe success!");
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2059 return 0;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2060
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2061 error_media_entity:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2062 media_entity_cleanup(&kmb_imx327->sd.entity);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2063 error_handler_free:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2064 v4l2_ctrl_handler_free(kmb_imx327->sd.ctrl_handler);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2065 error_unregister_i2c_dev:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2066 if (kmb_imx327->client)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2067 i2c_unregister_device(kmb_imx327->client);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2068 error_sensor_power_off:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2069 kmb_imx327_power_off(kmb_imx327);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2070 error_put_detect_gpios:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2071 if (detect_gpios)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2072 gpiod_put_array(detect_gpios);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2073 error_mutex_destroy:
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2074 mutex_destroy(&kmb_imx327->mutex);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2075
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2076 return ret;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2077 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2078
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2079 /**
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2080 * kmb_imx327_remove - Platform device unbinding
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2081 * @pdev: pointer to platform device
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2082 *
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2083 * Return: 0 if successful
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2084 */
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2085 static int kmb_imx327_pdev_remove(struct platform_device *pdev)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 @2086 {
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2087 struct kmb_imx327 *kmb_imx327 = platform_get_drvdata(pdev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2088 struct v4l2_subdev *sd = &kmb_imx327->sd;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2089
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2090 v4l2_async_unregister_subdev(sd);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2091 media_entity_cleanup(&sd->entity);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2092 v4l2_ctrl_handler_free(sd->ctrl_handler);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2093
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2094 pm_runtime_disable(&pdev->dev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2095 pm_runtime_suspended(&pdev->dev);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2096
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2097 if (kmb_imx327->client)
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2098 i2c_unregister_device(kmb_imx327->client);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2099
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2100 mutex_destroy(&kmb_imx327->mutex);
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2101
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2102 return 0;
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2103 }
ca29c95cf4cd93d Gjorgji Rosikopulos 2020-06-26 2104
:::::: The code at line 1962 was first introduced by commit
:::::: ca29c95cf4cd93d1e33013951d9ab731e49e1bce media: kmb-imx327-sensor: Add Keem Bay IMX327 sensor
:::::: TO: Gjorgji Rosikopulos <gjorgjix.rosikopulos(a)intel.com>
:::::: CC: nanli2x <nanx.li(a)intel.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v3 3/5] drm/i915: support 64K GTT pages for discrete cards
by kernel test robot
Hi Robert,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-tip/drm-tip]
[cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next drm/drm-next tegra-drm/drm/tegra/for-next airlied/drm-next v5.16 next-20220121]
[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/0day-ci/linux/commits/Robert-Beckett/discrete-card-64K...
base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
config: i386-randconfig-a013 (https://download.01.org/0day-ci/archive/20220121/202201212115.DdoJRHbQ-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 7b3d30728816403d1fd73cc5082e9fb761262bce)
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/0day-ci/linux/commit/7547f27d09bac4e34155aca7ed017c04b...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Robert-Beckett/discrete-card-64K-page-support/20220121-044244
git checkout 7547f27d09bac4e34155aca7ed017c04b7531e52
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/drm/i915/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/gem/i915_gem_object.c:807:
>> drivers/gpu/drm/i915/gem/selftests/huge_pages.c:1508:10: warning: variable 'err' is uninitialized when used here [-Wuninitialized]
return err;
^~~
drivers/gpu/drm/i915/gem/selftests/huge_pages.c:1485:9: note: initialize the variable 'err' to silence this warning
int err;
^
= 0
1 warning generated.
vim +/err +1508 drivers/gpu/drm/i915/gem/selftests/huge_pages.c
1480
1481 static int igt_ppgtt_compact(void *arg)
1482 {
1483 struct drm_i915_private *i915 = arg;
1484 struct drm_i915_gem_object *obj;
1485 int err;
1486
1487 /*
1488 * Simple test to catch issues with compact 64K pages -- since the pt is
1489 * compacted to 256B that gives us 32 entries per pt, however since the
1490 * backing page for the pt is 4K, any extra entries we might incorrectly
1491 * write out should be ignored by the HW. If ever hit such a case this
1492 * test should catch it since some of our writes would land in scratch.
1493 */
1494
1495 if (!HAS_64K_PAGES(i915)) {
1496 pr_info("device lacks compact 64K page support, skipping\n");
1497 return 0;
1498 }
1499
1500 if (!HAS_LMEM(i915)) {
1501 pr_info("device lacks LMEM support, skipping\n");
1502 return 0;
1503 }
1504
1505 /* We want the range to cover multiple page-table boundaries. */
1506 obj = i915_gem_object_create_lmem(i915, SZ_4M, 0);
1507 if (IS_ERR(obj))
> 1508 return err;
1509
1510 err = i915_gem_object_pin_pages_unlocked(obj);
1511 if (err)
1512 goto out_put;
1513
1514 if (obj->mm.page_sizes.phys < I915_GTT_PAGE_SIZE_64K) {
1515 pr_info("LMEM compact unable to allocate huge-page(s)\n");
1516 goto out_unpin;
1517 }
1518
1519 /*
1520 * Disable 2M GTT pages by forcing the page-size to 64K for the GTT
1521 * insertion.
1522 */
1523 obj->mm.page_sizes.sg = I915_GTT_PAGE_SIZE_64K;
1524
1525 err = igt_write_huge(i915, obj);
1526 if (err)
1527 pr_err("LMEM compact write-huge failed\n");
1528
1529 out_unpin:
1530 i915_gem_object_unpin_pages(obj);
1531 out_put:
1532 i915_gem_object_put(obj);
1533
1534 if (err == -ENOMEM)
1535 err = 0;
1536
1537 return err;
1538 }
1539
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v3 3/5] powerpc/vdso: Merge vdso64 and vdso32 into a single directory
by kernel test robot
Hi Christophe,
I love your patch! Yet something to improve:
[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.16 next-20220121]
[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/0day-ci/linux/commits/Christophe-Leroy/powerpc-vdso-au...
base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-randconfig-r021-20220120 (https://download.01.org/0day-ci/archive/20220121/202201212112.bkq4ClWB-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 7b3d30728816403d1fd73cc5082e9fb761262bce)
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
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# https://github.com/0day-ci/linux/commit/c972dc96e668e5c689a217abaeaba2461...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Christophe-Leroy/powerpc-vdso-augment-VDSO32-functions-to-support-64-bits-build/20220121-180748
git checkout c972dc96e668e5c689a217abaeaba246147a8666
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=powerpc prepare
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
arch/powerpc/include/asm/io.h:557:56: note: expanded from macro '__do_insw'
#define __do_insw(p, b, n) readsw((PCI_IO_ADDR)_IO_BASE+(p), (b), (n))
~~~~~~~~~~~~~~~~~~~~~^
In file included from arch/powerpc/kernel/asm-offsets.c:21:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:11:
In file included from arch/powerpc/include/asm/hardirq.h:6:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/powerpc/include/asm/io.h:619:
arch/powerpc/include/asm/io-defs.h:47:1: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
DEF_PCI_AC_NORET(insl, (unsigned long p, void *b, unsigned long c),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/powerpc/include/asm/io.h:616:3: note: expanded from macro 'DEF_PCI_AC_NORET'
__do_##name al; \
^~~~~~~~~~~~~~
<scratch space>:198:1: note: expanded from here
__do_insl
^
arch/powerpc/include/asm/io.h:558:56: note: expanded from macro '__do_insl'
#define __do_insl(p, b, n) readsl((PCI_IO_ADDR)_IO_BASE+(p), (b), (n))
~~~~~~~~~~~~~~~~~~~~~^
In file included from arch/powerpc/kernel/asm-offsets.c:21:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:11:
In file included from arch/powerpc/include/asm/hardirq.h:6:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/powerpc/include/asm/io.h:619:
arch/powerpc/include/asm/io-defs.h:49:1: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
DEF_PCI_AC_NORET(outsb, (unsigned long p, const void *b, unsigned long c),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/powerpc/include/asm/io.h:616:3: note: expanded from macro 'DEF_PCI_AC_NORET'
__do_##name al; \
^~~~~~~~~~~~~~
<scratch space>:200:1: note: expanded from here
__do_outsb
^
arch/powerpc/include/asm/io.h:559:58: note: expanded from macro '__do_outsb'
#define __do_outsb(p, b, n) writesb((PCI_IO_ADDR)_IO_BASE+(p),(b),(n))
~~~~~~~~~~~~~~~~~~~~~^
In file included from arch/powerpc/kernel/asm-offsets.c:21:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:11:
In file included from arch/powerpc/include/asm/hardirq.h:6:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/powerpc/include/asm/io.h:619:
arch/powerpc/include/asm/io-defs.h:51:1: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
DEF_PCI_AC_NORET(outsw, (unsigned long p, const void *b, unsigned long c),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/powerpc/include/asm/io.h:616:3: note: expanded from macro 'DEF_PCI_AC_NORET'
__do_##name al; \
^~~~~~~~~~~~~~
<scratch space>:202:1: note: expanded from here
__do_outsw
^
arch/powerpc/include/asm/io.h:560:58: note: expanded from macro '__do_outsw'
#define __do_outsw(p, b, n) writesw((PCI_IO_ADDR)_IO_BASE+(p),(b),(n))
~~~~~~~~~~~~~~~~~~~~~^
In file included from arch/powerpc/kernel/asm-offsets.c:21:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:26:
In file included from include/linux/kernel_stat.h:9:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:11:
In file included from arch/powerpc/include/asm/hardirq.h:6:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/powerpc/include/asm/io.h:619:
arch/powerpc/include/asm/io-defs.h:53:1: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
DEF_PCI_AC_NORET(outsl, (unsigned long p, const void *b, unsigned long c),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/powerpc/include/asm/io.h:616:3: note: expanded from macro 'DEF_PCI_AC_NORET'
__do_##name al; \
^~~~~~~~~~~~~~
<scratch space>:204:1: note: expanded from here
__do_outsl
^
arch/powerpc/include/asm/io.h:561:58: note: expanded from macro '__do_outsl'
#define __do_outsl(p, b, n) writesl((PCI_IO_ADDR)_IO_BASE+(p),(b),(n))
~~~~~~~~~~~~~~~~~~~~~^
6 warnings generated.
>> arch/powerpc/kernel/vdso/gettimeofday.S:75:8: error: unsupported directive '.stabs'
.stabs "_restgpr_31_x:F-1",36,0,0,_restgpr_31_x; .globl _restgpr_31_x; _restgpr_31_x:
^
arch/powerpc/kernel/vdso/gettimeofday.S:76:8: error: unsupported directive '.stabs'
.stabs "_rest32gpr_31_x:F-1",36,0,0,_rest32gpr_31_x; .globl _rest32gpr_31_x; _rest32gpr_31_x:
^
make[2]: *** [arch/powerpc/kernel/vdso/Makefile:71: arch/powerpc/kernel/vdso/gettimeofday-32.o] Error 1
make[2]: Target 'include/generated/vdso32-offsets.h' not remade because of errors.
make[1]: *** [arch/powerpc/Makefile:423: vdso_prepare] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:219: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
vim +75 arch/powerpc/kernel/vdso/gettimeofday.S
08c18b63d9656e arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2021-03-09 70
08c18b63d9656e arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2021-03-09 71 /* Routines for restoring integer registers, called by the compiler. */
08c18b63d9656e arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2021-03-09 72 /* Called with r11 pointing to the stack header word of the caller of the */
08c18b63d9656e arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2021-03-09 73 /* function, just beyond the end of the integer restore area. */
fe1b4be04d2583 arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2022-01-21 74 #ifndef __powerpc64__
08c18b63d9656e arch/powerpc/kernel/vdso32/gettimeofday.S Christophe Leroy 2021-03-09 @75 _GLOBAL(_restgpr_31_x)
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v11 4/5] clocksource/drivers: Add a goldfish-timer clocksource
by kernel test robot
Hi Laurent,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on tip/timers/core]
[also build test ERROR on linux/master linus/master next-20220121]
[cannot apply to geert-m68k/for-next v5.16]
[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/0day-ci/linux/commits/Laurent-Vivier/m68k-Add-Virtual-...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 35e13e9da9afbce13c1d36465504ece4e65f24fe
config: arc-allyesconfig (https://download.01.org/0day-ci/archive/20220121/202201212032.qL41ty3V-lk...)
compiler: arceb-elf-gcc (GCC) 11.2.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/0day-ci/linux/commit/ea1170b472532011ab0c8208b6414a368...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Laurent-Vivier/m68k-Add-Virtual-M68k-Machine/20220120-160832
git checkout ea1170b472532011ab0c8208b6414a36810f45ab
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/clocksource/timer-goldfish.c: In function 'goldfish_timer_read':
>> drivers/clocksource/timer-goldfish.c:52:20: error: implicit declaration of function 'gf_ioread32'; did you mean 'ioread32'? [-Werror=implicit-function-declaration]
52 | time_low = gf_ioread32(base + TIMER_TIME_LOW);
| ^~~~~~~~~~~
| ioread32
drivers/clocksource/timer-goldfish.c: In function 'goldfish_timer_set_oneshot':
>> drivers/clocksource/timer-goldfish.c:65:9: error: implicit declaration of function 'gf_iowrite32'; did you mean 'iowrite32'? [-Werror=implicit-function-declaration]
65 | gf_iowrite32(0, base + TIMER_ALARM_HIGH);
| ^~~~~~~~~~~~
| iowrite32
cc1: some warnings being treated as errors
vim +52 drivers/clocksource/timer-goldfish.c
40
41 static u64 goldfish_timer_read(struct clocksource *cs)
42 {
43 struct goldfish_timer *timerdrv = cs_to_gf(cs);
44 void __iomem *base = timerdrv->base;
45 u32 time_low, time_high;
46 u64 ticks;
47
48 /*
49 * time_low: get low bits of current time and update time_high
50 * time_high: get high bits of time at last time_low read
51 */
> 52 time_low = gf_ioread32(base + TIMER_TIME_LOW);
53 time_high = gf_ioread32(base + TIMER_TIME_HIGH);
54
55 ticks = ((u64)time_high << 32) | time_low;
56
57 return ticks;
58 }
59
60 static int goldfish_timer_set_oneshot(struct clock_event_device *evt)
61 {
62 struct goldfish_timer *timerdrv = ced_to_gf(evt);
63 void __iomem *base = timerdrv->base;
64
> 65 gf_iowrite32(0, base + TIMER_ALARM_HIGH);
66 gf_iowrite32(0, base + TIMER_ALARM_LOW);
67 gf_iowrite32(1, base + TIMER_IRQ_ENABLED);
68
69 return 0;
70 }
71
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v8 1/4] phy: cadence: Add Cadence D-PHY Rx driver
by kernel test robot
Hi Pratyush,
I love your patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next]
[also build test WARNING on linus/master v5.16 next-20220121]
[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/0day-ci/linux/commits/Pratyush-Yadav/Rx-mode-support-f...
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20220121/202201211916.mtjEWmKi-lk...)
compiler: m68k-linux-gcc (GCC) 11.2.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/0day-ci/linux/commit/a27020695f773ffff7f462e53f7ff7ca3...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Pratyush-Yadav/Rx-mode-support-for-Cadence-DPHY/20220121-174104
git checkout a27020695f773ffff7f462e53f7ff7ca39109cd0
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=m68k SHELL=/bin/bash drivers/phy/cadence/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/device.h:15,
from include/linux/phy/phy.h:15,
from drivers/phy/cadence/cdns-dphy-rx.c:11:
drivers/phy/cadence/cdns-dphy-rx.c: In function 'cdns_dphy_rx_probe':
>> drivers/phy/cadence/cdns-dphy-rx.c:223:30: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long int' [-Wformat=]
223 | dev_err(dev, "Failed to create PHY: %d\n", PTR_ERR(dphy->phy));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
144 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
drivers/phy/cadence/cdns-dphy-rx.c:223:17: note: in expansion of macro 'dev_err'
223 | dev_err(dev, "Failed to create PHY: %d\n", PTR_ERR(dphy->phy));
| ^~~~~~~
drivers/phy/cadence/cdns-dphy-rx.c:223:54: note: format string is defined here
223 | dev_err(dev, "Failed to create PHY: %d\n", PTR_ERR(dphy->phy));
| ~^
| |
| int
| %ld
In file included from include/linux/device.h:15,
from include/linux/phy/phy.h:15,
from drivers/phy/cadence/cdns-dphy-rx.c:11:
drivers/phy/cadence/cdns-dphy-rx.c:230:30: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long int' [-Wformat=]
230 | dev_err(dev, "Failed to register PHY provider: %d\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:144:56: note: in expansion of macro 'dev_fmt'
144 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
drivers/phy/cadence/cdns-dphy-rx.c:230:17: note: in expansion of macro 'dev_err'
230 | dev_err(dev, "Failed to register PHY provider: %d\n",
| ^~~~~~~
drivers/phy/cadence/cdns-dphy-rx.c:230:65: note: format string is defined here
230 | dev_err(dev, "Failed to register PHY provider: %d\n",
| ~^
| |
| int
| %ld
vim +223 drivers/phy/cadence/cdns-dphy-rx.c
203
204 static int cdns_dphy_rx_probe(struct platform_device *pdev)
205 {
206 struct device *dev = &pdev->dev;
207 struct phy_provider *provider;
208 struct cdns_dphy_rx *dphy;
209
210 dphy = devm_kzalloc(dev, sizeof(*dphy), GFP_KERNEL);
211 if (!dphy)
212 return -ENOMEM;
213
214 dev_set_drvdata(dev, dphy);
215 dphy->dev = dev;
216
217 dphy->regs = devm_platform_ioremap_resource(pdev, 0);
218 if (IS_ERR(dphy->regs))
219 return PTR_ERR(dphy->regs);
220
221 dphy->phy = devm_phy_create(dev, NULL, &cdns_dphy_rx_ops);
222 if (IS_ERR(dphy->phy)) {
> 223 dev_err(dev, "Failed to create PHY: %d\n", PTR_ERR(dphy->phy));
224 return PTR_ERR(dphy->phy);
225 }
226
227 phy_set_drvdata(dphy->phy, dphy);
228 provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
229 if (IS_ERR(provider)) {
230 dev_err(dev, "Failed to register PHY provider: %d\n",
231 PTR_ERR(provider));
232 return PTR_ERR(provider);
233 }
234
235 return 0;
236 }
237
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v3 4/5] drm/i915: add gtt misalignment test
by kernel test robot
Hi Robert,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip]
[cannot apply to drm-intel/for-linux-next v5.16 next-20220121]
[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/0day-ci/linux/commits/Robert-Beckett/discrete-card-64K...
base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
config: i386-randconfig-a005-20220117 (https://download.01.org/0day-ci/archive/20220121/202201211905.O4FxXbsX-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d4baf3b1322b84816aa623d8e8cb45a49cb68b84)
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/0day-ci/linux/commit/e7a6d547c26b78357dbad93dc9d8df90b...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Robert-Beckett/discrete-card-64K-page-support/20220121-044244
git checkout e7a6d547c26b78357dbad93dc9d8df90bc51d025
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/drm/i915/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/i915_gem_gtt.c:296:
>> drivers/gpu/drm/i915/selftests/i915_gem_gtt.c:1146:49: error: implicit conversion from enumeration type 'enum intel_region_id' to different enumeration type 'enum intel_memory_type' [-Werror,-Wenum-conversion]
u64 min_alignment = i915_vm_min_alignment(vm, id);
~~~~~~~~~~~~~~~~~~~~~ ^~
1 error generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for SND_SOC_CS35L41_SPI
Depends on SOUND && !UML && SND && SND_SOC && SPI_MASTER
Selected by
- SND_SOC_AMD_VANGOGH_MACH && SOUND && !UML && SND && SND_SOC && SND_SOC_AMD_ACP5x && I2C
vim +1146 drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
1131
1132 static int misaligned_pin(struct i915_address_space *vm,
1133 u64 hole_start, u64 hole_end,
1134 unsigned long end_time)
1135 {
1136 struct intel_memory_region *mr;
1137 enum intel_region_id id;
1138 unsigned long flags = PIN_OFFSET_FIXED | PIN_USER;
1139 int err = 0;
1140 u64 hole_size = hole_end - hole_start;
1141
1142 if (i915_is_ggtt(vm))
1143 flags |= PIN_GLOBAL;
1144
1145 for_each_memory_region(mr, vm->i915, id) {
> 1146 u64 min_alignment = i915_vm_min_alignment(vm, id);
1147 u64 size = min_alignment;
1148 u64 addr = round_up(hole_start + (hole_size / 2), min_alignment);
1149
1150 /* we can't test < 4k alignment due to flags being encoded in lower bits */
1151 if (min_alignment != I915_GTT_PAGE_SIZE_4K) {
1152 err = misaligned_case(vm, mr, addr + (min_alignment / 2), size, flags);
1153 /* misaligned should error with -EINVAL*/
1154 if (!err)
1155 err = -EBADSLT;
1156 if (err != -EINVAL)
1157 return err;
1158 }
1159
1160 /* test for vma->size expansion to min page size */
1161 err = misaligned_case(vm, mr, addr, PAGE_SIZE, flags);
1162 if (min_alignment > hole_size) {
1163 if (!err)
1164 err = -EBADSLT;
1165 else if (err == -ENOSPC)
1166 err = 0;
1167 }
1168 if (err)
1169 return err;
1170
1171 /* test for intermediate size not expanding vma->size for large alignments */
1172 err = misaligned_case(vm, mr, addr, size / 2, flags);
1173 if (min_alignment > hole_size) {
1174 if (!err)
1175 err = -EBADSLT;
1176 else if (err == -ENOSPC)
1177 err = 0;
1178 }
1179 if (err)
1180 return err;
1181 }
1182
1183 return 0;
1184 }
1185
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH v3 3/5] drm/i915: support 64K GTT pages for discrete cards
by kernel test robot
Hi Robert,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip]
[cannot apply to drm-intel/for-linux-next v5.16 next-20220121]
[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/0day-ci/linux/commits/Robert-Beckett/discrete-card-64K...
base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
config: i386-randconfig-a005-20220117 (https://download.01.org/0day-ci/archive/20220121/202201211804.7ErZmoIV-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d4baf3b1322b84816aa623d8e8cb45a49cb68b84)
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/0day-ci/linux/commit/7547f27d09bac4e34155aca7ed017c04b...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Robert-Beckett/discrete-card-64K-page-support/20220121-044244
git checkout 7547f27d09bac4e34155aca7ed017c04b7531e52
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/drm/i915/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/gem/i915_gem_object.c:807:
>> drivers/gpu/drm/i915/gem/selftests/huge_pages.c:1508:10: error: variable 'err' is uninitialized when used here [-Werror,-Wuninitialized]
return err;
^~~
drivers/gpu/drm/i915/gem/selftests/huge_pages.c:1485:9: note: initialize the variable 'err' to silence this warning
int err;
^
= 0
1 error generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for SND_SOC_CS35L41_SPI
Depends on SOUND && !UML && SND && SND_SOC && SPI_MASTER
Selected by
- SND_SOC_AMD_VANGOGH_MACH && SOUND && !UML && SND && SND_SOC && SND_SOC_AMD_ACP5x && I2C
vim +/err +1508 drivers/gpu/drm/i915/gem/selftests/huge_pages.c
1480
1481 static int igt_ppgtt_compact(void *arg)
1482 {
1483 struct drm_i915_private *i915 = arg;
1484 struct drm_i915_gem_object *obj;
1485 int err;
1486
1487 /*
1488 * Simple test to catch issues with compact 64K pages -- since the pt is
1489 * compacted to 256B that gives us 32 entries per pt, however since the
1490 * backing page for the pt is 4K, any extra entries we might incorrectly
1491 * write out should be ignored by the HW. If ever hit such a case this
1492 * test should catch it since some of our writes would land in scratch.
1493 */
1494
1495 if (!HAS_64K_PAGES(i915)) {
1496 pr_info("device lacks compact 64K page support, skipping\n");
1497 return 0;
1498 }
1499
1500 if (!HAS_LMEM(i915)) {
1501 pr_info("device lacks LMEM support, skipping\n");
1502 return 0;
1503 }
1504
1505 /* We want the range to cover multiple page-table boundaries. */
1506 obj = i915_gem_object_create_lmem(i915, SZ_4M, 0);
1507 if (IS_ERR(obj))
> 1508 return err;
1509
1510 err = i915_gem_object_pin_pages_unlocked(obj);
1511 if (err)
1512 goto out_put;
1513
1514 if (obj->mm.page_sizes.phys < I915_GTT_PAGE_SIZE_64K) {
1515 pr_info("LMEM compact unable to allocate huge-page(s)\n");
1516 goto out_unpin;
1517 }
1518
1519 /*
1520 * Disable 2M GTT pages by forcing the page-size to 64K for the GTT
1521 * insertion.
1522 */
1523 obj->mm.page_sizes.sg = I915_GTT_PAGE_SIZE_64K;
1524
1525 err = igt_write_huge(i915, obj);
1526 if (err)
1527 pr_err("LMEM compact write-huge failed\n");
1528
1529 out_unpin:
1530 i915_gem_object_unpin_pages(obj);
1531 out_put:
1532 i915_gem_object_put(obj);
1533
1534 if (err == -ENOMEM)
1535 err = 0;
1536
1537 return err;
1538 }
1539
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months
Re: [PATCH 1/2] media: rc-core: rename ir_raw_event_reset to ir_raw_event_overflow
by kernel test robot
Hi Sean,
I love your patch! Yet something to improve:
[auto build test ERROR on media-tree/master]
[also build test ERROR on next-20220121]
[cannot apply to sunxi/sunxi/for-next v5.16]
[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/0day-ci/linux/commits/Sean-Young/media-rc-core-rename-...
base: git://linuxtv.org/media_tree.git master
config: hexagon-randconfig-r041-20220120 (https://download.01.org/0day-ci/archive/20220121/202201211605.1ccb0HNg-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project f7b7138a62648f4019c55e4671682af1f851f295)
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/0day-ci/linux/commit/5b0115b915832b54ebe085c923d73209b...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Sean-Young/media-rc-core-rename-ir_raw_event_reset-to-ir_raw_event_overflow/20220121-001937
git checkout 5b0115b915832b54ebe085c923d73209b1abb364
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/media/rc/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> drivers/media/rc/mtk-cir.c:220:2: error: implicit declaration of function 'ir_raw_event_reset' [-Werror,-Wimplicit-function-declaration]
ir_raw_event_reset(ir->rc);
^
drivers/media/rc/mtk-cir.c:220:2: note: did you mean 'ir_raw_event_store'?
include/media/rc-core.h:317:5: note: 'ir_raw_event_store' declared here
int ir_raw_event_store(struct rc_dev *dev, struct ir_raw_event *ev);
^
1 error generated.
vim +/ir_raw_event_reset +220 drivers/media/rc/mtk-cir.c
6691e7b9a57c24 Sean Wang 2017-01-13 202
6691e7b9a57c24 Sean Wang 2017-01-13 203 static irqreturn_t mtk_ir_irq(int irqno, void *dev_id)
6691e7b9a57c24 Sean Wang 2017-01-13 204 {
6691e7b9a57c24 Sean Wang 2017-01-13 205 struct mtk_ir *ir = dev_id;
6691e7b9a57c24 Sean Wang 2017-01-13 206 u8 wid = 0;
6691e7b9a57c24 Sean Wang 2017-01-13 207 u32 i, j, val;
183e19f5b9ee18 Sean Young 2018-08-21 208 struct ir_raw_event rawir = {};
6691e7b9a57c24 Sean Wang 2017-01-13 209
6691e7b9a57c24 Sean Wang 2017-01-13 210 /*
6691e7b9a57c24 Sean Wang 2017-01-13 211 * Reset decoder state machine explicitly is required
6691e7b9a57c24 Sean Wang 2017-01-13 212 * because 1) the longest duration for space MTK IR hardware
6691e7b9a57c24 Sean Wang 2017-01-13 213 * could record is not safely long. e.g 12ms if rx resolution
6691e7b9a57c24 Sean Wang 2017-01-13 214 * is 46us by default. There is still the risk to satisfying
6691e7b9a57c24 Sean Wang 2017-01-13 215 * every decoder to reset themselves through long enough
6691e7b9a57c24 Sean Wang 2017-01-13 216 * trailing spaces and 2) the IRQ handler guarantees that
6691e7b9a57c24 Sean Wang 2017-01-13 217 * start of IR message is always contained in and starting
50c3c1ba171f3f Sean Wang 2017-06-30 218 * from register mtk_chkdata_reg(ir, i).
6691e7b9a57c24 Sean Wang 2017-01-13 219 */
6691e7b9a57c24 Sean Wang 2017-01-13 @220 ir_raw_event_reset(ir->rc);
6691e7b9a57c24 Sean Wang 2017-01-13 221
6691e7b9a57c24 Sean Wang 2017-01-13 222 /* First message must be pulse */
6691e7b9a57c24 Sean Wang 2017-01-13 223 rawir.pulse = false;
6691e7b9a57c24 Sean Wang 2017-01-13 224
6691e7b9a57c24 Sean Wang 2017-01-13 225 /* Handle all pulse and space IR controller captures */
6691e7b9a57c24 Sean Wang 2017-01-13 226 for (i = 0 ; i < MTK_CHKDATA_SZ ; i++) {
50c3c1ba171f3f Sean Wang 2017-06-30 227 val = mtk_r32(ir, mtk_chkdata_reg(ir, i));
6691e7b9a57c24 Sean Wang 2017-01-13 228 dev_dbg(ir->dev, "@reg%d=0x%08x\n", i, val);
6691e7b9a57c24 Sean Wang 2017-01-13 229
6691e7b9a57c24 Sean Wang 2017-01-13 230 for (j = 0 ; j < 4 ; j++) {
6691e7b9a57c24 Sean Wang 2017-01-13 231 wid = (val & (MTK_WIDTH_MASK << j * 8)) >> j * 8;
6691e7b9a57c24 Sean Wang 2017-01-13 232 rawir.pulse = !rawir.pulse;
6691e7b9a57c24 Sean Wang 2017-01-13 233 rawir.duration = wid * (MTK_IR_SAMPLE + 1);
6691e7b9a57c24 Sean Wang 2017-01-13 234 ir_raw_event_store_with_filter(ir->rc, &rawir);
6691e7b9a57c24 Sean Wang 2017-01-13 235 }
6691e7b9a57c24 Sean Wang 2017-01-13 236 }
6691e7b9a57c24 Sean Wang 2017-01-13 237
6691e7b9a57c24 Sean Wang 2017-01-13 238 /*
6691e7b9a57c24 Sean Wang 2017-01-13 239 * The maximum number of edges the IR controller can
6691e7b9a57c24 Sean Wang 2017-01-13 240 * hold is MTK_CHKDATA_SZ * 4. So if received IR messages
6691e7b9a57c24 Sean Wang 2017-01-13 241 * is over the limit, the last incomplete IR message would
6691e7b9a57c24 Sean Wang 2017-01-13 242 * be appended trailing space and still would be sent into
6691e7b9a57c24 Sean Wang 2017-01-13 243 * ir-rc-raw to decode. That helps it is possible that it
6691e7b9a57c24 Sean Wang 2017-01-13 244 * has enough information to decode a scancode even if the
6691e7b9a57c24 Sean Wang 2017-01-13 245 * trailing end of the message is missing.
6691e7b9a57c24 Sean Wang 2017-01-13 246 */
6691e7b9a57c24 Sean Wang 2017-01-13 247 if (!MTK_IR_END(wid, rawir.pulse)) {
6691e7b9a57c24 Sean Wang 2017-01-13 248 rawir.pulse = false;
6691e7b9a57c24 Sean Wang 2017-01-13 249 rawir.duration = MTK_MAX_SAMPLES * (MTK_IR_SAMPLE + 1);
6691e7b9a57c24 Sean Wang 2017-01-13 250 ir_raw_event_store_with_filter(ir->rc, &rawir);
6691e7b9a57c24 Sean Wang 2017-01-13 251 }
6691e7b9a57c24 Sean Wang 2017-01-13 252
6691e7b9a57c24 Sean Wang 2017-01-13 253 ir_raw_event_handle(ir->rc);
6691e7b9a57c24 Sean Wang 2017-01-13 254
6691e7b9a57c24 Sean Wang 2017-01-13 255 /*
6691e7b9a57c24 Sean Wang 2017-01-13 256 * Restart controller for the next receive that would
6691e7b9a57c24 Sean Wang 2017-01-13 257 * clear up all CHKDATA registers
6691e7b9a57c24 Sean Wang 2017-01-13 258 */
50c3c1ba171f3f Sean Wang 2017-06-30 259 mtk_w32_mask(ir, 0x1, MTK_IRCLR, ir->data->regs[MTK_IRCLR_REG]);
6691e7b9a57c24 Sean Wang 2017-01-13 260
6691e7b9a57c24 Sean Wang 2017-01-13 261 /* Clear interrupt status */
50c3c1ba171f3f Sean Wang 2017-06-30 262 mtk_w32_mask(ir, 0x1, MTK_IRINT_CLR,
50c3c1ba171f3f Sean Wang 2017-06-30 263 ir->data->regs[MTK_IRINT_CLR_REG]);
6691e7b9a57c24 Sean Wang 2017-01-13 264
6691e7b9a57c24 Sean Wang 2017-01-13 265 return IRQ_HANDLED;
6691e7b9a57c24 Sean Wang 2017-01-13 266 }
6691e7b9a57c24 Sean Wang 2017-01-13 267
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months