| Message ID | f5d4d71cd59f25b80889ef88fa044aa3a4268d46.1711288736.git.stano.jakubek@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/2] dt-bindings: arm: qcom: Add Motorola Moto G (2013) | expand |
On 24.03.2024 3:04 PM, Stanislav Jakubek wrote: > Add a device tree for the Motorola Moto G (2013) smartphone based > on the Qualcomm MSM8226 SoC. > > Initially supported features: > - Buttons (Volume Down/Up, Power) > - eMMC > - Hall Effect Sensor > - SimpleFB display > - TMP108 temperature sensor > - Vibrator > > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> > --- [...] > + hob-ram@f500000 { > + reg = <0x0f500000 0x40000>, > + <0x0f540000 0x2000>; > + no-map; > + }; Any reason it's in two parts? Should it be one contiguous region, or two separate nodes? lgtm otherwise
On Mon, Mar 25, 2024 at 08:28:27PM +0100, Konrad Dybcio wrote: > On 24.03.2024 3:04 PM, Stanislav Jakubek wrote: > > Add a device tree for the Motorola Moto G (2013) smartphone based > > on the Qualcomm MSM8226 SoC. > > > > Initially supported features: > > - Buttons (Volume Down/Up, Power) > > - eMMC > > - Hall Effect Sensor > > - SimpleFB display > > - TMP108 temperature sensor > > - Vibrator > > > > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> > > --- > > [...] > > > + hob-ram@f500000 { > > + reg = <0x0f500000 0x40000>, > > + <0x0f540000 0x2000>; > > + no-map; > > + }; > > Any reason it's in two parts? Should it be one contiguous region, or > two separate nodes? > > lgtm otherwise Hi Konrad, I copied this from downstream as-is. According to the downstream docs [1]: HOB RAM MMAP Device provides ability for userspace to access the hand over block memory to read out modem related parameters. And the two regs are the "DHOB partition" and "SHOB partition". I suppose this is something Motorola (firmware?) specific (since the downstream compatible is mmi,hob_ram [2]). Should I split this into 2 nodes - dhob@f500000 and shob@f540000? Stanislav [1] https://github.com/LineageOS/android_kernel_motorola_msm8226/blob/cm-14.1/Documentation/devicetree/bindings/misc/hob_ram.txt [2] https://github.com/LineageOS/android_kernel_motorola_msm8226/blob/cm-14.1/arch/arm/boot/dts/msm8226-moto-common.dtsi#L258
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml index 66beaac60e1d..d2910982ae86 100644 --- a/Documentation/devicetree/bindings/arm/qcom.yaml +++ b/Documentation/devicetree/bindings/arm/qcom.yaml @@ -137,6 +137,7 @@ properties: - microsoft,dempsey - microsoft,makepeace - microsoft,moneypenny + - motorola,falcon - samsung,s3ve3g - const: qcom,msm8226
Document the Motorola Moto G (2013), which is a smartphone based on the Qualcomm MSM8226 SoC. Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> --- Documentation/devicetree/bindings/arm/qcom.yaml | 1 + 1 file changed, 1 insertion(+)