mbox series

[v2,0/8] Add ipq806x missing bindings

Message ID 20220707010943.20857-1-ansuelsmth@gmail.com (mailing list archive)
Headers show
Series Add ipq806x missing bindings | expand

Message

Christian Marangi July 7, 2022, 1:09 a.m. UTC
This is a respin of "Multiple addition to ipq8064 dtsi" series
with major changes and some commit removed.

This series try to add some of the missing bindings for ipq806x.

This still lacks of the cpu bindings and all the bindings required
to scale cpu clk or L2. These will come later as the driver and
documentation require some changes.

So for now we try to add bindings that can directly applied without
making changes to any drivers.

Changes:
v2:
- Dropped "add sic non secure node for ipq8064" (no user found)
- Dropped "add smem node for ipq8064" (require changes to the driver
  to remove syscon node and use regs directly, will be added in a
  different series)
- Dropped "fix dtc warning for missing #address-cells for ipq8064" (
  doesn't actually fix any warning)
- Changed "enable usb phy by default for ipq8064" to disable usb phy
  by default
- Reworked rpm smb208 declaration to a saparate dtsi
- Fixed order with compatible-reg-status
- Squashed "remove redundant binding from ipq8064 rb3011 dts" with
  "add multiple missing pin definition for ipq8064" to prevent dtc
  error.
- Add review tag for snps patch

Changes from "Multiple addition to ipq8064 dtsi":
v3:
- Fix error in rb3011 dts
- Add tested-by tag on the entire series
v2:
- Added missing patch
- Added additional gsbi6 spi
- Added extra description for L2 cache opp
- Fxied smb208 enabled by default that is problematic for rb3011 devices

Christian Marangi (8):
  ARM: dts: qcom: add multiple missing pin definition for ipq8064
  ARM: dts: qcom: add gsbi6 missing definition for ipq8064
  ARM: dts: qcom: add specific ipq8064 dtsi with smb208 rpm regulators
  ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064
  ARM: dts: qcom: disable usb phy by default for ipq8064
  ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
  ARM: dts: qcom: fix and add some missing gsbi node for ipq8064
  ARM: dts: qcom: add speedbin efuse nvmem node

 arch/arm/boot/dts/qcom-ipq8064-ap148.dts   |   6 -
 arch/arm/boot/dts/qcom-ipq8064-rb3011.dts  |   9 --
 arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi |  37 +++++
 arch/arm/boot/dts/qcom-ipq8064.dtsi        | 150 +++++++++++++++++++--
 4 files changed, 179 insertions(+), 23 deletions(-)
 create mode 100644 arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi

Comments

Bjorn Andersson July 17, 2022, 2:46 a.m. UTC | #1
On Wed 06 Jul 20:09 CDT 2022, Christian Marangi wrote:

> This is a respin of "Multiple addition to ipq8064 dtsi" series
> with major changes and some commit removed.
> 
> This series try to add some of the missing bindings for ipq806x.
> 
> This still lacks of the cpu bindings and all the bindings required
> to scale cpu clk or L2. These will come later as the driver and
> documentation require some changes.
> 
> So for now we try to add bindings that can directly applied without
> making changes to any drivers.
> 
> Changes:
> v2:
> - Dropped "add sic non secure node for ipq8064" (no user found)
> - Dropped "add smem node for ipq8064" (require changes to the driver
>   to remove syscon node and use regs directly, will be added in a
>   different series)
> - Dropped "fix dtc warning for missing #address-cells for ipq8064" (
>   doesn't actually fix any warning)
> - Changed "enable usb phy by default for ipq8064" to disable usb phy
>   by default
> - Reworked rpm smb208 declaration to a saparate dtsi
> - Fixed order with compatible-reg-status
> - Squashed "remove redundant binding from ipq8064 rb3011 dts" with
>   "add multiple missing pin definition for ipq8064" to prevent dtc
>   error.
> - Add review tag for snps patch
> 
> Changes from "Multiple addition to ipq8064 dtsi":
> v3:
> - Fix error in rb3011 dts
> - Add tested-by tag on the entire series
> v2:
> - Added missing patch
> - Added additional gsbi6 spi
> - Added extra description for L2 cache opp
> - Fxied smb208 enabled by default that is problematic for rb3011 devices
> 
> Christian Marangi (8):
>   ARM: dts: qcom: add multiple missing pin definition for ipq8064
>   ARM: dts: qcom: add gsbi6 missing definition for ipq8064
>   ARM: dts: qcom: add specific ipq8064 dtsi with smb208 rpm regulators
>   ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064
>   ARM: dts: qcom: disable usb phy by default for ipq8064
>   ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
>   ARM: dts: qcom: fix and add some missing gsbi node for ipq8064
>   ARM: dts: qcom: add speedbin efuse nvmem node

Please prefix your ipa8064-specific patches with
  "ARM: dts: qcom: ipq8064: ..."

It makes the git history easier to read, so I fixed it up while applying
these.

Thanks,
Bjorn

> 
>  arch/arm/boot/dts/qcom-ipq8064-ap148.dts   |   6 -
>  arch/arm/boot/dts/qcom-ipq8064-rb3011.dts  |   9 --
>  arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi |  37 +++++
>  arch/arm/boot/dts/qcom-ipq8064.dtsi        | 150 +++++++++++++++++++--
>  4 files changed, 179 insertions(+), 23 deletions(-)
>  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi
> 
> -- 
> 2.36.1
>
Bjorn Andersson July 17, 2022, 3:08 a.m. UTC | #2
On Thu, 7 Jul 2022 03:09:34 +0200, Christian Marangi wrote:
> This is a respin of "Multiple addition to ipq8064 dtsi" series
> with major changes and some commit removed.
> 
> This series try to add some of the missing bindings for ipq806x.
> 
> This still lacks of the cpu bindings and all the bindings required
> to scale cpu clk or L2. These will come later as the driver and
> documentation require some changes.
> 
> [...]

Applied, thanks!

[1/8] ARM: dts: qcom: add multiple missing pin definition for ipq8064
      commit: 4af1defb305798d1a064a5ea0d0c9b30e5eee185
[2/8] ARM: dts: qcom: add gsbi6 missing definition for ipq8064
      commit: d883a12a547b6d42e795ff3b5ac87cfd013b5423
[3/8] ARM: dts: qcom: add specific ipq8064 dtsi with smb208 rpm regulators
      commit: 5c47a46d5e942ea6b041c8b7727b201817c1ff76
[4/8] ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064
      commit: 0ce34e0c13e99c239cce6099f64b0e95697f36b1
[5/8] ARM: dts: qcom: disable usb phy by default for ipq8064
      commit: d63d3124c0a5cdbe8b91d81b922fe56b2462e1b9
[6/8] ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
      commit: 8fafb7e5c041814876266259e5e439f93571dcef
[7/8] ARM: dts: qcom: fix and add some missing gsbi node for ipq8064
      commit: 6c421a9c08286389bb331fe783e2625c9efcc187
[8/8] ARM: dts: qcom: add speedbin efuse nvmem node
      commit: 7f5aecdd4ffcc018f73171bc0e028cd4e3361acd

Best regards,
Christian Marangi July 17, 2022, 9:20 a.m. UTC | #3
On Sat, Jul 16, 2022 at 09:46:23PM -0500, Bjorn Andersson wrote:
> On Wed 06 Jul 20:09 CDT 2022, Christian Marangi wrote:
> 
> > This is a respin of "Multiple addition to ipq8064 dtsi" series
> > with major changes and some commit removed.
> > 
> > This series try to add some of the missing bindings for ipq806x.
> > 
> > This still lacks of the cpu bindings and all the bindings required
> > to scale cpu clk or L2. These will come later as the driver and
> > documentation require some changes.
> > 
> > So for now we try to add bindings that can directly applied without
> > making changes to any drivers.
> > 
> > Changes:
> > v2:
> > - Dropped "add sic non secure node for ipq8064" (no user found)
> > - Dropped "add smem node for ipq8064" (require changes to the driver
> >   to remove syscon node and use regs directly, will be added in a
> >   different series)
> > - Dropped "fix dtc warning for missing #address-cells for ipq8064" (
> >   doesn't actually fix any warning)
> > - Changed "enable usb phy by default for ipq8064" to disable usb phy
> >   by default
> > - Reworked rpm smb208 declaration to a saparate dtsi
> > - Fixed order with compatible-reg-status
> > - Squashed "remove redundant binding from ipq8064 rb3011 dts" with
> >   "add multiple missing pin definition for ipq8064" to prevent dtc
> >   error.
> > - Add review tag for snps patch
> > 
> > Changes from "Multiple addition to ipq8064 dtsi":
> > v3:
> > - Fix error in rb3011 dts
> > - Add tested-by tag on the entire series
> > v2:
> > - Added missing patch
> > - Added additional gsbi6 spi
> > - Added extra description for L2 cache opp
> > - Fxied smb208 enabled by default that is problematic for rb3011 devices
> > 
> > Christian Marangi (8):
> >   ARM: dts: qcom: add multiple missing pin definition for ipq8064
> >   ARM: dts: qcom: add gsbi6 missing definition for ipq8064
> >   ARM: dts: qcom: add specific ipq8064 dtsi with smb208 rpm regulators
> >   ARM: dts: qcom: add missing snps,dwmac compatible for gmac ipq8064
> >   ARM: dts: qcom: disable usb phy by default for ipq8064
> >   ARM: dts: qcom: reduce pci IO size to 64K for ipq8064
> >   ARM: dts: qcom: fix and add some missing gsbi node for ipq8064
> >   ARM: dts: qcom: add speedbin efuse nvmem node
> 
> Please prefix your ipa8064-specific patches with
>   "ARM: dts: qcom: ipq8064: ..."
> 
> It makes the git history easier to read, so I fixed it up while applying
> these.
>

Oh ok! I wasn't aware of this tag looking at the old commits. Will do
that in the next commits.

> > 
> >  arch/arm/boot/dts/qcom-ipq8064-ap148.dts   |   6 -
> >  arch/arm/boot/dts/qcom-ipq8064-rb3011.dts  |   9 --
> >  arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi |  37 +++++
> >  arch/arm/boot/dts/qcom-ipq8064.dtsi        | 150 +++++++++++++++++++--
> >  4 files changed, 179 insertions(+), 23 deletions(-)
> >  create mode 100644 arch/arm/boot/dts/qcom-ipq8064-smb208.dtsi
> > 
> > -- 
> > 2.36.1
> >