From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Fri, 24 Aug 2018 20:30:59 +0530 Subject: [U-Boot] [PATCH 01/25] firmware: Add basic support for TI System Control Interface (TI SCI) protocol In-Reply-To: <20180824141125.GR26633@bill-the-cat> References: <20180821143203.29142-1-lokeshvutla@ti.com> <20180821143203.29142-2-lokeshvutla@ti.com> <20180824141125.GR26633@bill-the-cat> Message-ID: <465c17ad-b60f-7477-0f30-8f7784f9afb3@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Friday 24 August 2018 07:41 PM, Tom Rini wrote: > On Tue, Aug 21, 2018 at 08:01:39PM +0530, Lokesh Vutla wrote: > >> Texas Instrument's System Control Interface (TI SCI) message protocol is >> used in Texas Instrument's System on Chip (SoC) such as those in the K3 >> family AM654 SoC to communicate between various compute processors with >> a central system controller entity. >> >> The TI SCI message protocol provides support for management of various >> hardware entities within the SoC. Add support driver to allow >> communication with system controller entity within the SoC using the >> mailbox client. >> >> This is mostly derived from the TI SCI driver in Linux located at >> drivers/firmware/ti_sci.c. > > Since K3 support for the kernel is also being posted now, please make > sure any feedback to ti_sci there for K3 is worked in here as well when > applicable, thanks! ti_sci linux driver is already upstreamed and this is a port of the upstream driver. > >> >> Signed-off-by: Lokesh Vutla >> Signed-off-by: Nishanth Menon >> Signed-off-by: Andreas Dannenberg > > Content wise, seems fine: > > Reviewed-by: Tom Rini > > [snip] >> +++ b/include/linux/soc/ti/ti_sci_protocol.h >> @@ -0,0 +1,69 @@ >> +/* >> + * Texas Instruments System Control Interface Protocol >> + * Based on include/linux/soc/ti/ti_sci_protocol.h from Linux. >> + * >> + * Copyright (C) 2018 Texas Instruments Incorporated - http://www.ti.com/ >> + * Nishanth Menon >> + * Lokesh Vutla >> + * >> + * SPDX-License-Identifier: GPL-2.0+ >> + */ > > Since you've got one here, please run checkpatch.pl on the whole series > to check for other places where the SDPX header isn't in the right spot > (and of course other style/etc problems checkpatch complains about). > ahh my bad. Will fix it everywhere. Thanks and regards, Lokesh