Re: [PATCH v2 5/5] usb: dwc3: Drop support for "snps, ref-clock-period-ns" DT property

[Sean Anderson <seanga2@gmail.com>]

On 11/7/22 20:40, Marek Vasut wrote:
> Drop support for quickly deprecated DT property "snps,ref-clock-period-ns"
> to prevent its proliferation.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Angus Ainslie <angus@akkea.ca>
> Cc: Bin Meng <bmeng.cn@gmail.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> Cc: Michal Simek <michal.simek@xilinx.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Sean Anderson <sean.anderson@seco.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
> V2: New patch
> ---
>   drivers/usb/dwc3/core.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
> index fcd062dc898..a0d3a9d5eea 100644
> --- a/drivers/usb/dwc3/core.c
> +++ b/drivers/usb/dwc3/core.c
> @@ -140,9 +140,6 @@ static void dwc3_ref_clk_period(struct dwc3 *dwc)
>   		if (!rate)
>   			return;
>   		period = NSEC_PER_SEC / rate;
> -	} else if (dwc->ref_clk_per) {
> -		period = dwc->ref_clk_per;
> -		rate = NSEC_PER_SEC / period;
>   	} else {
>   		return;
>   	}
> @@ -1120,7 +1117,6 @@ void dwc3_of_parse(struct dwc3 *dwc)
>   		| (dwc->is_utmi_l1_suspend << 4);
>   
>   	dev_read_u32(dev, "snps,quirk-frame-length-adjustment", &dwc->fladj);
> -	dev_read_u32(dev, "snps,ref-clock-period-ns", &dwc->ref_clk_per);
>   
>   	/*
>   	 * Handle property "snps,incr-burst-type-adjustment".

Reviewed-by: Sean Anderson <seanga2@gmail.com>