Commit c0c725d7 authored by Bo Liu's avatar Bo Liu Committed by Johan Hovold
Browse files

gnss: replace ida_simple API



Use ida_alloc_xxx()/ida_free() instead of
ida_simple_get()/ida_simple_remove(), which has been deprecated.

Note that the upper bound is now inclusive.

Signed-off-by: default avatarBo Liu <liubo03@inspur.com>
Link: https://lore.kernel.org/r/20220616044733.3605-1-liubo03@inspur.com


Reviewed-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
[ johan: amend commit message ]
Signed-off-by: default avatarJohan Hovold <johan@kernel.org>
parent a111daf0
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -217,7 +217,7 @@ static void gnss_device_release(struct device *dev)

	kfree(gdev->write_buf);
	kfifo_free(&gdev->read_fifo);
	ida_simple_remove(&gnss_minors, gdev->id);
	ida_free(&gnss_minors, gdev->id);
	kfree(gdev);
}

@@ -232,7 +232,7 @@ struct gnss_device *gnss_allocate_device(struct device *parent)
	if (!gdev)
		return NULL;

	id = ida_simple_get(&gnss_minors, 0, GNSS_MINORS, GFP_KERNEL);
	id = ida_alloc_max(&gnss_minors, GNSS_MINORS - 1, GFP_KERNEL);
	if (id < 0) {
		kfree(gdev);
		return NULL;