On Thu, Apr 7, 2016 at 8:06 PM, Leonid Moiseichuk
<leonid.moiseichuk@xxxxxxxxx> wrote:
> For generic subvendor has sense to use generic subdevice.
> If subdevice ID not equal to 0x9480/0x9485 mvsas will be not activated.
>
> Tested on ASUS P9A-I/C2550/SAS/4L which uses vendor-specific 1043:8635.
Minors below.
FWIW:
Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
>
> Signed-off-by: Leonid Moiseichuk <leonid.moiseichuk@xxxxxxxxx>
> ---
Missed changelog vfrom v1 to v2.
> drivers/scsi/mvsas/mv_init.c | 19 +------------------
> 1 file changed, 1 insertion(+), 18 deletions(-)
>
> diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
> index c7c250519c4b..a6a4f09df0be 100644
> --- a/drivers/scsi/mvsas/mv_init.c
> +++ b/drivers/scsi/mvsas/mv_init.c
> @@ -704,24 +704,7 @@ static struct pci_device_id mvs_pci_table[] = {
> .class_mask = 0,
> .driver_data = chip_9445,
> },
> - {
> - .vendor = PCI_VENDOR_ID_MARVELL_EXT,
> - .device = 0x9485,
> - .subvendor = PCI_ANY_ID,
> - .subdevice = 0x9480,
> - .class = 0,
> - .class_mask = 0,
> - .driver_data = chip_9485,
> - },
> - {
> - .vendor = PCI_VENDOR_ID_MARVELL_EXT,
> - .device = 0x9485,
> - .subvendor = PCI_ANY_ID,
> - .subdevice = 0x9485,
> - .class = 0,
> - .class_mask = 0,
> - .driver_data = chip_9485,
> - },
> + { PCI_VDEVICE(MARVELL_EXT, 0x9485), chip_9485 },
+ commentary line?
... /* MARVELL bla-bla-bla */
> { PCI_VDEVICE(OCZ, 0x1021), chip_9485}, /* OCZ RevoDrive3 */
> { PCI_VDEVICE(OCZ, 0x1022), chip_9485}, /* OCZ RevoDrive3/zDriveR4 (exact model unknown) */
> { PCI_VDEVICE(OCZ, 0x1040), chip_9485}, /* OCZ RevoDrive3/zDriveR4 (exact model unknown) */
> --
> 2.8.0.rc3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
