diff mbox series

[1/2] dt-bindings: arm: qcom: Document google,senor board

Message ID 1619674827-26650-1-git-send-email-rnayak@codeaurora.org
State Not Applicable, archived
Headers show
Series [1/2] dt-bindings: arm: qcom: Document google,senor board | expand

Checks

Context Check Description
robh/checkpatch success

Commit Message

Rajendra Nayak April 29, 2021, 5:40 a.m. UTC
Document the google,senor board based on sc7280 SoC

Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Matthias Kaehlcke April 29, 2021, 12:43 p.m. UTC | #1
On Thu, Apr 29, 2021 at 11:10:26AM +0530, Rajendra Nayak wrote:
> Document the google,senor board based on sc7280 SoC
> 
> Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>

Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Matthias Kaehlcke April 29, 2021, 12:46 p.m. UTC | #2
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>
Doug Anderson April 29, 2021, 1:39 p.m. UTC | #3
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>
Doug Anderson April 29, 2021, 1:39 p.m. UTC | #4
Hi,

On Wed, Apr 28, 2021 at 10:41 PM Rajendra Nayak <rnayak@codeaurora.org> wrote:
>
> Document the google,senor board based on sc7280 SoC
>
> Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Douglas Anderson <dianders@chromium.org>
Robert P. J. Day April 29, 2021, 1:43 p.m. UTC | #5
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
Rajendra Nayak April 30, 2021, 4:19 a.m. UTC | #6
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
>
Rob Herring May 3, 2021, 7:32 p.m. UTC | #7
On Thu, 29 Apr 2021 11:10:26 +0530, Rajendra Nayak wrote:
> Document the google,senor board based on sc7280 SoC
> 
> Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 9b27e99..2babb95 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -178,6 +178,7 @@  properties:
       - items:
           - enum:
               - qcom,sc7280-idp
+              - google,senor
           - const: qcom,sc7280
 
       - items: