[Skiboot] [PATCH 3/7] npu2-opencapi: Rename functions used to reset an adapter

Andrew Donnellan andrew.donnellan at au1.ibm.com
Tue Mar 5 17:30:59 AEDT 2019


On 2/3/19 12:52 am, Frederic Barrat wrote:
> This is really to avoid confusion with a later patch and clarify
> whether we're resetting the ODL or the adapter.
> 
> Signed-off-by: Frederic Barrat <fbarrat at linux.ibm.com>

Reviewed-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

> ---
>   hw/npu2-opencapi.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/hw/npu2-opencapi.c b/hw/npu2-opencapi.c
> index 6d642cde..9fe5efd6 100644
> --- a/hw/npu2-opencapi.c
> +++ b/hw/npu2-opencapi.c
> @@ -814,7 +814,7 @@ static void otl_enabletx(uint32_t gcid, uint32_t scom_base,
>   	/* TODO: Abort if credits are zero */
>   }
>   
> -static void assert_reset(struct npu2_dev *dev)
> +static void assert_adapter_reset(struct npu2_dev *dev)
>   {
>   	uint8_t pin, data;
>   	int rc;
> @@ -869,7 +869,7 @@ err:
>   	OCAPIERR(dev, "Error writing I2C reset signal: %d\n", rc);
>   }
>   
> -static void deassert_reset(struct npu2_dev *dev)
> +static void deassert_adapter_reset(struct npu2_dev *dev)
>   {
>   	uint8_t data;
>   	int rc;
> @@ -1164,14 +1164,14 @@ static int64_t npu2_opencapi_freset(struct pci_slot *slot)
>   		/* fall-through */
>   	case OCAPI_SLOT_FRESET_INIT:
>   		reset_odl(chip_id, dev);
> -		assert_reset(dev);
> +		assert_adapter_reset(dev);
>   		pci_slot_set_state(slot,
>   				OCAPI_SLOT_FRESET_ASSERT_DELAY);
>   		/* assert for 5ms */
>   		return pci_slot_set_sm_timeout(slot, msecs_to_tb(5));
>   
>   	case OCAPI_SLOT_FRESET_ASSERT_DELAY:
> -		deassert_reset(dev);
> +		deassert_adapter_reset(dev);
>   		pci_slot_set_state(slot,
>   				OCAPI_SLOT_FRESET_DEASSERT_DELAY);
>   		/* give another 5ms to device to be ready */
> 

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited



More information about the Skiboot mailing list