Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1582372
| From | Chunyan Zhang <chunyan.zhang@spreadtrum.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G |
| Date | 2017-02-16 09:10 +0100 |
| Message-ID | <tbpix-3t4-5@gated-at.bofh.it> (permalink) |
| References | <taHUd-8vw-19@gated-at.bofh.it> <taHUd-8vw-31@gated-at.bofh.it> <tbpix-3t4-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 二, 2月 14, 2017 at 03:57:43下午 +0000, Mark Rutland wrote:
> Hi,
>
> On Tue, Feb 14, 2017 at 05:19:08PM +0800, Chunyan Zhang wrote:
>
> > + timer {
> > + compatible = "arm,armv8-timer";
> > + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(8)
> > + | IRQ_TYPE_LEVEL_LOW)>,
> > + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(8)
> > + | IRQ_TYPE_LEVEL_LOW)>,
> > + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(8)
> > + | IRQ_TYPE_LEVEL_LOW)>,
> > + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(8)
> > + | IRQ_TYPE_LEVEL_LOW)>;
> > + clock-frequency = <26000000>;
> > + };
>
> Your FW should program CNTFRQ on all CPUs, and this clock-frequency
> property should not be necessary. Please remove it.
>
Will remove it.
> > +
> > + pmu {
> > + compatible = "arm,armv8-pmuv3";
>
> Please list "arm,cortex-a53-pmu" first.
>
> > + interrupts = <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 154 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 155 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 156 IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 157 IRQ_TYPE_LEVEL_HIGH>;
> > + };
>
> As you are using SPIs, please add an interrupt-affinity property, per
> Documentation/devicetree/bindings/arm/pmu.txt.
Added, would appear in the next revision.
>
> > diff --git a/arch/arm64/boot/dts/sprd/sp9860g-1h10.dts b/arch/arm64/boot/dts/sprd/sp9860g-1h10.dts
> > new file mode 100644
> > index 0000000..59390fc
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/sprd/sp9860g-1h10.dts
> > @@ -0,0 +1,58 @@
> > +/*
> > + * Spreadtrum SP9860g board DTS file
> > + *
> > + * Copyright (C) 2017, Spreadtrum Communications Inc.
> > + *
> > + * This file is licensed under a dual GPLv2 or X11 license.
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "sc9860.dtsi"
> > +
> > +/ {
> > + model = "Spreadtrum SP9860G 3GFHD Board";
> > +
> > + compatible = "sprd,sc9860", "sprd,sp9860g-1h10";
> > +
> > + sprd,sc-id = <9860 1 0x20000>;
>
> What is this? I can't see any documentation or user of this property in
> mainline.
>
I will remove this property in the next iteration, actually this is for Spreadtrum's dtb tools only, I just forgot to remove it before sending this patch out.
Thanks,
Chunyan
>
> Thanks,
> Mark.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/5] Add Spreadtrum SP9860G support Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-14 10:50 +0100
[PATCH 5/5] serial: sprd: adjust TIMEOUT to a big value Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-14 10:50 +0100
[PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-14 10:50 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Sudeep Holla <sudeep.holla@arm.com> - 2017-02-14 17:50 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-17 08:40 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Sudeep Holla <sudeep.holla@arm.com> - 2017-02-17 11:30 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-20 10:50 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Sudeep Holla <sudeep.holla@arm.com> - 2017-02-20 11:50 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-20 14:00 +0100
Re: [PATCH 1/5] arm64: dts: Add basic DT to support Spreadtrum's SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-16 09:10 +0100
[PATCH 4/5] sprd_serial: switch comptible string to sc-uart Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-14 10:50 +0100
Re: [PATCH 4/5] sprd_serial: switch comptible string to sc-uart Arnd Bergmann <arnd@arndb.de> - 2017-02-16 14:40 +0100
Re: [PATCH 4/5] sprd_serial: switch comptible string to sc-uart Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-17 08:40 +0100
[PATCH 2/5] Documentation: sprd: Add bindings for SP9860G Chunyan Zhang <chunyan.zhang@spreadtrum.com> - 2017-02-14 10:50 +0100
csiph-web