On Mon, 6 Nov 2023 13:50:58 +0100 Nico Boehr <nrb@xxxxxxxxxxxxx> wrote: > These comments do not follow the kernel-doc style, hence they should not > start with /**. > > Signed-off-by: Nico Boehr <nrb@xxxxxxxxxxxxx> > --- > powerpc/emulator.c | 2 +- > powerpc/spapr_hcall.c | 6 +++--- > powerpc/spapr_vpa.c | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/powerpc/emulator.c b/powerpc/emulator.c > index 65ae4b65e655..39dd59645368 100644 > --- a/powerpc/emulator.c > +++ b/powerpc/emulator.c > @@ -71,7 +71,7 @@ static void test_64bit(void) > report_prefix_pop(); > } > > -/** > +/* > * Test 'Load String Word Immediate' instruction > */ this should have the name of the function first: * test_lswi() - Test 'Load String ... (same for all the other functions here) > static void test_lswi(void) > diff --git a/powerpc/spapr_hcall.c b/powerpc/spapr_hcall.c > index 0d0f25afe9f6..e9b5300a3912 100644 > --- a/powerpc/spapr_hcall.c > +++ b/powerpc/spapr_hcall.c > @@ -16,7 +16,7 @@ > #define H_ZERO_PAGE (1UL << (63-48)) > #define H_COPY_PAGE (1UL << (63-49)) > > -/** > +/* > * Test the H_SET_SPRG0 h-call by setting some values and checking whether > * the SPRG0 register contains the correct values afterwards > */ > @@ -46,7 +46,7 @@ static void test_h_set_sprg0(int argc, char **argv) > sprg0_orig); > } > > -/** > +/* > * Test the H_PAGE_INIT h-call by using it to clear and to copy a page, and > * by checking for the correct values in the destination page afterwards > */ > @@ -97,7 +97,7 @@ static int h_random(uint64_t *val) > return r3; > } > > -/** > +/* > * Test H_RANDOM by calling it a couple of times to check whether all bit > * positions really toggle (there should be no "stuck" bits in the output) > */ > diff --git a/powerpc/spapr_vpa.c b/powerpc/spapr_vpa.c > index 5586eb8d3341..6a3fe5e3f974 100644 > --- a/powerpc/spapr_vpa.c > +++ b/powerpc/spapr_vpa.c > @@ -53,7 +53,7 @@ static void print_vpa(struct vpa *vpa) > #define SUBFUNC_REGISTER (1ULL << 45) > #define SUBFUNC_DEREGISTER (5ULL << 45) > > -/** > +/* > * Test the H_REGISTER_VPA h-call register/deregister calls. > */ > static void test_register_vpa(void) > @@ -111,7 +111,7 @@ static void test_register_vpa(void) > report_prefix_pop(); > } > > -/** > +/* > * Test some VPA fields. > */ > static void test_vpa(void)