From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) (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 36BBC38BF62 for ; Mon, 2 Feb 2026 18:01:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770055313; cv=none; b=AqoiOTo3Dd24dOa+aYzOZ0rjFhnETLywK3XdUsxGFtxnsop2guZjs10W19w+iB+mISTzaXsocDQpT71W/1KNsPZ/BRmj+xOa8jr1o/mesK8z1riKiZLDu7fS9hfbB6sPjrWhVlWxjxg6F6iEF9WGZxApVscgdJzp3yWxJX9Ynzg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770055313; c=relaxed/simple; bh=hTtgkhOWMpNlBXr1nPDMrRziEVVQp/MwMYD4QueQQpg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=N4Vd3NGPAhxxrWlVDcmOXEk29V0JDFu7yrSP8eY5Yo1F8YJxxvJXKElQnM+Rzm6og51rWF7H7BveJn75mYjhOUOwIgQ8tNDz88s1do/b3OXDesaCRB8Stp6rYOqFvNKaR/L2Ir7SWWsb5+QsnypJ3N69nHNrw4TeWr6nbJ6+MZs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=jMPagI2B; arc=none smtp.client-ip=209.85.167.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="jMPagI2B" Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-59de0b7c28aso5514535e87.1 for ; Mon, 02 Feb 2026 10:01:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770055309; x=1770660109; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=sFABA++ZvkbyuC08URcMyM+fmFe85L0vnPidtUOeUZA=; b=jMPagI2BaSTgwApEPadMupRuXGmZULICw2dkfzBIlw/3Y62rJKorbZZdFdP8vQLPpT QJ79Q4qLJkKZ6rxEbQAp5yJTJdgGR3sv8hSnIWK2Z/tfor2To5n4bgxEU7v5SuqF0POf 24J1wQluxfgfE0JT2AV7wZFdwtGyiJ25BuG3CLXth/sauSqYzMmic2EgrlKjSwErQK0P aygK5F8dzS06vHtfWn6JmAL43U0nc83sn49e+Lg7RUr3sJ3naSSTF7qA2qdNwP9xib0c qSJA1tacedVLWbXYwDwMjbUQOKh1G/pEYLTFNwKUFn+FYjDK+92EFlrSR4Jj1FC9G8as +1cg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770055309; x=1770660109; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=sFABA++ZvkbyuC08URcMyM+fmFe85L0vnPidtUOeUZA=; b=B8FjdifgXN9iXhW1gtEUZOc3Er7eiG+GRWuwfOBRqMHdWCSKiWROybRxYYhCIzBqON p969GLles1Ynx3j/HWklOtsms80i/vkpDOe3DSG3/Qz6tbhc3LcSNydojuNaw8sIUt/p YBm2src1XENxv/aWdHFzxewjNn1nrDfq2eogFdEZqaJZCYsx/CCVW8zgOoD9kfA4esfz LfOu5CreuJfaHTfojNetiPMok4tfKLYG9IYxHKe8O3PX1846aMlOs++LnaNTZguQ29dh L8SxgQ8GSL2xXTjzemPrSQs1hrq81miiONMh+4KJc19EyL8HHPnxfBfhn/KWI2J1MHLR p+RQ== X-Forwarded-Encrypted: i=1; AJvYcCWc0s9H+gDD+ay5fK27j4qdqZiDC4nmzXuyYIwm7qTQw125Xwjj/mTmeO6+rBES/QV+2s5GL2YN8aIT76o=@vger.kernel.org X-Gm-Message-State: AOJu0YygsGhj2V5osXbBXLp7mNhQ3mLylm0lOMnvY1xv8Sf0HY6pHFPX LaNHAVVD3WCn55qQjzNHqXKZ2yz0SeCYqADpEWHzKDwPrCGWzM2ULiHI X-Gm-Gg: AZuq6aJe+i9ZU5zOU/7I+YczVN9PQHtMj/MP6pETQX01dfwL2YJamyRsSb2qrILrWSf WnHn+qcoecaSq9o/4VL9IiLip74diDpw59QlxLEXUcmb6JpDV2JfA8BkjvZvmpaDb+TcDfxVNrT PxUnUgIc2y+wBEHPUmEGjSRKm4bqA6VeOsS1Oty89NcJ734HZozChio7xccwPM7hvMlJoqcWWcx 06Ri2oFQT71rkRpXFFg+2hqBWndKtU/qGpQbdX+so+Cn6fUCRoQ2xLsDFl1wwYA5oTnj0qc0IHj 5owKAp0zHsnoJgd4bM0C3Ils6wCIEPVifH/+cib7Z64WpHNFNqokrDKcI6ny4vdbRV17xyRHF4z Ae4GBZvNKxDO6vi2um07M/Dno8+0O/9YcTXa5iICYfVxRM9Fnr+XqZDhfy1XcWA7eVjjXB+bl64 aEuo3xTZt0tv3VMHaOUB7XqnzlDtgdUY/E6VC5pwqijZm/5vnPVfL3LG8pAkmF0UFZ+n2U3w== X-Received: by 2002:a05:6512:4023:b0:59d:fd1d:ddff with SMTP id 2adb3069b0e04-59e164304e0mr6191268e87.45.1770055308708; Mon, 02 Feb 2026 10:01:48 -0800 (PST) Received: from ?IPV6:2a00:1fa0:4246:14fa:feaa:638c:335c:bdbe? ([2a00:1fa0:4246:14fa:feaa:638c:335c:bdbe]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-59e074bdabesm3761777e87.93.2026.02.02.10.01.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 02 Feb 2026 10:01:47 -0800 (PST) Message-ID: Date: Mon, 2 Feb 2026 21:01:36 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 2/4] usb: phy: tegra: return error value from utmi_wait_register To: Svyatoslav Ryhel Cc: Greg Kroah-Hartman , Mikko Perttunen , Thierry Reding , Jonathan Hunter , linux-usb@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260202080526.23487-1-clamor95@gmail.com> <20260202080526.23487-3-clamor95@gmail.com> <74268def-31fd-4015-8be6-faebd6676667@gmail.com> Content-Language: en-US From: Sergey Shtylyov In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 2/2/26 3:14 PM, Svyatoslav Ryhel wrote: > пн, 2 лют. 2026 р. о 14:05 Sergey Shtylyov пише: >> >> On 2/2/26 11:05 AM, Svyatoslav Ryhel wrote: >> >>> Return exact error value from utmi_wait_register during HSIC power on. >>> >>> Signed-off-by: Svyatoslav Ryhel >>> --- >>> drivers/usb/phy/phy-tegra-usb.c | 9 ++++++--- >>> 1 file changed, 6 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/usb/phy/phy-tegra-usb.c b/drivers/usb/phy/phy-tegra-usb.c >>> index 3a7a74f01d1c..6173b240c3ea 100644 >>> --- a/drivers/usb/phy/phy-tegra-usb.c >>> +++ b/drivers/usb/phy/phy-tegra-usb.c >>> @@ -891,6 +891,7 @@ static int uhsic_phy_power_on(struct tegra_usb_phy *phy) >>> struct tegra_utmip_config *config = phy->config; >>> void __iomem *base = phy->regs; >>> u32 val; >>> + int err = 0; >> >> This initialization seems pointless -- the newly added variable gets overwritten >> by you later... >> > > So? let it better be initialized and rewritten then later on catch errors. I'm not sure what errors you mean here. To me, it (contrariwise) seems to mask the possible errors when you forget to set err to e.g. -ENOMEM before returning (when adding a call to kmalloc() or any other function that doesn't return an error code itself)... I'm pretty sure gcc will drop this initialization when generating the object code and (what's worse) the static analyzers will trip on this code telling you that the value 0 is unused... [...] MBR, Sergey