[Skiboot] [PATCH] libpore: Sync p8 files, remove erroneous "IBM Confidential"

Andrew Donnellan andrew.donnellan at au1.ibm.com
Tue Nov 27 14:41:53 AEDT 2018



On 27/11/18 11:19 am, Stewart Smith wrote:
> We also had some rogue "IBM Confidential" strings that we failed to
> remove with the original change of Copyright headers for open sourcing.
> Do this by synchronising with the hostboot copy of the code, which
> removed the Confidential string when their copyright headers changed for
> initial open sourcing of the code back in 2014. See hostboot commit
> 3bcf5b7982bb8a2d9227dbff7be4ff2ce5fec05c where the HWP copyright headers
> were updated.
> 
> We likely missed this as we did a similar process inside the skiboot
> repository, but likely only on the (C) headers themselves.
> 
> The libpore changes that we were missing *look* minor, but we need to
> throw some testing at them at least, as there *are* changes that we were
> missing.
> 
> Reported-by: Dawn Sylvia <ddzubak at us.ibm.com>
> Signed-off-by: Stewart Smith <stewart at linux.ibm.com>

> --- a/libpore/p8_pore_table_static_data.c
> +++ b/libpore/p8_pore_table_static_data.c
> @@ -291,7 +293,7 @@ typedef uint32_t PoreInlineLocation;
>   /// PORE inline assembler context
>   ///
>   /// See the documentation page \ref pore_inline_assembler and the function
> -/// pore_inline_context_create() for further details.
> +/// pore_inline_context_create() for futher details.

...how are we adding more typos? hmm...

Everything else looks minor and sensible enough.

Reviewed-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited



More information about the Skiboot mailing list