[PATCH v2 2/4] net: ravb: Add optional reset deassertion

[Paul Barker]

On 24/03/2025 01:08, Marek Vasut wrote:
> On 3/19/25 1:03 PM, Paul Barker wrote:
> 
> [...]
> 
>>   static int ravb_probe(struct udevice *dev)
>>   {
>> +	struct ravb_device_ops *device_ops =
>> +		(struct ravb_device_ops *)dev_get_driver_data(dev);
>>   	struct eth_pdata *pdata = dev_get_plat(dev);
>>   	struct ravb_priv *eth = dev_get_priv(dev);
>>   	struct mii_dev *mdiodev;
>> @@ -648,16 +653,32 @@ static int ravb_probe(struct udevice *dev)
>>   	if (ret)
>>   		goto err_clk_enable;
>>   
>> +	if (device_ops->has_reset) {
>> +		ret = reset_get_by_index(dev, 0, &eth->rst);
> 
> Can devm_reset_control_get_optional() maybe be used here instead of the 
> flag ?

The Ethernet device tree nodes for RZ/G2{H,M,N,E} also contain a reset,
so we can't use the _optional() function here to perform the reset only
for RZ/G2L.

Thanks,

-- 
Paul Barker
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0x27F4B3459F002257.asc
Type: application/pgp-keys
Size: 3520 bytes
Desc: OpenPGP public key
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250324/5b45351a/attachment.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250324/5b45351a/attachment.sig>