Received-SPF: pass (google.com: domain of linux-kernel+bounces-94741-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: <Woojung.Huh@microchip.com>
To: <Parthiban.Veerasooran@microchip.com>, <davem@davemloft.net>,
	<edumazet@google.com>, <kuba@kernel.org>, <pabeni@redhat.com>,
	<horms@kernel.org>, <saeedm@nvidia.com>, <anthony.l.nguyen@intel.com>,
	<netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <andrew@lunn.ch>,
	<corbet@lwn.net>, <linux-doc@vger.kernel.org>, <robh+dt@kernel.org>,
	<krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>,
	<devicetree@vger.kernel.org>, <Horatiu.Vultur@microchip.com>,
	<ruanjinjie@huawei.com>, <Steen.Hegelund@microchip.com>,
	<vladimir.oltean@nxp.com>
CC: <UNGLinuxDriver@microchip.com>, <Thorsten.Kummermehr@microchip.com>,
	<Pier.Beruto@onsemi.com>, <Selvamani.Rajagopal@onsemi.com>,
	<Nicolas.Ferre@microchip.com>, <benjamin.bigler@bernformulastudent.ch>
Subject: RE: [PATCH net-next v3 10/12] net: ethernet: oa_tc6: implement
 mac-phy interrupt
Thread-Topic: [PATCH net-next v3 10/12] net: ethernet: oa_tc6: implement
 mac-phy interrupt
Thread-Index: AQHab6Ob/tWfers5YEGx5SyM6BefPLErX8qw
Date: Wed, 6 Mar 2024 23:42:16 +0000
Message-ID: <BL0PR11MB2913A429E53B85ABC67F9657E7212@BL0PR11MB2913.namprd11.prod.outlook.com>
References: <20240306085017.21731-1-Parthiban.Veerasooran@microchip.com>
 <20240306085017.21731-11-Parthiban.Veerasooran@microchip.com>
In-Reply-To: <20240306085017.21731-11-Parthiban.Veerasooran@microchip.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?5wttjtQUtZvFN9CJMF/TaQBx+u+Y1azyP7qOPrljaexQ6DETwlEE3JFHkw9+?=
 =?us-ascii?Q?P1Km0uj5uPWCmbVvfaJU+UfUXWZWe1BkEqJbVwOOcTLfVQ1uTIykW2WYnBPl?=
 =?us-ascii?Q?K/ryQC+uskY566gK+tms1EouAF7gtIxTfy4q9ugmli0dnIW3lH0la+QU2gKA?=
 =?us-ascii?Q?qvTYM/1rhnWqYQArHzrdz1ROnpis4p5uj8J6ejVXrnH0h0BzLIfxk2MWaYmr?=
 =?us-ascii?Q?aPOSkyNpVxC/wtf9gAr9yQvKGJNBQg2kVYhtRWlyvpqqI+ou8wEudp0tmSmJ?=
 =?us-ascii?Q?kUZQ+qU3VsBi7k3zOXjqgKqCpDNAoy88OZtRfQVMXmVqDYs2wxgw0/1PNW89?=
 =?us-ascii?Q?69VWTORVpwLFm+XB6DWZqYomoux9Crg0V8XT/xeEg730xEK7ino1gpOEBwQE?=
 =?us-ascii?Q?UJCaDCkitoic7vL6bNv1oDVGp4dkBz1msf3Ry785ZDHxLAh2fa/uocNbGJCf?=
 =?us-ascii?Q?l9+oL+i7UIUlvw/3BSdsmMvUhc0HXgaX3YPIRuvVRmjIwBIx/srZsFZ1kxcC?=
 =?us-ascii?Q?LQog5N1uX6BZ7ISAKk4nx11TFz012QDdW2XTn1v46kjqSQkRQkmap5FCV7jB?=
 =?us-ascii?Q?8wHGjpeZyKUvaeihnl6IKiyFBqGHCX84leXeq/D+gU/WeOxVSGiKbsm+I6AL?=
 =?us-ascii?Q?D0k0yiCKhZYwD/nRGGJ0I2IutJLWBFlL4yKMHfcIjVGxscJFXTIG00h+QmSv?=
 =?us-ascii?Q?G+M2vBtk496o3o379HNnMx2e+VunXEBtEC6C22ofF6a71agpahQrVT914fOM?=
 =?us-ascii?Q?q3Nwf6yp3YVoqJQkllDWttnzQXT+dlepS3em3WdA2qWKaqNQW4F6tLtuIFnB?=
 =?us-ascii?Q?njP8t2il7l1S8lU/KW61WHZmCkZoFFTqjPd9+IPlmZmGBi4jY31l/FnVdUic?=
 =?us-ascii?Q?BBlT77zDJhxpBYx6FXCq0pMdQ0KNICRgMZtrnZZNez3XzmRecLwAF6T6i+KT?=
 =?us-ascii?Q?7RvmIodFqy7ayqHXAemx13WXfSljaT4AFNKleTc5MH6vn1q4IJ36OBz/Fa4A?=
 =?us-ascii?Q?cyfQK8BwOALyR3OsSIqX7NZPNgYtbnyPa9NLQCr4Vwe5qtwyAZBBYxj27k+p?=
 =?us-ascii?Q?yelaFw8p7KqqilQDs+hGGrIC9DxVv1NwhhA7edeMJ5FMLORvVu06xSTEBTBf?=
 =?us-ascii?Q?mZN0Vz1BaD2q7KRu1ZzcVCFqeL1qaqJB9w104T2KGMFICOXOQeey6DZtVYBJ?=
 =?us-ascii?Q?R+4FLYA1W/+WewIqsctrDVxydd4G+xV51s0FvLPtrYp18CCeveHDP8ASJWV9?=
 =?us-ascii?Q?Im7JoVfFWwKn9+b19wHCZndr2SiusCnguKz4eiOX8GOHu7QkMhKBVGQ62JGW?=
 =?us-ascii?Q?GWqE4XN3BtyOcei4qs2UIjIzcCs9EbhstNyiOZ9DJgCHYtYQDI67uESUMuI1?=
 =?us-ascii?Q?ShHFQ6MgMYYBN2YHDST9wLkxPub8PDywq3mdt1bOWXKWdO3x5CskhC7Hzfyx?=
 =?us-ascii?Q?eTOtvm0ORgGLl/0Yi4uFIEsvgzslcgldJiQnKB4rsN8DQG69bKPCf3fwWWkn?=
 =?us-ascii?Q?JKaI3lFHHAvrIkkgJotqLRUNsLz/8Or10ava+x5cHYGnOdpkczjONKHlJp4r?=
 =?us-ascii?Q?h6yTPTUI9VKSJ3b6GthFWQGg/zNt1p4D+9Yu2m+f?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BL0PR11MB2913.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 60f7df91-5319-4fb0-ef20-08dc3e370e4e
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Mar 2024 23:42:16.3313
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: YnX9L5d5tHHiaWgtz/WQw9o4IA0xoY1jLoEgTq1qC3pGKXB+EOtOjuU/kW2mYrVS+a5au9T7lwygPyXK6IjQQjJwix4U/lgpQ5yhChN3+AI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4598

Hi Parthiban,

..

> +static irqreturn_t oa_tc6_macphy_isr(int irq, void *data)
> +{
> +	struct oa_tc6 *tc6 =3D data;
> +
> +	/* MAC-PHY interrupt can occur for the following reasons.
> +	 * - availability of tx credits if it was 0 before and not reported
> in
> +	 *   the previous rx footer.

Per description above, it may be typo of "the previous rx footer"

> +	 * - availability of rx chunks if it was 0 before and not reported
> in
> +	 *   the previous rx footer.
> +	 * - extended status event not reported in the previous rx footer.
> +	 */
> +	tc6->int_flag =3D true;
> +	/* Wake spi kthread to perform spi transfer */
> +	wake_up_interruptible(&tc6->spi_wq);
> +
> +	return IRQ_HANDLED;
> +}
> +