Re: [PATCH 08/15] dt-bindings: arm: cpus: Document qcom,msm8916-smp enable-method

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 04, 2021 at 01:37:58PM -0500, Rob Herring wrote:
> On Mon, Oct 4, 2021 at 1:34 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> >
> > On Tue, Sep 28, 2021 at 07:12:24PM +0200, Stephan Gerhold wrote:
> > > Document the qcom,msm8916-smp enable method. It is actually just
> > > an alias for qcom,msm8226-smp since it should be implemented identically.
> > >
> > > Signed-off-by: Stephan Gerhold <stephan@xxxxxxxxxxx>
> > > ---
> > >  Documentation/devicetree/bindings/arm/cpus.yaml | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
> > > index 11e3e09da2e5..5602a8f3c513 100644
> > > --- a/Documentation/devicetree/bindings/arm/cpus.yaml
> > > +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
> > > @@ -211,6 +211,7 @@ properties:
> > >            - qcom,kpss-acc-v1
> > >            - qcom,kpss-acc-v2
> > >            - qcom,msm8226-smp
> > > +          - qcom,msm8916-smp
> >
> > I thought arm64 is PSCI only.
> 
> I see the explanation now. Please add that to this commit as well and
> a comment here that this is for 32-bit only.
> 

I'm not sure why this is literally the only patch in the series where
I omitted the explanation. Sorry for the confusion! :)

I clarified this in v2 and added the comment as you suggested.

Thanks!
Stephan



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux