| Message ID | 1573068840-13098-1-git-send-email-agross@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL] Qualcomm ARM64 Defconfig updates for 5.5 | expand |
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
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
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
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
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
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.
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(+)