| Message ID | 1619674827-26650-2-git-send-email-rnayak@codeaurora.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 9d6e639cbaa1f27fedccb456dd2f70ecc39b5aca |
| Headers | show |
| Series | [1/2] dt-bindings: arm: qcom: Document google,senor board | expand |
On Thu, Apr 29, 2021 at 11:10:27AM +0530, Rajendra Nayak wrote: > The sc7280 IDP board is also called senor in the Chrome OS builds. > Add the "google,senor" compatible so coreboot/depthcharge knows what > device tree blob to pick > > Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Hi, On Wed, Apr 28, 2021 at 10:40 PM Rajendra Nayak <rnayak@codeaurora.org> wrote: > > The sc7280 IDP board is also called senor in the Chrome OS builds. > Add the "google,senor" compatible so coreboot/depthcharge knows what > device tree blob to pick > > Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org> > --- > arch/arm64/boot/dts/qcom/sc7280-idp.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Douglas Anderson <dianders@chromium.org>
On Thu, 29 Apr 2021, Doug Anderson wrote: > Hi, > > On Wed, Apr 28, 2021 at 10:40 PM Rajendra Nayak <rnayak@codeaurora.org> wrote: > > > > The sc7280 IDP board is also called senor in the Chrome OS builds. > > Add the "google,senor" compatible so coreboot/depthcharge knows what > > device tree blob to pick > > > > Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org> > > --- > > arch/arm64/boot/dts/qcom/sc7280-idp.dts | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Reviewed-by: Douglas Anderson <dianders@chromium.org> subject line contains "google,senor" rday
On 4/29/2021 7:13 PM, Robert P. J. Day wrote: > On Thu, 29 Apr 2021, Doug Anderson wrote: > >> Hi, >> >> On Wed, Apr 28, 2021 at 10:40 PM Rajendra Nayak <rnayak@codeaurora.org> wrote: >>> >>> The sc7280 IDP board is also called senor in the Chrome OS builds. >>> Add the "google,senor" compatible so coreboot/depthcharge knows what >>> device tree blob to pick >>> >>> Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org> >>> --- >>> arch/arm64/boot/dts/qcom/sc7280-idp.dts | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> Reviewed-by: Douglas Anderson <dianders@chromium.org> > > subject line contains "google,senor" are you suggesting it should not? > > rday >
diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dts b/arch/arm64/boot/dts/qcom/sc7280-idp.dts index 54d2cb3..a79fb5a 100644 --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dts +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dts @@ -11,7 +11,7 @@ / { model = "Qualcomm Technologies, Inc. sc7280 IDP platform"; - compatible = "qcom,sc7280-idp", "qcom,sc7280"; + compatible = "qcom,sc7280-idp", "google,senor", "qcom,sc7280"; aliases { serial0 = &uart5;
The sc7280 IDP board is also called senor in the Chrome OS builds. Add the "google,senor" compatible so coreboot/depthcharge knows what device tree blob to pick Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org> --- arch/arm64/boot/dts/qcom/sc7280-idp.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)