mbox

[GIT,PULL] Amlogic fixes for v4.10-rc4

Message ID m2inpi6d09.fsf@baylibre.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-amlogic.git tags/amlogic-fixes-2

Message

Kevin Hilman Jan. 13, 2017, 6:17 p.m. UTC
Arnd, Olof,

Please pull a couple fixes for v4.10-rc.

The branch is based on v4.10-rc3 to not conflict with another DT fix
that when in for -rc2.

Kevin


The following changes since commit a121103c922847ba5010819a3f250f1f7fc84ab8:

  Linux 4.10-rc3 (2017-01-08 14:18:17 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-amlogic.git tags/amlogic-fixes-2

for you to fetch changes up to 64a8225e9f9dec0e5cd2f5640743699056da4e4e:

  ARM64: dts: meson-gx: Add reserved memory zone and usable memory range (2017-01-12 17:00:50 -0800)

----------------------------------------------------------------
Amlogic fixes for v4.10-rc4
- odroid-c2: disable DVFS to avoid crashes due to broken SCPI
- fix misc. crashes by adding known reserved memory regions

----------------------------------------------------------------
Neil Armstrong (2):
      ARM64: dts: meson-gxbb-odroidc2: Disable SCPI DVFS
      ARM64: dts: meson-gx: Add reserved memory zone and usable memory range

 arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi       |  5 -----
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi                 | 12 ++++++++++++
 arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts    |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts       | 10 +++++++++-
 arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts           |  9 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dts           |  9 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi          |  5 -----
 arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-meta.dts  |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-pro.dts   |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-telos.dts |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi               |  2 +-
 arch/arm64/boot/dts/amlogic/meson-gxl-nexbox-a95x.dts     |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dts      |  9 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p231.dts      |  9 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dts      |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts       |  6 +++++-
 arch/arm64/boot/dts/amlogic/meson-gxm-s912-q200.dts       |  9 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxm-s912-q201.dts       |  9 +++++++++
 18 files changed, 111 insertions(+), 19 deletions(-)

Comments

Andreas Färber Jan. 16, 2017, 2:24 p.m. UTC | #1
Am 13.01.2017 um 19:17 schrieb Kevin Hilman:
> ----------------------------------------------------------------
> Amlogic fixes for v4.10-rc4
> - odroid-c2: disable DVFS to avoid crashes due to broken SCPI
> - fix misc. crashes by adding known reserved memory regions
> 
> ----------------------------------------------------------------
> Neil Armstrong (2):
>       ARM64: dts: meson-gxbb-odroidc2: Disable SCPI DVFS

>       ARM64: dts: meson-gx: Add reserved memory zone and usable memory range

Objection against the latter patch introducing widespread usage of
hardcoded linux,usable-memory properties - please review closely before
considering to merge.

Thanks,
Andreas

> 
>  arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi       |  5 -----
>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi                 | 12 ++++++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts    |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts       | 10 +++++++++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts           |  9 +++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dts           |  9 +++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi          |  5 -----
>  arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-meta.dts  |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-pro.dts   |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-telos.dts |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi               |  2 +-
>  arch/arm64/boot/dts/amlogic/meson-gxl-nexbox-a95x.dts     |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dts      |  9 +++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p231.dts      |  9 +++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dts      |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts       |  6 +++++-
>  arch/arm64/boot/dts/amlogic/meson-gxm-s912-q200.dts       |  9 +++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxm-s912-q201.dts       |  9 +++++++++
>  18 files changed, 111 insertions(+), 19 deletions(-)
> 
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-amlogic
>
Olof Johansson Jan. 17, 2017, 6:04 a.m. UTC | #2
On Mon, Jan 16, 2017 at 03:24:59PM +0100, Andreas F??rber wrote:
> Am 13.01.2017 um 19:17 schrieb Kevin Hilman:
> > ----------------------------------------------------------------
> > Amlogic fixes for v4.10-rc4
> > - odroid-c2: disable DVFS to avoid crashes due to broken SCPI
> > - fix misc. crashes by adding known reserved memory regions
> > 
> > ----------------------------------------------------------------
> > Neil Armstrong (2):
> >       ARM64: dts: meson-gxbb-odroidc2: Disable SCPI DVFS
> 
> >       ARM64: dts: meson-gx: Add reserved memory zone and usable memory range
> 
> Objection against the latter patch introducing widespread usage of
> hardcoded linux,usable-memory properties - please review closely before
> considering to merge.

Holding off until that discussion has concluded. Following up on that thread as
well.


-Olof
Kevin Hilman Jan. 17, 2017, 4:07 p.m. UTC | #3
Hi Olof,

Olof Johansson <olof@lixom.net> writes:

> On Mon, Jan 16, 2017 at 03:24:59PM +0100, Andreas F??rber wrote:
>> Am 13.01.2017 um 19:17 schrieb Kevin Hilman:
>> > ----------------------------------------------------------------
>> > Amlogic fixes for v4.10-rc4
>> > - odroid-c2: disable DVFS to avoid crashes due to broken SCPI
>> > - fix misc. crashes by adding known reserved memory regions
>> > 
>> > ----------------------------------------------------------------
>> > Neil Armstrong (2):
>> >       ARM64: dts: meson-gxbb-odroidc2: Disable SCPI DVFS
>> 
>> >       ARM64: dts: meson-gx: Add reserved memory zone and usable memory range
>> 
>> Objection against the latter patch introducing widespread usage of
>> hardcoded linux,usable-memory properties - please review closely before
>> considering to merge.
>
> Holding off until that discussion has concluded. Following up on that thread as
> well.

While we rework the reserved memory fix, could you cherry-pick the SCPI
fix from this branch into fixes?  That one is independent from this, and
is also a nasty crash.

Thanks,

Kevin
Olof Johansson Jan. 17, 2017, 8:11 p.m. UTC | #4
On Tue, Jan 17, 2017 at 08:07:50AM -0800, Kevin Hilman wrote:
> Hi Olof,
> 
> Olof Johansson <olof@lixom.net> writes:
> 
> > On Mon, Jan 16, 2017 at 03:24:59PM +0100, Andreas F??rber wrote:
> >> Am 13.01.2017 um 19:17 schrieb Kevin Hilman:
> >> > ----------------------------------------------------------------
> >> > Amlogic fixes for v4.10-rc4
> >> > - odroid-c2: disable DVFS to avoid crashes due to broken SCPI
> >> > - fix misc. crashes by adding known reserved memory regions
> >> > 
> >> > ----------------------------------------------------------------
> >> > Neil Armstrong (2):
> >> >       ARM64: dts: meson-gxbb-odroidc2: Disable SCPI DVFS
> >> 
> >> >       ARM64: dts: meson-gx: Add reserved memory zone and usable memory range
> >> 
> >> Objection against the latter patch introducing widespread usage of
> >> hardcoded linux,usable-memory properties - please review closely before
> >> considering to merge.
> >
> > Holding off until that discussion has concluded. Following up on that thread as
> > well.
> 
> While we rework the reserved memory fix, could you cherry-pick the SCPI
> fix from this branch into fixes?  That one is independent from this, and
> is also a nasty crash.

Sure! Applied to fixes.


Thanks,

-Olof