mbox series

[v1,00/17] Revise Facebook Minerva BMC DTS

Message ID 20240626130332.929534-1-yangchen.openbmc@gmail.com
Headers show
Series Revise Facebook Minerva BMC DTS | expand

Message

Yang Chen June 26, 2024, 1:03 p.m. UTC 
Revise the Linux device tree entry related to Facebook platform Minerva
specific devices connected to the Aspeed AST2600 BMC.

Changelog:
- v1:
  - Modify the properties of spi to match the schema.

Yang Chen (17):
  ARM: dts: aspeed: minerva: change the address of tmp75
  ARM: dts: aspeed: minerva: change aliases for uart
  ARM: dts: aspeed: minerva: add eeprom on i2c bus
  ARM: dts: aspeed: minerva: change RTC reference
  ARM: dts: aspeed: minerva: enable mdio3
  ARM: dts: aspeed: minerva: remove unused bus and device
  ARM: dts: aspeed: minerva: Define the LEDs node name
  ARM: dts: aspeed: minerva: Add adc sensors for fan board
  ARM: dts: aspeed: minerva: add linename of two pins
  ARM: dts: aspeed: minerva: enable ehci0 for USB
  ARM: dts: aspeed: minerva: add tmp75 sensor
  ARM: dts: aspeed: minerva: add power monitor xdp710
  ARM: dts: aspeed: minerva: revise sgpio line name
  ARM: dts: aspeed: minerva: Switch the i2c bus number
  ARM: dts: aspeed: minerva: remove unused power device
  ARM: dts: aspeed: minerva: add ltc4287 device
  ARM: dts: aspeed: minerva: Add spi-gpio

 .../aspeed/aspeed-bmc-facebook-minerva.dts    | 516 +++++++++++++-----
 1 file changed, 373 insertions(+), 143 deletions(-)

Comments

Rob Herring (Arm) June 26, 2024, 1:38 p.m. UTC | #1 
On Wed, Jun 26, 2024 at 7:05 AM Yang Chen <yangchen.openbmc@gmail.com> wrote:
>
> Revise the Linux device tree entry related to Facebook platform Minerva
> specific devices connected to the Aspeed AST2600 BMC.
>
> Changelog:
> - v1:

You already sent v1. This is v2.

>   - Modify the properties of spi to match the schema.
>
> Yang Chen (17):
>   ARM: dts: aspeed: minerva: change the address of tmp75
>   ARM: dts: aspeed: minerva: change aliases for uart
>   ARM: dts: aspeed: minerva: add eeprom on i2c bus
>   ARM: dts: aspeed: minerva: change RTC reference
>   ARM: dts: aspeed: minerva: enable mdio3
>   ARM: dts: aspeed: minerva: remove unused bus and device
>   ARM: dts: aspeed: minerva: Define the LEDs node name
>   ARM: dts: aspeed: minerva: Add adc sensors for fan board
>   ARM: dts: aspeed: minerva: add linename of two pins
>   ARM: dts: aspeed: minerva: enable ehci0 for USB
>   ARM: dts: aspeed: minerva: add tmp75 sensor
>   ARM: dts: aspeed: minerva: add power monitor xdp710
>   ARM: dts: aspeed: minerva: revise sgpio line name
>   ARM: dts: aspeed: minerva: Switch the i2c bus number
>   ARM: dts: aspeed: minerva: remove unused power device
>   ARM: dts: aspeed: minerva: add ltc4287 device
>   ARM: dts: aspeed: minerva: Add spi-gpio
>
>  .../aspeed/aspeed-bmc-facebook-minerva.dts    | 516 +++++++++++++-----
>  1 file changed, 373 insertions(+), 143 deletions(-)
>
> --
> 2.34.1
>
>
Yang Chen June 26, 2024, 2:01 p.m. UTC | #2 
On Wed, Jun 26, 2024 at 9:38 PM Rob Herring <robh@kernel.org> wrote:
>
> On Wed, Jun 26, 2024 at 7:05 AM Yang Chen <yangchen.openbmc@gmail.com> wrote:
> >
> > Revise the Linux device tree entry related to Facebook platform Minerva
> > specific devices connected to the Aspeed AST2600 BMC.
> >
> > Changelog:
> > - v1:
>
> You already sent v1. This is v2.
Hi Rob,

I appreciate your reminder. Should I send another patch series and use v3?

Thanks,
Yang Chen

>
> >   - Modify the properties of spi to match the schema.
> >
> > Yang Chen (17):
> >   ARM: dts: aspeed: minerva: change the address of tmp75
> >   ARM: dts: aspeed: minerva: change aliases for uart
> >   ARM: dts: aspeed: minerva: add eeprom on i2c bus
> >   ARM: dts: aspeed: minerva: change RTC reference
> >   ARM: dts: aspeed: minerva: enable mdio3
> >   ARM: dts: aspeed: minerva: remove unused bus and device
> >   ARM: dts: aspeed: minerva: Define the LEDs node name
> >   ARM: dts: aspeed: minerva: Add adc sensors for fan board
> >   ARM: dts: aspeed: minerva: add linename of two pins
> >   ARM: dts: aspeed: minerva: enable ehci0 for USB
> >   ARM: dts: aspeed: minerva: add tmp75 sensor
> >   ARM: dts: aspeed: minerva: add power monitor xdp710
> >   ARM: dts: aspeed: minerva: revise sgpio line name
> >   ARM: dts: aspeed: minerva: Switch the i2c bus number
> >   ARM: dts: aspeed: minerva: remove unused power device
> >   ARM: dts: aspeed: minerva: add ltc4287 device
> >   ARM: dts: aspeed: minerva: Add spi-gpio
> >
> >  .../aspeed/aspeed-bmc-facebook-minerva.dts    | 516 +++++++++++++-----
> >  1 file changed, 373 insertions(+), 143 deletions(-)
> >
> > --
> > 2.34.1
> >
> >
Rob Herring (Arm) June 26, 2024, 2:52 p.m. UTC | #3 
On Wed, Jun 26, 2024 at 8:02 AM Yang Chen <yangchen.openbmc@gmail.com> wrote:
>
> On Wed, Jun 26, 2024 at 9:38 PM Rob Herring <robh@kernel.org> wrote:
> >
> > On Wed, Jun 26, 2024 at 7:05 AM Yang Chen <yangchen.openbmc@gmail.com> wrote:
> > >
> > > Revise the Linux device tree entry related to Facebook platform Minerva
> > > specific devices connected to the Aspeed AST2600 BMC.
> > >
> > > Changelog:
> > > - v1:
> >
> > You already sent v1. This is v2.
> Hi Rob,
>
> I appreciate your reminder. Should I send another patch series and use v3?

Not until you have changes on this version.

Rob
Andrew Jeffery June 27, 2024, 2:48 a.m. UTC | #4 
Hello Yang Chen,

On Wed, 2024-06-26 at 21:03 +0800, Yang Chen wrote:
> Revise the Linux device tree entry related to Facebook platform Minerva
> specific devices connected to the Aspeed AST2600 BMC.
> 
> Changelog:
> - v1:
>   - Modify the properties of spi to match the schema.

I've put this series into a tree for Joel to pick up.

Thanks,

Andrew