Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        Richard Cochran <richardcochran@gmail.com>,
        Antoine Tenart <atenart@kernel.org>,
        linux-kernel@vger.kernel.org
Subject: [PATCH net 0/3] Fix PTP packet drops with ocelot-8021q DSA tag protocol
Date:   Mon, 26 Jun 2023 18:40:00 +0300
Message-Id: <20230626154003.3153076-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?heM4ee8N2Kb0SrqhOUN2eVU7TvZFKqMX3mwxDA7wqGita5lcwd0TL/2R70mp?=
 =?us-ascii?Q?mQb9aywS+o/zmeOjI7jp9pR/HtsDfotN8ASM5e1JqCa5Sa+CpwzzIou43Lq+?=
 =?us-ascii?Q?60WTa3Ofz+emWXE2ic9J5cpKRQpCel3bSiqstFfsZjIRWDJLTwOw4ftYsTqH?=
 =?us-ascii?Q?IhygPr7Sg/aiuaZDzWa+nhKYxMeXp0xbGEY0u8Acck02Bc49qbH7iflKRooT?=
 =?us-ascii?Q?GDTNNyY276pT5/MSdOkm2sNuuAU7qXRSu/w5zhWOOQNBRGAhk6nAbCJU342s?=
 =?us-ascii?Q?D/hTkcMgZMUCY4kaH0K/lDdDbhp+Frnw+XrthG6ylzQM9n7F/KDBbhVbxzl5?=
 =?us-ascii?Q?lX6nX3ftUW8+tagfq734gNhDHOadgpawcM+PAxyjY9avdbbM6YQcHcbdOq7g?=
 =?us-ascii?Q?xDh47CZJLZH6mlVlpJ0S5bPH8K1z94wZQEUAShzKo8QqRtmFIsmsrRA+w/kc?=
 =?us-ascii?Q?FOwDCT5dvrGALI8nmz0XSJowiplwVwfXH7kwMOBwkted19ht77do/AJfYR6A?=
 =?us-ascii?Q?wJvBNt2opG8S024GeJnFu25SJ/deaUWtizJNcEY8qMuGquA1AqBf62EnOui8?=
 =?us-ascii?Q?G4bZuhOPXNkurhdaoLawt+NYNcFQLrkedxUW4UkEH/SEetJiOC3k9hdc3XI6?=
 =?us-ascii?Q?rI8pyUj/1Gn8oTkIcmUY/Vt7qFfLiP78silUnqnOl8lTCC+B7kXNIL028HnR?=
 =?us-ascii?Q?DPH8IXGkQVVXT+Ge8SeTBmEhZhojFHaNiqzQbpzs3/K5u7wq0GiVi86DUdpa?=
 =?us-ascii?Q?ADkbwQvJaaJpxTDQGW9+j8PbERHAMxlpJlBUqglcpAPlOqGgwJ3BC1E069KT?=
 =?us-ascii?Q?Y5OxZ77JOi7BJOsydWMfHOypONBR7aE3hNCKM6Gp+lYTP7ykxrpPtolNmWQ8?=
 =?us-ascii?Q?FR7OIkRmvDJlrVp7pVAE80jSXkbCUrJ5qCLR8PkAX/bX/KkvW2EjfUwIKfYE?=
 =?us-ascii?Q?um4ni2kfsI9b0AU/u8qpKNrxbYWUXfw0VNeCVqZ08P0jG8WITC8SXvdroTDz?=
 =?us-ascii?Q?DNUlVPBLb2g3xQKdlJCQt+MSCvnvzGxz5fgorSw+QPAFoM227lJlAEwJetm1?=
 =?us-ascii?Q?Ex4WDW5jmKgUHlJywosRdPlsyTzr/RBx/s7O74ToqDN3Nzwtx79zCn2fuh+c?=
 =?us-ascii?Q?sYJO9hGxlJlSvdE1tpABd8L9cw/mBR+OKdjJPatpvgEH8d4rERNmUk2H5lqI?=
 =?us-ascii?Q?vyasyYworS/4pI9KnRQEf/iXSEwFFXhGSTsjaP7kcNMr/Q8m0rLEdn8wo7W2?=
 =?us-ascii?Q?jrncmiu8zmnkJu0PDpqKvv9xKV6vz8slYifVIDWSVDR9QNE0oam2rqgFjXQh?=
 =?us-ascii?Q?4fuHj7CvEcb4vv7/KgOpVyJycFRoFJuRb4ADxn3FCpXiahn7vgFltIpaVyl5?=
 =?us-ascii?Q?DVOXKEnIahnV2zBFtjDdxtu7HBSj7MsHVaoVgA7aTxbSs4MwmbL7pw2RkiLq?=
 =?us-ascii?Q?Lam0OzFdw6kzp8QeMrE5irMQWUYGkUgBHO8Z29O+0xpygE/Vx5FngNsWCq5X?=
 =?us-ascii?Q?MgMK6usIVKacXnsSDlVq+RDuxZiemZYEmDobOENAGr0DkJnf7jJJAhEo2OdU?=
 =?us-ascii?Q?zNBZFAbmCQn0kdHFPKR8eoemR6sqV7uTAlxarIYA+O3dOt9pEUFM14lxRGuh?=
 =?us-ascii?Q?Aw=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c49ccfe7-d7af-4032-a7db-08db765ba5a2
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jun 2023 15:40:19.7806
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: mZfDKvIkx8GtiN6AEjnqjsAdpn5R/qxwUwrHJkNlnOoGTQAVF/R55Qx4LaqbO77sqH3tQQ2lSUUOZ22ZPbX+yQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8638
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Patch 3/3 fixes an issue with the ocelot/felix driver, where it would
drop PTP traffic on RX unless hardware timestamping was enabled.

Fixing that requires the driver to know whether it had previously
configured the hardware to timestamp PTP packets on that port. But it
cannot correctly determine that today using the existing code structure,
so patches 1/3 and 2/3 fix the control path of the code such that
ocelot->ports[port]->ptp_rx_filter faithfully reflects whether that
configuration took place.

Vladimir Oltean (3):
  net: mscc: ocelot: don't report that RX timestamping is enabled by
    default
  net: mscc: ocelot: don't keep PTP configuration of all ports in single
    structure
  net: dsa: felix: don't drop PTP frames with tag_8021q when RX
    timestamping is disabled

 drivers/net/dsa/ocelot/felix.c         |  3 +++
 drivers/net/ethernet/mscc/ocelot.c     |  1 -
 drivers/net/ethernet/mscc/ocelot_ptp.c | 35 +++++++++++++++-----------
 include/soc/mscc/ocelot.h              |  5 ++--
 4 files changed, 25 insertions(+), 19 deletions(-)

-- 
2.34.1