On 25/08/23 13:22, Roger Quadros wrote: > Siddharth, > > On 25/08/2023 08:42, Siddharth Vadapalli wrote: >> Roger, >> >> On 25/08/23 00:39, Roger Quadros wrote: >>> >>> >>> On 24/08/2023 21:24, Tom Rini wrote: >>>> On Thu, Aug 24, 2023 at 11:34:29PM +0530, Siddharth Vadapalli wrote: >>>>> Hello Roger, >>>>> >>>>> On 22-08-2023 17:43, Roger Quadros wrote: ... >> > > This doesn't sound like a clean solution. > All hardware specific details (like reset ready-time) should come from the > device tree and not passed around in PHY APIs. > > Why to do such an intrusive change instead of fixing the am65-cpsw-nuss driver > to properly use the PHY uclass driver model and encode the required delays > in the device tree? Encoding delays in device-tree and using them is better. Kindly ignore my suggestion. > -- Regards, Siddharth.
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workarou... Siddharth Vadapalli
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Work... Roger Quadros
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: ... Siddharth Vadapalli
- Re: [PATCH 1/2] net: ti: am65-cpsw-nu... Roger Quadros
- Re: [PATCH 1/2] net: ti: am65-cp... Roger Quadros
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workarou... Siddharth Vadapalli
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Work... Tom Rini
- Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: ... Roger Quadros
- Re: [PATCH 1/2] net: ti: am65-cpsw-nu... Siddharth Vadapalli
- Re: [PATCH 1/2] net: ti: am65-cp... Roger Quadros
- Re: [PATCH 1/2] net: ti: am6... Siddharth Vadapalli
- [PATCH 2/2] net: phy: Change "PHY not found"... Roger Quadros
- Re: [PATCH 2/2] net: phy: Change "PHY not fo... Siddharth Vadapalli
- Re: [PATCH 2/2] net: phy: Change "PHY no... Roger Quadros