Message-ID: <199a5aff-fee8-71d3-bfcb-c0492667de1b@oracle.com>
Date:   Fri, 3 Feb 2023 10:47:34 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [External] : RE: [EXT] [PATCH v2 1/1] PCI: layerscape: Add EP
 mode support for ls1028a
Content-Language: en-US
To:     Frank Li <frank.li@nxp.com>, "M.H. Lian" <minghuan.lian@nxp.com>,
        Mingkai Hu <mingkai.hu@nxp.com>, Roy Zang <roy.zang@nxp.com>,
        Lorenzo Pieralisi <lpieralisi@kernel.org>,
        Rob Herring <robh@kernel.org>,
        =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= <kw@linux.com>,
        Bjorn Helgaas <bhelgaas@google.com>,
        "open list:PCI DRIVER FOR FREESCALE LAYERSCAPE" 
        <linuxppc-dev@lists.ozlabs.org>,
        "open list:PCI DRIVER FOR FREESCALE LAYERSCAPE" 
        <linux-pci@vger.kernel.org>,
        "moderated list:PCI DRIVER FOR FREESCALE LAYERSCAPE" 
        <linux-arm-kernel@lists.infradead.org>,
        open list <linux-kernel@vger.kernel.org>
Cc:     "imx@lists.linux.dev" <imx@lists.linux.dev>
References: <20230112193442.1513372-1-Frank.Li@nxp.com>
 <HE1PR0401MB2331C19D7474E6EAA4DC29C888D69@HE1PR0401MB2331.eurprd04.prod.outlook.com>
From:   ALOK TIWARI <alok.a.tiwari@oracle.com>
Organization: Oracle Corporation
In-Reply-To: <HE1PR0401MB2331C19D7474E6EAA4DC29C888D69@HE1PR0401MB2331.eurprd04.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TnlRWmYxOC8wU1BUVnVDam9ZNXFiTVV5cDdkVEZKQmhsYnRHQjNLNUJuTFF3?=
 =?utf-8?B?UW9CaWRKang0TGJRdzVDdVVjVVhlTmU2U1F2Sms2SU9SakZQenBvT3N1N3A1?=
 =?utf-8?B?aTEwQkI3RER5YUtBUWFBdW9KWnZXSmJNWlNNd2VwVlQzS2RsVnI2Zm45RzdR?=
 =?utf-8?B?NkJhWVg4TDZkMVozL2hmV3J6Z0J2ckk0VXlDQzBPUXJqT2RmMW0rcW95UnNG?=
 =?utf-8?B?ajQraDNGSHhNVEVkZjhYNnY0citIcmREdW1SWURoVkVCVUhFdmQ5ZlpUS05R?=
 =?utf-8?B?TnlndWpna1lXS2lndWl1cjFYZWlnUjkvU3lVOW9wRmtVMnlndzc0b2JOazd5?=
 =?utf-8?B?QTJaNnRRMzVqb0lHQ1ZQNERNS3g5MTkxNkpaT2JENC9QWDM5TE93S0tndFhl?=
 =?utf-8?B?VDVWc1NMUEhmZS8rYyttVlF0dDQxamNZNit0R2lqUlRtaG95TFFPVzBkeDdZ?=
 =?utf-8?B?bForaXZybkdCVTYvS0htM3psQVNyTlRuMFp5MWRjUTkzWUFHN3krZ3B2Sllr?=
 =?utf-8?B?T0RpeElnSjE3a0krbG1teWMyRUZleU1haGx6OXRGRHFNVThFYmRsOFRBc243?=
 =?utf-8?B?bm4vMzd5cnJoZEhSUHpxWkhJb0gvWFlzQWdnajdTZWVvNFY0R3dPL3lCRW5U?=
 =?utf-8?B?aDZtUWlodFdCb0JTMG4xQVZOMEE4RWpGWml6VFY4Z2FYZ09DMFpLQ0N6MUxJ?=
 =?utf-8?B?cjhtRnFNcTEyMVdvL29uckdIMjZNWVhOS0NPU3hvYmxSeTZuMVUzWUUxS1Zo?=
 =?utf-8?B?NzR1LzZvNVJtYVA3RFFIMWV1THdwK3hqb0JFT09IcithNFdIVTRscldUd1lS?=
 =?utf-8?B?S2F3WVdJczNNNktrWUwveWRwSlloTFptdXFxa3dRTzVsOEZXN09VcnI1UzNB?=
 =?utf-8?B?QjNVTEE2dVI2THo3VnJFSWhzSysrbktUUnB4VTM0aWdXWDdEWDRTcXIwUTlH?=
 =?utf-8?B?Rmg3VEpqWDF4S3ZvZWh5MU92cVJlcE5mK3A4cjdPbkJRUlRSbEh1K014UXF4?=
 =?utf-8?B?bDBhL0tUdnMxTFpPaXRnZW5NeVUyZzdmUWV5NVI2MTFoR1VPM21ndHFCRFVh?=
 =?utf-8?B?T0lUbmRQUFBxTGkyTnpFUHY4eGF1bGJuaWdhS1YwZit4T0NYL3dZeDArc1pv?=
 =?utf-8?B?TGw4RVpGc0NPa3V0Qk16STlBeXUzeFJxTUdTRlN4RW90TGxnQ3pGMmRjRWxN?=
 =?utf-8?B?Wmc5WVppY0lkTlF4alBqQXdtakVNeUZsSlFQSUJNSURodW96Z0p5QjZaSFNQ?=
 =?utf-8?B?VHpzWHlxTEpIRER3dmZwUDhYd05iVkh3cjF0NUVvS0czaDI3cUwvRFZ0Y0RI?=
 =?utf-8?B?N2crV2h1bFRpZDU5eXlPaVZNN2RuOW9JVkxxeGtzSzI5KzdONXgzbTBPUzRP?=
 =?utf-8?B?QlZEc3F4aDVKWDZOL3hFKzVGVWhKSmZ4K2tncHoxZlhBNW93TW1zSG91NytI?=
 =?utf-8?B?MXk2QmE5MEd1SFZLUHV2dTNzbnd0bG45ZjBnMWtWTHdmQmo5ZC9BVWRUR21J?=
 =?utf-8?B?dU9SekZlWk91L0tMNGNkTTlVdjFuRHhhajA5N2R1d0RTOUtMZWhQVVlhd3Bq?=
 =?utf-8?B?MGM5TGQ3QzI0eFVYbjdLZUVNb1AwUTYzTzZ4Y1VYZHdoRklFMk8vb3BqTTFU?=
 =?utf-8?B?UzdmNFpXL1BMM09YWFJXbyswaFZRczFvWHJsMFpRYjRsMEU3ZzF1UHBhaFNB?=
 =?utf-8?B?eW9sU0xJNHFrVnpPcWdhVlI2NXRoSmVPRkp0VVREODNqSnVyc2J3T01QK1VD?=
 =?utf-8?B?S29UbWlTQXVTYldRVzZ0T081SnhqcmZRZG1TOUkwQnVCelMrS1RDNzhRY0pU?=
 =?utf-8?B?L2p5WE5UN3o0ZE1TajlJZ3c5WWNuMW9rN21wWnhSME9TUyswQmhHUWI3dldy?=
 =?utf-8?B?Z0tVNWYxNnYvTlg3OURPOGVaa3ZKck9lY2tJV1NDT1YrMGxFc0NTYTUxa3VV?=
 =?utf-8?B?azNrWGVLYXAxVy9MNlpLQzdzU2RuWXpIbWxUYVp6ZWx0Z21QTXBGTTFSYk5U?=
 =?utf-8?B?MUJTNVJ3QTR6OG52RkJiODRMZ1pYVmRaZDc3S2xUTHA2UDUwQ0tNQ2JzZ0Nv?=
 =?utf-8?B?cWp2UFFFOFZLaVFRS1p5SDBFSWVuSUR2UTJaRnJpUGtRckZpeGZXOTdza3Nv?=
 =?utf-8?B?bWF1bFVZWmdYZ3FKcUZ4WnNET2VsOFN0dzk4NUk0aHp6ak1mbVMxN2plR1BJ?=
 =?utf-8?B?Z3c9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?utf-8?B?ZzR6M0dva0NZMWg1SzlQM0dGYWhzdGRRWHBXR0Y1YUtmcDdaVmNnbGFySXdm?=
 =?utf-8?B?VGpnRHpHcUdZZWhBMkIzWEUwUkRDR0ZTSTRyczFjTU5yS3QvTTBWTXZkQnVz?=
 =?utf-8?B?QlkvbmJieDJ0VUNmaGVNRmlwUGFvRzVvY296RnphYW1FTTAvVUkxb0hOdUlK?=
 =?utf-8?B?c1M5UmlyUHhUNFF2QTdqMjAyd3hlOTVsQkJ0S2NRNzZ6bDlWaFlpaGxIemRy?=
 =?utf-8?B?aXdOenhKSm1aZkdSclNqenB2R2dZYVRTaTc1aEgxcXkxY0hZTHZlU3cxUkdG?=
 =?utf-8?B?UmoxdGFUc2NwOUk5RkpneEwxRy9BN3dPbzRYbHZpdk5VZmhPOVZBeFhibEgz?=
 =?utf-8?B?ZUIycjB4S3hVbFN0bWlrTmNlMzVwQVZPb0FNbnc3UkY3dmx0Y3JjcTB1dlFu?=
 =?utf-8?B?MEYweUdMWFNpOFpySHN6SGMyM1hwSFRoTlpLd2M3ZVByM2hIR0JncDZ1L1hk?=
 =?utf-8?B?NnRVZi8xcGZ1MHlkbkhNRmpiQkwyYkFPeEF4YWthblJyTHlsamhGOWgzeXNu?=
 =?utf-8?B?TUdoM2JNL1BnOFI4Z3kxU2NHQ1VhOWMrc1dGTlJiSm91SU9vcW92YU9VZE9V?=
 =?utf-8?B?NkREVmgzYStISFY1VThhYUJ1R0tQWEhYTHpsUWtPZ0JNUURHdHNVVStpVjdt?=
 =?utf-8?B?NC9vZHFNd1JGOW1BVHV6Ylp4VVRBUzRPYStDRDFqd3RRS1RxeDlSdHB1Q1Ju?=
 =?utf-8?B?WGd3S0VUdDcxYTRPekMwc1FqbXFjRXZvNk5rM2JZSTlEZ1pIZXdJa21tTU5u?=
 =?utf-8?B?Z1pyVmNDV2RwRC83blZiL3E3SkoxZ2kwbDd4Ukh6SVFXRXZFTS96Q3crbmtk?=
 =?utf-8?B?MU1kbWFlalJPWFRTdGY2cVpHTnZ1dGFoMHF3Q21hTVpFdHhxTk1qZGhBbDg1?=
 =?utf-8?B?RTN1L1VlSzVNeG5aWFQ3WWxIdnEvMTZmVFpiTDNyOXFiTHVaTktPYThBZWdX?=
 =?utf-8?B?UVBRZWRnSU5vZllCNUl1RkY5SkVjcENvVGd4U29yeEJJWHptNGdlYThoUEdK?=
 =?utf-8?B?R0tIa0RoakRnZVBrdXJ1MVVnWXNlU0N4dE1QdjFGM1ZaY2x3NzV0WmN4dHZt?=
 =?utf-8?B?SjZpdkU4azFFUjVXM05hM0J1bHhyZHdaNHJ6dkhDelRuSzdsaXhqM1NkQXJJ?=
 =?utf-8?B?OUk2WnJEZzRxT0RscmVIejdjQzN1UllERy9pS0pLcWdtR3BMN2hiYlZvd3Y5?=
 =?utf-8?B?YVoxOGR4Q0pEWjZ5WmtrTjFQa3pJMlNCY1FrQTROOWVNWHhEQUF5QmJGTUZ6?=
 =?utf-8?B?TnZWTnFYV3hzN2tZZW9vQlhha1M1Z3V0QjlRcU9NaVlaWHRCRzRVTjl6eGdt?=
 =?utf-8?Q?mlBtTHRRBqgmA/k4hiJvLhmsZz7411GWZw?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: afc8216c-be80-40f9-027c-08db05a5fc5c
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4581.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Feb 2023 05:17:46.8594
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: EYazspEt8PGoQGN4OGHE+YPzEPlDFf+CKUciQBYzFqgzNl5txY4nTh20ctUyZq1oYrdnZVThTaQCfAVA9hWMnARIH7tGQyvtpn1VbVgOelc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR10MB4704
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.122.1
 definitions=2023-02-03_02,2023-02-02_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 phishscore=0
 malwarescore=0 mlxscore=0 adultscore=0 suspectscore=0 spamscore=0
 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2212070000 definitions=main-2302030047
X-Proofpoint-ORIG-GUID: a7gw57zwRRHEYtfEdbxTYiUdjRk7HTqm
X-Proofpoint-GUID: a7gw57zwRRHEYtfEdbxTYiUdjRk7HTqm
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

         { .compatible = "fsl,ls1046a-pcie-ep", .data = &ls1_ep_drvdata },
+       { .compatible = "fsl,ls1028a-pcie-ep", .data = &ls1_ep_drvdata },
	{ .compatible = "fsl,ls1088a-pcie-ep", .data = &ls2_ep_drvdata },

can it be like this for better readability. ?

Thanks,
Alok

On 2/2/2023 11:11 PM, Frank Li wrote:
>> Subject: [EXT] [PATCH v2 1/1] PCI: layerscape: Add EP mode support for
>> ls1028a
>>
>> Caution: EXT Email
>>
>> From: Xiaowei Bao <xiaowei.bao@nxp.com>
>>
>> Add PCIe EP mode support for ls1028a.
>>
>> Signed-off-by: Xiaowei Bao <xiaowei.bao@nxp.com>
>> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>> Signed-off-by: Frank Li <Frank.Li@nxp>
>> Acked-by:  Roy Zang <Roy.Zang@nxp.com>
>>
> [Frank Li] ping
>
>> ---
>>
>> Added
>> Signed-off-by: Frank Li <Frank.Li@nxp>
>> Acked-by:  Roy Zang <Roy.Zang@nxp.com>
>>
>>
>> All other patches were already accepte by maintainer in
>> https://urldefense.com/v3/__https://eur01.safelinks.protection.outlook.com/?url=https*3A*2F*2Flore.k__;JSUl!!ACWV5N9M2RV99hQ!PIKq0-lb3ccjBcAZNFo48kfWvbGPlh7_Wiws1olaNdTthUzhcGFrpLbw3vOMDaeops5LkajDO_y_vGPyuhE$
>> ernel.org%2Flkml%2F20211112223457.10599-1-
>> leoyang.li%40nxp.com%2F&data=05%7C01%7Cfrank.li%40nxp.com%7C29d1
>> 5c05d59346e552be08daf4d573e5%7C686ea1d3bc2b4c6fa92cd99c5c301635%
>> 7C0%7C0%7C638091494850726163%7CUnknown%7CTWFpbGZsb3d8eyJWIjo
>> iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C30
>> 00%7C%7C%7C&sdata=0wnHTyuX%2FyLAiioKNvlFbBM83nVyF%2FCdhqsEmV
>> f2sI4%3D&reserved=0
>>
>> But missed this one.
>>
>> Re-post.
>>
>>   drivers/pci/controller/dwc/pci-layerscape-ep.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c
>> b/drivers/pci/controller/dwc/pci-layerscape-ep.c
>> index ad99707b3b99..ed5cfc9408d9 100644
>> --- a/drivers/pci/controller/dwc/pci-layerscape-ep.c
>> +++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c
>> @@ -112,6 +112,7 @@ static const struct ls_pcie_ep_drvdata
>> lx2_ep_drvdata = {
>>   static const struct of_device_id ls_pcie_ep_of_match[] = {
>>          { .compatible = "fsl,ls1046a-pcie-ep", .data = &ls1_ep_drvdata },
>>          { .compatible = "fsl,ls1088a-pcie-ep", .data = &ls2_ep_drvdata },
>> +       { .compatible = "fsl,ls1028a-pcie-ep", .data = &ls1_ep_drvdata },
>>          { .compatible = "fsl,ls2088a-pcie-ep", .data = &ls2_ep_drvdata },
>>          { .compatible = "fsl,lx2160ar2-pcie-ep", .data = &lx2_ep_drvdata },
>>          { },
>> --
>> 2.34.1