On Tue, 2026-06-23 at 19:49 +0530, Aditya Gupta wrote:
+static const char *pnv_get_machine_type(enum PnvChipType chip_type) +{ + if (chip_type == PNV_CHIP_POWER8) { + return "powernv8"; + } else if (chip_type == PNV_CHIP_POWER9) { + return "powernv9"; + } else if (chip_type == PNV_CHIP_POWER10) { + return "powernv10"; + } else if (chip_type == PNV_CHIP_POWER11) { + return "powernv11"; + } else { + g_assert_not_reached(); + } +}
How about refactoring to the following? static const char *const machine_types[] = { [PNV_CHIP_POWER8] = "powernv8", [PNV_CHIP_POWER9] = "powernv9", [PNV_CHIP_POWER10] = "powernv10", [PNV_CHIP_POWER11] = "powernv11", }; return machine_types[x]; ~Shivang.