From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f45.google.com (mail-ej1-f45.google.com [209.85.218.45]) (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 793F9352009 for ; Sat, 15 Aug 2026 19:12:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786821131; cv=none; b=JliKadwR39F4ywaGSfuykPNtidq5c8C5/MD+vk9MDYYZ+0SDN1g5yN8Qcz1GoQ7Xyv/H3/nvYgtfb0VEYsjySTGhKT3Fpkja3pYByMwslEgSgtviEtuWtAVxVMQYBGZjdtJKSXYyBx6A7OuJDNYybgUbd3PdypktCfeXmkitmR4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786821131; c=relaxed/simple; bh=5RkMUJ1444X2cAUbWcMBKjhTBMhgCS7gOIBcS9lYMPs=; h=Message-ID:Date:MIME-Version:Subject:To:References:From: In-Reply-To:Content-Type; b=uPc/7kDL4nymeqJ+5woxE8+vSMbVcU+/jgjm5duIdxAd4uwVQ7LurnHPiIPHNORpGKMU3RXm0fGiYk9E6lEnmM1qVjGKnM47zfUzT2WqonyR4jBUCzt/nR0NquQdz4UzWFQwf7fDEncyJX+eQGulNZqtmTL1Nw18NjCFB1jYKSw= 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=OGvpzoKv; arc=none smtp.client-ip=209.85.218.45 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="OGvpzoKv" Received: by mail-ej1-f45.google.com with SMTP id a640c23a62f3a-c167aa9500dso295850466b.3 for ; Sat, 15 Aug 2026 12:12:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786821128; x=1787425928; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:to:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to:content-type; bh=88izO+QaB/O00yTBhu9wQoygITPFWyCSorKc++U3zBY=; b=OGvpzoKvC/8HTbmaFB/nErXD6qnlhk/zmsXYD1OoN7t5w35IW4M0y+EYsAKrhWKfyR ufdqcKHJCBbLpLj1obAkoaZkYYY1zrjg8byCGapnQDz/fViPHjZ8ihPMv2Rl3rk1Urou MWX5hktfymmp3wuZlZXPITJfxQA1nVgtyU2O5Bom3kdZh9kDedUJuFfrdwceIOJSqeTu hzpQE9FPQktvXr2EhyD53G57Kbe3eG6BkXbu5Sm/MjTJo2emI50CU60khHg4ZyHZ4w29 NlWfVs+GMpGmmVsoKZO45DrVYz+IuNGsk0HjiL6lrnjbwuF7ZWpdYvvOQZRMWm13Oic4 g0sQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786821128; x=1787425928; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references: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:content-type; bh=88izO+QaB/O00yTBhu9wQoygITPFWyCSorKc++U3zBY=; b=HDvzEZzhNa1wMQZUJ9hPh8HJ7ySCTGNgD4ZF+Tugu1b6k10YflhABq8mJp5hMaZWBK EH4vTE3rz9ixPFe/PqN0ZsfiwPQsg2u5v1qHITQnz1caD0MERG/fI+BG1vE/1qx6joAc oYKYmQu2jwPvy/JbGOe+dZ3BP64xelZ9D60fhJKxPZFVThvQChKjNKzRKg5JcBIO0eUQ OhKUuHbq2zweq1lSMfffYOvedDLsauBc5YnYEyUA3YfJPYXSA/X4jcmXWywoecCIKj6P WGk2rCFwEE4gMTkwl7QS2xOj5ikIY+weeePyHNVJdl+OW1nl36ciVlT99bug0pqmS+9F DDug== X-Forwarded-Encrypted: i=1; AHgh+RqvDvSYAJElBF0k0Yt4MkXpfps2z9/omaN/QrtgyQy7auST60B8CtggSYsqrFJjz8l/lAIS3ns=@vger.kernel.org X-Gm-Message-State: AOJu0YyEZIZxoGG1D6EbjlaL1bBIZ4rwd9TGDlBPKZdojyED8KBvcv5j cD7RqwwsClwbdMI7vLm7+zC737ptqadqbi1qAfKEfvGeR85QHybv/X3nQO5GLY/Y X-Gm-Gg: AR+sD11twNBnvWfsJGoWzsZhDYKpNRwe+oqWZ5JaLPgyZEPzDg/NEYN3jkwovonMn0L EbrBb5Q0bW7vrKoTLEXJN7udzEop1KsCqLbRYjrhktkbLcTnUk1cZL4MDhr+WEXhTHHpiax9xlY ZZKzHoGVcM8mG0Yjz1/YGf12QXn6wDq0xaENF/iYBHgUkcnFkA0HcqAvhiyRncWIRGVAy0vnxCL AXzbBIG+QKdH7LcQZmBflGJyZ7tmv+DIIpgGlFmbUsn3wAJb6CbRJ5a2cXSuqdpKKmxlwUwt80v tUHHdUc50DkC5gvcYLhxAEnqIT7xhy2an5Kds4hOYXun1ElF2dzsfNWrcGZb3OLZ9cyHS4qMUu+ oSm8Cv5vKrpQGzRHQblmo+MAUsB1flGkg54fKVgnlvIXKz3Ii4b1aA/BA/KkqZdY6ZXxIfNNwq6 Lh59mkJvNEm8VvlGfvDdwZ+lhi7IcC3N4W8J5fmn62HmEWr4yvUFxrMXaZQdDRnrNQ/FtEb3XDX zYCYg0O0OOXmfxE/1H4emJASlRkcL3sOxsdIaTUtw== X-Received: by 2002:a17:906:eeca:b0:c1a:5d49:f99d with SMTP id a640c23a62f3a-c2129ab6c24mr672768166b.7.1786821127686; Sat, 15 Aug 2026 12:12:07 -0700 (PDT) Received: from ?IPV6:2a00:1fa0:28f:426f:2aa1:8f7:f8a6:4f89? ([2a00:1fa0:28f:426f:2aa1:8f7:f8a6:4f89]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c212339f744sm252129866b.10.2026.08.15.12.12.03 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 15 Aug 2026 12:12:06 -0700 (PDT) Message-ID: Date: Sat, 15 Aug 2026 22:12:03 +0300 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 v2 10/10] net: ethernet: ravb: Add gPTP support for Gen4 To: =?UTF-8?Q?Niklas_S=C3=B6derlund?= , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Geert Uytterhoeven , Magnus Damm , Richard Cochran , Andrew Lunn , "DavidS. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Vadim Fedorenko , linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org References: <20260811160200.2049987-1-niklas.soderlund+renesas@ragnatech.se> <20260811160200.2049987-11-niklas.soderlund+renesas@ragnatech.se> Content-Language: en-US From: Sergey Shtylyov In-Reply-To: <20260811160200.2049987-11-niklas.soderlund+renesas@ragnatech.se> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 8/11/26 7:02 PM, Niklas Söderlund wrote: > While driver advertises gPTP support on Gen4 platforms it is in fact > completely broken. On R-Car Gen4 devices the RAVB module have no > internal gPTP clock as generations before it. Instead it utilizes a > system-wide gPTP clock. > > This change utilizes the refactoring of the RAVB gPTP code to add > support for a system-wide clock and stops the Gen4 devices trying to use > the non-existing internal gPTP clock. > > To remain backward compatible the ptp-timer property is optional.If not > present, or not enabled, the RAVB driver will no longer advertise gPTP > support to user-space. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Sergey Shtylyov [...] > diff --git a/drivers/net/ethernet/renesas/ravb.h b/drivers/net/ethernet/renesas/ravb.h > index b4df5cab0e5c..4c8dc02022e2 100644 > --- a/drivers/net/ethernet/renesas/ravb.h > +++ b/drivers/net/ethernet/renesas/ravb.h > @@ -249,6 +249,8 @@ enum APSR_BIT { > APSR_RDM = 0x00002000, > APSR_TDM = 0x00004000, > APSR_MIISELECT = 0x01000000, /* R-Car V4M only */ > + APSR_GPTPTIMER_SOURCE = BIT(25), /* Gen4 */ > + APSR_GPTPCLOCK = BIT(29), /* Gen4 */ Again, I don't quite like that you switched to using BIT() all of a sudden... [...]> diff --git a/drivers/net/ethernet/renesas/ravb_main.c b/drivers/net/ethernet/renesas/ravb_main.c > index 12877a8488ed..30170e92d3d7 100644 > --- a/drivers/net/ethernet/renesas/ravb_main.c > +++ b/drivers/net/ethernet/renesas/ravb_main.c > @@ -2752,6 +2752,59 @@ static const struct ravb_hw_info ravb_gen3_hw_info = { > .magic_pkt = 1, > }; > > +static int ravb_gen4_ptp_probe(struct net_device *ndev) > +{ > + struct ravb_private *priv = netdev_priv(ndev); > + > + priv->of_gptp = > + of_parse_phandle(priv->pdev->dev.of_node, "ptp-timer", 0); You now have full 100 columns, no need to break that line... :-) > + if (!priv->of_gptp) > + return 0; > + > + if (!of_device_is_available(priv->of_gptp)) { > + of_node_put(priv->of_gptp); > + priv->of_gptp = NULL; > + return 0; This *return* seems superfluous... > + } > + > + return 0; > +}[...] MBR, Sergey