From nobody Fri Jun 19 08:16:19 2026 Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) (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 4148D175A9D for ; Sun, 26 Apr 2026 03:11:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777173066; cv=none; b=nw2cXb+qLs0gEsKXpoCKRTYVZqOAlxlUFtTavfKmlVs03g4AS7ITZ0VVk03bxgLMHN4Gs1/f/fkz/pGIor7sd/UcWeLfy5g69hSXOVZFlNh7Tki0UEpHARcQxKrxS9lkjbjq6IDsQNa/Yn856KmXbYWVwPCMNHDHK0dIV1s4zHg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777173066; c=relaxed/simple; bh=bND4NZ92gkOhzz2yhi0CQGsTVGEBKQQPVVGPNQWwArA=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=rXdLW67My+gxDzCcqDXy1dR5YDHRrxK+C5BU8ihzLKHVhN3CjTdNKuDsJUuBCSjpBC/SH9+ELuQLOiLUj3ESKRnM6nrIYYhWkJUuthLuvnBOxH7877dymFGoq37DfDC0pi3uPNwzU6S7cHGdlKJbWX6dMDEEBFOEsNfy2cIb2hs= 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=WpMwVqR2; arc=none smtp.client-ip=209.85.214.170 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="WpMwVqR2" Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-2b299b3c739so41983515ad.3 for ; Sat, 25 Apr 2026 20:11:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777173064; x=1777777864; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=jXNdM0nDs1ysyRJQ3ZFY4/74xWe4tcjuZYdpv/C1sCY=; b=WpMwVqR2nEnPb/zRCB9fOpjqUC9vMKq0g14/v8kcn92smNmsK6yco6hQ1NxVzgGC4x YL4OR1W3DAKAqGPa852uLi/G+Td3KW+ygC9HvvtSUggGGDt5cJRCPD+FKa6Tpf9KO+TK Ne/zsMqCD4axjKLKq0YH0Gfu3GZkrMoYsx9ug2kPfq51hTvQcpw8uVk+oQBGy8f7yTmq G6v9LO7MrrQGkC7dhPpgOk6H5rdmbrTUu8FolSYsaqnpIGuSkOvBbsFwe2nBhWyXwAy2 7w07jgZcsZtZ+mPHzT0qciIY6lGEUtyboWbAKe8iw/mQuUr6XLecb/qQ/Ci+MKi6MxSf BmsA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777173064; x=1777777864; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=jXNdM0nDs1ysyRJQ3ZFY4/74xWe4tcjuZYdpv/C1sCY=; b=fD+n7s6ota1niAUsefXBKeYHDbdMsen60kA7yZsBQt0EHWZgo25hIABKcy65wk1hDj LXfBck1DBnagWnuZdtCZjZUhquBK1cx/WqhMdwsHKXsJID8KujQYbDdqlZ5dyK3Ha6/v kUjmuklbNwY3LF1WmQY1bPvMf1Vnlv0yUZgMOPuTDDYwzasNCEOLBOUJTSQ+Reh6qOY9 hLm3vhFXDTWlZibow/pGbAkvXZuQWU9qor+1yXkRJoiAjX14s/2A/GA/oxylswXLhgZV YOFIzxmsFh581HAx9h4IooL1Jat1BxfKIxQ7aajl1Q2uLQddXtxmtkco7nB7V59hW6nd GyAg== X-Forwarded-Encrypted: i=1; AFNElJ/7US1kiGhyMy7sc9hSwTwDFFKjYUhV8JndZ6jvKdPQvgSxvBNkkGMtBo9JLgBQBea7NZDdcDk5Yqiu7DY=@vger.kernel.org X-Gm-Message-State: AOJu0Yx5jcDiiFBbRkqrWc8gq0IEAeziF+OqyqFiStt/IJrtDpP5UBbx WeBljtunOb6yJ1VtYBtvLlIND58DhhGFx3W5XmnhogaLzPftUq6Lni4M X-Gm-Gg: AeBDieu3NxlyhPnGaHB7o5vSMs/M+4+uBgjServ0FrlxXhiJMqeShA9sBSlRDS6oy7s 4fzcu49CTjXx32sAi58dwvyPza3kLfaPsfit08SXHUpVTxfQRnas6Hamy8h3o/dsIL7KflXs9j2 BoPY6T9ZRK+NJhXclipBSPlIYIBsYc/MahRi7zD0xttk12ocv+ZzinHFXntuE/AUVxH6HEXPLUN VcwNHgNNd08Z3jjr+3Zlcfcls+CCW0Qe4DD5L314LD1+Cq2xH4g6N23vn2Xnfd3e10fWCZITyPy YDi5NUXJevrFa8+6AarVhhxiKRA+M4o19WqcRSlkolKYYVHagUaqYhc0km5cTMX49gt7puwHhk0 BPj0pVjG0bWe7kV7F+B+iCeM3/HoYqOWQVFgdmVNDk0nJC3hmvqWtL8olT55EIaAEOelZujwk8x R/d+t3s1z1+xVQ+PHkDFBXJdQNzwfHFAd1AVd+7CpK2w8gYYSfKCHpgwfbtoLYkJCH X-Received: by 2002:a17:903:8c6:b0:2b0:c45a:bc2 with SMTP id d9443c01a7336-2b5f9eddc7emr406105735ad.16.1777173064481; Sat, 25 Apr 2026 20:11:04 -0700 (PDT) Received: from DESKTOP-R76JGJA.localdomain (vps-25d44ef0.vps.ovh.us. [2604:2dc0:202:300::22df]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b5fab405f3sm263701005ad.78.2026.04.25.20.11.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 25 Apr 2026 20:11:03 -0700 (PDT) From: Eric Wu To: Greg Kroah-Hartman Cc: Yoelvis Oliveros , Len Bao , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH] staging: octeon: remove typedefs from octeon-stubs.h Date: Sun, 26 Apr 2026 11:11:00 +0800 Message-ID: <20260426031100.97996-1-kunjinkao.jp@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The Linux kernel coding style discourages the use of typedefs for enums and structs. checkpatch reports seven "do not add new typedefs" warnings on drivers/staging/octeon/octeon-stubs.h. Drop the typedef wrappers in octeon-stubs.h and convert the affected users in ethernet.c and octeon-ethernet.h to use the underlying 'enum cvmx_*' / 'struct cvmx_*' tags directly. No functional change. Compile-tested with CONFIG_COMPILE_TEST=3Dy and CONFIG_OCTEON_ETHERNET=3Dm on x86_64; both default and W=3D1 builds are clean. Signed-off-by: Eric Wu --- drivers/staging/octeon/ethernet.c | 6 +-- drivers/staging/octeon/octeon-ethernet.h | 2 +- drivers/staging/octeon/octeon-stubs.h | 48 ++++++++++++------------ 3 files changed, 28 insertions(+), 28 deletions(-) diff --git a/drivers/staging/octeon/ethernet.c b/drivers/staging/octeon/eth= ernet.c index eadb74fc14c8..d85a9991faf6 100644 --- a/drivers/staging/octeon/ethernet.c +++ b/drivers/staging/octeon/ethernet.c @@ -201,8 +201,8 @@ EXPORT_SYMBOL(cvm_oct_free_work); */ static struct net_device_stats *cvm_oct_common_get_stats(struct net_device= *dev) { - cvmx_pip_port_status_t rx_status; - cvmx_pko_port_status_t tx_status; + struct cvmx_pip_port_status rx_status; + struct cvmx_pko_port_status tx_status; struct octeon_ethernet *priv =3D netdev_priv(dev); =20 if (priv->port < CVMX_PIP_NUM_INPUT_PORTS) { @@ -798,7 +798,7 @@ static int cvm_oct_probe(struct platform_device *pdev) =20 num_interfaces =3D cvmx_helper_get_number_of_interfaces(); for (interface =3D 0; interface < num_interfaces; interface++) { - cvmx_helper_interface_mode_t imode =3D + enum cvmx_helper_interface_mode imode =3D cvmx_helper_interface_get_mode(interface); int num_ports =3D cvmx_helper_ports_on_interface(interface); int port; diff --git a/drivers/staging/octeon/octeon-ethernet.h b/drivers/staging/oct= eon/octeon-ethernet.h index a6140705706f..d369c97c1332 100644 --- a/drivers/staging/octeon/octeon-ethernet.h +++ b/drivers/staging/octeon/octeon-ethernet.h @@ -57,7 +57,7 @@ struct octeon_ethernet { struct net_device *netdev; /* * Type of port. This is one of the enums in - * cvmx_helper_interface_mode_t + * cvmx_helper_interface_mode */ int imode; /* PHY mode */ diff --git a/drivers/staging/octeon/octeon-stubs.h b/drivers/staging/octeon= /octeon-stubs.h index 291eaffd2543..c39e7e307346 100644 --- a/drivers/staging/octeon/octeon-stubs.h +++ b/drivers/staging/octeon/octeon-stubs.h @@ -215,14 +215,14 @@ enum cvmx_fau_op_size { CVMX_FAU_OP_SIZE_64 =3D 3 }; =20 -typedef enum { +enum cvmx_spi_mode { CVMX_SPI_MODE_UNKNOWN =3D 0, CVMX_SPI_MODE_TX_HALFPLEX =3D 1, CVMX_SPI_MODE_RX_HALFPLEX =3D 2, CVMX_SPI_MODE_DUPLEX =3D 3 -} cvmx_spi_mode_t; +}; =20 -typedef enum { +enum cvmx_helper_interface_mode { CVMX_HELPER_INTERFACE_MODE_DISABLED, CVMX_HELPER_INTERFACE_MODE_RGMII, CVMX_HELPER_INTERFACE_MODE_GMII, @@ -233,20 +233,20 @@ typedef enum { CVMX_HELPER_INTERFACE_MODE_PICMG, CVMX_HELPER_INTERFACE_MODE_NPI, CVMX_HELPER_INTERFACE_MODE_LOOP, -} cvmx_helper_interface_mode_t; +}; =20 -typedef enum { +enum cvmx_pow_wait { CVMX_POW_WAIT =3D 1, CVMX_POW_NO_WAIT =3D 0, -} cvmx_pow_wait_t; +}; =20 -typedef enum { +enum cvmx_pko_lock { CVMX_PKO_LOCK_NONE =3D 0, CVMX_PKO_LOCK_ATOMIC_TAG =3D 1, CVMX_PKO_LOCK_CMD_QUEUE =3D 2, -} cvmx_pko_lock_t; +}; =20 -typedef enum { +enum cvmx_pko_status { CVMX_PKO_SUCCESS, CVMX_PKO_INVALID_PORT, CVMX_PKO_INVALID_QUEUE, @@ -254,7 +254,7 @@ typedef enum { CVMX_PKO_NO_MEMORY, CVMX_PKO_PORT_ALREADY_SETUP, CVMX_PKO_CMD_QUEUE_INIT_ERROR -} cvmx_pko_status_t; +}; =20 enum cvmx_pow_tag_type { CVMX_POW_TAG_TYPE_ORDERED =3D 0L, @@ -386,7 +386,7 @@ union cvmx_ipd_sub_port_qos_cnt { } s; }; =20 -typedef struct { +struct cvmx_pip_port_status { u32 dropped_octets; u32 dropped_packets; u32 pci_raw_packets; @@ -409,13 +409,13 @@ typedef struct { u32 inb_packets; u64 inb_octets; u16 inb_errors; -} cvmx_pip_port_status_t; +}; =20 -typedef struct { +struct cvmx_pko_port_status { u32 packets; u64 octets; u64 doorbell; -} cvmx_pko_port_status_t; +}; =20 union cvmx_pip_frm_len_chkx { u64 u64; @@ -1260,14 +1260,14 @@ static inline int octeon_is_simulation(void) } =20 static inline void cvmx_pip_get_port_status(u64 port_num, u64 clear, - cvmx_pip_port_status_t *status) + struct cvmx_pip_port_status *status) { } =20 static inline void cvmx_pko_get_port_status(u64 port_num, u64 clear, - cvmx_pko_port_status_t *status) + struct cvmx_pko_port_status *status) { } =20 -static inline cvmx_helper_interface_mode_t cvmx_helper_interface_get_mode(= int +static inline enum cvmx_helper_interface_mode cvmx_helper_interface_get_mo= de(int interface) { return 0; @@ -1344,11 +1344,11 @@ static inline unsigned int cvmx_get_core_num(void) } =20 static inline void cvmx_pow_work_request_async_nocheck(int scr_addr, - cvmx_pow_wait_t wait) + enum cvmx_pow_wait wait) { } =20 static inline void cvmx_pow_work_request_async(int scr_addr, - cvmx_pow_wait_t wait) + enum cvmx_pow_wait wait) { } =20 static inline struct cvmx_wqe *cvmx_pow_work_response_async(int scr_addr) @@ -1358,13 +1358,13 @@ static inline struct cvmx_wqe *cvmx_pow_work_respon= se_async(int scr_addr) return wqe; } =20 -static inline struct cvmx_wqe *cvmx_pow_work_request_sync(cvmx_pow_wait_t = wait) +static inline struct cvmx_wqe *cvmx_pow_work_request_sync(enum cvmx_pow_wa= it wait) { return (void *)(unsigned long)wait; } =20 static inline int cvmx_spi_restart_interface(int interface, - cvmx_spi_mode_t mode, int timeout) + enum cvmx_spi_mode mode, int timeout) { return 0; } @@ -1383,12 +1383,12 @@ static inline union cvmx_gmxx_rxx_rx_inbnd cvmx_spi= 4000_check_speed(int interfac } =20 static inline void cvmx_pko_send_packet_prepare(u64 port, u64 queue, - cvmx_pko_lock_t use_locking) + enum cvmx_pko_lock use_locking) { } =20 -static inline cvmx_pko_status_t cvmx_pko_send_packet_finish(u64 port, +static inline enum cvmx_pko_status cvmx_pko_send_packet_finish(u64 port, u64 queue, union cvmx_pko_command_word0 pko_command, - union cvmx_buf_ptr packet, cvmx_pko_lock_t use_locking) + union cvmx_buf_ptr packet, enum cvmx_pko_lock use_locking) { return 0; } --=20 2.43.0