lkml.org 
[lkml]   [2021]   [Dec]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v6] ARM: dts: aspeed: Adding Facebook Bletchley BMC
Date
Hi Joel

Bletchley platform needs to support fan control.
Since Billy's patch for upstream is still under review, I can't add pwm/tach node into dts or the build would fail.

As you know, I had committed the pwm/tach driver for ast26xx to OpenBMC linux, could I commit the dts to OpenBMC linux instead of upstream for early build?

Howard

> -----Original Message-----
> From: Joel Stanley <joel@jms.id.au>
> Sent: Tuesday, December 7, 2021 6:52 AM
> To: Patrick Williams <patrick@stwcx.xyz>
> Cc: Howard Chiu <howard10703049@gmail.com>; Arnd Bergmann
> <arnd@arndb.de>; Olof Johansson <olof@lixom.net>; SoC Team
> <soc@kernel.org>; Rob Herring <robh+dt@kernel.org>; Andrew Jeffery
> <andrew@aj.id.au>; Linux ARM <linux-arm-kernel@lists.infradead.org>;
> devicetree <devicetree@vger.kernel.org>; Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org>; linux-aspeed <linux-aspeed@lists.ozlabs.org>;
> Howard Chiu (邱冠睿) <Howard.Chiu@quantatw.com>
> Subject: Re: [PATCH v6] ARM: dts: aspeed: Adding Facebook Bletchley BMC
>
> On Mon, 6 Dec 2021 at 21:25, Patrick Williams <patrick@stwcx.xyz> wrote:
> >
> > On Wed, Dec 01, 2021 at 11:37:38AM +0800, Howard Chiu wrote:
> > > Initial introduction of Facebook Bletchley equipped with
> > > Aspeed 2600 BMC SoC.
> > >
> > > Signed-off-by: Howard Chiu <howard.chiu@quantatw.com>
> > >
> >
> > ...
> >
> > > Change since v2:
> > > - Remove uart5 workaround
> > > - Remove gpio nodes of pca9552/pca9539
> > > - Modify gpio-line-name of led/power/presence pins with openbmc
> pattern
> >
> > A number of the GPIOs do not have defined openbmc patterns for them yet.
> The
> > names you have chosen are ok for now, but we will be changing them as we
> refine
> > development of this machine further.
> >
> > > + gpio-line-names =
> > > +
> "SLED0_MS_DETECT1","SLED0_VBUS_BMC_EN","SLED0_INA230_ALERT","SLE
> D0_P12V_STBY_ALERT",
> > > +
> "SLED0_SSD_ALERT","SLED0_MS_DETECT0","SLED0_RST_CCG5","SLED0_FUSB
> 302_INT",
> > > +
> "SLED0_MD_STBY_RESET","SLED0_MD_IOEXP_EN_FAULT","SLED0_MD_DIR","
> SLED0_MD_DECAY",
> > > +
> "SLED0_MD_MODE1","SLED0_MD_MODE2","SLED0_MD_MODE3","power-ho
> st0";
> >
> > Such as these...
> >
> > > --
> > > 2.25.1
> > >
> >
> > Otherwise,
> >
> > Reviewed-by: Patrick Williams <patrick@stwcx.xyz>
>
> Thanks for reviewing.
>
> Howard, I'm merging patches for v5.17 but I can't apply this until you
> fix the warnings I mentioned.
>
> Cheers,
>
> Jeol
> >
> > --
> > Patrick Williams
\
 
 \ /
  Last update: 2021-12-07 03:55    [W:0.241 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site