[GIT,PULL] Qualcomm ARM64 Defconfig updates for 5.5
mbox series

Message ID 1573068840-13098-1-git-send-email-agross@kernel.org
State New
Headers show
Series
  • [GIT,PULL] Qualcomm ARM64 Defconfig updates for 5.5
Related show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5

Message

Andy Gross Nov. 6, 2019, 7:33 p.m. UTC
The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:

  Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5

for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:

  arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)

----------------------------------------------------------------
Qualcomm ARM64 Based defconfig Updates for v5.5

* Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
* Enable SN65DSI86 display bridge
* Enable QCA Bluetooth over Uart
* Enable various Qualcomm remoteproc dependencies

----------------------------------------------------------------
Bjorn Andersson (6):
      arm64: defconfig: Enable Qualcomm remoteproc dependencies
      arm64: defconfig: Enable Qualcomm SPI and QSPI controller
      arm64: defconfig: Enable Qualcomm socinfo driver
      arm64: defconfig: Enable Qualcomm CPUfreq HW driver
      arm64: defconfig: Enable Qualcomm pseudo rng
      arm64: defconfig: Enable Qualcomm watchdog driver

Jeffrey Hugo (2):
      arm64: defconfig: Enable QCA Bluetooth over UART
      arm64: defconfig: Enable SN65DSI86 display bridge

 arch/arm64/configs/defconfig | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Olof Johansson Nov. 6, 2019, 10:04 p.m. UTC | #1
On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> 
>   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> 
> for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> 
>   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> 
> ----------------------------------------------------------------
> Qualcomm ARM64 Based defconfig Updates for v5.5
> 
> * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> * Enable SN65DSI86 display bridge
> * Enable QCA Bluetooth over Uart
> * Enable various Qualcomm remoteproc dependencies
> 
> ----------------------------------------------------------------
> Bjorn Andersson (6):
>       arm64: defconfig: Enable Qualcomm remoteproc dependencies
>       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
>       arm64: defconfig: Enable Qualcomm socinfo driver
>       arm64: defconfig: Enable Qualcomm CPUfreq HW driver


Hi, this turns on the driver as =y, when the option is tristate. Other
cpufreq drivers are also modules. Is this driver truly needed to be
a built-in for a generic defconfig?


-Olof
Bjorn Andersson Nov. 6, 2019, 11:18 p.m. UTC | #2
On Wed 06 Nov 14:04 PST 2019, Olof Johansson wrote:

> On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> > The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> > 
> >   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> > 
> > for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> > 
> >   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> > 
> > ----------------------------------------------------------------
> > Qualcomm ARM64 Based defconfig Updates for v5.5
> > 
> > * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> > * Enable SN65DSI86 display bridge
> > * Enable QCA Bluetooth over Uart
> > * Enable various Qualcomm remoteproc dependencies
> > 
> > ----------------------------------------------------------------
> > Bjorn Andersson (6):
> >       arm64: defconfig: Enable Qualcomm remoteproc dependencies
> >       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
> >       arm64: defconfig: Enable Qualcomm socinfo driver
> >       arm64: defconfig: Enable Qualcomm CPUfreq HW driver
> 
> 
> Hi, this turns on the driver as =y, when the option is tristate. Other
> cpufreq drivers are also modules. Is this driver truly needed to be
> a built-in for a generic defconfig?
> 

linux-next has 7 CPUfreq drivers =y and 3 =m, so I picked =y for the
Qualcomm by majority choice. And the same driver is used by a number of
different Qualcomm platforms, so the growth looks to be bounded.

It should work as =m as well, but we would need to validate that.

Regards,
Bjorn
Bjorn Andersson Dec. 6, 2019, 5:02 a.m. UTC | #3
On Wed 06 Nov 14:04 PST 2019, Olof Johansson wrote:

> On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> > The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> > 
> >   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> > 
> > for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> > 
> >   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> > 
> > ----------------------------------------------------------------
> > Qualcomm ARM64 Based defconfig Updates for v5.5
> > 
> > * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> > * Enable SN65DSI86 display bridge
> > * Enable QCA Bluetooth over Uart
> > * Enable various Qualcomm remoteproc dependencies
> > 
> > ----------------------------------------------------------------
> > Bjorn Andersson (6):
> >       arm64: defconfig: Enable Qualcomm remoteproc dependencies
> >       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
> >       arm64: defconfig: Enable Qualcomm socinfo driver
> >       arm64: defconfig: Enable Qualcomm CPUfreq HW driver
> 
> 
> Hi, this turns on the driver as =y, when the option is tristate. Other
> cpufreq drivers are also modules. Is this driver truly needed to be
> a built-in for a generic defconfig?
> 

I see Linus merged the arm defconfig PR today, but afaict the patches
from this branch is missing. Please advice on how you would like us to
proceed.

Regards,
Bjorn
Olof Johansson Dec. 6, 2019, 4:32 p.m. UTC | #4
On Thu, Dec 05, 2019 at 09:02:22PM -0800, Bjorn Andersson wrote:
> On Wed 06 Nov 14:04 PST 2019, Olof Johansson wrote:
> 
> > On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> > > The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> > > 
> > >   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> > > 
> > > for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> > > 
> > >   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> > > 
> > > ----------------------------------------------------------------
> > > Qualcomm ARM64 Based defconfig Updates for v5.5
> > > 
> > > * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> > > * Enable SN65DSI86 display bridge
> > > * Enable QCA Bluetooth over Uart
> > > * Enable various Qualcomm remoteproc dependencies
> > > 
> > > ----------------------------------------------------------------
> > > Bjorn Andersson (6):
> > >       arm64: defconfig: Enable Qualcomm remoteproc dependencies
> > >       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
> > >       arm64: defconfig: Enable Qualcomm socinfo driver
> > >       arm64: defconfig: Enable Qualcomm CPUfreq HW driver
> > 
> > 
> > Hi, this turns on the driver as =y, when the option is tristate. Other
> > cpufreq drivers are also modules. Is this driver truly needed to be
> > a built-in for a generic defconfig?
> > 
> 
> I see Linus merged the arm defconfig PR today, but afaict the patches
> from this branch is missing. Please advice on how you would like us to
> proceed.

I think I was waiting on the promised validation of using =m for the cpufreq
drivers and an updated pull request.


-Olof
Bjorn Andersson Dec. 6, 2019, 5:28 p.m. UTC | #5
On Fri 06 Dec 08:32 PST 2019, Olof Johansson wrote:

> On Thu, Dec 05, 2019 at 09:02:22PM -0800, Bjorn Andersson wrote:
> > On Wed 06 Nov 14:04 PST 2019, Olof Johansson wrote:
> > 
> > > On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> > > > The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> > > > 
> > > >   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> > > > 
> > > > are available in the git repository at:
> > > > 
> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> > > > 
> > > > for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> > > > 
> > > >   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> > > > 
> > > > ----------------------------------------------------------------
> > > > Qualcomm ARM64 Based defconfig Updates for v5.5
> > > > 
> > > > * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> > > > * Enable SN65DSI86 display bridge
> > > > * Enable QCA Bluetooth over Uart
> > > > * Enable various Qualcomm remoteproc dependencies
> > > > 
> > > > ----------------------------------------------------------------
> > > > Bjorn Andersson (6):
> > > >       arm64: defconfig: Enable Qualcomm remoteproc dependencies
> > > >       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
> > > >       arm64: defconfig: Enable Qualcomm socinfo driver
> > > >       arm64: defconfig: Enable Qualcomm CPUfreq HW driver
> > > 
> > > 
> > > Hi, this turns on the driver as =y, when the option is tristate. Other
> > > cpufreq drivers are also modules. Is this driver truly needed to be
> > > a built-in for a generic defconfig?
> > > 
> > 
> > I see Linus merged the arm defconfig PR today, but afaict the patches
> > from this branch is missing. Please advice on how you would like us to
> > proceed.
> 
> I think I was waiting on the promised validation of using =m for the cpufreq
> drivers and an updated pull request.
> 

Sounds reasonable, I would however like the CPUfreq driver to be =y;
there are several other drivers doing this and lately work has gone into
ensuring that this driver probes as early as possible.

In either case, I presume we would change the =y to a =m in an
incremental patch, in order to not have to rewrite the branch?


@Amit, have we done any validation with the SDM845 CPUfreq driver built
as a module?

Regards,
Bjorn
Amit Kucheria Dec. 6, 2019, 5:57 p.m. UTC | #6
On Fri, Dec 6, 2019 at 10:58 PM Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
>
> On Fri 06 Dec 08:32 PST 2019, Olof Johansson wrote:
>
> > On Thu, Dec 05, 2019 at 09:02:22PM -0800, Bjorn Andersson wrote:
> > > On Wed 06 Nov 14:04 PST 2019, Olof Johansson wrote:
> > >
> > > > On Wed, Nov 06, 2019 at 01:33:56PM -0600, Andy Gross wrote:
> > > > > The following changes since commit 54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c:
> > > > >
> > > > >   Linux 5.4-rc1 (2019-09-30 10:35:40 -0700)
> > > > >
> > > > > are available in the git repository at:
> > > > >
> > > > >   git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-defconfig-for-5.5
> > > > >
> > > > > for you to fetch changes up to a36612b7469cf76953cf237a2b9a62f6c97730bb:
> > > > >
> > > > >   arm64: defconfig: Enable Qualcomm watchdog driver (2019-10-20 23:04:59 -0700)
> > > > >
> > > > > ----------------------------------------------------------------
> > > > > Qualcomm ARM64 Based defconfig Updates for v5.5
> > > > >
> > > > > * Enable Qualcomm cpufreq, watchdog, prng, socinfo, SPI, and QSPI controllers
> > > > > * Enable SN65DSI86 display bridge
> > > > > * Enable QCA Bluetooth over Uart
> > > > > * Enable various Qualcomm remoteproc dependencies
> > > > >
> > > > > ----------------------------------------------------------------
> > > > > Bjorn Andersson (6):
> > > > >       arm64: defconfig: Enable Qualcomm remoteproc dependencies
> > > > >       arm64: defconfig: Enable Qualcomm SPI and QSPI controller
> > > > >       arm64: defconfig: Enable Qualcomm socinfo driver
> > > > >       arm64: defconfig: Enable Qualcomm CPUfreq HW driver
> > > >
> > > >
> > > > Hi, this turns on the driver as =y, when the option is tristate. Other
> > > > cpufreq drivers are also modules. Is this driver truly needed to be
> > > > a built-in for a generic defconfig?
> > > >
> > >
> > > I see Linus merged the arm defconfig PR today, but afaict the patches
> > > from this branch is missing. Please advice on how you would like us to
> > > proceed.
> >
> > I think I was waiting on the promised validation of using =m for the cpufreq
> > drivers and an updated pull request.
> >
>
> Sounds reasonable, I would however like the CPUfreq driver to be =y;
> there are several other drivers doing this and lately work has gone into
> ensuring that this driver probes as early as possible.
>
> In either case, I presume we would change the =y to a =m in an
> incremental patch, in order to not have to rewrite the branch?
>
>
> @Amit, have we done any validation with the SDM845 CPUfreq driver built
> as a module?

Yes, the driver works as a module.