Merge branch 'controller-registry-shared-notify' into integration

esphome/core/controller_registry.cpp

@@ -10,20 +10,23 @@ StaticVector<Controller *, CONTROLLER_REGISTRY_MAX> ControllerRegistry::controll
 
 void ControllerRegistry::register_controller(Controller *controller) { controllers.push_back(controller); }
 
+void ControllerRegistry::notify(void *obj, DispatchFunc dispatch) {
+  for (auto *controller : controllers) {
+    dispatch(controller, obj);
+  }
+}
+
 // Macro for standard registry notification dispatch - calls on_<entity_name>_update()
+// Each wrapper passes a small trampoline lambda that calls the correct virtual method.
 #define CONTROLLER_REGISTRY_NOTIFY(entity_type, entity_name) \
   void ControllerRegistry::notify_##entity_name##_update(entity_type *obj) { /* NOLINT(bugprone-macro-parentheses) */ \
-    for (auto *controller : controllers) { \
-      controller->on_##entity_name##_update(obj); \
-    } \
+    notify(obj, [](Controller *c, void *o) { c->on_##entity_name##_update(static_cast<entity_type *>(o)); }); \
   }
 
 // Macro for entities where controller method has no "_update" suffix (Event, Update)
 #define CONTROLLER_REGISTRY_NOTIFY_NO_UPDATE_SUFFIX(entity_type, entity_name) \
   void ControllerRegistry::notify_##entity_name(entity_type *obj) { /* NOLINT(bugprone-macro-parentheses) */ \
-    for (auto *controller : controllers) { \
-      controller->on_##entity_name(obj); \
-    } \
+    notify(obj, [](Controller *c, void *o) { c->on_##entity_name(static_cast<entity_type *>(o)); }); \
   }
 
 #ifdef USE_BINARY_SENSOR

esphome/core/controller_registry.h

@@ -162,6 +162,21 @@ class ControllerRegistry {
    */
   static void register_controller(Controller *controller);
 
+  /** Type-erased dispatch function pointer.
+   *
+   * Each notify method passes a small trampoline that calls the
+   * correct virtual method on Controller. The shared notify() loop
+   * iterates controllers once, calling the trampoline for each.
+   */
+  using DispatchFunc = void (*)(Controller *, void *);
+
+  /** Shared dispatch loop - iterates controllers and calls dispatch for each.
+   *
+   * Marked noinline to ensure only one copy of the loop exists in flash,
+   * rather than being duplicated into each notify_*_update wrapper.
+   */
+  static void __attribute__((noinline)) notify(void *obj, DispatchFunc dispatch);
+
 #ifdef USE_BINARY_SENSOR
   static void notify_binary_sensor_update(binary_sensor::BinarySensor *obj);
 #endif