Re: [PATCH] x86/rtc: Simplify PNP ids check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 14, 2022 at 10:25 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
>
> compare_pnp_id() already iterates over the single linked pnp_ids list
> starting with the id past to it.
>
> So there is no need for add_rtc_cmos() to call compare_pnp_id()
> for each id on the list.
>
> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>

> ---
>  arch/x86/kernel/rtc.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/arch/x86/kernel/rtc.c b/arch/x86/kernel/rtc.c
> index 349046434513..1309b9b05338 100644
> --- a/arch/x86/kernel/rtc.c
> +++ b/arch/x86/kernel/rtc.c
> @@ -138,15 +138,12 @@ static __init int add_rtc_cmos(void)
>         static const char * const ids[] __initconst =
>             { "PNP0b00", "PNP0b01", "PNP0b02", };
>         struct pnp_dev *dev;
> -       struct pnp_id *id;
>         int i;
>
>         pnp_for_each_dev(dev) {
> -               for (id = dev->id; id; id = id->next) {
> -                       for (i = 0; i < ARRAY_SIZE(ids); i++) {
> -                               if (compare_pnp_id(id, ids[i]) != 0)
> -                                       return 0;
> -                       }
> +               for (i = 0; i < ARRAY_SIZE(ids); i++) {
> +                       if (compare_pnp_id(dev->id, ids[i]) != 0)
> +                               return 0;
>                 }
>         }
>  #endif
> --
> 2.38.1
>



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux