lkml.org 
[lkml]   [2006]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH 1/2] Add SATA support to libsas
From
Date
On Thu, 2006-08-10 at 19:19 -0700, Darrick J. Wong wrote:
> +static inline void sas_mark_dev_sata(struct domain_device *dev)
> +{
> + dev->rphy->identify = dev->port->phy->identify;
> + dev->rphy->identify.initiator_port_protocols =
> SAS_PROTOCOL_SATA;
> + dev->rphy->identify.target_port_protocols = SAS_PROTOCOL_SATA;
> + dev->rphy->identify.device_type = SAS_END_DEVICE;
> + memcpy(&dev->rphy->identify.sas_address, dev->sas_addr,
> SAS_ADDR_SIZE);
> +}

Actually, this is wrong: you can't memcpy from dev->sas_addr to
identify.sas_addr the former is a big endian u8[8] and the latter is a
u64. However, the function is unnecessary anyway. We already have a
sas_fill_in_rphy that does all of this.

James

Index: BUILD-2.6/drivers/scsi/libsas/sas_discover.c
===================================================================
--- BUILD-2.6.orig/drivers/scsi/libsas/sas_discover.c 2006-08-21 21:30:38.000000000 -0500
+++ BUILD-2.6/drivers/scsi/libsas/sas_discover.c 2006-08-21 21:49:22.000000000 -0500
@@ -398,15 +398,6 @@
spin_unlock_irqrestore(&port->phy_list_lock, flags);
}

-static inline void sas_mark_dev_sata(struct domain_device *dev)
-{
- dev->rphy->identify = dev->port->phy->identify;
- dev->rphy->identify.initiator_port_protocols = SAS_PROTOCOL_SATA;
- dev->rphy->identify.target_port_protocols = SAS_PROTOCOL_SATA;
- dev->rphy->identify.device_type = SAS_END_DEVICE;
- memcpy(&dev->rphy->identify.sas_address, dev->sas_addr, SAS_ADDR_SIZE);
-}
-
#define ATA_IDENTIFY_DEV 0xEC
#define ATA_IDENTIFY_PACKET_DEV 0xA1
#define ATA_SET_FEATURES 0xEF
@@ -490,7 +481,7 @@
sas_satl_register_dev(dev);
*/

- sas_mark_dev_sata(dev);
+ sas_fill_in_rphy(dev, dev->rphy);

res = sas_rphy_add(dev->rphy);
if (res)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-08-22 19:29    [W:0.108 / U:2.240 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site