Hi Kever,

On 2023-09-01 09:49, Kever Yang wrote:
> Hi Jonas,
> 
>      Why is this patch 6/6 and didn't see other patches?

Patch 1-3 moves a workaround made in usb dwc3 driver into generic phy
core so entire series was sent to usb maintainer and cc mailing list.

Patch 4-6 was trivial and had no real dependencies on prior patches but
was related to the work made in patch 1-3, so sent these to the listed
maintainers and cc mailing list.

Full series can also be found at:
https://patchwork.ozlabs.org/project/uboot/list/?series=371248&state=*

Regards,
Jonas

> 
> On 2023/9/1 06:16, Jonas Karlman wrote:
>> The documentation for struct phy state that "The content of the
>> structure is managed solely by the PHY API and PHY drivers".
>>
>> Change to use the generic_phy_valid() helper to check if phy is valid.
>>
>> Fixes: b7d8d40346f2 ("video: rockchip: dw_mipi_dsi: Fix external phy 
>> existence check")
>> Signed-off-by: Jonas Karlman <jo...@kwiboo.se>
> Reviewed-by: Kever Yang <kever.y...@rock-chips.com>
> 
> Thanks,
> - Kever
>> ---
>>   drivers/video/rockchip/dw_mipi_dsi_rockchip.c | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/video/rockchip/dw_mipi_dsi_rockchip.c 
>> b/drivers/video/rockchip/dw_mipi_dsi_rockchip.c
>> index 0852b53ebed5..1a5ab781e3f1 100644
>> --- a/drivers/video/rockchip/dw_mipi_dsi_rockchip.c
>> +++ b/drivers/video/rockchip/dw_mipi_dsi_rockchip.c
>> @@ -377,7 +377,7 @@ static int dsi_phy_init(void *priv_data)
>>      struct dw_rockchip_dsi_priv *dsi = dev_get_priv(dev);
>>      int ret, i, vco;
>>   
>> -    if (dsi->phy.dev) {
>> +    if (generic_phy_valid(&dsi->phy)) {
>>              ret = generic_phy_configure(&dsi->phy, &dsi->phy_opts);
>>              if (ret) {
>>                      dev_err(dsi->dsi_host,
>> @@ -559,7 +559,7 @@ dw_mipi_dsi_get_lane_mbps(void *priv_data, struct 
>> display_timing *timings,
>>      }
>>   
>>      /* for external phy only the mipi_dphy_config is necessary */
>> -    if (dsi->phy.dev) {
>> +    if (generic_phy_valid(&dsi->phy)) {
>>              phy_mipi_dphy_get_default_config(timings->pixelclock.typ  * 10 
>> / 8,
>>                                               bpp, lanes,
>>                                               &dsi->phy_opts);
>> @@ -859,7 +859,7 @@ static int dw_mipi_dsi_rockchip_probe(struct udevice 
>> *dev)
>>      }
>>   
>>      /* Get a ref clock only if not using an external phy. */
>> -    if (priv->phy.dev) {
>> +    if (generic_phy_valid(&priv->phy)) {
>>              dev_dbg(dev, "setting priv->ref to NULL\n");
>>              priv->ref = NULL;
>>   

Reply via email to