From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D6BE9D531; Sun, 19 Jan 2025 02:17:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.9 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737253065; cv=none; b=MoS5NS+pMxyRe+BrJtKpIYp+D1K04rm3Vt3lF7WLW0AkBzm9TOdQWUgCu5lH+5Eh+iBSGgXPDEP9bjO40wiumbxnVk/2rhx3NKu8gTcwo3OcBrnH9F3qTFui+D335BNuKw3Cw1gU9h+spXIqz/JWNsyRnOwWXU3oJ5Reb2Y7sDE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737253065; c=relaxed/simple; bh=aWnj56UpZhT1vPPFWTZ97+LjjElp/AAkeSGppMBcfzE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ToIpzWRNr6w1M5XsZdnvow9HcVS5BZFkDPPnJIrAmYOcXgMFyjIFcGUHolXj/cpaDLLzL8xxFH2Il/QclmDyaEe9Rq8IV+N5yypbXh49Gkh8oWIYA+O6ndo3petLhHydfHmtckBUpJJcE2jMa6923rTvCPf5I4E5oX6+0MFOrdY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=B+GrcK7U; arc=none smtp.client-ip=198.175.65.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="B+GrcK7U" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1737253064; x=1768789064; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=aWnj56UpZhT1vPPFWTZ97+LjjElp/AAkeSGppMBcfzE=; b=B+GrcK7UDDJO3MMk332sFmL1YIiqznxDVAoxue8FMlJDL8tmSL1pmRzy ij5v1sBurD39/TI82wpuZWPFcHPjDsDmwl8g1HLyAaSY20tSW+jNxEVep 8qdr+y9WTC6Viy+/rkR67/qP7GwAIPW8hBUUg9q2wc9AWKf5KS9D9gKMx FXYDDu5iH/5xWSbz/9XtoXn30Swv64ifvD9sM8B2IunBhkZng8xg8A7Eb 4WvFp9CeJ4VgmbQcGt3KRLrhaKm8aMhw66danfxcJ84mMLZIzyXWzY0Gi CyFc1wbX7VQcj/Tk35bcVN+APh7O2zdCggwtW+788pLHTMW6eQPaEfxhz g==; X-CSE-ConnectionGUID: 4aH3td8kQTKnmGJBOLng6Q== X-CSE-MsgGUID: GPKxh/cQT02UR6Cxgq9K8A== X-IronPort-AV: E=McAfee;i="6700,10204,11319"; a="60118770" X-IronPort-AV: E=Sophos;i="6.13,216,1732608000"; d="scan'208";a="60118770" Received: from orviesa009.jf.intel.com ([10.64.159.149]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Jan 2025 18:17:44 -0800 X-CSE-ConnectionGUID: xoAJLbEHTsego64iyTBy2g== X-CSE-MsgGUID: xm7fJQONSfq9278QjYBfWQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.13,216,1732608000"; d="scan'208";a="105955320" Received: from lkp-server01.sh.intel.com (HELO d63d4d77d921) ([10.239.97.150]) by orviesa009.jf.intel.com with ESMTP; 18 Jan 2025 18:17:36 -0800 Received: from kbuild by d63d4d77d921 with local (Exim 4.96) (envelope-from ) id 1tZKsX-000V1D-2O; Sun, 19 Jan 2025 02:17:33 +0000 Date: Sun, 19 Jan 2025 10:16:51 +0800 From: kernel test robot To: Shree Ramamoorthy , lgirdwood@gmail.com, broonie@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, aaro.koskinen@iki.fi, andreas@kemnade.info, khilman@baylibre.com, rogerq@kernel.org, tony@atomide.com, jerome.neanne@baylibre.com, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Cc: oe-kbuild-all@lists.linux.dev, m-leonard@ti.com, praneeth@ti.com, christophe.jaillet@wanadoo.fr Subject: Re: [PATCH v3 3/4] regulator: tps65215: Add support for TPS65215 regulator resources Message-ID: <202501191022.syh1x4cP-lkp@intel.com> References: <20250113231018.125426-4-s-ramamoorthy@ti.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250113231018.125426-4-s-ramamoorthy@ti.com> Hi Shree, kernel test robot noticed the following build errors: [auto build test ERROR on broonie-regulator/for-next] [also build test ERROR on next-20250117] [cannot apply to robh/for-next tmlind-omap/for-next linus/master v6.13-rc7] [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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Shree-Ramamoorthy/regulator-tps65215-Update-struct-names/20250114-071259 base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next patch link: https://lore.kernel.org/r/20250113231018.125426-4-s-ramamoorthy%40ti.com patch subject: [PATCH v3 3/4] regulator: tps65215: Add support for TPS65215 regulator resources config: microblaze-allmodconfig (https://download.01.org/0day-ci/archive/20250119/202501191022.syh1x4cP-lkp@intel.com/config) compiler: microblaze-linux-gcc (GCC) 14.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250119/202501191022.syh1x4cP-lkp@intel.com/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot | Closes: https://lore.kernel.org/oe-kbuild-all/202501191022.syh1x4cP-lkp@intel.com/ All errors (new ones prefixed by >>): >> drivers/regulator/tps65219-regulator.c:261:44: error: 'TPS65215_LDO_2' undeclared here (not in a function); did you mean 'TPS65219_LDO_2'? 261 | TPS65219_REGULATOR("LDO2", "ldo2", TPS65215_LDO_2, | ^~~~~~~~~~~~~~ drivers/regulator/tps65219-regulator.c:99:43: note: in definition of macro 'TPS65219_REGULATOR' 99 | .id = _id, \ | ^~~ >> drivers/regulator/tps65219-regulator.c:263:28: error: 'TPS65215_REG_LDO2_VOUT' undeclared here (not in a function); did you mean 'TPS65219_REG_LDO2_VOUT'? 263 | TPS65215_REG_LDO2_VOUT, | ^~~~~~~~~~~~~~~~~~~~~~ drivers/regulator/tps65219-regulator.c:104:43: note: in definition of macro 'TPS65219_REGULATOR' 104 | .vsel_reg = _vr, \ | ^~~ >> drivers/regulator/tps65219-regulator.c:266:28: error: 'TPS65215_ENABLE_LDO2_EN_MASK' undeclared here (not in a function); did you mean 'TPS65219_ENABLE_LDO2_EN_MASK'? 266 | TPS65215_ENABLE_LDO2_EN_MASK, 0, 0, tps65215_ldo_2_range, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/regulator/tps65219-regulator.c:111:43: note: in definition of macro 'TPS65219_REGULATOR' 111 | .enable_mask = _em, \ | ^~~ >> drivers/regulator/tps65219-regulator.c:321:10: error: 'TPS65215' undeclared here (not in a function); did you mean 'TPS65219'? 321 | [TPS65215] = { | ^~~~~~~~ | TPS65219 >> drivers/regulator/tps65219-regulator.c:321:10: error: array index in initializer not of integer type drivers/regulator/tps65219-regulator.c:321:10: note: (near initialization for 'chip_info_table') drivers/regulator/tps65219-regulator.c: In function 'tps65219_regulator_probe': >> drivers/regulator/tps65219-regulator.c:348:14: error: variable 'chip' has initializer but incomplete type 348 | enum pmic_id chip = platform_get_device_id(pdev)->driver_data; | ^~~~~~~ >> drivers/regulator/tps65219-regulator.c:348:22: error: storage size of 'chip' isn't known 348 | enum pmic_id chip = platform_get_device_id(pdev)->driver_data; | ^~~~ drivers/regulator/tps65219-regulator.c:348:22: warning: unused variable 'chip' [-Wunused-variable] vim +261 drivers/regulator/tps65219-regulator.c 258 259 static const struct regulator_desc tps65215_regs[] = { 260 // TPS65215's LDO2 is the same as TPS65219's LDO3 > 261 TPS65219_REGULATOR("LDO2", "ldo2", TPS65215_LDO_2, 262 REGULATOR_VOLTAGE, ldos_3_4_ops, 64, > 263 TPS65215_REG_LDO2_VOUT, 264 TPS65219_BUCKS_LDOS_VOUT_VSET_MASK, 265 TPS65219_REG_ENABLE_CTRL, > 266 TPS65215_ENABLE_LDO2_EN_MASK, 0, 0, tps65215_ldo_2_range, 267 3, 0, 0, NULL, 0, 0), 268 }; 269 270 static const struct regulator_desc tps65219_regs[] = { 271 TPS65219_REGULATOR("LDO2", "ldo2", TPS65219_LDO_2, 272 REGULATOR_VOLTAGE, ldos_1_2_ops, 64, 273 TPS65219_REG_LDO2_VOUT, 274 TPS65219_BUCKS_LDOS_VOUT_VSET_MASK, 275 TPS65219_REG_ENABLE_CTRL, 276 TPS65219_ENABLE_LDO2_EN_MASK, 0, 0, tps65219_ldo_2_range, 277 2, 0, 0, NULL, 0, TPS65219_LDOS_BYP_CONFIG_MASK), 278 TPS65219_REGULATOR("LDO3", "ldo3", TPS65219_LDO_3, 279 REGULATOR_VOLTAGE, ldos_3_4_ops, 64, 280 TPS65219_REG_LDO3_VOUT, 281 TPS65219_BUCKS_LDOS_VOUT_VSET_MASK, 282 TPS65219_REG_ENABLE_CTRL, 283 TPS65219_ENABLE_LDO3_EN_MASK, 0, 0, tps65219_ldos_3_4_range, 284 3, 0, 0, NULL, 0, 0), 285 TPS65219_REGULATOR("LDO4", "ldo4", TPS65219_LDO_4, 286 REGULATOR_VOLTAGE, ldos_3_4_ops, 64, 287 TPS65219_REG_LDO4_VOUT, 288 TPS65219_BUCKS_LDOS_VOUT_VSET_MASK, 289 TPS65219_REG_ENABLE_CTRL, 290 TPS65219_ENABLE_LDO4_EN_MASK, 0, 0, tps65219_ldos_3_4_range, 291 3, 0, 0, NULL, 0, 0), 292 }; 293 294 static irqreturn_t tps65219_regulator_irq_handler(int irq, void *data) 295 { 296 struct tps65219_regulator_irq_data *irq_data = data; 297 298 if (irq_data->type->event_name[0] == '\0') { 299 /* This is the timeout interrupt no specific regulator */ 300 dev_err(irq_data->dev, 301 "System was put in shutdown due to timeout during an active or standby transition.\n"); 302 return IRQ_HANDLED; 303 } 304 305 regulator_notifier_call_chain(irq_data->rdev, 306 irq_data->type->event, NULL); 307 308 dev_err(irq_data->dev, "Error IRQ trap %s for %s\n", 309 irq_data->type->event_name, irq_data->type->regulator_name); 310 return IRQ_HANDLED; 311 } 312 313 struct tps65219_chip_data { 314 size_t rdesc_size; 315 size_t common_rdesc_size; 316 const struct regulator_desc *rdesc; 317 const struct regulator_desc *common_rdesc; 318 }; 319 320 static struct tps65219_chip_data chip_info_table[] = { > 321 [TPS65215] = { 322 .rdesc = tps65215_regs, 323 .rdesc_size = ARRAY_SIZE(tps65215_regs), 324 .common_rdesc = common_regs, 325 .common_rdesc_size = ARRAY_SIZE(common_regs), 326 }, 327 [TPS65219] = { 328 .rdesc = tps65219_regs, 329 .rdesc_size = ARRAY_SIZE(tps65219_regs), 330 .common_rdesc = common_regs, 331 .common_rdesc_size = ARRAY_SIZE(common_regs), 332 }, 333 }; 334 335 static int tps65219_regulator_probe(struct platform_device *pdev) 336 { 337 struct tps65219_regulator_irq_data *irq_data; 338 struct tps65219_regulator_irq_type *irq_type; 339 340 struct tps65219_chip_data *pmic; 341 struct regulator_dev *rdev; 342 int error; 343 int irq; 344 int i; 345 346 struct tps65219 *tps = dev_get_drvdata(pdev->dev.parent); 347 struct regulator_config config = { }; > 348 enum pmic_id chip = platform_get_device_id(pdev)->driver_data; 349 pmic = &chip_info_table[chip]; 350 351 config.dev = tps->dev; 352 config.driver_data = tps; 353 config.regmap = tps->regmap; 354 355 for (i = 0; i < pmic->common_rdesc_size; i++) { 356 rdev = devm_regulator_register(&pdev->dev, &pmic->common_rdesc[i], 357 &config); 358 if (IS_ERR(rdev)) 359 return dev_err_probe(tps->dev, PTR_ERR(rdev), 360 "Failed to register %s regulator\n", 361 pmic->common_rdesc[i].name); 362 } 363 364 for (i = 0; i < pmic->rdesc_size; i++) { 365 rdev = devm_regulator_register(&pdev->dev, &pmic->rdesc[i], 366 &config); 367 if (IS_ERR(rdev)) 368 return dev_err_probe(tps->dev, PTR_ERR(rdev), 369 "Failed to register %s regulator\n", 370 pmic->rdesc[i].name); 371 } 372 373 irq_data = devm_kmalloc(tps->dev, 374 ARRAY_SIZE(tps65219_regulator_irq_types) * 375 sizeof(struct tps65219_regulator_irq_data), 376 GFP_KERNEL); 377 if (!irq_data) 378 return -ENOMEM; 379 380 for (i = 0; i < ARRAY_SIZE(tps65219_regulator_irq_types); ++i) { 381 irq_type = &tps65219_regulator_irq_types[i]; 382 383 irq = platform_get_irq_byname(pdev, irq_type->irq_name); 384 if (irq < 0) 385 return -EINVAL; 386 387 irq_data[i].dev = tps->dev; 388 irq_data[i].type = irq_type; 389 390 error = devm_request_threaded_irq(tps->dev, irq, NULL, 391 tps65219_regulator_irq_handler, 392 IRQF_ONESHOT, 393 irq_type->irq_name, 394 &irq_data[i]); 395 if (error) { 396 dev_err(tps->dev, "failed to request %s IRQ %d: %d\n", 397 irq_type->irq_name, irq, error); 398 return error; 399 } 400 } 401 402 return 0; 403 } 404 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki