From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DA6C52DC765 for ; Mon, 20 Oct 2025 07:13:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760944409; cv=none; b=tg6dA+HEO2Bq5sqeYbq5g3wqzcA+vezlODgGZS5XeZOgfrXhNQg51fRbwgeB+2mFVOzS1LSMmk15ArzHy5ePpLxJIN2nGuz5+txQkDBWR2d7lRfPaXbBhPd5PjOOmeiS4Y8uJ8LsT1S34RLamOoah0vrp2Zq8DufHhKWGoNX9vQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760944409; c=relaxed/simple; bh=PeGTn3iFxDHG7Zzquzk+8Eb8iZsO1eg0YMwOWsI8x/Q=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:In-Reply-To; b=ps4FnuTxEvQ5TV6tbLapRv/INV+Pmq0/SA9tK3FdtkFc+u9dFUGHax7ZmWn0Q6+TGSFoHeXiL73awghMl1ifwSz25t2ujDEOAGAOHfSTpNg7CjJV2uAgFpoYZggSJy7/6BHhU3YKQ1K8f+o8tW8wV6lnQB4grIx0bjh6Udnrj0Y= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Dr4M8hUh; arc=none smtp.client-ip=209.85.221.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Dr4M8hUh" Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-421851bca51so3260026f8f.1 for ; Mon, 20 Oct 2025 00:13:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1760944406; x=1761549206; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=v29GrQR+ncc3DZUAcuhlbn2tQqEiegD397LZB82cRuo=; b=Dr4M8hUhE+/cBGZLTsBkQyyIzNFDW+rmb/Dxy7sgROQE8OhRNw5grubqMfjp+Ex6v/ Ll6ADkHS2z/2ZvABzPxP7kb4Hm9NRwrPZayLIfFAYRsXkejSVtZl5vmuK1jOi88O/yuI 4/4sZrjNVOo5KxgCD2Rf3Psqy2KjHYvivDDIFStEF8gzVFXnif/eSTQQMe/lOMq10f2j odhRfjzS6LqWs2fD2h/x55gLv80Wp7cjIKzQA0qAXz2bTP/PBaxZCAz5sE7BfITdKCV8 WxnBhMY0vTgtQmPGav8ySENXn0MGBjvvpIaEBJi83FtymZUEx16IJnBSQ4D2auG9DSlM Nm1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760944406; x=1761549206; h=in-reply-to:content-disposition:mime-version:message-id:subject:cc :to:from:date:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=v29GrQR+ncc3DZUAcuhlbn2tQqEiegD397LZB82cRuo=; b=jGwyOw6Juwg0q7EXYuKL8naxDKLmt6XUllnBdb0VFcxaYk1Frx4DqLEqw+IGyGwKO+ xF2fyfxiouclYcyyJFi3xnK6TK4x8hMAOJoHPIJDDOOWQs90KS/0aLouKfOYEZn5ZwZt +6dy2KyudC5GgFSqUPfMAnD8VcqT5e5cJDJFgG4V0XYFA52bWRVUZKQXLIfaqn8UlIZ8 bEmfupyYBEaP87FVjI5ERF2o4YfMRcVgJqiVj5dUZdYmtggYYqa0iz9gyq2Ccz0sKg6v P5djG1kONxM9TYRLABP0dB3sF364bm1YVSiDXNFDWuaQyghV6RizaHB+k3ovwTlNJSOm iq3A== X-Forwarded-Encrypted: i=1; AJvYcCU3is0FX24UTsdkyP6Ojd/TP2czj3Q2x1C01vgPy7m0qfNNqh7YT4uJGuaRtVFAdw1vVEvyEEG5fhm2@vger.kernel.org X-Gm-Message-State: AOJu0YzHFCETPerflCuZy/KTaArGpqtK5HaMCC/tRZ8/bYZqE5+MTZbG vH701zFjQ6O3o7Ezx1HHymFANkcoAvUxOembqWVQUFxrUfanX/Ouril4bLW7jUQAqAc= X-Gm-Gg: ASbGncs+CqPwlEuZGgOGCViKS9dto2u7bXds4k3BNhZu4r6iOAl4gwmmO809d8W9PjS gs1oLCZ9xTgprEMppSOYdLENMiCHNT3ddZFTZ+rmMKRGCr8rAGE/FC05izPYCKwW0eBotvHjhsX 3ytQ33yPJoIfecHq/K8RWMsijS3gpO64n4AWD0jtd43wDxkNC3TG/4BoPz5Ix4XWHtbWRA+MeKr n0yl2rI19qB+C99osjLU192ass8vgp/xJUwM+HhEIvLuDpdzx/oY7rJmcvMRrCHsVwVc4+K5lqa 9NR5Dc8V5Rbiw92HetJpa/tV2HPFCHGSV71mYA3fNJByf4sEilY3eW4Od26AUjVZda/12wVayzw ZiGjXEFJIuO+eBynKJr1ZRDTn70e9CI0++Adt8T1p3yUZNcSLRimiT0icDLPSAq8saQzXOSwyeV iRCAkjCiRxjDQoM7/S X-Google-Smtp-Source: AGHT+IGCWcj+ZeSsBGtAZjJoabbi0svPUJxfv7bQF2wz0Kk+abDltn5J9+7dAFKFJ6moPAKqLR/Rtg== X-Received: by 2002:a05:6000:25fd:b0:427:587:d9ae with SMTP id ffacd0b85a97d-4270587d9c1mr6413104f8f.9.1760944406004; Mon, 20 Oct 2025 00:13:26 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-427f00b9f71sm13715262f8f.37.2025.10.20.00.13.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Oct 2025 00:13:25 -0700 (PDT) Date: Mon, 20 Oct 2025 10:13:22 +0300 From: Dan Carpenter To: oe-kbuild@lists.linux.dev, Jonathan Brophy , lee Jones , Pavel Machek , Jonathan Brophy , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Radoslav Tsvetkov Cc: lkp@intel.com, oe-kbuild-all@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org Subject: Re: [PATCH v3 4/4] leds: Add virtualcolor LED group driver Message-ID: <202510201457.uOrhkKly-lkp@intel.com> Precedence: bulk X-Mailing-List: linux-leds@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: <20251019092331.49531-5-professorjonny98@gmail.com> Hi Jonathan, kernel test robot noticed the following build warnings: https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Jonathan-Brophy/dt-bindings-Add-virtualcolor-class-dt-bindings-documentation/20251019-172647 base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git for-leds-next patch link: https://lore.kernel.org/r/20251019092331.49531-5-professorjonny98%40gmail.com patch subject: [PATCH v3 4/4] leds: Add virtualcolor LED group driver config: parisc-randconfig-r072-20251020 (https://download.01.org/0day-ci/archive/20251020/202510201457.uOrhkKly-lkp@intel.com/config) compiler: hppa-linux-gcc (GCC) 8.5.0 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 | Reported-by: Dan Carpenter | Closes: https://lore.kernel.org/r/202510201457.uOrhkKly-lkp@intel.com/ smatch warnings: drivers/leds/rgb/leds-group-virtualcolor.c:303 parse_monochromatic_leds() warn: 'mono_count' unsigned <= 0 drivers/leds/rgb/leds-group-virtualcolor.c:303 parse_monochromatic_leds() warn: error code type promoted to positive: 'mono_count' vim +/mono_count +303 drivers/leds/rgb/leds-group-virtualcolor.c bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 296 static int parse_monochromatic_leds(struct device *dev, struct device_node *child, bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 297 struct virtual_led *vled) bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 298 { bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 299 u32 mono_count; This needs to be an int. bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 300 int ret, i; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 301 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 302 mono_count = of_property_count_elems_of_size(child, "leds", sizeof(u32)); bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 @303 if (mono_count <= 0) { ^^^^^^^^^^^^^^^ bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 304 vled->num_monochromatics = 0; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 305 return 0; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 306 } bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 307 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 308 vled->num_monochromatics = mono_count; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 309 vled->monochromatics = devm_kcalloc(dev, vled->num_monochromatics, bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 310 sizeof(*vled->monochromatics), GFP_KERNEL); bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 311 if (!vled->monochromatics) bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 312 return -ENOMEM; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 313 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 314 for (i = 0; i < vled->num_monochromatics; i++) { bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 315 struct led_classdev *led_cdev; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 316 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 317 led_cdev = devm_of_led_get(dev, i); bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 318 if (IS_ERR(led_cdev)) { bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 319 ret = PTR_ERR(led_cdev); bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 320 return dev_err_probe(dev, ret, bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 321 "Failed to get monochromatic LED %d\n", i); bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 322 } bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 323 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 324 vled->monochromatics[i] = led_cdev; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 325 } bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 326 bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 327 return 0; bc8d39d8adf81b2 Jonathan Brophy 2025-10-19 328 } -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki