Date:   Wed, 15 Dec 2021 20:48:55 +0100
From:   Francesco Dolcini <francesco.dolcini@toradex.com>
To:     Andrew Lunn <andrew@lunn.ch>
Cc:     Francesco Dolcini <francesco.dolcini@toradex.com>,
        "Russell King (Oracle)" <linux@armlinux.org.uk>,
        Joakim Zhang <qiangqing.zhang@nxp.com>,
        Philippe Schenker <philippe.schenker@toradex.com>,
        netdev@vger.kernel.org, "David S . Miller" <davem@davemloft.net>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Fabio Estevam <festevam@gmail.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next 3/3] net: fec: reset phy on resume after power-up
Message-ID: <20211215194855.GA114534@francesco-nb.int.toradex.com>
References: <DB8PR04MB679570A356B655A5D6BFE818E6769@DB8PR04MB6795.eurprd04.prod.outlook.com>
 <YbnDc/snmb1WYVCt@shell.armlinux.org.uk>
 <Ybm3NDeq96TSjh+k@lunn.ch>
 <20211215110139.GA64001@francesco-nb.int.toradex.com>
 <YbpDKH6CaHzm119W@lunn.ch>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YbpDKH6CaHzm119W@lunn.ch>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?jUNSJT+1cdpVADIavVkZES7ASxeoKxiKW+vv/6RgWZxEjEtwNd5iW526Xcm4?=
 =?us-ascii?Q?IVu41d1+eOVwCmJABuPTjhe5C1ea5+ocRdOU7IWzcQObIoOrQIRbOqRlUcx1?=
 =?us-ascii?Q?biXeTBTQD/AB7/CFgYnl68bGPPGkvPwReIIDI1F2DiVyFPNdSe6vrL6gSY4l?=
 =?us-ascii?Q?uBhDngYsK6JOsQoWiITMQ3vr1TFIvlf3+qzCoatC5sdsvDD7QpEhLBP+vqlV?=
 =?us-ascii?Q?vOR9CqzhRxmEej8Q5aj/+c30/ERg/GrH8sIWoQ94zODDMURLxwN86P92if7u?=
 =?us-ascii?Q?KtHGVFLNyRCuIAq7I5Nf6X1CPXUNCn0ome/SeRbUzuFxy23hobrAW3lO+Irt?=
 =?us-ascii?Q?rr5R5FPqhvMjgagP3CBasjlR62HoLk9BqPWTpPzXdhVxolQFQ7aHILAiXxhN?=
 =?us-ascii?Q?JorxC+qRroP6YMbKOF4kkspmnm5fTx49ZS+mDwFThbJZtx0OlMVOHZA7yxkw?=
 =?us-ascii?Q?ZC3uiiwzwsUOHFoVv8hmp8TnpIUb53szVe+/Au2736k1wHoCLKzWc2rvSO8h?=
 =?us-ascii?Q?NqbUN4JPHGoVj9KkK/kbzGOcHhAfC5leu66tRDNRpzn4Z/dJwlYkV8I/uNXS?=
 =?us-ascii?Q?PUYMboIF4uFcfolbvUxuzLLH4k9hULWryUM9Vwp+0NWn8vRwFYzm1HXsuaT3?=
 =?us-ascii?Q?bTxuykB5Q4UEV2Gmh4hNTHN+V/0QwgxMcD3OAtNq4FTO3L8UnBa0DMq4GIep?=
 =?us-ascii?Q?kmqqd3Duef9+nc8nYCY42qzaQL8jaXrSTihsAKVW4qyZ3PbwtVvOww61sBhh?=
 =?us-ascii?Q?E1dCchc7Ek/fk+/ZG0n8lmpOMAPdNzxJLdvdqVxtfsMHeQCB5GL3T/kB1FM+?=
 =?us-ascii?Q?qe3jeTckg8U3gwClbXpbZyalYlUwHCoytM77oTJ2vsuL8d8kZhbTZsnmEOsj?=
 =?us-ascii?Q?956r4ptedc3zI0Wt2W9TSW6/yaaXynbZrtrupauDVLLyLR0Lf9UbalGkKWnl?=
 =?us-ascii?Q?G410n8GBdg9DojsLHmEy7ISFS2r1zouPZCkyttnRJNX4+rZ1EppSoV4kFSg1?=
 =?us-ascii?Q?06qm2v4ea5CshKQF+AFFNXzPN3htHjdT1SIidxEiLXQBIRV9vFQ5wo+RN3ED?=
 =?us-ascii?Q?75OmnKwjX3a3TcLXpWYrI48xXv1sCIe7dt6DPt23N0811Pkm7K3TaZ5dAqYf?=
 =?us-ascii?Q?Qd6QEE6hX1qrxjL3B0/g5reQxtAfHxrOEdos9WR2k+ExT082QVrUZShEV/Ts?=
 =?us-ascii?Q?p3Pa6AlBw93RaXMUP0OdbhuCaoAq/F5GcsnJxB3Mzf6Bt0Y/gDhrPpXoXE4a?=
 =?us-ascii?Q?vM55LvA9SEA8+4PJGQU0grSrnuJcTbsvfhOiTUS2CcKtYo53KBunhkl+/kSc?=
 =?us-ascii?Q?LJV1qanWkrl5po6JpGwAm8f+rlGs2qH8GPPopSXKuuRlh71FVNg5DgaubieG?=
 =?us-ascii?Q?giHkl9i6DAbj+RqzwZGxrBqz9lDMtWzxdZOsQFNtFcbBN7UkoWbNeCrWyZ2r?=
 =?us-ascii?Q?9O85qkzovRE2TpUV/8QxrWI3Ho4G7zkT2ihzcBfhV51MCeGfNdIGC1rrnFfo?=
 =?us-ascii?Q?zt4ht0Dc81/Zlic692GRUmXGGPps7CN+ewP6KlqtBQXwxo1SXVo2R/p3P7fN?=
 =?us-ascii?Q?DT8zqmY9SbB4r5ChXzqBWfvJok1HMtLeBcuEXxutw4ARgkQ5YhhdyOtd+FVC?=
 =?us-ascii?Q?HlOabqHQz1cILVvVQGPLjHgb6g4nZ1mdqdT7+M1zFtbNDBrD95Mlz6q9ZKFt?=
 =?us-ascii?Q?IqZt+Q=3D=3D?=
X-OriginatorOrg: toradex.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4b1a0e0f-e52f-4e5f-e4d7-08d9c003f71d
X-MS-Exchange-CrossTenant-AuthSource: ZRAP278MB0642.CHEP278.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2021 19:49:11.4007
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: d9995866-0d9b-4251-8315-093f062abab4
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LOwrpdZt3qbAL5r3beOQ7X88jA2nMsASjSIsRJiz04arlCocIHyXi0G6oZoU8Ppd4M/WienrisXEGHkBe8mh6LZIlNxAWrKFIljl8bQ+M3k=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: ZRAP278MB0416
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Dec 15, 2021 at 08:34:00PM +0100, Andrew Lunn wrote:
> > The reason is that the power consumption in reset is higher in reset
> > compared to the normal PHY software power down.
> 
> Does the datasheet have numbers for in lower power mode and held in
> reset? We only have an issue if held in reset when in low power mode
> consumes more power than simply in low power mode.

The numbers for KSZ9131, Table 6-3: Power Consumption from datasheet [0]

61.2mW in reset, 24.4mW in software power down (3.3VDD)
40.9mW in reset, 12.5mW in software power down (2.5VDD)

Francesco

[0] https://ww1.microchip.com/downloads/en/DeviceDoc/00002841B.pdf