intel_th: remove usage of the deprecated ida_simple_xx() API
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 14 Apr 2024 09:28:13 +0000 (11:28 +0200)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 29 Apr 2024 15:20:06 +0000 (08:20 -0700)
ida_alloc() and ida_free() should be preferred to the deprecated
ida_simple_get() and ida_simple_remove().

This is less verbose.

Link: https://lkml.kernel.org/r/2aca50a9d061faecfd4ded80b5874cd3be9b855d.1713086613.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
drivers/hwtracing/intel_th/core.c

index cc7f879bb175d4285d7b873533c24bcd15f58e81..86c8efecd7c26444e478b9e37555f8d9ec8d9327 100644 (file)
@@ -871,7 +871,7 @@ intel_th_alloc(struct device *dev, const struct intel_th_drvdata *drvdata,
        if (!th)
                return ERR_PTR(-ENOMEM);
 
-       th->id = ida_simple_get(&intel_th_ida, 0, 0, GFP_KERNEL);
+       th->id = ida_alloc(&intel_th_ida, GFP_KERNEL);
        if (th->id < 0) {
                err = th->id;
                goto err_alloc;
@@ -931,7 +931,7 @@ err_chrdev:
                            "intel_th/output");
 
 err_ida:
-       ida_simple_remove(&intel_th_ida, th->id);
+       ida_free(&intel_th_ida, th->id);
 
 err_alloc:
        kfree(th);
@@ -964,7 +964,7 @@ void intel_th_free(struct intel_th *th)
        __unregister_chrdev(th->major, 0, TH_POSSIBLE_OUTPUTS,
                            "intel_th/output");
 
-       ida_simple_remove(&intel_th_ida, th->id);
+       ida_free(&intel_th_ida, th->id);
 
        kfree(th);
 }