From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f178.google.com (mail-pf1-f178.google.com [209.85.210.178]) (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 59F983C6A3C for ; Wed, 4 Mar 2026 17:35:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772645745; cv=none; b=El1MJeW9TzOiGhrhBcc3qow0f6dQHdj0Dod1tNgur8CPlaLeU1ysEym0Sto5YYYnIUkFy1KbGiELKGHlanxtBqU6VCleGWe15nUJBkmgCF0UEL/S3EPGzOhbKZt7YzuNflSNs+3xgCKyAKimbWdzQWROEsHKV5qAFD7/NMh86w0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772645745; c=relaxed/simple; bh=VlEH3lAnxtUSxpMHwf0dxl3KstRwnQssP0ryb48SfxM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=puWE+pFzVolEhOTJ2s2jZwZDzLfP4bjOIYfuOvl1jufJn+5+SDkUBxoLMUEW0Ws7kraS1tz1fUfMlp8oodUgGyO79vI5MCj1smcKsNT5hU+w03cK+HgNwa4Rwf9MI8PLvIph6xwiTFDTz+eBMf4uENgaLHaZT3YjhnDzqp7CI3Y= 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=bdK/V5fe; arc=none smtp.client-ip=209.85.210.178 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="bdK/V5fe" Received: by mail-pf1-f178.google.com with SMTP id d2e1a72fcca58-8272c559597so7117060b3a.1 for ; Wed, 04 Mar 2026 09:35:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772645744; x=1773250544; 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=92zh51YpPKp93KuezxDm90TJZ2/tmL+TqtOQmshr6EU=; b=bdK/V5fe1529VnHbFxFywbfK/gC87aH3xuKD9Ibo/OEXooF7r0Dj8FwGYHBGBUkaaj cNj0KYGeM3OsHLSm9Ee2Xtle5AErD778BjI5mcX5TlWQMyE18yoRrLrIDvc8qc8eZ9kA IGKlCy4brZ3CvoyJnr8p+7nSJJad1+8wYoj+I9kCU2vG/Wpszy7x8sZT36VHB9n27s36 l4fMasFUIjnhq65jPMYL4eXRFWc7MB5i6s8ZiNf3Mb4sCPJakbqllW1oDhSfCy7NsRUx ZODBPZAgDkbwiWoE9XTLGoFGjp7eFmd7e4bcAPjN1ua3/L/85RWC5eDcobrnCAuw2c2h k3lg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772645744; x=1773250544; 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=92zh51YpPKp93KuezxDm90TJZ2/tmL+TqtOQmshr6EU=; b=IgcLyYo0mO5wZjR6OppMveLkr2ulAIOw+40WX/DNKEbKm7++Ph31ON9h8slpckhNCp GeJtlHTk+TeesUmApQX3YPmWc6iq2yOip9a4PnB8RmpngPx/ChaQhsQIye6NB+uwYb+8 r71Sam2Uyr/8IhSo+GWFR8Egwm+Lhb+ZaAjbtFC+1/0vLMotCGsMb13BvPMTo+qibUby axQYQ8qhosiLi3EBNmNY2gL7B53GTrzLgz/nkWzpVL24G4DI8Aik2C3ShMH0JRByxzWh HEwp8zWWMPvYtJJTiQ0FwIQOD+BtZQKiZTnCnvRyd4phcip83uF9LrN4XFY1NUZUZ3FV ArFg== X-Forwarded-Encrypted: i=1; AJvYcCW/qW5qO9giYMpuIrWssGdlHQZQQVOCMfXbEi9diqUEFcsleMlQ8RkzWxyTe2CjXBNVUBVb0/DUHiF16NI=@vger.kernel.org X-Gm-Message-State: AOJu0YyKu66ZDVdbru5WYmEBBxhAOiZt9ZaEz64obhuWqwbVeAh3uaIX AjP273dH7/eqTCM4juem5hyrSOS0hzBDDbbIge3n3XQOqDgev8LlRm2s X-Gm-Gg: ATEYQzwPEGoeW8lSygNFJ8yg90DoToRN9ZQ9PzGYeJH+ZCjyQnK1rlaC9VWVoUF8B/K IkfuY+UksuHvV8NwhrAvNyyCDEYN1TxREF4ltol5M2aw81SHiGeQwHBXFuVUTjMK1NdBiQhoNoz +fXb1DIIX+9eURm6hpaEp7URc2qgLC6GbMmdohVmrl1TMbO77xVQwnNckUbdUy/nAhRggCQpkQw 1W4arNmUGRCQOhs2KEoj0+3ZPdaINiAWJV0Gwepe8UZrcyQFzV49KWuLgux+YpLUN7dtf60wPhy VetUJD9JWpGB+K8/XE0hRL7q9wAU0O4dySj05I9+9xtEmzCTLklYIwWiVe1vhyfr4lgGSATU6Ci B/Gt0FR+6CHHzpMSXNsPr9zXMDcrFeAqxere0tUo9xepMgXZ8bjoaBhE0vOgLHx3Hh3t/dMmTNH lTQek9EEtsaFIrbrvjKz1pW3Q6sRl8wekJBh7PGOeRqPD+THAy/A03KFzUWacgFhyx53gUzPyN6 nTPIMWvrnLbAwz8lpJGzxU+9w31fg== X-Received: by 2002:a05:6a20:4327:b0:394:8455:d1a8 with SMTP id adf61e73a8af0-3982ddb7520mr2712238637.13.1772645743652; Wed, 04 Mar 2026 09:35:43 -0800 (PST) Received: from [192.168.1.109] (c-73-37-64-97.hsd1.or.comcast.net. [73.37.64.97]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c70fa8059bcsm17463809a12.18.2026.03.04.09.35.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 04 Mar 2026 09:35:43 -0800 (PST) Message-ID: Date: Wed, 4 Mar 2026 09:35:42 -0800 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 v3] usb: core: remove unnecessary goto in usb_create_newid_files() To: Greg KH Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260304155812.4559-1-marusik.adamenko@gmail.com> <2026030412-trace-sneer-60bf@gregkh> Content-Language: en-US From: Mark Adamenko In-Reply-To: <2026030412-trace-sneer-60bf@gregkh> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Thanks for responding, I wasn't suggested by any tool. I was advised that if I want to become a contributor, I should start by finding something small, so I came across this file and found the goto. I understand it's not a big change, but the kernel style guide says: "The goto statement comes in handy when a function exits from multiple locations and some common work such as cleanup has to be done. If there is no cleanup needed then just return directly." The original function only exits at the exit case and at the end of the function, without extra cleanup. On 3/4/26 08:11, Greg KH wrote: > On Wed, Mar 04, 2026 at 07:58:12AM -0800, Mark Adamenko wrote: >> The goto to 'exit' only returns the already initialized variable >> 'error', which would be 0. At that point it could not yet be anything >> else. Replacing with an early return 0 removes the >> need for the label entirely. > > Odd text wrapping :( > >> >> Signed-off-by: Mark Adamenko >> --- >> v3: fix missing return statement at end of function >> --- > > You forgot the other versions listed here :( > >> drivers/usb/core/driver.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > But first, why? There's no real need for this change, why make it at > all? It follows the "standard" way of jumping to the end of the > function to exit. What tool asked you to make this change? > > thanks, > > greg k-h