Re: [PATCH net-next 3/5] net: wwan: t7xx: PCIe reset rescan

["Kumar, M Chetan" <m.chetan.kumar@linux.intel.com>]

On 9/7/2022 3:49 AM, Sergey Ryazanov wrote:
> On Fri, Sep 2, 2022 at 7:50 AM Kumar, M Chetan
> <m.chetan.kumar@linux.intel.com> wrote:
>> On 8/30/2022 7:32 AM, Sergey Ryazanov wrote:
>>> On Tue, Aug 16, 2022 at 7:12 AM <m.chetan.kumar@intel.com> wrote:
>>>> From: Haijun Liu <haijun.liu@mediatek.com>
>>>>
>>>> PCI rescan module implements "rescan work queue". In firmware flashing
>>>> or coredump collection procedure WWAN device is programmed to boot in
>>>> fastboot mode and a work item is scheduled for removal & detection.
>>>> The WWAN device is reset using APCI call as part driver removal flow.
>>>> Work queue rescans pci bus at fixed interval for device detection,
>>>> later when device is detect work queue exits.
>>>
>>> [skipped]
>>>
>>>> diff --git a/drivers/net/wwan/t7xx/t7xx_pci_rescan.c b/drivers/net/wwan/t7xx/t7xx_pci_rescan.c
>>>> new file mode 100644
>>>> index 000000000000..045777d8a843
>>>> --- /dev/null
>>>> +++ b/drivers/net/wwan/t7xx/t7xx_pci_rescan.c
>>>
>>> [skipped]
>>>
>>>> +static void t7xx_remove_rescan(struct work_struct *work)
>>>> +{
>>>> +       struct pci_dev *pdev;
>>>> +       int num_retries = RESCAN_RETRIES;
>>>> +       unsigned long flags;
>>>> +
>>>> +       spin_lock_irqsave(&g_mtk_rescan_context.dev_lock, flags);
>>>> +       g_mtk_rescan_context.rescan_done = 0;
>>>> +       pdev = g_mtk_rescan_context.dev;
>>>> +       spin_unlock_irqrestore(&g_mtk_rescan_context.dev_lock, flags);
>>>> +
>>>> +       if (pdev) {
>>>> +               pci_stop_and_remove_bus_device_locked(pdev);
>>>
>>> What is the purpose of removing the device then trying to find it by
>>> rescanning the bus? Would not it be easier to save a PCI device
>>> configuration, reset the device, and then restore the configuration?
>>
>> If hotplug is disabled, the device is not removed on reset. So in this
>> case driver need to handle the device removal and rescan.
> 
> I still can not understand this part and need a clue. Why should the
> driver disable the hotplug?

This is a platform configuration, it could be set to enable/disable.
We can find this option in BIOS settings.

> And is there a more gentle way to reset the firmware without the
> device object removing?

Device reset causes WWAN device to fall off the BUS. If device had not 
fallen off the bus then we could have reused.

Without these changes, we need to manually execute device remove & 
rescan commands.

Could you please suggest how can we proceed here ?

-- 
Chetan