[PATCH 02/15] hw/alpha/dp264: Fix block comment style problems

Yodel Eldar posted 15 patches 4 weeks ago
Maintainers: Richard Henderson <richard.henderson@linaro.org>
[PATCH 02/15] hw/alpha/dp264: Fix block comment style problems
Posted by Yodel Eldar 4 weeks ago
Use leading "/*", internal "*", and trailing "*/" for block comments.

Signed-off-by: Yodel Eldar <yodel.eldar@yodel.dev>
---
 hw/alpha/dp264.c | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/hw/alpha/dp264.c b/hw/alpha/dp264.c
index 98219f0456..27fbcee637 100644
--- a/hw/alpha/dp264.c
+++ b/hw/alpha/dp264.c
@@ -26,12 +26,14 @@ static uint64_t cpu_alpha_superpage_to_phys(void *opaque, uint64_t addr)
     return addr;
 }
 
-/* Note that there are at least 3 viewpoints of IRQ numbers on Alpha systems.
-    (0) The dev_irq_n lines into the cpu, which we totally ignore,
-    (1) The DRIR lines in the typhoon chipset,
-    (2) The "vector" aka mangled interrupt number reported by SRM PALcode,
-    (3) The interrupt number assigned by the kernel.
-   The following function is concerned with (1) only.  */
+/*
+ * Note that there are at least 3 viewpoints of IRQ numbers on Alpha systems.
+ *  (0) The dev_irq_n lines into the cpu, which we totally ignore,
+ *  (1) The DRIR lines in the typhoon chipset,
+ *  (2) The "vector" aka mangled interrupt number reported by SRM PALcode,
+ *  (3) The interrupt number assigned by the kernel.
+ * The following function is concerned with (1) only.
+ */
 
 static int clipper_pci_map_irq(PCIDevice *d, int irq_num)
 {
@@ -132,9 +134,11 @@ static void clipper_init(MachineState *machine)
     pci_dev = pci_create_simple(pci_bus, -1, "cmd646-ide");
     pci_ide_create_devs(pci_dev);
 
-    /* Load PALcode.  Given that this is not "real" cpu palcode,
-       but one explicitly written for the emulation, we might as
-       well load it directly from and ELF image.  */
+    /*
+     * Load PALcode.  Given that this is not "real" cpu palcode,
+     * but one explicitly written for the emulation, we might as
+     * well load it directly from and ELF image.
+     */
     palcode_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS,
                                       machine->firmware ?: "palcode-clipper");
     if (palcode_filename == NULL) {

-- 
2.53.0
Re: [PATCH 02/15] hw/alpha/dp264: Fix block comment style problems
Posted by Richard Henderson 1 week, 6 days ago
On 3/11/26 08:31, Yodel Eldar wrote:
> Use leading "/*", internal "*", and trailing "*/" for block comments.
> 
> Signed-off-by: Yodel Eldar <yodel.eldar@yodel.dev>
> ---
>   hw/alpha/dp264.c | 22 +++++++++++++---------
>   1 file changed, 13 insertions(+), 9 deletions(-)


Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

> 
> diff --git a/hw/alpha/dp264.c b/hw/alpha/dp264.c
> index 98219f0456..27fbcee637 100644
> --- a/hw/alpha/dp264.c
> +++ b/hw/alpha/dp264.c
> @@ -26,12 +26,14 @@ static uint64_t cpu_alpha_superpage_to_phys(void *opaque, uint64_t addr)
>       return addr;
>   }
>   
> -/* Note that there are at least 3 viewpoints of IRQ numbers on Alpha systems.
> -    (0) The dev_irq_n lines into the cpu, which we totally ignore,
> -    (1) The DRIR lines in the typhoon chipset,
> -    (2) The "vector" aka mangled interrupt number reported by SRM PALcode,
> -    (3) The interrupt number assigned by the kernel.
> -   The following function is concerned with (1) only.  */
> +/*
> + * Note that there are at least 3 viewpoints of IRQ numbers on Alpha systems.
> + *  (0) The dev_irq_n lines into the cpu, which we totally ignore,
> + *  (1) The DRIR lines in the typhoon chipset,
> + *  (2) The "vector" aka mangled interrupt number reported by SRM PALcode,
> + *  (3) The interrupt number assigned by the kernel.
> + * The following function is concerned with (1) only.
> + */
>   
>   static int clipper_pci_map_irq(PCIDevice *d, int irq_num)
>   {
> @@ -132,9 +134,11 @@ static void clipper_init(MachineState *machine)
>       pci_dev = pci_create_simple(pci_bus, -1, "cmd646-ide");
>       pci_ide_create_devs(pci_dev);
>   
> -    /* Load PALcode.  Given that this is not "real" cpu palcode,
> -       but one explicitly written for the emulation, we might as
> -       well load it directly from and ELF image.  */
> +    /*
> +     * Load PALcode.  Given that this is not "real" cpu palcode,
> +     * but one explicitly written for the emulation, we might as
> +     * well load it directly from and ELF image.
> +     */
>       palcode_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS,
>                                         machine->firmware ?: "palcode-clipper");
>       if (palcode_filename == NULL) {
>