[PATCH] phy: rockchip-inno-usb2: cleanup comments and make ops static

[Jagan Teki]

On Mon, Jun 19, 2023 at 2:39 PM Marek Vasut <marex at denx.de> wrote:
>
> On 6/19/23 09:24, Xavier Drudis Ferran wrote:
> > Thanks to Jagan Teki for noting that three functions should be static
> > and comments from uclass_clk are redundant.
>
> The change log should be at below --- , the commit message should be above.
>
> > Signed-off-by: Xavier Drudis Ferran <xdrudis at tinet.cat>
> > ---
> >   drivers/phy/rockchip/phy-rockchip-inno-usb2.c | 25 +++----------------
> >   1 file changed, 3 insertions(+), 22 deletions(-)
> >
> > diff --git a/drivers/phy/rockchip/phy-rockchip-inno-usb2.c b/drivers/phy/rockchip/phy-rockchip-inno-usb2.c
> > index be5f79490c..2b33b6f03c 100644
> > --- a/drivers/phy/rockchip/phy-rockchip-inno-usb2.c
> > +++ b/drivers/phy/rockchip/phy-rockchip-inno-usb2.c
> > @@ -182,25 +182,12 @@ static struct phy_ops rockchip_usb2phy_ops = {
> >       .of_xlate = rockchip_usb2phy_of_xlate,
> >   };
> >
> > -/**
> > - * round_rate() - Adjust a rate to the exact rate a clock can provide.
> > - * @clk:     The clock to manipulate.
> > - * @rate:    Desidered clock rate in Hz.
> > - *
> > - * Return: rounded rate in Hz, or -ve error code.
> > - */
> > -ulong rockchip_usb2phy_clk_round_rate(struct clk *clk, ulong rate)
>
> Why is the documentation dropped ?

As these comments seem redundant as in clk-uclass.h definitions, so I
asked to drop the same from the driver.

Xavier, can you add the details in the commit message?

Jagan.