* [PATCH 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region @ 2023-07-05 5:39 Komal Bajaj 2023-07-05 5:39 ` [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: " Komal Bajaj 2023-07-05 5:39 ` [PATCH 2/2] arm64: dts: qcom: qru1000-idp: " Komal Bajaj 0 siblings, 2 replies; 10+ messages in thread From: Komal Bajaj @ 2023-07-05 5:39 UTC (permalink / raw) To: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel, Komal Bajaj Update reserved memory region for QDU1000 and QRU1000 SoCs. Komal Bajaj (2): arm64: dts: qcom: qdu1000-idp: Update reserved memory region arm64: dts: qcom: qru1000-idp: Update reserved memory region arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ arch/arm64/boot/dts/qcom/qru1000-idp.dts | 16 +++++++++++++++ 2 files changed, 42 insertions(+) -- 2.40.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 5:39 [PATCH 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region Komal Bajaj @ 2023-07-05 5:39 ` Komal Bajaj 2023-07-05 5:49 ` Dmitry Baryshkov 2023-07-05 6:13 ` Krzysztof Kozlowski 2023-07-05 5:39 ` [PATCH 2/2] arm64: dts: qcom: qru1000-idp: " Komal Bajaj 1 sibling, 2 replies; 10+ messages in thread From: Komal Bajaj @ 2023-07-05 5:39 UTC (permalink / raw) To: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel, Komal Bajaj Add missing reserved regions as described in QDU1000 memory map. Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> --- arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts index 1d22f87fd238..3f5512ec0a90 100644 --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts @@ -448,6 +448,32 @@ &qupv3_id_0 { status = "okay"; }; +&reserved_memory{ + #address-cells = <2>; + #size-cells = <2>; + ranges; + + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ + no-map; + reg = <0x0 0xe0000000 0x0 0x20000000>; + }; + + harq_buffer_mem:harq_buffer_region@800000000{ + no-map; + reg = <0x8 0x0 0x0 0x80000000>; + }; + + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ + no-map; + reg = <0x8 0x80000000 0x0 0x50000000>; + }; + + fapi_buffer_mem:fapi_buffer_region@8d0000000{ + no-map; + reg = <0x8 0xd0000000 0x0 0x20000000>; + }; +}; + &sdhc { pinctrl-0 = <&sdc_on_state>; pinctrl-1 = <&sdc_off_state>; -- 2.40.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 5:39 ` [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: " Komal Bajaj @ 2023-07-05 5:49 ` Dmitry Baryshkov 2023-07-05 7:06 ` Komal Bajaj 2023-07-05 6:13 ` Krzysztof Kozlowski 1 sibling, 1 reply; 10+ messages in thread From: Dmitry Baryshkov @ 2023-07-05 5:49 UTC (permalink / raw) To: Komal Bajaj Cc: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: > > Add missing reserved regions as described in QDU1000 memory map. > > Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> > --- > arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > index 1d22f87fd238..3f5512ec0a90 100644 > --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > @@ -448,6 +448,32 @@ &qupv3_id_0 { > status = "okay"; > }; > > +&reserved_memory{ > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ no_underscores_in_node_names. Ever. Also, if you have checked other platforms, you'd have seen that other platforms use a much more generic node name for 'memory' nodes (which you should have used too). > + no-map; > + reg = <0x0 0xe0000000 0x0 0x20000000>; > + }; > + > + harq_buffer_mem:harq_buffer_region@800000000{ > + no-map; > + reg = <0x8 0x0 0x0 0x80000000>; > + }; > + > + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ > + no-map; > + reg = <0x8 0x80000000 0x0 0x50000000>; > + }; > + > + fapi_buffer_mem:fapi_buffer_region@8d0000000{ > + no-map; > + reg = <0x8 0xd0000000 0x0 0x20000000>; > + }; > +}; > + > &sdhc { > pinctrl-0 = <&sdc_on_state>; > pinctrl-1 = <&sdc_off_state>; > -- > 2.40.1 > -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 5:49 ` Dmitry Baryshkov @ 2023-07-05 7:06 ` Komal Bajaj 2023-07-05 9:42 ` Dmitry Baryshkov 0 siblings, 1 reply; 10+ messages in thread From: Komal Bajaj @ 2023-07-05 7:06 UTC (permalink / raw) To: Dmitry Baryshkov Cc: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On 7/5/2023 11:19 AM, Dmitry Baryshkov wrote: > On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >> Add missing reserved regions as described in QDU1000 memory map. >> >> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >> index 1d22f87fd238..3f5512ec0a90 100644 >> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >> @@ -448,6 +448,32 @@ &qupv3_id_0 { >> status = "okay"; >> }; >> >> +&reserved_memory{ >> + #address-cells = <2>; >> + #size-cells = <2>; >> + ranges; >> + >> + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ > no_underscores_in_node_names. Ever. > > Also, if you have checked other platforms, you'd have seen that other > platforms use a much more generic node name for 'memory' nodes (which > you should have used too). These memory nodes are new to QDU platform, so will it be okay if I keep these names without region suffix? Thanks Komal > >> + no-map; >> + reg = <0x0 0xe0000000 0x0 0x20000000>; >> + }; >> + >> + harq_buffer_mem:harq_buffer_region@800000000{ >> + no-map; >> + reg = <0x8 0x0 0x0 0x80000000>; >> + }; >> + >> + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ >> + no-map; >> + reg = <0x8 0x80000000 0x0 0x50000000>; >> + }; >> + >> + fapi_buffer_mem:fapi_buffer_region@8d0000000{ >> + no-map; >> + reg = <0x8 0xd0000000 0x0 0x20000000>; >> + }; >> +}; >> + >> &sdhc { >> pinctrl-0 = <&sdc_on_state>; >> pinctrl-1 = <&sdc_off_state>; >> -- >> 2.40.1 >> > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 7:06 ` Komal Bajaj @ 2023-07-05 9:42 ` Dmitry Baryshkov 2023-07-05 11:02 ` Komal Bajaj 2023-07-05 11:08 ` Konrad Dybcio 0 siblings, 2 replies; 10+ messages in thread From: Dmitry Baryshkov @ 2023-07-05 9:42 UTC (permalink / raw) To: Komal Bajaj Cc: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On Wed, 5 Jul 2023 at 10:06, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: > > > > On 7/5/2023 11:19 AM, Dmitry Baryshkov wrote: > > On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: > >> Add missing reserved regions as described in QDU1000 memory map. > >> > >> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> > >> --- > >> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ > >> 1 file changed, 26 insertions(+) > >> > >> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > >> index 1d22f87fd238..3f5512ec0a90 100644 > >> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > >> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > >> @@ -448,6 +448,32 @@ &qupv3_id_0 { > >> status = "okay"; > >> }; > >> > >> +&reserved_memory{ > >> + #address-cells = <2>; > >> + #size-cells = <2>; > >> + ranges; > >> + > >> + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ > > no_underscores_in_node_names. Ever. > > > > Also, if you have checked other platforms, you'd have seen that other > > platforms use a much more generic node name for 'memory' nodes (which > > you should have used too). > > These memory nodes are new to QDU platform, so will it be okay if I keep > these names without region suffix? Just use 'memory@abcd'. > > Thanks > Komal > > > > >> + no-map; > >> + reg = <0x0 0xe0000000 0x0 0x20000000>; > >> + }; > >> + > >> + harq_buffer_mem:harq_buffer_region@800000000{ > >> + no-map; > >> + reg = <0x8 0x0 0x0 0x80000000>; > >> + }; > >> + > >> + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ > >> + no-map; > >> + reg = <0x8 0x80000000 0x0 0x50000000>; > >> + }; > >> + > >> + fapi_buffer_mem:fapi_buffer_region@8d0000000{ > >> + no-map; > >> + reg = <0x8 0xd0000000 0x0 0x20000000>; > >> + }; > >> +}; > >> + > >> &sdhc { > >> pinctrl-0 = <&sdc_on_state>; > >> pinctrl-1 = <&sdc_off_state>; > >> -- > >> 2.40.1 > >> > > > -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 9:42 ` Dmitry Baryshkov @ 2023-07-05 11:02 ` Komal Bajaj 2023-07-05 11:08 ` Konrad Dybcio 1 sibling, 0 replies; 10+ messages in thread From: Komal Bajaj @ 2023-07-05 11:02 UTC (permalink / raw) To: Dmitry Baryshkov Cc: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On 7/5/2023 3:12 PM, Dmitry Baryshkov wrote: > On Wed, 5 Jul 2023 at 10:06, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >> >> >> On 7/5/2023 11:19 AM, Dmitry Baryshkov wrote: >>> On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >>>> Add missing reserved regions as described in QDU1000 memory map. >>>> >>>> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> >>>> --- >>>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ >>>> 1 file changed, 26 insertions(+) >>>> >>>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> index 1d22f87fd238..3f5512ec0a90 100644 >>>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> @@ -448,6 +448,32 @@ &qupv3_id_0 { >>>> status = "okay"; >>>> }; >>>> >>>> +&reserved_memory{ >>>> + #address-cells = <2>; >>>> + #size-cells = <2>; >>>> + ranges; >>>> + >>>> + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ >>> no_underscores_in_node_names. Ever. >>> >>> Also, if you have checked other platforms, you'd have seen that other >>> platforms use a much more generic node name for 'memory' nodes (which >>> you should have used too). >> These memory nodes are new to QDU platform, so will it be okay if I keep >> these names without region suffix? > Just use 'memory@abcd'. Sure, will update like that. > >> Thanks >> Komal >> >>>> + no-map; >>>> + reg = <0x0 0xe0000000 0x0 0x20000000>; >>>> + }; >>>> + >>>> + harq_buffer_mem:harq_buffer_region@800000000{ >>>> + no-map; >>>> + reg = <0x8 0x0 0x0 0x80000000>; >>>> + }; >>>> + >>>> + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ >>>> + no-map; >>>> + reg = <0x8 0x80000000 0x0 0x50000000>; >>>> + }; >>>> + >>>> + fapi_buffer_mem:fapi_buffer_region@8d0000000{ >>>> + no-map; >>>> + reg = <0x8 0xd0000000 0x0 0x20000000>; >>>> + }; >>>> +}; >>>> + >>>> &sdhc { >>>> pinctrl-0 = <&sdc_on_state>; >>>> pinctrl-1 = <&sdc_off_state>; >>>> -- >>>> 2.40.1 >>>> > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 9:42 ` Dmitry Baryshkov 2023-07-05 11:02 ` Komal Bajaj @ 2023-07-05 11:08 ` Konrad Dybcio 2023-07-05 14:19 ` Krzysztof Kozlowski 1 sibling, 1 reply; 10+ messages in thread From: Konrad Dybcio @ 2023-07-05 11:08 UTC (permalink / raw) To: Dmitry Baryshkov, Komal Bajaj Cc: agross, andersson, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On 5.07.2023 11:42, Dmitry Baryshkov wrote: > On Wed, 5 Jul 2023 at 10:06, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >> >> >> >> On 7/5/2023 11:19 AM, Dmitry Baryshkov wrote: >>> On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >>>> Add missing reserved regions as described in QDU1000 memory map. >>>> >>>> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> >>>> --- >>>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ >>>> 1 file changed, 26 insertions(+) >>>> >>>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> index 1d22f87fd238..3f5512ec0a90 100644 >>>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>> @@ -448,6 +448,32 @@ &qupv3_id_0 { >>>> status = "okay"; >>>> }; >>>> >>>> +&reserved_memory{ >>>> + #address-cells = <2>; >>>> + #size-cells = <2>; >>>> + ranges; >>>> + >>>> + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ >>> no_underscores_in_node_names. Ever. >>> >>> Also, if you have checked other platforms, you'd have seen that other >>> platforms use a much more generic node name for 'memory' nodes (which >>> you should have used too). >> >> These memory nodes are new to QDU platform, so will it be okay if I keep >> these names without region suffix? > > Just use 'memory@abcd'. Documentation/devicetree/bindings/reserved-memory/reserved-memory.yaml Following the generic-names recommended practice, node names should reflect the purpose of the node (ie. "framebuffer" or "dma-pool"). Unit address (@<address>) should be appended to the name if the node is a static allocation. Konrad > >> >> Thanks >> Komal >> >>> >>>> + no-map; >>>> + reg = <0x0 0xe0000000 0x0 0x20000000>; >>>> + }; >>>> + >>>> + harq_buffer_mem:harq_buffer_region@800000000{ >>>> + no-map; >>>> + reg = <0x8 0x0 0x0 0x80000000>; >>>> + }; >>>> + >>>> + tenx_sp_buffer_mem:tenx_sp_buffer_region@880000000{ >>>> + no-map; >>>> + reg = <0x8 0x80000000 0x0 0x50000000>; >>>> + }; >>>> + >>>> + fapi_buffer_mem:fapi_buffer_region@8d0000000{ >>>> + no-map; >>>> + reg = <0x8 0xd0000000 0x0 0x20000000>; >>>> + }; >>>> +}; >>>> + >>>> &sdhc { >>>> pinctrl-0 = <&sdc_on_state>; >>>> pinctrl-1 = <&sdc_off_state>; >>>> -- >>>> 2.40.1 >>>> >>> >> > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 11:08 ` Konrad Dybcio @ 2023-07-05 14:19 ` Krzysztof Kozlowski 0 siblings, 0 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2023-07-05 14:19 UTC (permalink / raw) To: Konrad Dybcio, Dmitry Baryshkov, Komal Bajaj Cc: agross, andersson, robh+dt, krzysztof.kozlowski+dt, conor+dt, linux-arm-msm, devicetree, linux-kernel On 05/07/2023 13:08, Konrad Dybcio wrote: > On 5.07.2023 11:42, Dmitry Baryshkov wrote: >> On Wed, 5 Jul 2023 at 10:06, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >>> >>> >>> >>> On 7/5/2023 11:19 AM, Dmitry Baryshkov wrote: >>>> On Wed, 5 Jul 2023 at 08:40, Komal Bajaj <quic_kbajaj@quicinc.com> wrote: >>>>> Add missing reserved regions as described in QDU1000 memory map. >>>>> >>>>> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> >>>>> --- >>>>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ >>>>> 1 file changed, 26 insertions(+) >>>>> >>>>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>>> index 1d22f87fd238..3f5512ec0a90 100644 >>>>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts >>>>> @@ -448,6 +448,32 @@ &qupv3_id_0 { >>>>> status = "okay"; >>>>> }; >>>>> >>>>> +&reserved_memory{ >>>>> + #address-cells = <2>; >>>>> + #size-cells = <2>; >>>>> + ranges; >>>>> + >>>>> + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ >>>> no_underscores_in_node_names. Ever. >>>> >>>> Also, if you have checked other platforms, you'd have seen that other >>>> platforms use a much more generic node name for 'memory' nodes (which >>>> you should have used too). >>> >>> These memory nodes are new to QDU platform, so will it be okay if I keep >>> these names without region suffix? >> >> Just use 'memory@abcd'. > Documentation/devicetree/bindings/reserved-memory/reserved-memory.yaml > > Following the generic-names recommended practice, node names should > reflect the purpose of the node (ie. "framebuffer" or "dma-pool"). > Unit address (@<address>) should be appended to the name if the node > is a static allocation. > Yeah, two minutes too late :) Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region 2023-07-05 5:39 ` [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: " Komal Bajaj 2023-07-05 5:49 ` Dmitry Baryshkov @ 2023-07-05 6:13 ` Krzysztof Kozlowski 1 sibling, 0 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2023-07-05 6:13 UTC (permalink / raw) To: Komal Bajaj, agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel On 05/07/2023 07:39, Komal Bajaj wrote: > Add missing reserved regions as described in QDU1000 memory map. > > Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> > --- > arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 26 ++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > index 1d22f87fd238..3f5512ec0a90 100644 > --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts > @@ -448,6 +448,32 @@ &qupv3_id_0 { > status = "okay"; > }; > > +&reserved_memory{ > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; Drop, you should not neeed it. > + > + ecc_meta_data_reserved_mem:ecc_meta_data_reserved_region@e0000000{ That's not correct style. Missing space after ':', no underscores in node names. reg is first property, not second. Please open existing files and look how it is done there. Srsly, just open qdu1000.dtsi which you are extending. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm64: dts: qcom: qru1000-idp: Update reserved memory region 2023-07-05 5:39 [PATCH 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region Komal Bajaj 2023-07-05 5:39 ` [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: " Komal Bajaj @ 2023-07-05 5:39 ` Komal Bajaj 1 sibling, 0 replies; 10+ messages in thread From: Komal Bajaj @ 2023-07-05 5:39 UTC (permalink / raw) To: agross, andersson, konrad.dybcio, robh+dt, krzysztof.kozlowski+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel, Komal Bajaj Add missing reserved regions as described in QRU1000 memory map. Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com> --- arch/arm64/boot/dts/qcom/qru1000-idp.dts | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts index 2cc893ae4d10..836804020946 100644 --- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts +++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts @@ -448,6 +448,22 @@ &qupv3_id_0 { status = "okay"; }; +&reserved_memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + ecc_meta_data_reserved_mem: ecc_meta_data_reserved_region@f0000000 { + no-map; + reg = <0x0 0xf0000000 0x0 0x10000000>; + }; + + tenx_sp_mem: tenx_sp_region@800000000 { + no-map; + reg = <0x8 0x0 0x0 0x80000000>; + }; +}; + &uart7 { status = "okay"; }; -- 2.40.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-07-05 14:19 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-05 5:39 [PATCH 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region Komal Bajaj 2023-07-05 5:39 ` [PATCH 1/2] arm64: dts: qcom: qdu1000-idp: " Komal Bajaj 2023-07-05 5:49 ` Dmitry Baryshkov 2023-07-05 7:06 ` Komal Bajaj 2023-07-05 9:42 ` Dmitry Baryshkov 2023-07-05 11:02 ` Komal Bajaj 2023-07-05 11:08 ` Konrad Dybcio 2023-07-05 14:19 ` Krzysztof Kozlowski 2023-07-05 6:13 ` Krzysztof Kozlowski 2023-07-05 5:39 ` [PATCH 2/2] arm64: dts: qcom: qru1000-idp: " Komal Bajaj
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).