On 31.8.2016 15:10, Johannes Thumshirn wrote:
> On Wed, Aug 31, 2016 at 02:59:16PM +0200, Tomas Henzl wrote:
>> On 26.8.2016 18:44, Don Brace wrote:
>>> From: Kevin Barnett <kevin.barn...@microsemi.com>
>>>
>>> Removed the workaround for the transition to spanning.
>>>
>>> Reviewed-by: Scott Teel <scott.t...@microsemi.com>
>>> Reviewed-by: Scott Benesh <scott.ben...@microsemi.com>
>>> Signed-off-by: Kevin Barnett <kevin.barn...@microsemi.com>
>>> Signed-off-by: Don Brace <don.br...@microsemi.com>
>>> ---
>>> drivers/scsi/smartpqi/smartpqi_init.c | 38
>>> +++++++++++++++++----------------
>>> 1 file changed, 20 insertions(+), 18 deletions(-)
>>>
>>> diff --git a/drivers/scsi/smartpqi/smartpqi_init.c
>>> b/drivers/scsi/smartpqi/smartpqi_init.c
>>> index 6eab38e..9922e31 100644
>>> --- a/drivers/scsi/smartpqi/smartpqi_init.c
>>> +++ b/drivers/scsi/smartpqi/smartpqi_init.c
>>> @@ -3663,6 +3663,18 @@ static int pqi_validate_device_capability(struct
>>> pqi_ctrl_info *ctrl_info)
>>> return -EINVAL;
>>> }
>>>
>>> + if (!ctrl_info->inbound_spanning_supported) {
>>> + dev_err(&ctrl_info->pci_dev->dev,
>>> + "the controller does not support inbound spanning\n");
>>> + return -EINVAL;
>>> + }
>>> +
>>> + if (ctrl_info->outbound_spanning_supported) {
>>> + dev_err(&ctrl_info->pci_dev->dev,
>>> + "the controller supports outbound spanning but this
>>> driver does not\n");
>>> + return -EINVAL;
>>> + }
>>> +
>>> return 0;
>>> }
>> This looks like you are dropping support for a probably older firmware
>> devices,
>> is that necessary - can't you continue a keep the support with something
>> similar to that
>> 'TEMPHACk' below ?
> The question is, if any of these older firmware devices ever left the
> Microsemi buildings. If not I _think_ it's save to drop it, as no real
> users are harmed.
Well yes, likely it was used only during development - in that case also
if all devices support inbound_spanning it is not needed to test at all and
that bit could have been used for anything else (it's too late now for this
though.)
Also if Microsemi doesn't want support future devices with this driver
a better option is to not use PCI_ANY_ID in pci_device_id instead of testing
the outbound_spanning_supported.
All that is minor, so -
Reviewed-by: Tomas Henzl <the...@redhat.com>
>
> Byte,
> Johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
