Re: [PATCH v3 5/9] hw/hppa: Move software power button address back into PDC

[Richard Henderson]

On 1/12/24 21:29, del...@kernel.org wrote:
+static hwaddr soft_power_reg;
  
  #define enable_lasi_lan()       0
  
@@ -45,7 +45,6 @@ static DeviceState *lasi_dev;
  
  static void hppa_powerdown_req(Notifier *n, void *opaque)
  {
-    hwaddr soft_power_reg = HPA_POWER_BUTTON;
      uint32_t val;
  
      val = ldl_be_phys(&address_space_memory, soft_power_reg);
@@ -221,7 +220,7 @@ static FWCfgState *create_fw_cfg(MachineState *ms, PCIBus 
*pci_bus,
      fw_cfg_add_file(fw_cfg, "/etc/hppa/machine",
                      g_memdup(mc->name, len), len);
  
-    val = cpu_to_le64(HPA_POWER_BUTTON);
+    val = cpu_to_le64(soft_power_reg);

I think it would be better to pass this as a parameter to create_fw_cfg, or to drop the 
translated FW_CFG_IO_BASE parameter and merely pass in translate itself.


r~