.. | .. |
---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
---|
1 | 2 | /* OMAP SSI driver. |
---|
2 | 3 | * |
---|
3 | 4 | * Copyright (C) 2010 Nokia Corporation. All rights reserved. |
---|
4 | 5 | * Copyright (C) 2014 Sebastian Reichel <sre@kernel.org> |
---|
5 | 6 | * |
---|
6 | 7 | * Contact: Carlos Chinea <carlos.chinea@nokia.com> |
---|
7 | | - * |
---|
8 | | - * This program is free software; you can redistribute it and/or |
---|
9 | | - * modify it under the terms of the GNU General Public License |
---|
10 | | - * version 2 as published by the Free Software Foundation. |
---|
11 | | - * |
---|
12 | | - * This program is distributed in the hope that it will be useful, but |
---|
13 | | - * WITHOUT ANY WARRANTY; without even the implied warranty of |
---|
14 | | - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
---|
15 | | - * General Public License for more details. |
---|
16 | | - * |
---|
17 | | - * You should have received a copy of the GNU General Public License |
---|
18 | | - * along with this program; if not, write to the Free Software |
---|
19 | | - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA |
---|
20 | | - * 02110-1301 USA |
---|
21 | 8 | */ |
---|
22 | 9 | |
---|
23 | 10 | #include <linux/compiler.h> |
---|
.. | .. |
---|
48 | 35 | static DEFINE_IDA(platform_omap_ssi_ida); |
---|
49 | 36 | |
---|
50 | 37 | #ifdef CONFIG_DEBUG_FS |
---|
51 | | -static int ssi_debug_show(struct seq_file *m, void *p __maybe_unused) |
---|
| 38 | +static int ssi_regs_show(struct seq_file *m, void *p __maybe_unused) |
---|
52 | 39 | { |
---|
53 | 40 | struct hsi_controller *ssi = m->private; |
---|
54 | 41 | struct omap_ssi_controller *omap_ssi = hsi_controller_drvdata(ssi); |
---|
.. | .. |
---|
63 | 50 | return 0; |
---|
64 | 51 | } |
---|
65 | 52 | |
---|
66 | | -static int ssi_debug_gdd_show(struct seq_file *m, void *p __maybe_unused) |
---|
| 53 | +static int ssi_gdd_regs_show(struct seq_file *m, void *p __maybe_unused) |
---|
67 | 54 | { |
---|
68 | 55 | struct hsi_controller *ssi = m->private; |
---|
69 | 56 | struct omap_ssi_controller *omap_ssi = hsi_controller_drvdata(ssi); |
---|
.. | .. |
---|
117 | 104 | return 0; |
---|
118 | 105 | } |
---|
119 | 106 | |
---|
120 | | -static int ssi_regs_open(struct inode *inode, struct file *file) |
---|
121 | | -{ |
---|
122 | | - return single_open(file, ssi_debug_show, inode->i_private); |
---|
123 | | -} |
---|
124 | | - |
---|
125 | | -static int ssi_gdd_regs_open(struct inode *inode, struct file *file) |
---|
126 | | -{ |
---|
127 | | - return single_open(file, ssi_debug_gdd_show, inode->i_private); |
---|
128 | | -} |
---|
129 | | - |
---|
130 | | -static const struct file_operations ssi_regs_fops = { |
---|
131 | | - .open = ssi_regs_open, |
---|
132 | | - .read = seq_read, |
---|
133 | | - .llseek = seq_lseek, |
---|
134 | | - .release = single_release, |
---|
135 | | -}; |
---|
136 | | - |
---|
137 | | -static const struct file_operations ssi_gdd_regs_fops = { |
---|
138 | | - .open = ssi_gdd_regs_open, |
---|
139 | | - .read = seq_read, |
---|
140 | | - .llseek = seq_lseek, |
---|
141 | | - .release = single_release, |
---|
142 | | -}; |
---|
| 107 | +DEFINE_SHOW_ATTRIBUTE(ssi_regs); |
---|
| 108 | +DEFINE_SHOW_ATTRIBUTE(ssi_gdd_regs); |
---|
143 | 109 | |
---|
144 | 110 | static int ssi_debug_add_ctrl(struct hsi_controller *ssi) |
---|
145 | 111 | { |
---|
.. | .. |
---|
330 | 296 | break; |
---|
331 | 297 | case ABORT_RATE_CHANGE: |
---|
332 | 298 | dev_dbg(&ssi->device, "abort rate change\n"); |
---|
333 | | - /* Fall through */ |
---|
| 299 | + fallthrough; |
---|
334 | 300 | case POST_RATE_CHANGE: |
---|
335 | 301 | dev_dbg(&ssi->device, "post rate change (%lu -> %lu)\n", |
---|
336 | 302 | clk_data->old_rate, clk_data->new_rate); |
---|
.. | .. |
---|
404 | 370 | if (err < 0) |
---|
405 | 371 | goto out_err; |
---|
406 | 372 | err = platform_get_irq_byname(pd, "gdd_mpu"); |
---|
407 | | - if (err < 0) { |
---|
408 | | - dev_err(&pd->dev, "GDD IRQ resource missing\n"); |
---|
| 373 | + if (err < 0) |
---|
409 | 374 | goto out_err; |
---|
410 | | - } |
---|
411 | 375 | omap_ssi->gdd_irq = err; |
---|
412 | 376 | tasklet_init(&omap_ssi->gdd_tasklet, ssi_gdd_tasklet, |
---|
413 | 377 | (unsigned long)ssi); |
---|
.. | .. |
---|
460 | 424 | struct omap_ssi_controller *omap_ssi = hsi_controller_drvdata(ssi); |
---|
461 | 425 | int err; |
---|
462 | 426 | |
---|
463 | | - err = pm_runtime_get_sync(ssi->device.parent); |
---|
| 427 | + err = pm_runtime_resume_and_get(ssi->device.parent); |
---|
464 | 428 | if (err < 0) { |
---|
465 | 429 | dev_err(&ssi->device, "runtime PM failed %d\n", err); |
---|
466 | 430 | return err; |
---|
.. | .. |
---|
538 | 502 | platform_set_drvdata(pd, ssi); |
---|
539 | 503 | |
---|
540 | 504 | err = ssi_add_controller(ssi, pd); |
---|
541 | | - if (err < 0) |
---|
| 505 | + if (err < 0) { |
---|
| 506 | + hsi_put_controller(ssi); |
---|
542 | 507 | goto out1; |
---|
| 508 | + } |
---|
543 | 509 | |
---|
544 | 510 | pm_runtime_enable(&pd->dev); |
---|
545 | 511 | |
---|
.. | .. |
---|
560 | 526 | if (!childpdev) { |
---|
561 | 527 | err = -ENODEV; |
---|
562 | 528 | dev_err(&pd->dev, "failed to create ssi controller port\n"); |
---|
| 529 | + of_node_put(child); |
---|
563 | 530 | goto out3; |
---|
564 | 531 | } |
---|
565 | 532 | } |
---|
.. | .. |
---|
571 | 538 | device_for_each_child(&pd->dev, NULL, ssi_remove_ports); |
---|
572 | 539 | out2: |
---|
573 | 540 | ssi_remove_controller(ssi); |
---|
| 541 | + pm_runtime_disable(&pd->dev); |
---|
574 | 542 | out1: |
---|
575 | 543 | platform_set_drvdata(pd, NULL); |
---|
576 | | - pm_runtime_disable(&pd->dev); |
---|
577 | 544 | |
---|
578 | 545 | return err; |
---|
579 | 546 | } |
---|
.. | .. |
---|
664 | 631 | if (ret) |
---|
665 | 632 | return ret; |
---|
666 | 633 | |
---|
667 | | - return platform_driver_register(&ssi_port_pdriver); |
---|
| 634 | + ret = platform_driver_register(&ssi_port_pdriver); |
---|
| 635 | + if (ret) { |
---|
| 636 | + platform_driver_unregister(&ssi_pdriver); |
---|
| 637 | + return ret; |
---|
| 638 | + } |
---|
| 639 | + |
---|
| 640 | + return 0; |
---|
668 | 641 | } |
---|
669 | 642 | module_init(ssi_init); |
---|
670 | 643 | |
---|