| .. | .. |
|---|
| 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 | |
|---|