summaryrefslogtreecommitdiffstats
path: root/drivers
Commit message (Collapse)AuthorAgeFilesLines
...
* ata: libata: drop ata_msg_drv()Hannes Reinecke2022-01-052-15/+7
| | | | | | | | Callers are already protected by ata_dev_print_info(), so no need to have an additional configuration parameter here. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_info()Hannes Reinecke2022-01-051-7/+3
| | | | | | | Convert the sole caller to ata_dev_dbg() and remove the definition. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_probe()Hannes Reinecke2022-01-053-33/+14
| | | | | | | All callsites have been converted to dynamic debugging. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_warn()Hannes Reinecke2022-01-051-11/+8
| | | | | | | The WARN level was always enabled, so drop ata_msg_warn(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_malloc()Hannes Reinecke2022-01-051-1/+1
| | | | | | | Unused. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_ctl()Hannes Reinecke2022-01-051-5/+2
| | | | | | | The one caller have been converted to dynamic debugging. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: drop ata_msg_error() and ata_msg_intr()Hannes Reinecke2022-01-051-3/+3
| | | | | | | Unused. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_sx4: add module parameter 'dimm_test'Hannes Reinecke2022-01-051-20/+14
| | | | | | | Add module parameter 'dimm_test' to enable DIMM testing during startup. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_sx4: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke2022-01-051-71/+34
| | | | | | | | Drop pointless VPRINTK() calls for setting up SG tables and convert the remaining calls to structured logging. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_sil: Drop pointless VPRINTK() callsHannes Reinecke2022-01-051-1/+0
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_fsl: convert VPRINTK() calls to ata_port_dbg()Hannes Reinecke2022-01-051-47/+33
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_nv: drop pointless VPRINTK() calls and convert remaining onesHannes Reinecke2022-01-051-24/+14
| | | | | | | | | Quite some information from the VPRINTK() is already covered by tracepoints, so remove the pointless calls and convert the remaining ones to structured logging. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_mv: Drop pointless VPRINTK() call and convert the remaining oneHannes Reinecke2022-01-051-6/+1
| | | | | | | Drop pointless VPRINTK() call and convert the remaining one to dev_dbg(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_inic162x: Drop pointless VPRINTK() callsHannes Reinecke2022-01-051-2/+0
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_rcar: Drop pointless VPRINTK() callsHannes Reinecke2022-01-051-16/+1
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_qstor: Drop pointless VPRINTK() callsHannes Reinecke2022-01-051-11/+0
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_promise: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke2022-01-051-23/+8
| | | | | | | | Drop pointless VPRINTK() calls for entering and existing interrupt routines and convert the remaining calls to ata_port_dbg(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pata_via: Drop pointless VPRINTK() callsHannes Reinecke2022-01-051-12/+0
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pata_octeon_cf: Drop pointless VPRINTK() calls and convert the ↵Hannes Reinecke2022-01-051-21/+1
| | | | | | | | | | remaining one Drop pointless VPRINTK() calls and convert the remaining calls to the existing bmdma tracepoint. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pdc_adma: Drop pointless VPRINTK() calls and remove disabled NCQ debuggingHannes Reinecke2022-01-051-29/+2
| | | | | | | | Drop pointless VPRINTK() calls for entering routines and setting up sg tables. And while we're at it, remove the disabled debugging messages. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: ahci: Drop pointless VPRINTK() calls and convert the remaining onesHannes Reinecke2022-01-053-21/+5
| | | | | | | | Drop pointless VPRINTK() calls for entering and existing interrupt routines and convert the remaining calls to dev_dbg(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: remove pointless VPRINTK() callsHannes Reinecke2022-01-055-52/+0
| | | | | | | | | | Most of the information is already covered by tracepoints (if not downright pointless), so remove the VPRINTK() calls. And while we're at it, remove ata_scsi_dump_cdb(), too, as this information can be retrieved from scsi tracing. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pata_pdc2027x: Replace PDPRINTK() with standard ata loggingHannes Reinecke2022-01-051-43/+28
| | | | | | | | Use standard ata logging macros instead of the hand-crafted PDPRINTK and remove duplicate logging messages. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_qstor: replace DPRINTK() with dev_dbg()Hannes Reinecke2022-01-051-2/+2
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_rcar: replace DPRINTK() with ata_port_dbg()Hannes Reinecke2022-01-051-1/+1
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_fsl: move DPRINTK to ata debuggingHannes Reinecke2022-01-051-44/+39
| | | | | | | Replace all DPRINTK calls with the ata_XXX_dbg functions. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pdc_adma: Remove DPRINTK callHannes Reinecke2022-01-051-2/+0
| | | | | | | | | The DPRINTK call doesn't print information which isn't already covered by tracepoints later on. Remove it. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: pata_octeon_cf: remove DPRINTK() macro in interrupt contextHannes Reinecke2022-01-051-2/+0
| | | | | | | | There is only so much information to be glanced when the interrupt routine is called and exited, so remove these DPRINTK() calls. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_mv: replace DPRINTK with dynamic debuggingHannes Reinecke2022-01-051-38/+38
| | | | | | | Move the DPRINTK calls over to dynamic debugging. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_mv: kill 'port' argument in mv_dump_all_regs()Hannes Reinecke2022-01-051-13/+6
| | | | | | | Always '-1', so drop it and simplify the function. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: move DPRINTK to ata debuggingHannes Reinecke2022-01-051-9/+6
| | | | | | | Replace all DPRINTK calls with ata_dev_dbg(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: revamp ata_get_cmd_descript()Hannes Reinecke2022-01-053-17/+11
| | | | | | | | Rename ata_get_cmd_descrip() to ata_get_cmd_name() and simplify it to return "unknown" instead of NULL. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: move ata_{port,link,dev}_dbg to standard pr_XXX() macrosHannes Reinecke2022-01-053-86/+29
| | | | | | | | Use standard pr_{debug,info,notice,warn,err} macros instead of the hand-crafted printk helpers. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: add tracepoints for ATA error handlingHannes Reinecke2022-01-053-30/+7
| | | | | | | Add tracepoints for ATA error handling. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata-scsi: drop DPRINTK calls for cdb translationHannes Reinecke2022-01-051-19/+1
| | | | | | | | Drop DPRINTK calls for cdb translation as they are already covered by other traces, and also drop the DPRINTK calls in ata_scsi_hotplug(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata-sff: tracepoints for HSM state machineHannes Reinecke2022-01-052-13/+36
| | | | | | | Add tracepoints for the HSM state machine and drop DPRINTK calls Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: tracepoints for bus-master DMAHannes Reinecke2022-01-0511-97/+76
| | | | | | | | Add tracepoints for bus-master DMA and taskfile related functions. That allows us to drop the relevant DPRINTK() calls. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: add qc_prep tracepointHannes Reinecke2022-01-041-0/+1
| | | | | | | | | Convert the existing ata_qc_issue() tracepoint into a template, and add tracepoints for ata_qc_prep() and ata_qc_issue() based on that template. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: add reset tracepointsHannes Reinecke2022-01-0410-57/+19
| | | | | | | | | | | | To follow the flow of control we should be using tracepoints, as they will tie in with the actual I/O flow and deliver a better overview about what it happening. This patch adds tracepoints for hard reset, soft reset, and postreset and adds them in the libata-eh control flow. With that we can drop the reset DPRINTK calls in the various drivers. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: sanitize ATA_HORKAGE_DUMP_IDHannes Reinecke2022-01-041-2/+2
| | | | | | | | | | | With moving ata_dev_dbg() over to dynamic debugging ATA_HORKAGE_DUMP_ID will now print out the raw IDENTIFY data without a header unless explicitly enable via dyndebug. So move the logging level up to INFO and have the header printed always. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: move ata_dump_id() to dynamic debuggingHannes Reinecke2022-01-041-28/+10
| | | | | | | | Use ata_dev_dbg() to print out the information in ata_dump_id() and remove the ata_msg_probe() conditional. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: Add ata_port_classify() helperHannes Reinecke2022-01-047-21/+40
| | | | | | | | | Add an ata_port_classify() helper to print out the results from the device classification and remove the debugging statements from ata_dev_classify(). Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: whitespace cleanupHannes Reinecke2022-01-042-10/+10
| | | | | Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: remove pointless debugging messagesHannes Reinecke2022-01-047-29/+0
| | | | | | | | | Debugging messages in pci init functions or sg setup are pretty much pointless, as the workflow pretty much decides what happened. So drop them. Signed-off-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libata: use min() to make code cleanerChangcheng Deng2022-01-041-8/+2
| | | | | | | | Use min() in order to make code cleaner. Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Changcheng Deng <deng.changcheng@zte.com.cn> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libahci_platform: Get rid of dup message when IRQ can't be retrievedAndy Shevchenko2022-01-041-4/+1
| | | | | | | | platform_get_irq() will print a message when it fails. No need to repeat this. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: libahci_platform: Remove bogus 32-bit DMA mask attemptAndy Shevchenko2022-01-041-7/+2
| | | | | | | | | | If 64-bit mask attempt fails, the 32-bit will fail by the very same reason. Don't even try the latter. It's a continuation of the changes that contains, e.g. dcc02c19cc06 ("sata_sil24: use dma_set_mask_and_coherent"). Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_dwc_460ex: Remove unused forward declarationAndy Shevchenko2022-01-041-1/+0
| | | | | | | | sata_dwc_port_stop() is not used before being defined, remove redundant forward declaration. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_dwc_460ex: Use temporary variable for struct deviceAndy Shevchenko2022-01-041-18/+19
| | | | | | | Use temporary variable for struct device to make code neater. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
* ata: sata_dwc_460ex: Use devm_platform_*ioremap_resource() APIsAndy Shevchenko2022-01-041-5/+2
| | | | | | | | | Use devm_platform_get_and_ioremap_resource() and devm_platform_ioremap_resource() APIs instead of their open coded analogues. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>