esp: Cleanup unused imports and declarations (#905)

tact1m4n3 · web-flow · commit 36a186ac2ce3 · 2026-02-12T15:22:20.000+02:00
diff --git a/examples/espressif/esp/build.zig b/examples/espressif/esp/build.zig
@@ -127,108 +127,3 @@ const Example = struct {
     file: []const u8,
     features: Features = .{},
 };
-
-const lwip_flags = [_][]const u8{ "-std=c99", "-fno-sanitize=undefined" };
-const lwip_files = [_][]const u8{
-    // Core files
-    "core/init.c",
-    "core/def.c",
-    "core/dns.c",
-    "core/inet_chksum.c",
-    "core/ip.c",
-    "core/mem.c",
-    "core/memp.c",
-    "core/netif.c",
-    "core/pbuf.c",
-    "core/raw.c",
-    "core/stats.c",
-    "core/sys.c",
-    "core/altcp.c",
-    "core/altcp_alloc.c",
-    "core/altcp_tcp.c",
-    "core/tcp.c",
-    "core/tcp_in.c",
-    "core/tcp_out.c",
-    "core/timeouts.c",
-    "core/udp.c",
-
-    // IPv4 implementation:
-    "core/ipv4/acd.c",
-    "core/ipv4/autoip.c",
-    "core/ipv4/dhcp.c",
-    "core/ipv4/etharp.c",
-    "core/ipv4/icmp.c",
-    "core/ipv4/igmp.c",
-    "core/ipv4/ip4_frag.c",
-    "core/ipv4/ip4.c",
-    "core/ipv4/ip4_addr.c",
-
-    // IPv6 implementation:
-    "core/ipv6/dhcp6.c",
-    "core/ipv6/ethip6.c",
-    "core/ipv6/icmp6.c",
-    "core/ipv6/inet6.c",
-    "core/ipv6/ip6.c",
-    "core/ipv6/ip6_addr.c",
-    "core/ipv6/ip6_frag.c",
-    "core/ipv6/mld6.c",
-    "core/ipv6/nd6.c",
-
-    // Interfaces
-    "netif/ethernet.c",
-
-    // Interfaces:
-    // "netif/bridgeif.c",
-    // "netif/ethernet.c",
-    // "netif/slipif.c",
-    // "netif/bridgeif_fdb.c",
-
-    // sequential APIs
-    // "api/err.c",
-    // "api/api_msg.c",
-    // "api/netifapi.c",
-    // "api/sockets.c",
-    // "api/netbuf.c",
-    // "api/api_lib.c",
-    // "api/tcpip.c",
-    // "api/netdb.c",
-    // "api/if_api.c",
-
-    // 6LoWPAN
-    // "netif/lowpan6.c",
-    // "netif/lowpan6_ble.c",
-    // "netif/lowpan6_common.c",
-    // "netif/zepif.c",
-
-    // PPP
-    // "netif/ppp/polarssl/arc4.c",
-    // "netif/ppp/polarssl/des.c",
-    // "netif/ppp/polarssl/md4.c",
-    // "netif/ppp/polarssl/sha1.c",
-    // "netif/ppp/polarssl/md5.c",
-    // "netif/ppp/ipcp.c",
-    // "netif/ppp/magic.c",
-    // "netif/ppp/pppoe.c",
-    // "netif/ppp/mppe.c",
-    // "netif/ppp/multilink.c",
-    // "netif/ppp/chap-new.c",
-    // "netif/ppp/auth.c",
-    // "netif/ppp/chap_ms.c",
-    // "netif/ppp/ipv6cp.c",
-    // "netif/ppp/chap-md5.c",
-    // "netif/ppp/upap.c",
-    // "netif/ppp/pppapi.c",
-    // "netif/ppp/pppos.c",
-    // "netif/ppp/eap.c",
-    // "netif/ppp/pppol2tp.c",
-    // "netif/ppp/demand.c",
-    // "netif/ppp/fsm.c",
-    // "netif/ppp/eui64.c",
-    // "netif/ppp/ccp.c",
-    // "netif/ppp/pppcrypt.c",
-    // "netif/ppp/utils.c",
-    // "netif/ppp/vj.c",
-    // "netif/ppp/lcp.c",
-    // "netif/ppp/ppp.c",
-    // "netif/ppp/ecp.c",
-};
diff --git a/examples/espressif/esp/src/blinky.zig b/examples/espressif/esp/src/blinky.zig
@@ -1,6 +1,5 @@
 const std = @import("std");
 const microzig = @import("microzig");
-const peripherals = microzig.chip.peripherals;
 const hal = microzig.hal;
 const gpio = hal.gpio;
 const usb_serial_jtag = hal.usb_serial_jtag;
diff --git a/examples/espressif/esp/src/gpio_input.zig b/examples/espressif/esp/src/gpio_input.zig
@@ -1,8 +1,6 @@
-const std = @import("std");
 const microzig = @import("microzig");
 
 const esp = microzig.hal;
-const time = esp.time;
 const gpio = esp.gpio;
 
 pub fn main() !void {
diff --git a/examples/espressif/esp/src/i2c_bus_scan.zig b/examples/espressif/esp/src/i2c_bus_scan.zig
@@ -5,7 +5,6 @@ const time = microzig.drivers.time;
 const esp = microzig.hal;
 const i2c = esp.i2c;
 const gpio = esp.gpio;
-const peripherals = microzig.chip.peripherals;
 
 var i2c0 = i2c.instance.num(0);
 
diff --git a/examples/espressif/esp/src/i2c_display_sh1106.zig b/examples/espressif/esp/src/i2c_display_sh1106.zig
@@ -1,12 +1,10 @@
 const std = @import("std");
 const microzig = @import("microzig");
-const time = microzig.drivers.time;
 const SH1106 = microzig.drivers.display.SH1106;
 
 const esp = microzig.hal;
 const i2c = esp.i2c;
 const gpio = esp.gpio;
-const peripherals = microzig.chip.peripherals;
 const I2C_Datagram_Device = esp.drivers.I2C_Datagram_Device;
 const sleep_ms = esp.time.sleep_ms;
 
diff --git a/examples/espressif/esp/src/i2c_temp.zig b/examples/espressif/esp/src/i2c_temp.zig
@@ -1,12 +1,10 @@
 const std = @import("std");
 const microzig = @import("microzig");
-const time = microzig.drivers.time;
 const TMP117 = microzig.drivers.sensor.TMP117;
 
 const esp = microzig.hal;
 const i2c = esp.i2c;
 const gpio = esp.gpio;
-const peripherals = microzig.chip.peripherals;
 const I2C_Device = esp.drivers.I2C_Device;
 const sleep_ms = esp.time.sleep_ms;
 
diff --git a/examples/espressif/esp/src/lwip/exports.zig b/examples/espressif/esp/src/lwip/exports.zig
@@ -1,7 +1,6 @@
 /// Platform dependent exports required by lwip
 ///
 const std = @import("std");
-const assert = std.debug.assert;
 
 const microzig = @import("microzig");
 const esp = microzig.hal;
diff --git a/examples/espressif/esp/src/rtos.zig b/examples/espressif/esp/src/rtos.zig
@@ -1,11 +1,7 @@
 const std = @import("std");
-const log = std.log;
 
 const microzig = @import("microzig");
-const peripherals = microzig.chip.peripherals;
 const esp = microzig.hal;
-const gpio = esp.gpio;
-const systimer = esp.systimer;
 const usb_serial_jtag = esp.usb_serial_jtag;
 const rtos = esp.rtos;
 
diff --git a/examples/espressif/esp/src/systimer.zig b/examples/espressif/esp/src/systimer.zig
@@ -1,8 +1,6 @@
 const std = @import("std");
 const microzig = @import("microzig");
-const peripherals = microzig.chip.peripherals;
 const hal = microzig.hal;
-const gpio = hal.gpio;
 const systimer = hal.systimer;
 const usb_serial_jtag = hal.usb_serial_jtag;
 
diff --git a/examples/espressif/esp/src/tcp_server.zig b/examples/espressif/esp/src/tcp_server.zig
@@ -1,8 +1,6 @@
 const std = @import("std");
-const assert = std.debug.assert;
 
 const microzig = @import("microzig");
-const interrupt = microzig.cpu.interrupt;
 const hal = microzig.hal;
 const rtos = hal.rtos;
 const radio = hal.radio;
diff --git a/port/espressif/esp/src/hal/clocks.zig b/port/espressif/esp/src/hal/clocks.zig
@@ -1,6 +1,3 @@
-const std = @import("std");
-const microzig = @import("microzig");
-
 const compatibility = @import("compatibility.zig");
 
 /// Clock config init error.
diff --git a/port/espressif/esp/src/hal/drivers.zig b/port/espressif/esp/src/hal/drivers.zig
@@ -2,7 +2,6 @@
 //! This file implements driver abstractions based on HAL devices.
 //!
 
-const std = @import("std");
 const microzig = @import("microzig");
 const hal = microzig.hal;
 const mdf = microzig.drivers;
diff --git a/port/espressif/esp/src/hal/i2c.zig b/port/espressif/esp/src/hal/i2c.zig
@@ -6,7 +6,6 @@ const peripherals = microzig.chip.peripherals;
 pub const I2C0 = peripherals.I2C0;
 
 const gpio = @import("gpio.zig");
-const clocks = @import("clocks.zig");
 const time = @import("time.zig");
 
 // TODO: How and why. Is this xtal? That clock is 40_000_000 according to the hal
diff --git a/port/espressif/esp/src/hal/radio/osi.zig b/port/espressif/esp/src/hal/radio/osi.zig
@@ -5,15 +5,13 @@ const builtin = @import("builtin");
 const c = @import("esp-wifi-driver");
 const microzig = @import("microzig");
 const enter_critical_section = microzig.interrupt.enter_critical_section;
-const time = microzig.drivers.time;
 const peripherals = microzig.chip.peripherals;
 const APB_CTRL = peripherals.APB_CTRL;
 const radio_interrupt = microzig.options.hal.radio.interrupt;
 
 const radio = @import("../radio.zig");
 const rng = @import("../rng.zig");
 const rtos = @import("../rtos.zig");
-const systimer = @import("../systimer.zig");
 const get_time_since_boot = @import("../time.zig").get_time_since_boot;
 const timer = @import("timer.zig");
 const wifi = @import("wifi.zig");
diff --git a/port/espressif/esp/src/hal/rom.zig b/port/espressif/esp/src/hal/rom.zig
@@ -1,5 +1,3 @@
-const microzig = @import("microzig");
-
 const compatibility = @import("compatibility.zig");
 
 /// Microsecond delay using rom function.
diff --git a/port/espressif/esp/src/hal/rtos.zig b/port/espressif/esp/src/hal/rtos.zig
@@ -1,18 +1,12 @@
 const std = @import("std");
 const assert = std.debug.assert;
-const Allocator = std.mem.Allocator;
 const log = std.log.scoped(.rtos);
-const builtin = @import("builtin");
 
 const microzig = @import("microzig");
-const CriticalSection = microzig.interrupt.CriticalSection;
 const enter_critical_section = microzig.interrupt.enter_critical_section;
 const TrapFrame = microzig.cpu.TrapFrame;
-const SYSTEM = microzig.chip.peripherals.SYSTEM;
-const time = microzig.drivers.time;
 const rtos_options = microzig.options.hal.rtos;
 
-const get_time_since_boot = @import("time.zig").get_time_since_boot;
 const system = @import("system.zig");
 const systimer = @import("systimer.zig");
 
diff --git a/port/espressif/esp/src/hal/time.zig b/port/espressif/esp/src/hal/time.zig
@@ -1,4 +1,3 @@
-const std = @import("std");
 const microzig = @import("microzig");
 const time = microzig.drivers.time;
 
diff --git a/port/espressif/esp/src/hal/uart.zig b/port/espressif/esp/src/hal/uart.zig
@@ -4,8 +4,6 @@ const peripherals = microzig.chip.peripherals;
 
 // TODO: chip independent. currently specific to esp32c3.
 
-const UART = peripherals.UART;
-
 fn reg(comptime index: comptime_int) @TypeOf(@field(peripherals, std.fmt.comptimePrint("UART{}", .{index}))) {
     return @field(peripherals, std.fmt.comptimePrint("UART{}", .{index}));
 }
