Received: by 2002:a05:7412:3b8b:b0:fc:a2b0:25d7 with SMTP id nd11csp918890rdb; Fri, 9 Feb 2024 05:20:43 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUuYWWRAy/Shdpg8+rAcTmi+N8WAy/PRzg+Rpq5EWO/42PMCYgqLjYMJ5puZC7GnAxPa8x5+nwBUdPeeq6eubSB65vvm7sNrY/l98425g== X-Google-Smtp-Source: AGHT+IFCTsPFy1wO4cC+9vPiFzO+R1viAvx4xG6CX9Aog7w3nXWI3Y6kUqvYDC8epwXl5sR+g9ba X-Received: by 2002:a05:6a20:e68c:b0:19e:b196:c8dc with SMTP id mz12-20020a056a20e68c00b0019eb196c8dcmr1825614pzb.20.1707484843420; Fri, 09 Feb 2024 05:20:43 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707484843; cv=pass; d=google.com; s=arc-20160816; b=awbvSZKwSuLv6a376KeYOjZ4qHg3od9mDyos1kPuUfrw89auqlLo91fT9bLdzLzwjg bBhKYLyEE0chmWYUJN8KkgUGsFjiyl11Ydtt5ZCklwUnUoCHOfsrRMpcY3fedVmSfaL0 NQMsV0YsLwunUG3vk6qqkzYELX+Eur3ogU4IhYiPxAGkUlsS9L8qixApkExef5lJIoN1 6gMP16N+MfZACKbbM+BOuumIBk4aXxePOYFK2KI6qVt514qwtRA9KlOq8wTC439xUCna D7raWTWkUu9m9ss9s5mTauUvXJSXO7W7MKf+LKvAqfqfGDFfc6lyHBknkERTkKkbx3nP u+3A== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=yHV4xD6vA5uk0R6Dh4WYIT0pSu9+jAt1mKXfWSKM2do=; fh=NQxnrfI0MWos31hMbZlou6xO6kwGsd6VesPl2zmrv+c=; b=MGK9NZ8FWhWdttdy9LbQrMqhiMkWalkbpXplWCwfys5m0eVbFJnBbf71Buk7Q0Ivoz pIqi2cwFMTnnV/IA295c+ZJq9E3JtGJCEngry7JpsObjYmyrb7o9l4GFkA8oIkMMXoQf Cu99tiITsgUR8ESzy24Rw4m9FUU5o2DFXwtVqB6kUS6UdHkWk9hE06pVN5lEqJMziA5+ sHwfN6x35pCh6MivXFB36oRjl1dC3jlJ6P8P0MOyrBBngw4NsToSCugVXlbBiCOeaTKJ 4g51mmwQh4NNVwewuY7O3UgWka/HObDdQDGzzcffFx0OwJj/BjQ8UR8Jt0fNJCnbD3yF jf/g==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=X24qYcl2; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-59351-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-59351-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org X-Forwarded-Encrypted: i=2; AJvYcCUH+yMMVeHqqnEpLl+Sdn3T+xSQMfE4NnscVERi8y4FcvAnWVjm06qzAT4PDZCFBDTmoDCt/L1aVPmP15PNRanW4dHNkSdSELZu88VWfQ== Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id cu15-20020a17090afa8f00b00296377bb025si1617028pjb.55.2024.02.09.05.20.43 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Feb 2024 05:20:43 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-59351-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=X24qYcl2; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-59351-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-59351-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 1AA74284B9F for ; Fri, 9 Feb 2024 13:20:43 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7656E381B9; Fri, 9 Feb 2024 13:20:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="X24qYcl2" Received: from mail-yw1-f178.google.com (mail-yw1-f178.google.com [209.85.128.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 28DA72E630 for ; Fri, 9 Feb 2024 13:20:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707484834; cv=none; b=SB/uO5LrNaoUdhrvRwch6sgA2ptkVzGZqbG7HiWzauIATtSfOo8+2EjExR8HHIe0WmGxHov3f8R+a1aCoDl1zAEZF8G7ZPJ+jHbvSzqLDHPqHVDGLXGIMEVPuY7+C15Mo13Ew5fS+b9hD0xWg4DoJm5ZXvfd+PVYrhiNifBt+rA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707484834; c=relaxed/simple; bh=yHV4xD6vA5uk0R6Dh4WYIT0pSu9+jAt1mKXfWSKM2do=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=GFuV1tx61y5W3XE2ObV65qJIqavCmzHdOROGARpNrKZ1hXqvTIrI7qxLTwud6J5hX8QUYEALqw6GLxuULgvg6d8pHAxIdYPKx0OweFQFKWpok1oXShmc/HmzQrOE75wUhucuzln4WWl+lOS+lUSgBhFzNF4X1ODaPUrPhhI5tFg= 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=X24qYcl2; arc=none smtp.client-ip=209.85.128.178 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 Received: by mail-yw1-f178.google.com with SMTP id 00721157ae682-6002317a427so9150287b3.2 for ; Fri, 09 Feb 2024 05:20:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1707484832; x=1708089632; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=yHV4xD6vA5uk0R6Dh4WYIT0pSu9+jAt1mKXfWSKM2do=; b=X24qYcl2NWLm9SXvPG0vqETFeGHM+OCT82MbulmC+FlrmY2aFDqt7dpsOn2qNuh0jf AciIwr+HvE7Y4h6t+6hC4PhrclT1qcTw5dBUWmOwafy7hrNX/gs4LOX/Ged0tqm8YQgJ PVrSYHlCtJR6SlryCMDZSFVVEqAX43V+l7QjSSPu5gtkh2y1t2uU/0T00LiTTVwAK7mL zruJ/BjAbyP8KeSwPgvmaang5PZpJNot5ANKAv4bFyOTtj2fsFYOTqOJpnSg7zQxPWmX 4Qo51Eq0lgS/2LveK61oSgknetek/RkniZNys4F2m1oz+4MLB/+e6YNoRhANuZbcgKdc ZLJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707484832; x=1708089632; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=yHV4xD6vA5uk0R6Dh4WYIT0pSu9+jAt1mKXfWSKM2do=; b=BDDq1RbGBJsQSR1EF6Ukseq7c/xF1ONZy1UAqJ6w/poRrfpwm79drkfbPz2aOOaAlQ o9tHe11MTCzmVN5FmjMz7TaBCMWry4KVIPwNHDFe2SgAcGJ8NrtSvL9FsTkvET6Yn6Ha tLuboRsWgghGOLbZgJU20d5RP05aS5NhO9Gv/+UNd9eeRzFABy4sqBZNp7+Z+01X8tlJ 3jmnsRJCnjuMR9TBR6zZE+NYr3IF+uWZMMssDZwgV/n0y8W3Ufj4PcHHk2kFpqFxtfy1 u0mEAQemypoCNkSuRkSOMWeC0ZJm6vSyXlHqXkL6i9g+az0+N0X1P0c3xLh4almnDf6a 3Z4g== X-Forwarded-Encrypted: i=1; AJvYcCVSfD+JI5VFUp+HclCNa0qTyNWP+ylJr/FMzsryfpKoxcaMTIvLov+M7/mIpyhLNYrxU+DeYaXSVdMBYWuCxhMQm1YQuOlD0fA8k4PE X-Gm-Message-State: AOJu0Yz4ElMzrbPTrjQs85BmJf5Xjslko0U+2y7oEny5FXOk1vsuqvfK uIMPnGFmU4NHSrXy6gqah9XeC5/vdAT1IXmn3TMkxhaYet0Icop9Loa+OgZV3oEI2Ug9yTI19eX bjqOpu78SPpPyK2JtmN3Pjk3OTLbFxyQNnIobvw== X-Received: by 2002:a0d:ccc3:0:b0:600:34b6:6bc8 with SMTP id o186-20020a0dccc3000000b0060034b66bc8mr1289933ywd.33.1707484832203; Fri, 09 Feb 2024 05:20:32 -0800 (PST) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240209-realtek_reverse-v6-0-0662f8cbc7b5@gmail.com> <20240209-realtek_reverse-v6-5-0662f8cbc7b5@gmail.com> In-Reply-To: <20240209-realtek_reverse-v6-5-0662f8cbc7b5@gmail.com> From: Linus Walleij Date: Fri, 9 Feb 2024 14:20:21 +0100 Message-ID: Subject: Re: [PATCH net-next v6 05/11] net: dsa: realtek: common rtl83xx module To: Luiz Angelo Daros de Luca Cc: =?UTF-8?Q?Alvin_=C5=A0ipraga?= , Andrew Lunn , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Russell King , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, Feb 9, 2024 at 6:04=E2=80=AFAM Luiz Angelo Daros de Luca wrote: > Some code can be shared between both interface modules (MDIO and SMI) > and among variants. These interface functions migrated to a common > module: > > - rtl83xx_lock > - rtl83xx_unlock > - rtl83xx_probe > - rtl83xx_register_switch > - rtl83xx_unregister_switch > - rtl83xx_shutdown > - rtl83xx_remove > > The reset during probe was moved to the end of the common probe. This way= , > we avoid a reset if anything else fails. > > Signed-off-by: Luiz Angelo Daros de Luca Sweet! Reviewed-by: Linus Walleij Yours, Linus Walleij