From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (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 7AA2822F38E for ; Fri, 21 Mar 2025 17:53:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742579639; cv=none; b=TuA14GSjIG7295s/XZKMP4U4WSYZnzEOZ4SFarwJrGDLp6W74wQ2GLuypfNOST5g0ru8tM4c9KqGjBC+wtyXnVzRBHMmJxrq3ZE3bI4PBFhST8TppedVD9rEXKECXe5UTvrMsAUyOrm2A1Hcs1MATb9h2Ng03iyJ3vwcudBkZ+s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742579639; c=relaxed/simple; bh=U6fktsyMuyMuZdm15c/lbWLdkSmOMUNXc8kCcIrjy5g=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZR+X1Py7kHl9Lwis+Tmh9zyVEcZDblGKt2VReyf9OuGkk4GcdDNQZpEOri1VV16oroEzA7y0RHFM0MXzwj1tHCfhShn59deaqdaPGbwRd3ob5w3/2jspOFey6/CD1eYP60x5qNa4hwAjrRqNrjK7iPKYK9G/reUHBK8HHsVRsj4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=ap9j5758; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="ap9j5758" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1742579636; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=O3nRKMratk622C9divVml2hIzMFZgLrlffCOuR6FHAA=; b=ap9j5758KELZ4iWcVIeBWOgyDEkE3Kmc9F2iLmk4W01PxYY49AMG8X1SExB5PNA/jGnBJ0 Hb19lZ7VGC+H3/gzJCezgm+8DzrDDgxzSmeCL04kRuN81noSUTPGX3dXEimzPPs8CA2GJo Yjs5egNl/m+CxnyQJMvvsYxpOecBpRU= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-571-b5Ow5mIDPnGMnTwwZjfPzQ-1; Fri, 21 Mar 2025 13:53:55 -0400 X-MC-Unique: b5Ow5mIDPnGMnTwwZjfPzQ-1 X-Mimecast-MFC-AGG-ID: b5Ow5mIDPnGMnTwwZjfPzQ_1742579634 Received: by mail-wm1-f71.google.com with SMTP id 5b1f17b1804b1-43d01024089so16324005e9.1 for ; Fri, 21 Mar 2025 10:53:54 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742579634; x=1743184434; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=O3nRKMratk622C9divVml2hIzMFZgLrlffCOuR6FHAA=; b=K6gG2dV2X/WKkW0KMC8UWZ4GAxegnkOZMO/Gyzg/jWg5mdB4FJXWA8pab5MlOIohMS QH8ROZ0t2KKGTFaC87AegmiymWeiW2YZ+PlArUPnmh8IuzBudHbjVgM1T6dlsMTtwoAo L/rAiy4wkI/iy4uvZ9/VUBsJrm5DdgOh8Nhpg6dct47MXtBEDqPvIgZf8CUslLdOJ6N9 rXeiljR3LbkCvBUhYErz5WEPO5XDgyyVliSQK4QOaHiitZjgXt0k5yiSyQD/RJfZxMt+ pq271DT+ncbaQujteur/5g0oMEtMTHeYU5oo4QY3fpXWB6PmkKKabG3q7tYUnJaaHvBO cbUQ== X-Gm-Message-State: AOJu0YwmD0KMLp7el6LdpyrBNzb+brZjfaw4HDeVAjvoCV2L20ckbAdz c6LtQPe4199VKAfN6GpgyW/FA5xmMhJqit8Szd4K3T61g+oaahrc3CFdvF5eqAY5uPIVvH0AUfx Hrf3Ue1qeiclLTTkFA5mPBYFqj59duXHkeMpT6jWhu9FxEHARJE2Tuw== X-Gm-Gg: ASbGnctA8dPoVJWzEpv9tKSsqVFE8H4fiv1icHgyNOPozM99WonioBE8KKYacAdJx/O +ZqcnCF1iRpI8fJZcCEKwp01IPd1UVdNfiPKqFksfUDIogIxyDFnC6R9yS8gsSz/LPaD6Vj1y5O cWQPFReCxlUG+tA41phXVjPR7Ujq2Jwngdd8GMehwsfXXaRDWpEHcsLE5G75Lxw3kvlCULIFedL PzHDBqyft8pMThP92JpKxn1fQfDVEMc5Fqo/MILoO2BfpQJqdwnlnBBo3+JxpWI9EsM0uwQzHYr /UobxfHZrz46YV6pZP+nbWs5NwxuN2ArnLrf7ceadMMjig== X-Received: by 2002:a05:600c:c07:b0:43d:45a:8fc1 with SMTP id 5b1f17b1804b1-43d509e4640mr40035085e9.4.1742579633820; Fri, 21 Mar 2025 10:53:53 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEi86ywD1M/RbAWOzMYKUQu56wBK07oi3Jw+JWYD17hgaWYXOejGMZOs7EmeoFI2yzpzEZPFA== X-Received: by 2002:a05:600c:c07:b0:43d:45a:8fc1 with SMTP id 5b1f17b1804b1-43d509e4640mr40034835e9.4.1742579633387; Fri, 21 Mar 2025 10:53:53 -0700 (PDT) Received: from [192.168.88.253] (146-241-77-210.dyn.eolo.it. [146.241.77.210]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43d440ed786sm83196635e9.38.2025.03.21.10.53.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 21 Mar 2025 10:53:52 -0700 (PDT) Message-ID: Date: Fri, 21 Mar 2025 18:53:51 +0100 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next v3 2/2] net: mdio: mdio-i2c: Add support for single-byte SMBus operations To: Maxime Chevallier , davem@davemloft.net, Andrew Lunn , Jakub Kicinski , Eric Dumazet , Russell King , Heiner Kallweit Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, thomas.petazzoni@bootlin.com, Florian Fainelli , =?UTF-8?Q?K=C3=B6ry_Maincent?= , Simon Horman , Romain Gantois , Antoine Tenart , =?UTF-8?Q?Marek_Beh=C3=BAn?= , Sean Anderson , =?UTF-8?Q?Bj=C3=B8rn_Mork?= References: <20250314162319.516163-1-maxime.chevallier@bootlin.com> <20250314162319.516163-3-maxime.chevallier@bootlin.com> Content-Language: en-US From: Paolo Abeni In-Reply-To: <20250314162319.516163-3-maxime.chevallier@bootlin.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 3/14/25 5:23 PM, Maxime Chevallier wrote: > diff --git a/drivers/net/mdio/mdio-i2c.c b/drivers/net/mdio/mdio-i2c.c > index da2001ea1f99..202f486e71f1 100644 > --- a/drivers/net/mdio/mdio-i2c.c > +++ b/drivers/net/mdio/mdio-i2c.c > @@ -106,6 +106,62 @@ static int i2c_mii_write_default_c22(struct mii_bus *bus, int phy_id, int reg, > return i2c_mii_write_default_c45(bus, phy_id, -1, reg, val); > } > > +static int smbus_byte_mii_read_default_c22(struct mii_bus *bus, int phy_id, > + int reg) > +{ > + struct i2c_adapter *i2c = bus->priv; > + union i2c_smbus_data smbus_data; > + int val = 0, ret; > + > + if (!i2c_mii_valid_phy_id(phy_id)) > + return 0; > + > + ret = i2c_smbus_xfer(i2c, i2c_mii_phy_addr(phy_id), 0, > + I2C_SMBUS_READ, reg, > + I2C_SMBUS_BYTE_DATA, &smbus_data); > + if (ret < 0) > + return ret; > + > + val = ((smbus_data.byte & 0xff) << 8); External brackets not needed. > + > + ret = i2c_smbus_xfer(i2c, i2c_mii_phy_addr(phy_id), 0, > + I2C_SMBUS_READ, reg, > + I2C_SMBUS_BYTE_DATA, &smbus_data); > + if (ret < 0) > + return ret; > + > + val |= (smbus_data.byte & 0xff); same here. > + > + return val; > +} > + > +static int smbus_byte_mii_write_default_c22(struct mii_bus *bus, int phy_id, > + int reg, u16 val) > +{ > + struct i2c_adapter *i2c = bus->priv; > + union i2c_smbus_data smbus_data; > + int ret; > + > + if (!i2c_mii_valid_phy_id(phy_id)) > + return 0; > + > + smbus_data.byte = ((val & 0xff00) >> 8); and here. > + > + ret = i2c_smbus_xfer(i2c, i2c_mii_phy_addr(phy_id), 0, > + I2C_SMBUS_WRITE, reg, > + I2C_SMBUS_BYTE_DATA, &smbus_data); > + if (ret < 0) > + return ret; > + > + smbus_data.byte = val & 0xff; I would not have noted the above if even this one carried additional brackets... Cheers, Paolo