Re: [PATCH v1 1/2] dt-bindings: clock: Add SC7280 GCC clock binding

[Taniya Das <tdas@xxxxxxxxxxxxxx>]

Thanks Stephen for your review comments.

On 1/13/2021 1:36 AM, Stephen Boyd wrote:

> > +  clock-names:
> > +    items:
> > +      - const: bi_tcxo
> > +      - const: bi_tcxo_ao
> > +      - const: sleep_clk
> > +      - const: pcie_0_pipe_clk
> > +      - const: pcie_1_pipe_clk
> > +      - const: usb3_phy_wrapper_gcc_usb30_pipe_clk
>
> Don't ufs phy clks also go into gcc?

The latest patch has this updated.

> > +
> > +  '#clock-cells':
> > +    const: 1
> > +
> > +  '#reset-cells':
> > +    const: 1
> > +
> > +  '#power-domain-cells':
> > +    const: 1
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  protected-clocks:
> > +    description:
> > +      Protected clock specifier list as per common clock binding.
>
> I suppose this is fine.

Removed the above in the latest patch.


--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation.

--