From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0083.outbound.protection.outlook.com [104.47.42.83]) by dpdk.org (Postfix) with ESMTP id 4E9AF107A for ; Thu, 28 Sep 2017 16:01:42 +0200 (CEST) Received: from CY1PR03CA0008.namprd03.prod.outlook.com (10.174.128.18) by CO2PR03MB2359.namprd03.prod.outlook.com (10.166.93.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Thu, 28 Sep 2017 14:01:41 +0000 Received: from BN1BFFO11FD024.protection.gbl (2a01:111:f400:7c10::1:197) by CY1PR03CA0008.outlook.office365.com (2603:10b6:600::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.56.8 via Frontend Transport; Thu, 28 Sep 2017 14:01:40 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=fail action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BN1BFFO11FD024.mail.protection.outlook.com (10.58.144.87) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.20.56.11 via Frontend Transport; Thu, 28 Sep 2017 14:01:40 +0000 Received: from [10.232.14.39] ([10.232.14.39]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id v8SE1cW4012646; Thu, 28 Sep 2017 07:01:38 -0700 To: CC: References: <20170928123000.1711-1-shreyansh.jain@nxp.com> <20170928141045.21762-1-shreyansh.jain@nxp.com> From: Shreyansh Jain Message-ID: <0dc71a1b-9c8e-847c-ad08-3227a7dd8a1d@nxp.com> Date: Thu, 28 Sep 2017 19:42:31 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <20170928141045.21762-1-shreyansh.jain@nxp.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-EOPAttributedMessage: 0 X-Matching-Connectors: 131510809005194317; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(979002)(6009001)(336005)(7966004)(346002)(39380400002)(376002)(39860400002)(2980300002)(1110001)(1109001)(339900001)(3190300001)(377454003)(24454002)(189002)(199003)(8676002)(83506001)(50466002)(97736004)(8936002)(81156014)(2906002)(81166006)(305945005)(230700001)(229853002)(36756003)(104016004)(575784001)(86362001)(5660300001)(65826007)(23676002)(189998001)(498600001)(31696002)(50986999)(76176999)(54356999)(68736007)(2950100002)(6916009)(316002)(105606002)(33646002)(47776003)(106466001)(2351001)(6246003)(77096006)(4326008)(64126003)(53936002)(85426001)(6666003)(356003)(65806001)(31686004)(65956001)(58126008)(969003)(989001)(999001)(1009001)(1019001); DIR:OUT; SFP:1101; SCL:1; SRVR:CO2PR03MB2359; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1BFFO11FD024; 1:RFLYBvX++urhD/7fMnu9+RjyFyYVfT/aYLnJWHUbKBvXzZG9DjLjEoGVB1BXvC1B8ekTpT2l4IIa6KJKmr90De6vflk+kyzP9yi38HTxnDHQeHr2uS7wmbQcrtseWyTe X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 868e55ad-9f07-484a-1568-08d50679714c X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017052603199)(201703131430075)(201703131517081); SRVR:CO2PR03MB2359; X-Microsoft-Exchange-Diagnostics: 1; CO2PR03MB2359; 3:LvudaAE0YwTVRqe/qwsJIYOLV9HhphHsKpVFgWXcRN9JTn7Nv9Ed3uL6ir6qViM99jKSt7mO58tdrU/ifAYoOdByPzkkUh2AAu5/lkptKAgEvKSS3wift5RpM2QLlcniALCJpFJiHwtxRIfqL3thP1demrZvJw0EEtnPTtfFlzVF48BseLrogk+gBgWSTnFCJ/7Y5SPxt9o/ryN/ViXfu0RQ2AeopLwL6AwPN7RTQSynZOKjHlC6M2VUBd2w9wErYvOHR4yRRTYyYhEePzA9y/iBjRN7qi1Ndci5jSZSeK9ZmeA+RV84st93cz8uAWFge4oe2jZPLAmJmhuim+86KuUNYmoXpcWeimoqk9icvJo=; 25:u8k37KeG5yDcWYdO4Qhu8nHT3o23EWV5+UBTOmMjL/h9NeTWeciFQ1lbWxGPVpj7dRiXNjv9v2VcVl90TKMAP+clzyqssywA5i+yHvjiHZGnISxyg5LFRmY2iONs1kkflFNgxsumzU7lcHEcHZmv7oRJVi8eQyTiAPY8b6V0E3ltq3mzix3oRH7iKKYmICrg0EimIVABI7Q7jOuV05v8J2FwtQjrGYOGQKaBo7x8mZXwm7P4CiBNmyb78AYgJelV7pgcOk2gVkVjKM2C0Xi/+hmcf43pTGZ5ElbJppzrIF4mpZXl85Y3Qc4qZRm1YXap8zIwnJlfHZ/I7bkEYUkrsg== X-MS-TrafficTypeDiagnostic: CO2PR03MB2359: X-Microsoft-Exchange-Diagnostics: 1; CO2PR03MB2359; 31:d3bTjkrT6hMgXQK8jXekvSCURDJ8sVPKHnio1AuOYD9QOADveFsCZSFY34qCL4S+uz1MeFtplQS96q30EdHAHzzIlRXqOA/SmGQcWAlSo5NztpdRNP/ZZY3Fhk0X1ECCMTOfOQmhBMi27ABE/J6AbCcR+N+Kh+zTG+tTpTeqMmMkDtsDSVWL6apvrDl8lFAIj19fsq4Vb1nstUWn2cRdmVjGlkLPsWOp+YQ1LsrYvws=; 4:VQ8UYJEX1ZemweNdodC5Z4hhHoY8J95TBKtKLpwzXBw6Z3hCfN+oArpUDkVSWHWl/poGd+GBO+szNsAy9bv4HpZOAliPryyWNfHUbGwH2VReYRUslPZuR7waMSREq2OTId7kBogqiwkM6am7h0IC36iQvNiJ1dtxWfa1bek4W+i7jvT4DEdHXijskXD+KRR5VOTlCeo7r+oQQcb94mqQv0fQNsnbK55yowto/JjO53fXCEDVVnmFBx5QRLP/bVRLttfNowvJelbTWzHA6452iNgvRwwa9LErAOLNQa+Dqj8= X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6095135)(2401047)(5005006)(8121501046)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(3002001)(6055026)(6096035)(201703131430075)(201703131441075)(201703131448075)(201703131433075)(201703161259150)(20161123561025)(20161123559100)(20161123565025)(20161123556025)(20161123563025)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CO2PR03MB2359; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(400006)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CO2PR03MB2359; X-Forefront-PRVS: 0444EB1997 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtDTzJQUjAzTUIyMzU5OzIzOlo4aGlLZ00zanhpeVJVTlFleklicTBPK2hq?= =?utf-8?B?cnhva3VQWnhrNzEvNUozZHdvcm1uZXhoOCs5REVFZDdJNmFqSTlaTC9KZFdU?= =?utf-8?B?akpJQnJJb3RseFJ2VExJT1UyU2wrSDFCRFUyT3JxSUN5NzI1WW80ZDlrWHZx?= =?utf-8?B?OGVsNGd6R3BMNFRvMzU5QXVDTDVhK3BRdGhqODR2cUFVVkdhc3B1M0VNYXVY?= =?utf-8?B?KzBmUGRxV25LQ09GWFVHK2dySmo0ekphMGNVT1owSEtqbGJmUW1ZQlhvODFp?= =?utf-8?B?M0VHVVBYdXROaFNWakp2WVQwcUU0VW1HVVc3WSt5cE50Rjg5VUtuTGtEUEFo?= =?utf-8?B?TUErczVqTDFHTitxOGVnVGZtc0NvUTkwc2wwZDd4RW1mVm51S0VJUHJ0djBq?= =?utf-8?B?L24wUk4yK3p5bDRSUnpNYlBOS0Z2NnhuRFFYcFpMVzNNTVg2MFkzZzN2Z2NU?= =?utf-8?B?MDhhNCt2ZlBEdUdmT1V5eHpvU1NLWDJFUHVUdElVLzdmbkNFM0szU1RoQjZp?= =?utf-8?B?ZkVxVEdZNzdCTzNZNlAyZkNEb0E5bFBkd3J3a0Qrb2VHZTBtTUFzZStJcER3?= =?utf-8?B?RFc3cmxQNFJoZ09xZjBWY3Z4YmlONGRoUFNEYTlVRC83ZXd4VUpPemJZQXcy?= =?utf-8?B?ZE5RNEpvdkZ1dGo1c2lIS1Z2TGFMY0tpMS84cGRBenRFZTMvK2hsS3MrMkpB?= =?utf-8?B?Y092U25sWTlFZ3E1YTFOVjRaZmhicTVrWVptdFk1aUNpTFZ1b3JQYTJlMkVp?= =?utf-8?B?MWE1cW5zSFJxekpZbE5TMDlVUFdkRHRZV2ZFOGhpa0pLaFVWTjlZSDRLWGJQ?= =?utf-8?B?WmZPSGphVFBDSjVrV2I5dWJtUlplZlVpVUJDSkErQ1FzZzFuanpubms1bStu?= =?utf-8?B?bVVJclNLaWtjVHJpbmVCZ1VwK2JWTGQ0YlEwbDZoaDBSeDVQRzdxaWIzUkN3?= =?utf-8?B?UGgzZGl2UVh0S0prL0ZzcER6ekoyOXRSNFg5YktDQ0p0M0dVM1V3Z0ZOYVYy?= =?utf-8?B?WTNodFRHVVpDcVB0d1dRaVNNallzK0pMbWVjb2ZNY1hiNHUzM1R0cURNMDhl?= =?utf-8?B?NmlaQy9Db2NSaGpJTU1oeDNUejF2MzVKa1crVURGT04xTlZyYm03QW9OTDJB?= =?utf-8?B?blpqSlo0L0VjRHA4aDdwZlV2MlRQa2RsNnBtOWpUTktIc0x1MWRGSVhZWTA4?= =?utf-8?B?aDBzNHFHL1ZWRnRlQ2syZE55c2lNdzdmenVOTUVLSE1Yek16NFpacFNMYks2?= =?utf-8?B?ZXczUjlCTUZrWVFPbHVvTXNuZjdNTHBrWjFEUVF4SWJjQVJrcXc5WnlJZEtj?= =?utf-8?B?TDdWMml5RG1PdW5YNHFYUXpTU0hVejJOZkJJU1NTUUxMdXlmSmYvU0dlMWZL?= =?utf-8?B?Zmhld09tQVhSLzVGc3FhQTNsbHlPdVREM3dvNkpxSzIyZks4ZnM4cmR4Qy9N?= =?utf-8?B?cjFyaDY4RmpKUGRldG9BQytCVUxyeG5Db1l2NUVuRE9LS1BWSzR1cVkwUzZR?= =?utf-8?B?clAyTlhxazdveUxpR3BlMXhvZGJtUUFHc09QUUI4RGZYMWhqakpVYzIzUHFU?= =?utf-8?B?ZHZGRTIrQ1lVNk9ybFBnMFNTNnhUMjRQYjlEUXlpTFBybG04aUdOalVvMDll?= =?utf-8?B?RTVUWkdRQTJSNzg5eTNKWms5UVRGbmxXUVQ4TC9IakNUUVhRQ2p6RjdQODA1?= =?utf-8?B?Q1VlRUxVN2tESCtDaTZoTmpPWWs3bkh4UVVZTkgyZWlsdFRiVjQwUVpYMml6?= =?utf-8?B?VGozL3JZanBxV1FZRUppVHlYU1BrQ1k0N0lDQ1M5bFhnbnRzZ0lQQ2k4Wk9m?= =?utf-8?B?Y3drdGhNY1UxOTk5cVJORHY1MlA1TlUxZ0VlZHVaVDl0WUdPbW9HM25rcmU5?= =?utf-8?B?dEU2dVdwNC9yM094aWtkYUZZS3JaTW1Xdzg0Rno3dFhZYVhaNmRhWE5RdHZk?= =?utf-8?B?MTRNazdRcnZVYXlMV3BiNkVOQzFWaFpUeFkxNEZEcFk3d0V5ZzIyL01qVVdo?= =?utf-8?B?SnhGVzlieHhQbWg5UUJFemtlNjc1bWhMM0RsS0VpRmlKQVAxWlp5UDFhbXc4?= =?utf-8?Q?BDvg=3D?= X-Microsoft-Exchange-Diagnostics: 1; CO2PR03MB2359; 6:Z/+h9+9+ZEyYFHIJjroQa7bw5cSzrCvWk2TDAQ1Iypo7+OUhiZD2cYakXAr3wwzuRi8+MT0yCz95GlWQpCttGCmyypW4OlxUDBLMhGKvZTgzS2GCr4rrokfWVTbTpJxoX6m9EK7G6Aco+ObAeo4Exx6gdfO2tqQDxu9ES0TUryvkFIUdoXNR+xi7H9EsAqZW172ADryr020u26it2EhZRJmqbSI+9j+pWL3I0dpJ48whGTHKgFKVPU+OVYgVXnKwp8i44tMgmx7oYD0JD6a+L+VssxDAWzbrZLqyRgpu7JLJFh40g2QXDkkLyUMJBonD5pa9zjx3yiuM+LztwmQl/Q==; 5:xgUYLVEPFaOJEhPXNBmfdbeDSzIbwoYBqi91GtBWwDXvj+SzlxF5u1PBQvX5uyEEJWxUqOKBMmPVl065dLzzcrL8DUT9QqClh8ADrUX+V/vNgT6R36CdwGsi9ppyxehZhznZGvBT/e8AOi9Xwyaj7Q==; 24:gB41+kxIFVKt05grUEWGyhgnV+k7o0YUT25HG8SXugToRatnYjo2H+iTnMMRY/Q5k6tKYB56ubJd5pj6Z3+KhgO6JaH0avFuctAZAHboZl8=; 7:1NcsCP9803UGY8Ozg2Fu2wxrBAPmKoMFNJvJuUtJnZ3RrcWO14kc52WYrMivB8/xygpXPeL1hzXNOxOFcwk8GaNXsmexOnHbVPxMLmyj1MxkvaFULSX3iF5XmM2OUJPmDqAm3zXb4UT9sCg2ryqVad7Mw0MQpVYHo35hgTlrL5uHWxO5eWQ5UJHzauZnzzCN+PoLrSqJl1LExlQ+HqprG2wr0rYgRVme5T2ppiEQWB8= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Sep 2017 14:01:40.2698 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO2PR03MB2359 Subject: Re: [dpdk-dev] [PATCH 1/2] bus/dpaa: fix incorrect ccsr mem allocation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Sep 2017 14:01:42 -0000 Hello Ferruh, On Thursday 28 September 2017 07:40 PM, Shreyansh Jain wrote: > Fixes: 5ad2d123be48 "(bus/dpaa: introducing FMan configurations)" > > Signed-off-by: Shreyansh Jain > --- > drivers/bus/dpaa/base/fman/fman.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/bus/dpaa/base/fman/fman.c b/drivers/bus/dpaa/base/fman/fman.c > index 2c6029e..d0a8ee4 100644 > --- a/drivers/bus/dpaa/base/fman/fman.c > +++ b/drivers/bus/dpaa/base/fman/fman.c > @@ -242,11 +242,11 @@ fman_if_init(const struct device_node *dpa_node) > if (!phys_addr) { > FMAN_ERR(-EINVAL, "of_translate_address(%s, %p)\n", > mname, regs_addr); > - __if->ccsr_map = mmap(NULL, __if->regs_size, > - PROT_READ | PROT_WRITE, MAP_SHARED, > - fman_ccsr_map_fd, phys_addr); > goto err; > } > + __if->ccsr_map = mmap(NULL, __if->regs_size, > + PROT_READ | PROT_WRITE, MAP_SHARED, > + fman_ccsr_map_fd, phys_addr); > if (__if->ccsr_map == MAP_FAILED) { > FMAN_ERR(-errno, "mmap(0x%"PRIx64")\n", phys_addr); > goto err; > While working on the v6, I missed two small changes. I have created patches against these and added the fixes. Can you please apply them? (I didn't want to send a complete v7 just for these). - Shreyansh