Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 30 Jun 2023 15:26:47 +0200
From:   Simon Horman <simon.horman@corigine.com>
To:     Hariprasad Kelam <hkelam@marvell.com>
Cc:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
        davem@davemloft.net, sgoutham@marvell.com, lcherian@marvell.com,
        gakula@marvell.com, jerinj@marvell.com, sbhatta@marvell.com
Subject: Re: [net Patch 1/4] octeontx2-af: cn10kb: fix interrupt csr addresses
Message-ID: <ZJ7YFxFOKC1NLMwa@corigine.com>
References: <20230630062845.26606-1-hkelam@marvell.com>
 <20230630062845.26606-2-hkelam@marvell.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230630062845.26606-2-hkelam@marvell.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?335AXMIPEsHXXw4RkRAUXSB1O8/ev/xHlwpU79Wu+ttrGjyNSORfo+ijbgeX?=
 =?us-ascii?Q?kTqW0Ngl9lXMLDHrsdQXID57YvjD58DVqan6qL3NXQF+rcudPfSn2DAOL9uN?=
 =?us-ascii?Q?57iwhsPWcnaesncXGZkjvO5K12Uym1cFDKXuYcrqkXxIk0TxRvnDQ8lCWGkv?=
 =?us-ascii?Q?NkAFU0Qotjp4E/6gPitn/O2P9x8l9B5sitDFxkI1ZqRq/oOfnQ2oWSuCQTGn?=
 =?us-ascii?Q?l2lPS6giiU4b9I5orcEa9/t7wEDEzmL3FY8vx5IzQU5r3XcgWsY0XQhIZKdK?=
 =?us-ascii?Q?jgQbPx6MiPZtgBrSVWLqJ1VWutnVAadn2w4/apVqzyDYbtQGAaCHR/OcyR6Y?=
 =?us-ascii?Q?jtYzYdE5Fy0ulGNZOk25edoUTNAtdsIoVy4QwLA9M56a6B3nYPo+SRcu+rzR?=
 =?us-ascii?Q?f/Gx5u8maee8hbEbS+JoU86XqNQrqCI/L/XuTG4FNByqNhD28O/2e0i4+lwn?=
 =?us-ascii?Q?Lj9qtjI6RzVK6r+eBd3aQRxivJaeAPgx/c0x/h0cwxTBK8KDB5mJR6a/RHEA?=
 =?us-ascii?Q?zfgvdgHYdU+fayzqMlyL1jxCF9z8lwFXIbLAiwo6rCGzej1BmdoUNjxsb96z?=
 =?us-ascii?Q?062peHK9u1DVclTm6qwaTCAB76ZK2uCL60FbAKZL+QzVyAKx0LUw44/EmlAb?=
 =?us-ascii?Q?2zCAbTCsjjyIIu6ziiw6FScBiBtx9xbAbKJ132E10E3ivQcVb7c75BDrmCgA?=
 =?us-ascii?Q?xD5uR99iurA1gM9E6YbyvUIDLYaDSYwVsC0v3Q8UskToTsS4/zYEtK6znmw9?=
 =?us-ascii?Q?wbTgAQRvMhUwpj27+AY/Z0ehmr6oZoX7BBbSSsVvLPYmlqmxpJPWHTHBTHal?=
 =?us-ascii?Q?tFVUR6YTAAVlt10oMeAWFuU0wFBor+ikd0+MGq7pGj8PX2mQQR6zbm5Dtq7K?=
 =?us-ascii?Q?WK1DfRAwenZNzMfnvareX0AlUyQcVmzrFKOX3svAsJUc/yFHZ/5ocq8eovFo?=
 =?us-ascii?Q?L3QQkw8534/KG494L1QiWwVZoY+/GPan2OX7gPvplCRJcHC9Z8bK8D75xuEp?=
 =?us-ascii?Q?cwI8uWaxPrSDL5phIgW5T9+d+f+2xIhvPYG2LCnrCyuyWQgV3P8cYrWnJ7oW?=
 =?us-ascii?Q?Sn/OlFBLcnd/hG0lyGCPjLvnuuS33d0G5u+CnWGhBXUSgcADMrK+ycrzDAfe?=
 =?us-ascii?Q?Hv5YM0Z1sclKWuKof8VHatK+VdhEX+W7Y0F8gcfdIxM1NMRC7Q3Cz6BPMaLt?=
 =?us-ascii?Q?eURDAyfURduhWhdzS+fKmlqPu+kArsJ2LPIwFz/JockorpBOlt226MT9/p72?=
 =?us-ascii?Q?fMAlZO0aOy+cOyl/8Iwa5jCxIG+9u893FDw9dfddR20iUIDaTUO0WY/OpMUJ?=
 =?us-ascii?Q?NqMfTiw4fhbqQzVNvXl/KMJ+fk30lXOBUXCgu1uxJVDcHlg7cRDr54CdMwA8?=
 =?us-ascii?Q?6ZR9FLOkGg0K3VOtQKGQoSsDh6tJyu3jGq4okWYIs8e6WrRwT+Y2M1sFgdAM?=
 =?us-ascii?Q?TeEXxY72QUug1+gKKWchEz/nx9cvCyeCpO6wb2PiGmmDtPfy3lQEzA9BFzBr?=
 =?us-ascii?Q?uCnpGeJxaM+pbhlj1sPbto5ePUaOABtp63TEVjL5VkwoWW33OUQGr+NEW9iM?=
 =?us-ascii?Q?m0qocbZmWi+IG0VK1StxEa0At1VTHhsjwH8B2QoZSKB9FpYHFkmdUUKtI5zQ?=
 =?us-ascii?Q?5AW+Zyl7xqxJRzZaGjuQXLIjDn+BlKIUYUNh7d+p4mN4nJghqlHkuxTjAA/T?=
 =?us-ascii?Q?rBmFNQ=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 99ebfd83-4309-49c7-44d3-08db796dabb3
X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jun 2023 13:26:54.3647
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: QAVI8BY8duMg4xLxY068X+msPGsHnfKx6/MqCBdMpAZNZvWYnFip9CQT0Hx3Z99IIG9NjV1eOp6Fal4iERDX/7keXvZCnfUTtq/PEg8WcAg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR13MB6100
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Jun 30, 2023 at 11:58:42AM +0530, Hariprasad Kelam wrote:
> The current design is that, for asynchronous events like link_up and
> link_down firmware raises the interrupt to kernel. The previous patch
> which added RPM_USX driver has a bug where it uses old csr addresses
> for configuring interrupts. Which is resulting in losing interrupts
> from source firmware.
> 
> This patch fixes the issue by correcting csr addresses.
> 
> Fixes: b9d0fedc6234 ("octeontx2-af: cn10kb: Add RPM_USX MAC support")
> Signed-off-by: Hariprasad Kelam <hkelam@marvell.com>
> Signed-off-by: Sunil Goutham <sgoutham@marvell.com>

Reviewed-by: Simon Horman <simon.horman@corigine.com>