From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lokesh Vutla <lokeshvutla@ti.com>
Date: Fri, 24 Aug 2018 20:32:49 +0530
Subject: [U-Boot] [PATCH 21/25] remoteproc: Introduce K3 system
	controller
In-Reply-To: <20180824141243.GL26633@bill-the-cat>
References: <20180821143203.29142-1-lokeshvutla@ti.com>
 <20180821143203.29142-22-lokeshvutla@ti.com>
 <20180824141243.GL26633@bill-the-cat>
Message-ID: <7c1d9d1a-b993-cdf0-9089-4512be7e735b@ti.com>
List-Id: <u-boot.lists.denx.de>
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:42 PM, Tom Rini wrote:
> On Tue, Aug 21, 2018 at 08:01:59PM +0530, Lokesh Vutla wrote:
> 
>> K3 specific SoCs have a dedicated microcontroller for doing
>> resource management. Any HLOS/firmware on compute clusters should
>> load a firmware to this microcontroller before accessing any resource.
>> Adding support for loading this firmware.
>>
>> After the K3 system controller got loaded with firmware and started
>> up it sends out a boot notification message through the secure proxy
>> facility using the TI SCI protocol. Intercept and receive this message
>> through the rproc start operation which will need to get invoked
>> explicitly after the firmware got loaded.
> [snip]
>> +/**
>> + * struct k3_sysctrler_msg_hdr - Generic Header for Messages and responses.
>> + * @cmd_id:	Message ID. One of K3_MSG_*
>> + * @host_id:	Host ID of the message
>> + * @seq_ne:	Message identifier indicating a transfer sequence.
>> + * @flags:	Flags for the message.
>> + */
>> +struct k3_sysctrler_msg_hdr {
>> +	u16 cmd_id;
>> +	u8 host_id;
>> +	u8 seq_nr;
>> +	u32 flags;
>> +} __attribute__ ((__packed__));
> 
> Here and elsewhere please use __packed.  Thanks!
> 

Sure. Will fix it in v2.

Thanks and regards,
Lokesh