[PATCH 2/2] phy: renesas: Add Renesas Multi-Protocol PHY driver for R-Car X5H

[Marek Vasut]

On 9/3/26 12:15 PM, Manivannan Sadhasivam wrote:

Hello Manivannan,

>> +/* Firmware update */
>> +#define MPPHY_FW_BASE			0x10000
>> +#define MPPHY_FW_CH_OFFSET		0x20000
>> +#define MPPHY_FW_NAME			"rcar_gen5_mp_phy.bin"
> 
> Is this firmware redistributable? I would assume not as like other Renesas IP
> firmware. If so, you should add a documentation about how an user can get the
> firmware, and package it. Like,
> Documentation/PCI/controller/rcar-pcie-firmware.rst

I will do so in V2 .

The Gen4 firmware is now part of linux-firmware, so I updated that piece 
of documentation and sent a patch.

[...]

>> +static int mp_phy_probe(struct platform_device *pdev)
>> +{

[...]

>> +	return pm_runtime_resume_and_get(dev);
> 
> Since the runtime usage count is incremented and not dropped anywhere else, the
> driver will never runtime suspend.
> 
>> +}

I do not think we can actually runtime suspend the PHY itself, so this 
is expected.

The rest of the feedback is handled in V2, thank you !

-- 
Best regards,
Marek Vasut