From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ang, Chee Hong Date: Thu, 9 May 2019 08:03:21 +0000 Subject: [U-Boot] [PATCH v1 0/3] Enable PSCI services for booting Linux In-Reply-To: <2b89edee-3ffb-db5d-ad11-f733168a88b2@denx.de> References: <1557206425-37534-1-git-send-email-chee.hong.ang@intel.com> <1557381141.102271.1.camel@intel.com> <2b89edee-3ffb-db5d-ad11-f733168a88b2@denx.de> Message-ID: <1557389000.102910.5.camel@intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Thu, 2019-05-09 at 08:59 +0200, Marek Vasut wrote: > On 5/9/19 7:52 AM, Ang, Chee Hong wrote: > > > > On Mon, 2019-05-06 at 22:20 -0700, chee.hong.ang at intel.com wrote: > > > > > > From: "Ang, Chee Hong" > > > > > > Add "SYTEM_RESET" (cold reset) and "CPU_ON" (SMP) PSCI support > > > for booting Linux on Stratix 10 platform. > > > > > > Ang, Chee Hong (3): > > >   ARM: socfpga: stratix10: Enable PSCI system reset > > >   ARM: socfpga: stratix10: Enable PSCI CPU_ON > > >   ARM: socfpga: stratix10: Enable PSCI support for Stratix 10 > > > > > >  arch/arm/mach-socfpga/Kconfig  |  9 ++++++- > > >  arch/arm/mach-socfpga/Makefile |  3 +++ > > >  arch/arm/mach-socfpga/psci.c   | 56 > > > ++++++++++++++++++++++++++++++++++++++++++ > > >  3 files changed, 67 insertions(+), 1 deletion(-) > > >  create mode 100644 arch/arm/mach-socfpga/psci.c > > > > > Hi Marek, > > > > Any comment on this ? These PSCI patches are needed for booting > > Linux > > on our S10 platform. > Same questions as in Re: [PATCH v1 0/3] Enable HPS execution stage > notification , can we stop adding more and more stuff to arch/ and > instead add it to drivers/ ? As you might have noticed, we are trying > to > remove stuff from arch/ and move as much as possible to drivers/ , > but > S10 is doing the exact opposite :( > Not all PSCI code fit in DM (drivers/). There should be a good place to keep the PSCI code for our platform. Can you advise us where should we place those PSCI implementation other than arch/ ?