From: Sandeep Patil <sspatil@android.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will@kernel.org>, Jiri Slaby <jslaby@suse.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Chunyan Zhang <chunyan.zhang@unisoc.com>,
"open list:SERIAL DRIVERS" <linux-serial@vger.kernel.org>,
Baolin Wang <baolin.wang7@gmail.com>,
Orson Zhai <orsonzhai@gmail.com>,
Linux ARM <linux-arm-kernel@lists.infradead.org>,
Android Kernel Team <kernel-team@android.com>,
saravanak@google.com
Subject: Re: [PATCH 1/2] arm64: change ARCH_SPRD Kconfig to tristate
Date: Mon, 9 Mar 2020 21:27:39 -0700 [thread overview]
Message-ID: <20200310042739.GB260998@google.com> (raw)
In-Reply-To: <20200310041903.GA260998@google.com>
On Mon, Mar 09, 2020 at 09:19:03PM -0700, Sandeep Patil wrote:
> Hi Geert,
>
> On Mon, Mar 09, 2020 at 11:32:06AM +0100, Geert Uytterhoeven wrote:
> > Hi Chunyan,
> >
> > On Mon, Mar 9, 2020 at 9:32 AM Chunyan Zhang <zhang.lyra@gmail.com> wrote:
> > > On Mon, 9 Mar 2020 at 16:03, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > > > On Thu, Mar 5, 2020 at 11:33 AM Chunyan Zhang <zhang.lyra@gmail.com> wrote:
> > > > > From: Chunyan Zhang <chunyan.zhang@unisoc.com>
> > > > >
> > > > > The default value of Kconfig for almost all sprd drivers are the same with
> > > > > ARCH_SPRD, making these drivers built as modules as default would be easier
> > > > > if we can set ARCH_SPRD as 'm', so this patch change ARCH_SPRD to tristate.
> > > > >
> > > > > Signed-off-by: Chunyan Zhang <chunyan.zhang@unisoc.com>
> > > >
> > > > Can you actually boot a kernel on a Spreadtrum platform when all platform
> > > > and driver support is modular?
> > >
> > > Yes, even if all drivers are modular.
> >
> > Cool. No hard dependencies on e.g. regulators that are turned off when
> > unused?
> >
> > > But I hope serial can be builtin, then I can have a console to see
> > > kernel output before loading modules.
> >
> > No dependency on the clock driver?
> > Oh, I see you have a hack in the serial driver, to assume default
> > values when the serial port's parent clock is not found. That may
> > limit use of the other serial ports, depending on the actual serial
> > hardware.
> > And on Sharkl64, the serial port's clock is a fixed-clock anyway, so
> > you don't even need the hack.
> >
> > But in general you cannot rely on that, especially if your SoC has clock
> > and/or power domains.
> >
> > BTW, what about the watchdog driver? That one does need a clock, and
> > loading it too late will reboot your system.
> >
> > > Also, this's what Google GKI [1] asked :)
> > >
> > > Regards,
> > > Chunyan
> > >
> > > [1] https://arstechnica.com/gadgets/2019/11/google-outlines-plans-for-mainline-linux-kernel-support-in-android/
> >
> > Let's see how having everything modular works out on an SoC where all
> > hardware is part of a clock and power domain.
>
> I'm curious, are there any problems that we should be aware of? We know about
> the regulator sync state and consumer-supplier dependencies. [1]
>
> (Adding Saravana inline)
>
(oops, forgot to paste the link to presentation)
1. https://linuxplumbersconf.org/event/4/contributions/402/attachments/320/544/Solving_issues_associated_with_modules_and_supplier-consumer_dependencies.pdf
WARNING: multiple messages have this Message-ID (diff)
From: Sandeep Patil <sspatil@android.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Android Kernel Team <kernel-team@android.com>,
Catalin Marinas <catalin.marinas@arm.com>,
saravanak@google.com,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Chunyan Zhang <zhang.lyra@gmail.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Chunyan Zhang <chunyan.zhang@unisoc.com>,
"open list:SERIAL DRIVERS" <linux-serial@vger.kernel.org>,
Jiri Slaby <jslaby@suse.com>,
Baolin Wang <baolin.wang7@gmail.com>,
Orson Zhai <orsonzhai@gmail.com>, Will Deacon <will@kernel.org>,
Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 1/2] arm64: change ARCH_SPRD Kconfig to tristate
Date: Mon, 9 Mar 2020 21:27:39 -0700 [thread overview]
Message-ID: <20200310042739.GB260998@google.com> (raw)
In-Reply-To: <20200310041903.GA260998@google.com>
On Mon, Mar 09, 2020 at 09:19:03PM -0700, Sandeep Patil wrote:
> Hi Geert,
>
> On Mon, Mar 09, 2020 at 11:32:06AM +0100, Geert Uytterhoeven wrote:
> > Hi Chunyan,
> >
> > On Mon, Mar 9, 2020 at 9:32 AM Chunyan Zhang <zhang.lyra@gmail.com> wrote:
> > > On Mon, 9 Mar 2020 at 16:03, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > > > On Thu, Mar 5, 2020 at 11:33 AM Chunyan Zhang <zhang.lyra@gmail.com> wrote:
> > > > > From: Chunyan Zhang <chunyan.zhang@unisoc.com>
> > > > >
> > > > > The default value of Kconfig for almost all sprd drivers are the same with
> > > > > ARCH_SPRD, making these drivers built as modules as default would be easier
> > > > > if we can set ARCH_SPRD as 'm', so this patch change ARCH_SPRD to tristate.
> > > > >
> > > > > Signed-off-by: Chunyan Zhang <chunyan.zhang@unisoc.com>
> > > >
> > > > Can you actually boot a kernel on a Spreadtrum platform when all platform
> > > > and driver support is modular?
> > >
> > > Yes, even if all drivers are modular.
> >
> > Cool. No hard dependencies on e.g. regulators that are turned off when
> > unused?
> >
> > > But I hope serial can be builtin, then I can have a console to see
> > > kernel output before loading modules.
> >
> > No dependency on the clock driver?
> > Oh, I see you have a hack in the serial driver, to assume default
> > values when the serial port's parent clock is not found. That may
> > limit use of the other serial ports, depending on the actual serial
> > hardware.
> > And on Sharkl64, the serial port's clock is a fixed-clock anyway, so
> > you don't even need the hack.
> >
> > But in general you cannot rely on that, especially if your SoC has clock
> > and/or power domains.
> >
> > BTW, what about the watchdog driver? That one does need a clock, and
> > loading it too late will reboot your system.
> >
> > > Also, this's what Google GKI [1] asked :)
> > >
> > > Regards,
> > > Chunyan
> > >
> > > [1] https://arstechnica.com/gadgets/2019/11/google-outlines-plans-for-mainline-linux-kernel-support-in-android/
> >
> > Let's see how having everything modular works out on an SoC where all
> > hardware is part of a clock and power domain.
>
> I'm curious, are there any problems that we should be aware of? We know about
> the regulator sync state and consumer-supplier dependencies. [1]
>
> (Adding Saravana inline)
>
(oops, forgot to paste the link to presentation)
1. https://linuxplumbersconf.org/event/4/contributions/402/attachments/320/544/Solving_issues_associated_with_modules_and_supplier-consumer_dependencies.pdf
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-03-10 4:27 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-05 10:32 [PATCH 1/2] arm64: change ARCH_SPRD Kconfig to tristate Chunyan Zhang
2020-03-05 10:32 ` Chunyan Zhang
2020-03-05 10:32 ` [PATCH 2/2] tty: serial: make SERIAL_SPRD not depend on ARCH_SPRD Chunyan Zhang
2020-03-05 10:32 ` Chunyan Zhang
2020-03-06 12:41 ` Geert Uytterhoeven
2020-03-06 12:41 ` Geert Uytterhoeven
2020-03-09 1:17 ` Chunyan Zhang
2020-03-09 1:17 ` Chunyan Zhang
2020-03-09 8:01 ` Geert Uytterhoeven
2020-03-09 8:01 ` Geert Uytterhoeven
2020-03-09 8:42 ` Chunyan Zhang
2020-03-09 8:42 ` Chunyan Zhang
2020-03-09 10:32 ` Geert Uytterhoeven
2020-03-09 10:32 ` Geert Uytterhoeven
2020-03-09 11:15 ` Chunyan Zhang
2020-03-09 11:15 ` Chunyan Zhang
2020-03-09 14:10 ` Orson Zhai
2020-03-09 14:10 ` Orson Zhai
2020-03-09 8:03 ` [PATCH 1/2] arm64: change ARCH_SPRD Kconfig to tristate Geert Uytterhoeven
2020-03-09 8:03 ` Geert Uytterhoeven
2020-03-09 8:32 ` Chunyan Zhang
2020-03-09 8:32 ` Chunyan Zhang
2020-03-09 10:32 ` Geert Uytterhoeven
2020-03-09 10:32 ` Geert Uytterhoeven
2020-03-10 4:19 ` Sandeep Patil
2020-03-10 4:19 ` Sandeep Patil
2020-03-10 4:27 ` Sandeep Patil [this message]
2020-03-10 4:27 ` Sandeep Patil
2020-03-10 8:47 ` Geert Uytterhoeven
2020-03-10 8:47 ` Geert Uytterhoeven
2020-03-10 9:41 ` Orson Zhai
2020-03-10 9:41 ` Orson Zhai
2020-03-10 9:52 ` Geert Uytterhoeven
2020-03-10 9:52 ` Geert Uytterhoeven
2020-03-10 10:13 ` Orson Zhai
2020-03-10 10:13 ` Orson Zhai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200310042739.GB260998@google.com \
--to=sspatil@android.com \
--cc=baolin.wang7@gmail.com \
--cc=catalin.marinas@arm.com \
--cc=chunyan.zhang@unisoc.com \
--cc=geert@linux-m68k.org \
--cc=gregkh@linuxfoundation.org \
--cc=jslaby@suse.com \
--cc=kernel-team@android.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=orsonzhai@gmail.com \
--cc=saravanak@google.com \
--cc=will@kernel.org \
--cc=zhang.lyra@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.