Re: [PATCH] arm64: dts: add support for the Pixel 2 XL
Hi Konrad,
On 06/03/2021 10:49 am, Konrad Dybcio wrote:
>
> On 05.03.2021 22:35, Caleb Connolly wrote:
>> Add a minimal devicetree capable of booting on the Pixel 2 XL MSM8998
>> device.
>>
>> It's currently possible to boot the device into postmarketOS with USB
>> networking, however the display panel depends on Display Stream
>> Compression which is not yet supported in the kernel.
>>
>> The bootloader also requires that the dtbo partition contains a device
>> tree overlay with a particular id which has to be overlayed onto the
>> existing dtb. It's possible to use a specially crafted dtbo partition to
>> workaround this, more information is available here:
>>
>> https://gitlab.com/calebccff/dtbo-google-wahoo-mainline
>>
>> Signed-off-by: Caleb Connolly
>> ---
>> It's possible to get wifi working by running Bjorns diag-router in the
>> background, without this the wifi firmware crashes every 10 seconds or
>> so. This is the same issue encountered on the OnePlus 5.
>>
>> arch/arm64/boot/dts/qcom/Makefile | 1 +
>> .../boot/dts/qcom/msm8998-google-taimen.dts | 14 +
>> .../boot/dts/qcom/msm8998-google-wahoo.dtsi | 391 ++
>> 3 files changed, 406 insertions(+)
>> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>>
>> diff --git a/arch/arm64/boot/dts/qcom/Makefile
>> b/arch/arm64/boot/dts/qcom/Makefile
>> index 5113fac80b7a..d942d3ec3928 100644
>> --- a/arch/arm64/boot/dts/qcom/Makefile
>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)+=
>> msm8994-msft-lumia-cityman.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8994-sony-xperia-kitakami-sumire.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8996-mtp.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-asus-novago-tp370ql.dtb
>> +dtb-$(CONFIG_ARCH_QCOM) += msm8998-google-taimen.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-hp-envy-x2.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-lenovo-miix-630.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-mtp.dtb
>> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> new file mode 100644
>> index ..ffaaafe14037
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> @@ -0,0 +1,14 @@
>> +// SPDX-License-Identifier: GPL-2.0-only
>> +/*
>> + * Copyright (c) 2020, Caleb Connolly
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include "msm8998-google-wahoo.dtsi"
>> +
>> +/ {
>> +model = "Google Pixel 2 XL";
>> +compatible = "google,taimen", "google,wahoo", "qcom,msm8998",
>> "qcom,msm8998-mtp";
> Drop the mtp compatible. Also, afaict wahoo is a shared platform name for
> P2/2XL, so perhaps using the same naming scheme we used for Xperias/Lumias
> (soc-vendor-platform-board) would clear up some confusion. In this case, I'm
> not sure about the wahoo compatible, but I reckon it's fine for it to stay so
> that it's easier to introduce potential quirks that concern both devices.
>
Yeah, I think it's useful to have a shared compatible for similar devices.
>> +qcom,msm-id = <0x124 0x20001>;
> Move it to the common dtsi, unless the other Pixel ships with a different SoC
> revision.
>
>
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> new file mode 100644
>> index ..0c221ead2df7
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> @@ -0,0 +1,391 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +/* Copyright (c) 2020 Caleb Connolly */
>> +
>> +#include "msm8998.dtsi"
>> +#include "pm8998.dtsi"
>> +#include "pmi8998.dtsi"
>> +#include "pm8005.dtsi"
>> +
>> +/delete-node/ &mpss_mem;
>> +/delete-node/ &venus_mem;
>> +/delete-node/ &mba_mem;
>> +/delete-node/ &slpi_mem;
>> +
>> +/ {
>> +aliases {
>> +};
>> +
>> +chosen {
>> +#address-cells = <2>;
>> +#size-cells = <2>;
>> +ranges;
>> +
>> +/* Add "earlycon" intended to be used in combination with UART
>> serial console */
>> +bootargs = "clk_ignore_unused earlycon
>> console=ttyGS0,115200";// loglevel=10 drm.debug=15 debug";
> clk_ignore_unused is a BIG hack!
Ah, I haven't touched this device in a while, seems like we don't need
this anymore on 5.11
>
> You should trace which clocks are important for it to stay alive and fix it
> on the driver side.
>
> What breaks if it's not there? Does it still happen with Angelo's clk patches
> that got in for the 5.12
>
> window?
>
> Aside from that, //loglevel... should also go.
>
>
>> +
>> +vph_pwr: vph-pwr-regulator {
>> +compatible = "regulator-fixed";
>> +regulator-name = "vph_pwr";
>> +regulator-always-on;
>> +regulator-boot-on;
>> +};
>> +};
> Don't you need to specify voltage here?
This is based on the MT
Re: [PATCH] arm64: dts: add support for the Pixel 2 XL
Hi Bjorn,
On 06/03/2021 3:37 am, Bjorn Andersson wrote:
> On Fri 05 Mar 15:35 CST 2021, Caleb Connolly wrote:
>
> Please add "qcom: " to $subject as well.
>
>> Add a minimal devicetree capable of booting on the Pixel 2 XL MSM8998
>> device.
>>
>> It's currently possible to boot the device into postmarketOS with USB
>> networking, however the display panel depends on Display Stream
>> Compression which is not yet supported in the kernel.
>>
>> The bootloader also requires that the dtbo partition contains a device
>> tree overlay with a particular id which has to be overlayed onto the
>> existing dtb. It's possible to use a specially crafted dtbo partition to
>> workaround this, more information is available here:
>>
>> https://gitlab.com/calebccff/dtbo-google-wahoo-mainline
>>
> So it's not possible to just erase the dto, like on most other devices?
That's correct, I initially tried that but the bootloader just hung when
I tried to boot anything.
It seems to require a DTO matching some revision-specific IDs, the
bootloader is very unhappy when it can't
find them.
>
>> Signed-off-by: Caleb Connolly
>> ---
>> It's possible to get wifi working by running Bjorns diag-router in the
>> background, without this the wifi firmware crashes every 10 seconds or
>> so. This is the same issue encountered on the OnePlus 5.
>>
>> arch/arm64/boot/dts/qcom/Makefile | 1 +
>> .../boot/dts/qcom/msm8998-google-taimen.dts | 14 +
>> .../boot/dts/qcom/msm8998-google-wahoo.dtsi | 391 ++
>> 3 files changed, 406 insertions(+)
>> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>>
>> diff --git a/arch/arm64/boot/dts/qcom/Makefile
>> b/arch/arm64/boot/dts/qcom/Makefile
>> index 5113fac80b7a..d942d3ec3928 100644
>> --- a/arch/arm64/boot/dts/qcom/Makefile
>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM)+=
>> msm8994-msft-lumia-cityman.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8994-sony-xperia-kitakami-sumire.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8996-mtp.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-asus-novago-tp370ql.dtb
>> +dtb-$(CONFIG_ARCH_QCOM) += msm8998-google-taimen.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-hp-envy-x2.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-lenovo-miix-630.dtb
>> dtb-$(CONFIG_ARCH_QCOM)+= msm8998-mtp.dtb
>> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> new file mode 100644
>> index ..ffaaafe14037
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
>> @@ -0,0 +1,14 @@
>> +// SPDX-License-Identifier: GPL-2.0-only
> Would you be willing to release these as BSD-3-Clause instead?
Yeah I'd be happy to, is this a new practice? As the device I submitted
previously is also GPL-2.0.
>
>> +/*
>> + * Copyright (c) 2020, Caleb Connolly
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include "msm8998-google-wahoo.dtsi"
>> +
>> +/ {
>> +model = "Google Pixel 2 XL";
>> +compatible = "google,taimen", "google,wahoo", "qcom,msm8998",
>> "qcom,msm8998-mtp";
>> +qcom,msm-id = <0x124 0x20001>;
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> new file mode 100644
>> index ..0c221ead2df7
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>> @@ -0,0 +1,391 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +/* Copyright (c) 2020 Caleb Connolly */
>> +
>> +#include "msm8998.dtsi"
>> +#include "pm8998.dtsi"
>> +#include "pmi8998.dtsi"
>> +#include "pm8005.dtsi"
>> +
>> +/delete-node/ &mpss_mem;
>> +/delete-node/ &venus_mem;
>> +/delete-node/ &mba_mem;
>> +/delete-node/ &slpi_mem;
>> +
>> +/ {
>> +aliases {
>> +};
>> +
>> +chosen {
>> +#address-cells = <2>;
>> +#size-cells = <2>;
>> +ranges;
>> +
>> +/* Add "earlycon" intended to be used in combination with UART
>> serial console */
>> +bootargs = "clk_ignore_unused earlycon
>> console=ttyGS0,115200";// loglevel=10 drm.debug=15 debug";
> Please drop earlycon from this list (user should be able to add it if
> they care later?) and use stdout-path to set the console, like we do on
> other devices.
>
>> +};
>> +
>> +reserved-memory {
>> +#address-cells = <2>;
>> +#size-cells = <2>;
>> +ranges;
>> +
>> +mpss_mem: memory@8cc0 {
>> +reg = <0 0x8cc0 0 0x780>;
>> +no-map;
>> +};
>> +
>> +venus_mem: memory@9440 {
>> +reg = <0 0x9440 0 0x50>;
>> +no-map;
>> +};
>> +
>> +mba_mem: memory@9410 {
>> +reg = <0 0x9490 0 0x20>;
>> +
Re: [PATCH] arm64: dts: add support for the Pixel 2 XL
On 05.03.2021 22:35, Caleb Connolly wrote:
> Add a minimal devicetree capable of booting on the Pixel 2 XL MSM8998
> device.
>
> It's currently possible to boot the device into postmarketOS with USB
> networking, however the display panel depends on Display Stream
> Compression which is not yet supported in the kernel.
>
> The bootloader also requires that the dtbo partition contains a device
> tree overlay with a particular id which has to be overlayed onto the
> existing dtb. It's possible to use a specially crafted dtbo partition to
> workaround this, more information is available here:
>
> https://gitlab.com/calebccff/dtbo-google-wahoo-mainline
>
> Signed-off-by: Caleb Connolly
> ---
> It's possible to get wifi working by running Bjorns diag-router in the
> background, without this the wifi firmware crashes every 10 seconds or
> so. This is the same issue encountered on the OnePlus 5.
>
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/msm8998-google-taimen.dts | 14 +
> .../boot/dts/qcom/msm8998-google-wahoo.dtsi | 391 ++
> 3 files changed, 406 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile
> b/arch/arm64/boot/dts/qcom/Makefile
> index 5113fac80b7a..d942d3ec3928 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM) +=
> msm8994-msft-lumia-cityman.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8994-sony-xperia-kitakami-sumire.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8996-mtp.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-asus-novago-tp370ql.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += msm8998-google-taimen.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-hp-envy-x2.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-lenovo-miix-630.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-mtp.dtb
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> new file mode 100644
> index ..ffaaafe14037
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> @@ -0,0 +1,14 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (c) 2020, Caleb Connolly
> + */
> +
> +/dts-v1/;
> +
> +#include "msm8998-google-wahoo.dtsi"
> +
> +/ {
> + model = "Google Pixel 2 XL";
> + compatible = "google,taimen", "google,wahoo", "qcom,msm8998",
> "qcom,msm8998-mtp";
Drop the mtp compatible. Also, afaict wahoo is a shared platform name for
P2/2XL, so perhaps using the same naming scheme we used for Xperias/Lumias
(soc-vendor-platform-board) would clear up some confusion. In this case, I'm
not sure about the wahoo compatible, but I reckon it's fine for it to stay so
that it's easier to introduce potential quirks that concern both devices.
> + qcom,msm-id = <0x124 0x20001>;
Move it to the common dtsi, unless the other Pixel ships with a different SoC
revision.
> +};
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> new file mode 100644
> index ..0c221ead2df7
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> @@ -0,0 +1,391 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/* Copyright (c) 2020 Caleb Connolly */
> +
> +#include "msm8998.dtsi"
> +#include "pm8998.dtsi"
> +#include "pmi8998.dtsi"
> +#include "pm8005.dtsi"
> +
> +/delete-node/ &mpss_mem;
> +/delete-node/ &venus_mem;
> +/delete-node/ &mba_mem;
> +/delete-node/ &slpi_mem;
> +
> +/ {
> + aliases {
> + };
> +
> + chosen {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + /* Add "earlycon" intended to be used in combination with UART
> serial console */
> + bootargs = "clk_ignore_unused earlycon
> console=ttyGS0,115200";// loglevel=10 drm.debug=15 debug";
clk_ignore_unused is a BIG hack!
You should trace which clocks are important for it to stay alive and fix it on
the driver side.
What breaks if it's not there? Does it still happen with Angelo's clk patches
that got in for the 5.12
window?
Aside from that, //loglevel... should also go.
> +
> + vph_pwr: vph-pwr-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "vph_pwr";
> + regulator-always-on;
> + regulator-boot-on;
> + };
> +};
Don't you need to specify voltage here?
> +
> +&blsp1_uart3 {
> + status = "disabled";
> +
> + bluetooth {
> + compatible = "qcom,wcn3990-bt";
> +
> + vddio-supply = <&vreg_s4a_1p8>;
> + vddxo-supply = <&vreg_l7a_1p8>;
> + vddrf-supply = <&vreg_l17a_1p3>;
> + vddch0-supply = <&vreg_l25a_3p3>;
> + max-speed = <320>;
> +
Re: [PATCH] arm64: dts: add support for the Pixel 2 XL
On Fri 05 Mar 15:35 CST 2021, Caleb Connolly wrote:
Please add "qcom: " to $subject as well.
> Add a minimal devicetree capable of booting on the Pixel 2 XL MSM8998
> device.
>
> It's currently possible to boot the device into postmarketOS with USB
> networking, however the display panel depends on Display Stream
> Compression which is not yet supported in the kernel.
>
> The bootloader also requires that the dtbo partition contains a device
> tree overlay with a particular id which has to be overlayed onto the
> existing dtb. It's possible to use a specially crafted dtbo partition to
> workaround this, more information is available here:
>
> https://gitlab.com/calebccff/dtbo-google-wahoo-mainline
>
So it's not possible to just erase the dto, like on most other devices?
> Signed-off-by: Caleb Connolly
> ---
> It's possible to get wifi working by running Bjorns diag-router in the
> background, without this the wifi firmware crashes every 10 seconds or
> so. This is the same issue encountered on the OnePlus 5.
>
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/msm8998-google-taimen.dts | 14 +
> .../boot/dts/qcom/msm8998-google-wahoo.dtsi | 391 ++
> 3 files changed, 406 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile
> b/arch/arm64/boot/dts/qcom/Makefile
> index 5113fac80b7a..d942d3ec3928 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM) +=
> msm8994-msft-lumia-cityman.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8994-sony-xperia-kitakami-sumire.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8996-mtp.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-asus-novago-tp370ql.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += msm8998-google-taimen.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-hp-envy-x2.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-lenovo-miix-630.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-mtp.dtb
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> new file mode 100644
> index ..ffaaafe14037
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
> @@ -0,0 +1,14 @@
> +// SPDX-License-Identifier: GPL-2.0-only
Would you be willing to release these as BSD-3-Clause instead?
> +/*
> + * Copyright (c) 2020, Caleb Connolly
> + */
> +
> +/dts-v1/;
> +
> +#include "msm8998-google-wahoo.dtsi"
> +
> +/ {
> + model = "Google Pixel 2 XL";
> + compatible = "google,taimen", "google,wahoo", "qcom,msm8998",
> "qcom,msm8998-mtp";
> + qcom,msm-id = <0x124 0x20001>;
> +};
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> new file mode 100644
> index ..0c221ead2df7
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
> @@ -0,0 +1,391 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/* Copyright (c) 2020 Caleb Connolly */
> +
> +#include "msm8998.dtsi"
> +#include "pm8998.dtsi"
> +#include "pmi8998.dtsi"
> +#include "pm8005.dtsi"
> +
> +/delete-node/ &mpss_mem;
> +/delete-node/ &venus_mem;
> +/delete-node/ &mba_mem;
> +/delete-node/ &slpi_mem;
> +
> +/ {
> + aliases {
> + };
> +
> + chosen {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + /* Add "earlycon" intended to be used in combination with UART
> serial console */
> + bootargs = "clk_ignore_unused earlycon
> console=ttyGS0,115200";// loglevel=10 drm.debug=15 debug";
Please drop earlycon from this list (user should be able to add it if
they care later?) and use stdout-path to set the console, like we do on
other devices.
> + };
> +
> + reserved-memory {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + mpss_mem: memory@8cc0 {
> + reg = <0 0x8cc0 0 0x780>;
> + no-map;
> + };
> +
> + venus_mem: memory@9440 {
> + reg = <0 0x9440 0 0x50>;
> + no-map;
> + };
> +
> + mba_mem: memory@9410 {
> + reg = <0 0x9490 0 0x20>;
> + no-map;
> + };
> +
> + slpi_mem: memory@94b0 {
> + reg = <0 0x94b0 0 0x70>;
> + no-map;
> + };
> +
> + ramoops: ramoops@a181 {
> + compatible = "ramoops";
> + reg = <0 0xa181 0 0x20>;
> + record-size = <0x2>;
> + console-size = <0x10>;
> + pmsg-size =
[PATCH] arm64: dts: add support for the Pixel 2 XL
Add a minimal devicetree capable of booting on the Pixel 2 XL MSM8998
device.
It's currently possible to boot the device into postmarketOS with USB
networking, however the display panel depends on Display Stream
Compression which is not yet supported in the kernel.
The bootloader also requires that the dtbo partition contains a device
tree overlay with a particular id which has to be overlayed onto the
existing dtb. It's possible to use a specially crafted dtbo partition to
workaround this, more information is available here:
https://gitlab.com/calebccff/dtbo-google-wahoo-mainline
Signed-off-by: Caleb Connolly
---
It's possible to get wifi working by running Bjorns diag-router in the
background, without this the wifi firmware crashes every 10 seconds or
so. This is the same issue encountered on the OnePlus 5.
arch/arm64/boot/dts/qcom/Makefile | 1 +
.../boot/dts/qcom/msm8998-google-taimen.dts | 14 +
.../boot/dts/qcom/msm8998-google-wahoo.dtsi | 391 ++
3 files changed, 406 insertions(+)
create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
create mode 100644 arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
diff --git a/arch/arm64/boot/dts/qcom/Makefile
b/arch/arm64/boot/dts/qcom/Makefile
index 5113fac80b7a..d942d3ec3928 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -16,6 +16,7 @@ dtb-$(CONFIG_ARCH_QCOM) +=
msm8994-msft-lumia-cityman.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8994-sony-xperia-kitakami-sumire.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8996-mtp.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8998-asus-novago-tp370ql.dtb
+dtb-$(CONFIG_ARCH_QCOM)+= msm8998-google-taimen.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8998-hp-envy-x2.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8998-lenovo-miix-630.dtb
dtb-$(CONFIG_ARCH_QCOM)+= msm8998-mtp.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
new file mode 100644
index ..ffaaafe14037
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8998-google-taimen.dts
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Copyright (c) 2020, Caleb Connolly
+ */
+
+/dts-v1/;
+
+#include "msm8998-google-wahoo.dtsi"
+
+/ {
+ model = "Google Pixel 2 XL";
+ compatible = "google,taimen", "google,wahoo", "qcom,msm8998",
"qcom,msm8998-mtp";
+ qcom,msm-id = <0x124 0x20001>;
+};
diff --git a/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
new file mode 100644
index ..0c221ead2df7
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8998-google-wahoo.dtsi
@@ -0,0 +1,391 @@
+// SPDX-License-Identifier: GPL-2.0
+/* Copyright (c) 2020 Caleb Connolly */
+
+#include "msm8998.dtsi"
+#include "pm8998.dtsi"
+#include "pmi8998.dtsi"
+#include "pm8005.dtsi"
+
+/delete-node/ &mpss_mem;
+/delete-node/ &venus_mem;
+/delete-node/ &mba_mem;
+/delete-node/ &slpi_mem;
+
+/ {
+ aliases {
+ };
+
+ chosen {
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
+
+ /* Add "earlycon" intended to be used in combination with UART
serial console */
+ bootargs = "clk_ignore_unused earlycon
console=ttyGS0,115200";// loglevel=10 drm.debug=15 debug";
+ };
+
+ reserved-memory {
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
+
+ mpss_mem: memory@8cc0 {
+ reg = <0 0x8cc0 0 0x780>;
+ no-map;
+ };
+
+ venus_mem: memory@9440 {
+ reg = <0 0x9440 0 0x50>;
+ no-map;
+ };
+
+ mba_mem: memory@9410 {
+ reg = <0 0x9490 0 0x20>;
+ no-map;
+ };
+
+ slpi_mem: memory@94b0 {
+ reg = <0 0x94b0 0 0x70>;
+ no-map;
+ };
+
+ ramoops: ramoops@a181 {
+ compatible = "ramoops";
+ reg = <0 0xa181 0 0x20>;
+ record-size = <0x2>;
+ console-size = <0x10>;
+ pmsg-size = <0x8>;
+ };
+ };
+
+ vph_pwr: vph-pwr-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "vph_pwr";
+ regulator-always-on;
+ regulator-boot-on;
+ };
+};
+
+&blsp1_uart3 {
+ status = "disabled";
+
+ bluetooth {
+ compatible = "qcom,wcn3990-bt";
+
+ vddio-supply = <&vreg_s4a_1p8>;
+ vddxo-supply = <&vreg_l7a_1p8>;
+ vddrf-supply = <&vreg_l17a_1p3>;
+ vddch0-supply = <&vr
