Re: [PATCH v6 04/14] x86: Secure Launch Resource Table header file

[Ross Philipson]

On 5/5/23 12:22, Simon Horman wrote:
> On Thu, May 04, 2023 at 02:50:13PM +0000, Ross Philipson wrote:
> > Introduce the Secure Launch Resource Table which forms the formal
> > interface between the pre and post launch code.
> >
> > Signed-off-by: Ross Philipson <ross.philipson@xxxxxxxxxx>
> > ---
> >   include/linux/slr_table.h | 270 ++++++++++++++++++++++++++++++++++++++++++++++
> >   1 file changed, 270 insertions(+)
> >   create mode 100644 include/linux/slr_table.h
> >
> > diff --git a/include/linux/slr_table.h b/include/linux/slr_table.h
>
> ...
>
> > +static inline void *slr_end_of_entrys(struct slr_table *table)
> > +{
> > +	return (((void *)table) + table->size);
> > +}
> > +
> > +static inline struct slr_entry_hdr *
> > +slr_next_entry(struct slr_table *table,
> > +		struct slr_entry_hdr *curr)
>
> nit: The indentation of the line above doesn't align with the '(' on
>       the line before it.

Yea I see there is another one above that is not aligned too. We will 
fix those.

Thanks
Ross