diff options
author | Dave Jiang <dave.jiang@intel.com> | 2011-03-26 19:11:51 -0400 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2011-07-03 07:00:36 -0400 |
commit | 09d7da135b34bc74a7996b5db373521557ddf3d4 (patch) | |
tree | 64b66ecb731cd0e609ef3fcb1fd4d460513aaa89 /drivers/scsi/isci/port.c | |
parent | 52ae18ac80fbdd268720b0daa27ac797a801500c (diff) |
isci: Remove event_* calls as they are just wrappers
Removed isci_event_* calls and call those functions directly.
Reported-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/scsi/isci/port.c')
-rw-r--r-- | drivers/scsi/isci/port.c | 51 |
1 files changed, 38 insertions, 13 deletions
diff --git a/drivers/scsi/isci/port.c b/drivers/scsi/isci/port.c index a5b2565ac5af..666076a2834e 100644 --- a/drivers/scsi/isci/port.c +++ b/drivers/scsi/isci/port.c | |||
@@ -279,10 +279,8 @@ void isci_port_link_up( | |||
279 | * @port: This parameter specifies the isci port with the active link. | 279 | * @port: This parameter specifies the isci port with the active link. |
280 | * | 280 | * |
281 | */ | 281 | */ |
282 | void isci_port_link_down( | 282 | void isci_port_link_down(struct isci_host *isci_host, struct isci_phy *isci_phy, |
283 | struct isci_host *isci_host, | 283 | struct isci_port *isci_port) |
284 | struct isci_phy *isci_phy, | ||
285 | struct isci_port *isci_port) | ||
286 | { | 284 | { |
287 | struct isci_remote_device *isci_device; | 285 | struct isci_remote_device *isci_device; |
288 | 286 | ||
@@ -358,9 +356,7 @@ void isci_port_formed( | |||
358 | * @port: This parameter specifies the sci port with the active link. | 356 | * @port: This parameter specifies the sci port with the active link. |
359 | * | 357 | * |
360 | */ | 358 | */ |
361 | void isci_port_ready( | 359 | void isci_port_ready(struct isci_host *isci_host, struct isci_port *isci_port) |
362 | struct isci_host *isci_host, | ||
363 | struct isci_port *isci_port) | ||
364 | { | 360 | { |
365 | dev_dbg(&isci_host->pdev->dev, | 361 | dev_dbg(&isci_host->pdev->dev, |
366 | "%s: isci_port = %p\n", __func__, isci_port); | 362 | "%s: isci_port = %p\n", __func__, isci_port); |
@@ -378,9 +374,7 @@ void isci_port_ready( | |||
378 | * @port: This parameter specifies the sci port with the active link. | 374 | * @port: This parameter specifies the sci port with the active link. |
379 | * | 375 | * |
380 | */ | 376 | */ |
381 | void isci_port_not_ready( | 377 | void isci_port_not_ready(struct isci_host *isci_host, struct isci_port *isci_port) |
382 | struct isci_host *isci_host, | ||
383 | struct isci_port *isci_port) | ||
384 | { | 378 | { |
385 | dev_dbg(&isci_host->pdev->dev, | 379 | dev_dbg(&isci_host->pdev->dev, |
386 | "%s: isci_port = %p\n", __func__, isci_port); | 380 | "%s: isci_port = %p\n", __func__, isci_port); |
@@ -394,9 +388,8 @@ void isci_port_not_ready( | |||
394 | * process. | 388 | * process. |
395 | * | 389 | * |
396 | */ | 390 | */ |
397 | void isci_port_hard_reset_complete( | 391 | void isci_port_hard_reset_complete(struct isci_port *isci_port, |
398 | struct isci_port *isci_port, | 392 | enum sci_status completion_status) |
399 | enum sci_status completion_status) | ||
400 | { | 393 | { |
401 | dev_dbg(&isci_port->isci_host->pdev->dev, | 394 | dev_dbg(&isci_port->isci_host->pdev->dev, |
402 | "%s: isci_port = %p, completion_status=%x\n", | 395 | "%s: isci_port = %p, completion_status=%x\n", |
@@ -480,3 +473,35 @@ int isci_port_perform_hard_reset( | |||
480 | 473 | ||
481 | return ret; | 474 | return ret; |
482 | } | 475 | } |
476 | |||
477 | /** | ||
478 | * isci_port_invalid_link_up() - This function informs the SCI Core user that | ||
479 | * a phy/link became ready, but the phy is not allowed in the port. In some | ||
480 | * situations the underlying hardware only allows for certain phy to port | ||
481 | * mappings. If these mappings are violated, then this API is invoked. | ||
482 | * @controller: This parameter represents the controller which contains the | ||
483 | * port. | ||
484 | * @port: This parameter specifies the SCI port object for which the callback | ||
485 | * is being invoked. | ||
486 | * @phy: This parameter specifies the phy that came ready, but the phy can't be | ||
487 | * a valid member of the port. | ||
488 | * | ||
489 | */ | ||
490 | void isci_port_invalid_link_up(struct scic_sds_controller *scic, | ||
491 | struct scic_sds_port *sci_port, | ||
492 | struct scic_sds_phy *phy) | ||
493 | { | ||
494 | struct isci_host *ihost = | ||
495 | (struct isci_host *)sci_object_get_association(scic); | ||
496 | |||
497 | dev_warn(&ihost->pdev->dev, "Invalid link up!\n"); | ||
498 | } | ||
499 | |||
500 | void isci_port_stop_complete(struct scic_sds_controller *scic, | ||
501 | struct scic_sds_port *sci_port, | ||
502 | enum sci_status completion_status) | ||
503 | { | ||
504 | struct isci_host *ihost = sci_object_get_association(scic); | ||
505 | |||
506 | dev_dbg(&ihost->pdev->dev, "Port stop complete\n"); | ||
507 | } | ||