Re: [PATCH v2] mmc: zynq_sdhci: support Renesas RZ/N1

On 8/25/26 9:47 PM, Ralph Siemsen wrote:

Hi,

>>>          return ret;
>>>      }
>>> -    clock = clk_get_rate(&clk);
>>> +    clock = clk_get_rate(&clks.clks[0]);
>>
>> Can the clock ever be non-first clock , i.e. do you need to handle 
>> clock-names DT property somehow ?
> 
> It's not clear to me if the DT binding specifies the order. It says 
> there must be either 2 or 3 clocks, as well as clock-names properties.
> 
> Existing DTS files (rzn1 and zynqmp) have it as the first clock.
> 
> We could try using the name ("clk_xin") and if that fails, fall back on 
> using the first clock.
Sounds good.