mailbox: omap: Use mbox_controller channel list directly
authorAndrew Davis <afd@ti.com>
Wed, 10 Apr 2024 13:59:39 +0000 (08:59 -0500)
committerJassi Brar <jassisinghbrar@gmail.com>
Mon, 20 May 2024 03:29:44 +0000 (22:29 -0500)
The driver stores a list of omap_mbox structs so it can later use it to
lookup the mailbox names in of_xlate. This same information is already
available in the mbox_controller passed into of_xlate. Simply use that
data and remove the extra allocation and storage of the omap_mbox list.

Signed-off-by: Andrew Davis <afd@ti.com>
Signed-off-by: Jassi Brar <jassisinghbrar@gmail.com>
drivers/mailbox/omap-mailbox.c

index 97f59d9..8e42266 100644 (file)
@@ -85,7 +85,6 @@ struct omap_mbox_device {
        u32 num_users;
        u32 num_fifos;
        u32 intr_type;
-       struct omap_mbox **mboxes;
 };
 
 struct omap_mbox {
@@ -356,25 +355,6 @@ static void omap_mbox_fini(struct omap_mbox *mbox)
        mbox_queue_free(mbox->rxq);
 }
 
-static struct omap_mbox *omap_mbox_device_find(struct omap_mbox_device *mdev,
-                                              const char *mbox_name)
-{
-       struct omap_mbox *_mbox, *mbox = NULL;
-       struct omap_mbox **mboxes = mdev->mboxes;
-       int i;
-
-       if (!mboxes)
-               return NULL;
-
-       for (i = 0; (_mbox = mboxes[i]); i++) {
-               if (!strcmp(_mbox->name, mbox_name)) {
-                       mbox = _mbox;
-                       break;
-               }
-       }
-       return mbox;
-}
-
 static int omap_mbox_chan_startup(struct mbox_chan *chan)
 {
        struct omap_mbox *mbox = mbox_chan_to_omap_mbox(chan);
@@ -539,6 +519,7 @@ static struct mbox_chan *omap_mbox_of_xlate(struct mbox_controller *controller,
        struct device_node *node;
        struct omap_mbox_device *mdev;
        struct omap_mbox *mbox;
+       int i;
 
        mdev = dev_get_drvdata(controller->dev);
        if (WARN_ON(!mdev))
@@ -551,16 +532,23 @@ static struct mbox_chan *omap_mbox_of_xlate(struct mbox_controller *controller,
                return ERR_PTR(-ENODEV);
        }
 
-       mbox = omap_mbox_device_find(mdev, node->name);
+       for (i = 0; i < controller->num_chans; i++) {
+               mbox = controller->chans[i].con_priv;
+               if (!strcmp(mbox->name, node->name)) {
+                       of_node_put(node);
+                       return &controller->chans[i];
+               }
+       }
+
        of_node_put(node);
-       return mbox ? mbox->chan : ERR_PTR(-ENOENT);
+       return ERR_PTR(-ENOENT);
 }
 
 static int omap_mbox_probe(struct platform_device *pdev)
 {
        int ret;
        struct mbox_chan *chnls;
-       struct omap_mbox **list, *mbox;
+       struct omap_mbox *mbox;
        struct omap_mbox_device *mdev;
        struct omap_mbox_fifo *fifo;
        struct device_node *node = pdev->dev.of_node;
@@ -608,12 +596,6 @@ static int omap_mbox_probe(struct platform_device *pdev)
        if (!mdev->irq_ctx)
                return -ENOMEM;
 
-       /* allocate one extra for marking end of list */
-       list = devm_kcalloc(&pdev->dev, info_count + 1, sizeof(*list),
-                           GFP_KERNEL);
-       if (!list)
-               return -ENOMEM;
-
        chnls = devm_kcalloc(&pdev->dev, info_count + 1, sizeof(*chnls),
                             GFP_KERNEL);
        if (!chnls)
@@ -675,7 +657,6 @@ static int omap_mbox_probe(struct platform_device *pdev)
                        return mbox->irq;
                mbox->chan = &chnls[i];
                chnls[i].con_priv = mbox;
-               list[i] = mbox++;
        }
 
        mutex_init(&mdev->cfg_lock);
@@ -683,7 +664,6 @@ static int omap_mbox_probe(struct platform_device *pdev)
        mdev->num_users = num_users;
        mdev->num_fifos = num_fifos;
        mdev->intr_type = intr_type;
-       mdev->mboxes = list;
 
        controller = devm_kzalloc(&pdev->dev, sizeof(*controller), GFP_KERNEL);
        if (!controller)