[Patch v2 1/2] clk: zynq: Add clock wizard driver
Michal Simek
michal.simek at xilinx.com
Fri May 7 10:18:50 CEST 2021
Hi,
snip.
>>> +
>>> + priv->base = (void __iomem *)addr;
>>
>> But this assignment should be done in probe where you copy data from
>> plat structures to priv structures.
>
> Do you mean priv->base = (void __iomem *)plat->addr?
yes.
>>> +
>>> + clk_in1 = dev_read_u32_default(dev, "clock-frequency", 0);
>>
>> This is not the part of DT binding. You should be able to get that
>> frequencies via clock framework.
>
> Can you provide some hints about this? I am new to clock driver.
You should look at drivers/serial/serial_zynq.c where you can find out
clk_get_by_index(), clk_get_rate(), clk_enable()
That clk_get_rate() is the function you should use to get that frequency.
snip.
>
>>> + { /* sentinel */ }
>>> +};
>>> +
>>> +U_BOOT_DRIVER(clk_wzrd) = {
>>> + .name = "zynq-clk-wizard",
>>> + .id = UCLASS_CLK,
>>> + .of_match = clk_wzrd_ids,
>>> + .ops = &clk_wzrd_ops,
>>> + .probe = clk_wzrd_probe,
>>> + .of_to_plat = clk_wzrd_of_to_plat,
>>> + .priv_auto = sizeof(struct clkwzd),
>>
>> you need to get memory for .plat_auto here.
>
> Okay.
>
> On the other hand, if we want to add set_phase feature to clock wizard,
> can you make some suggestions? I checked the clk-uclass and it does not
> seem to be supported.
Then you have to add it to uclass first but don't know content here.
>
> Thanks,
> Zhengxun
>
>
> CONFIDENTIALITY NOTE:
>
> This e-mail and any attachments may contain confidential information
> and/or personal data, which is protected by applicable laws. Please be
> reminded that duplication, disclosure, distribution, or use of this e-mail
> (and/or its attachments) or any part thereof is prohibited. If you receive
> this e-mail in error, please notify us immediately and delete this mail as
> well as its attachment(s) from your system. In addition, please be
> informed that collection, processing, and/or use of personal data is
> prohibited unless expressly permitted by personal data protection laws.
> Thank you for your attention and cooperation.
>
> Macronix International Co., Ltd.
>
> =====================================================================
>
>
>
> ============================================================================
>
> CONFIDENTIALITY NOTE:
>
> This e-mail and any attachments may contain confidential information and/or personal data, which is protected by applicable laws. Please be reminded that duplication, disclosure, distribution, or use of this e-mail (and/or its attachments) or any part thereof is prohibited. If you receive this e-mail in error, please notify us immediately and delete this mail as well as its attachment(s) from your system. In addition, please be informed that collection, processing, and/or use of personal data is prohibited unless expressly permitted by personal data protection laws. Thank you for your attention and cooperation.
>
> Macronix International Co., Ltd.
>
> =====================================================================
And get rid of these. Next time I won't reply to this email. Sorry.
Thanks,
Michal
More information about the U-Boot
mailing list