From mboxrd@z Thu Jan  1 00:00:00 1970
From: santosh <santosh.shukla@caviumnetworks.com>
Subject: Re: [PATCH v4 2/2] octeontx: move mbox to common folder
Date: Wed, 4 Apr 2018 10:36:09 +0530
Message-ID: <8daff581-a03f-0659-2bbb-ef999caaca46@caviumnetworks.com>
References: <20180319092726.10153-1-pbhagavatula@caviumnetworks.com>
 <20180402091000.9208-1-pbhagavatula@caviumnetworks.com>
 <20180402091000.9208-2-pbhagavatula@caviumnetworks.com>
 <20180404032919.GB8574@jerin>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: thomas@monjalon.net, anatoly.burakov@intel.com, lironh@marvell.com,
 bruce.richardson@intel.com, fiona.trahe@intel.com, shreyansh.jain@nxp.com,
 hemant.agrawal@nxp.com, dev@dpdk.org
To: Jerin Jacob <jerin.jacob@caviumnetworks.com>,
 Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
Return-path: <dev-bounces@dpdk.org>
Received: from NAM03-BY2-obe.outbound.protection.outlook.com
 (mail-by2nam03on0078.outbound.protection.outlook.com [104.47.42.78])
 by dpdk.org (Postfix) with ESMTP id D88391BB29
 for <dev@dpdk.org>; Wed,  4 Apr 2018 07:06:27 +0200 (CEST)
In-Reply-To: <20180404032919.GB8574@jerin>
Content-Language: en-US
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>


On Wednesday 04 April 2018 08:59 AM, Jerin Jacob wrote:
> -----Original Message-----
>> Date: Mon,  2 Apr 2018 14:40:00 +0530
>> From: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
>> To: jerin.jacob@caviumnetworks.com, santosh.shukla@caviumnetworks.com,
>>  thomas@monjalon.net, anatoly.burakov@intel.com, lironh@marvell.com,
>>  bruce.richardson@intel.com, fiona.trahe@intel.com, shreyansh.jain@nxp.com,
>>  hemant.agrawal@nxp.com
>> Cc: dev@dpdk.org, Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
>> Subject: [dpdk-dev] [PATCH v4 2/2] octeontx: move mbox to common folder
>> X-Mailer: git-send-email 2.16.3
>>
>> Move commonly used functions across mempool, event and net devices to a
>> common folder in drivers.
>>
>> Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
>> ---

snip

>> vfid == 0 ?
>>
>> +		ram_mbox_base = octeontx_ssovf_bar(OCTEONTX_SSO_HWS, 0, 4);

also ram_mbox_base may throw build warning in case vfid>0..right?

>> +		if (octeontx_mbox_set_ram_mbox_base(ram_mbox_base)) {
>> +			mbox_log_err("Invalid Failed to set ram mbox base");
>> +			return -EINVAL;
>> +		}
>> +	}
>> +
>>  	rte_wmb();
>>  	mbox_log_dbg("Domain=%d hws=%d total_ssowvfs=%d", res->domain,
>>  			res->vfid, sdev.total_ssowvfs);
>> @@ -213,6 +221,7 @@ ssovf_probe(struct rte_pci_driver *pci_drv, struct rte_pci_device *pci_dev)
>>  	uint16_t vfid;
>>  	uint8_t *idreg;
>>  	struct ssovf_res *res;
>> +	uint8_t *reg;
>>
>>  	RTE_SET_USED(pci_drv);
>>
>> @@ -246,6 +255,15 @@ ssovf_probe(struct rte_pci_driver *pci_drv, struct rte_pci_device *pci_dev)
>>  	res->domain = val & 0xffff;
>>
>>  	sdev.total_ssovfs++;
>> +	if (!vfid) {
> vfid == 0 ?
>
>> +		reg = octeontx_ssovf_bar(OCTEONTX_SSO_GROUP, 0, 0);

ditto for reg.

>> +		reg += SSO_VHGRP_PF_MBOX(1);
>> +		if (octeontx_mbox_set_reg(reg)) {
>> +			mbox_log_err("Invalid Failed to set mbox_reg");
>> +			return -EINVAL;
>> +		}
>> +	}
>> +
>>  	rte_wmb();
>>  	mbox_log_dbg("Domain=%d group=%d total_ssovfs=%d", res->domain,
>>  			res->vfid, sdev.total_ssovfs);
> With above changes:
>
> Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
>
Acked-by: Santosh Shukla <santosh.shukla@caviumnetworks.com>
Thanks.