All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board
@ 2022-07-28 21:33 Bob Moragues
  2022-07-28 21:33 ` [PATCH v3 2/2] arm64: dts: qcom: sc7280: Add support for zoglin Bob Moragues
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Bob Moragues @ 2022-07-28 21:33 UTC (permalink / raw)
  To: LKML
  Cc: swboyd, mka, dianders, Bob Moragues, Rob Herring, Andy Gross,
	Bjorn Andersson, Rob Herring, Stephen Boyd, devicetree,
	linux-arm-msm

Zoglin is a Hoglin Chromebook with SPI Flash reduced from 64MB to 8MB.
Zoglin is identical to Hoglin except for the SPI Flash.
The actual SPI Flash is dynamically probed at and not specified in DTS.

Reviewed-by: Douglas Anderson <dianders@chromium.org>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Bob Moragues <moragues@chromium.org>
---

Changes in v3:
- Fixed Author
- Fixed duplicate Signed-off-by

 Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 581485392404..63091df3cbb3 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -475,6 +475,7 @@ properties:
 
       - description: Qualcomm Technologies, Inc. sc7280 CRD platform (newest rev)
         items:
+          - const: google,zoglin
           - const: google,hoglin
           - const: qcom,sc7280
 
-- 
2.37.1.455.g008518b4e5-goog


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH v3 2/2] arm64: dts: qcom: sc7280: Add support for zoglin
  2022-07-28 21:33 [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Bob Moragues
@ 2022-07-28 21:33 ` Bob Moragues
  2022-07-28 21:47 ` [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Doug Anderson
  2022-08-29 23:46 ` (subset) " Bjorn Andersson
  2 siblings, 0 replies; 4+ messages in thread
From: Bob Moragues @ 2022-07-28 21:33 UTC (permalink / raw)
  To: LKML
  Cc: swboyd, mka, dianders, Bob Moragues, Andy Gross, Bjorn Andersson,
	Rob Herring, devicetree, linux-arm-msm

Zoglin is a Hoglin Chromebook with SPI Flash reduced from 64MB to 8MB.
Zoglin is identical to Hoglin except for the SPI Flash.
The actual SPI Flash is dynamically probed at and not specified in DTS.

Signed-off-by: Bob Moragues <moragues@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
---

Changes in v3:
- Reset Author
- Fixed duplicate Signed-off-by

 arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dts
index 6d324ac2dda1..98d09187fc69 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dts
@@ -13,7 +13,7 @@
 
 / {
 	model = "Qualcomm Technologies, Inc. sc7280 CRD platform (rev5+)";
-	compatible = "google,hoglin", "qcom,sc7280";
+	compatible = "google,zoglin", "google,hoglin", "qcom,sc7280";
 
 	/* FIXED REGULATORS */
 
-- 
2.37.1.455.g008518b4e5-goog


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board
  2022-07-28 21:33 [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Bob Moragues
  2022-07-28 21:33 ` [PATCH v3 2/2] arm64: dts: qcom: sc7280: Add support for zoglin Bob Moragues
@ 2022-07-28 21:47 ` Doug Anderson
  2022-08-29 23:46 ` (subset) " Bjorn Andersson
  2 siblings, 0 replies; 4+ messages in thread
From: Doug Anderson @ 2022-07-28 21:47 UTC (permalink / raw)
  To: Bob Moragues
  Cc: LKML, Stephen Boyd, Matthias Kaehlcke, Rob Herring, Andy Gross,
	Bjorn Andersson, Rob Herring, Stephen Boyd,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-arm-msm

Hi,

On Thu, Jul 28, 2022 at 2:33 PM Bob Moragues <moragues@chromium.org> wrote:
>
> Zoglin is a Hoglin Chromebook with SPI Flash reduced from 64MB to 8MB.
> Zoglin is identical to Hoglin except for the SPI Flash.
> The actual SPI Flash is dynamically probed at and not specified in DTS.
>
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Bob Moragues <moragues@chromium.org>

A nit from my point of view is that I would have expected your
Signed-off-by to be the first tag. As I understand it the tags are
usually supposed to be a top-to-bottom history of what happened. You
signed off on the patch and then people added reviews / acks. I don't
personally think this is important enough to spin a v4 for, though the
last word would be Bjorn.

> ---
>
> Changes in v3:
> - Fixed Author
> - Fixed duplicate Signed-off-by

Technically this should have been:

Changes in v3:
- Really really fixed Signed-off-by

Changes in v2:
- Fixed Author
- Fixed duplicate Signed-off-by

...but that's also not really worth spinning for.

I think this looks good now. Though unofficial (and not stable git
hashes), I've been trying to keep track of some patches during this
dead time till -rc1. I've put your two patches here:

https://github.com/dianders/kernel-staging/commits/qcom/arm64-staging

-Doug

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: (subset) [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board
  2022-07-28 21:33 [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Bob Moragues
  2022-07-28 21:33 ` [PATCH v3 2/2] arm64: dts: qcom: sc7280: Add support for zoglin Bob Moragues
  2022-07-28 21:47 ` [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Doug Anderson
@ 2022-08-29 23:46 ` Bjorn Andersson
  2 siblings, 0 replies; 4+ messages in thread
From: Bjorn Andersson @ 2022-08-29 23:46 UTC (permalink / raw)
  To: LKML, moragues
  Cc: mka, dianders, linux-arm-msm, robh+dt, swboyd, robh, devicetree,
	sboyd, agross

On Thu, 28 Jul 2022 14:33:20 -0700, Bob Moragues wrote:
> Zoglin is a Hoglin Chromebook with SPI Flash reduced from 64MB to 8MB.
> Zoglin is identical to Hoglin except for the SPI Flash.
> The actual SPI Flash is dynamically probed at and not specified in DTS.
> 
> 

Applied, thanks!

[1/2] dt-bindings: arm: qcom: document zoglin board
      commit: f72a9f525b98aca1ad134e163e070d614ff60a24
[2/2] arm64: dts: qcom: sc7280: Add support for zoglin
      commit: 2cb88ebe44531ba18c908e627811698c9461b4cb

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-08-29 23:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-28 21:33 [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Bob Moragues
2022-07-28 21:33 ` [PATCH v3 2/2] arm64: dts: qcom: sc7280: Add support for zoglin Bob Moragues
2022-07-28 21:47 ` [PATCH v3 1/2] dt-bindings: arm: qcom: document zoglin board Doug Anderson
2022-08-29 23:46 ` (subset) " Bjorn Andersson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.