Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

2017-08-30 Thread Christoph Hellwig 
Thanks,

applied to the uuid for-next tree.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

2017-07-26 Thread Ard Biesheuvel 
On 26 July 2017 at 08:52, Christoph Hellwig  wrote:
> On Tue, Jul 25, 2017 at 01:40:06PM +0300, Andy Shevchenko wrote:
>> Christoph, can we apply this one at least to move things forward?
>
> Id be happy to pick this up for 4.14.  Does everyone involved agree
> that the uuid tree is the right one?

Fine by me for EFI
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

2017-07-26 Thread Christoph Hellwig 
On Tue, Jul 25, 2017 at 01:40:06PM +0300, Andy Shevchenko wrote:
> Christoph, can we apply this one at least to move things forward?

Id be happy to pick this up for 4.14.  Does everyone involved agree
that the uuid tree is the right one?
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

2017-07-25 Thread Andy Shevchenko 
On Thu, 2017-07-20 at 13:18 +0100, Ard Biesheuvel wrote:
> On 19 July 2017 at 19:28, Andy Shevchenko
>  wrote:
> > There are new types and helpers that are supposed to be used in new
> > code.
> > 
> > As a preparation to get rid of legacy types and API functions do
> > the conversion here.
> > 
> > Cc: Matt Fleming 
> > Cc: Ard Biesheuvel 
> > Signed-off-by: Andy Shevchenko 
> 
> Acked-by: Ard Biesheuvel 


Thanks!

Christoph, can we apply this one at least to move things forward?

> 
> > ---
> >  drivers/firmware/efi/cper.c | 10 ++---
> >  include/linux/cper.h| 94 ++--
> > -
> >  include/linux/efi.h |  4 +-
> >  3 files changed, 54 insertions(+), 54 deletions(-)
> > 
> > diff --git a/drivers/firmware/efi/cper.c
> > b/drivers/firmware/efi/cper.c
> > index 48a8f69da42a..684e65c11dde 100644
> > --- a/drivers/firmware/efi/cper.c
> > +++ b/drivers/firmware/efi/cper.c
> > @@ -534,7 +534,7 @@ static void
> >  cper_estatus_print_section(const char *pfx, struct
> > acpi_hest_generic_data *gdata,
> >    int sec_no)
> >  {
> > -   uuid_le *sec_type = (uuid_le *)gdata->section_type;
> > +   guid_t *sec_type = (guid_t *)gdata->section_type;
> > __u16 severity;
> > char newpfx[64];
> > 
> > @@ -545,12 +545,12 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> > printk("%s""Error %d, type: %s\n", pfx, sec_no,
> >    cper_severity_str(severity));
> > if (gdata->validation_bits & CPER_SEC_VALID_FRU_ID)
> > -   printk("%s""fru_id: %pUl\n", pfx, (uuid_le *)gdata-
> > >fru_id);
> > +   printk("%s""fru_id: %pUl\n", pfx, gdata->fru_id);
> > if (gdata->validation_bits & CPER_SEC_VALID_FRU_TEXT)
> > printk("%s""fru_text: %.20s\n", pfx, gdata-
> > >fru_text);
> > 
> > snprintf(newpfx, sizeof(newpfx), "%s%s", pfx, INDENT_SP);
> > -   if (!uuid_le_cmp(*sec_type, CPER_SEC_PROC_GENERIC)) {
> > +   if (guid_equal(sec_type, _SEC_PROC_GENERIC)) {
> > struct cper_sec_proc_generic *proc_err =
> > acpi_hest_get_payload(gdata);
> > 
> > printk("%s""section_type: general processor
> > error\n", newpfx);
> > @@ -558,7 +558,7 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> > cper_print_proc_generic(newpfx, proc_err);
> > else
> > goto err_section_too_small;
> > -   } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PLATFORM_MEM)) {
> > +   } else if (guid_equal(sec_type, _SEC_PLATFORM_MEM)) {
> > struct cper_sec_mem_err *mem_err =
> > acpi_hest_get_payload(gdata);
> > 
> > printk("%s""section_type: memory error\n", newpfx);
> > @@ -568,7 +568,7 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> >    gdata->error_data_length);
> > else
> > goto err_section_too_small;
> > -   } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PCIE)) {
> > +   } else if (guid_equal(sec_type, _SEC_PCIE)) {
> > struct cper_sec_pcie *pcie =
> > acpi_hest_get_payload(gdata);
> > 
> > printk("%s""section_type: PCIe error\n", newpfx);
> > diff --git a/include/linux/cper.h b/include/linux/cper.h
> > index 4c671fc2081e..723e952fde0d 100644
> > --- a/include/linux/cper.h
> > +++ b/include/linux/cper.h
> > @@ -74,36 +74,36 @@ enum {
> >   * Corrected Machine Check
> >   */
> >  #define
> > CPER_NOTIFY_CMC 
> >    \
> > -   UUID_LE(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C,
> > 0xF4, \
> > -   0xEB, 0xD4, 0xF8, 0x90)
> > +   GUID_INIT(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C,
> > 0xF4,   \
> > + 0xEB, 0xD4, 0xF8, 0x90)
> >  /* Corrected Platform Error */
> >  #define
> > CPER_NOTIFY_CPE 
> >    \
> > -   UUID_LE(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4,
> > 0x81, \
> > -   0xF2, 0x7E, 0xBE, 0xEE)
> > +   GUID_INIT(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4,
> > 0x81,   \
> > + 0xF2, 0x7E, 0xBE, 0xEE)
> >  /* Machine Check Exception */
> >  #define
> > CPER_NOTIFY_MCE 
> >    \
> > -   UUID_LE(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65,
> > 0xAB, \
> > -   0xE1, 0x49, 0x13, 0xBB)
> > +   GUID_INIT(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65,
> > 0xAB,   \
> > + 0xE1, 0x49, 0x13, 0xBB)
> >  /* PCI Express Error */
> >  #define
> > CPER_NOTIFY_PCIE   \
>

Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

2017-07-20 Thread Ard Biesheuvel 
On 19 July 2017 at 19:28, Andy Shevchenko
 wrote:
> There are new types and helpers that are supposed to be used in new code.
>
> As a preparation to get rid of legacy types and API functions do
> the conversion here.
>
> Cc: Matt Fleming 
> Cc: Ard Biesheuvel 
> Signed-off-by: Andy Shevchenko 

Acked-by: Ard Biesheuvel 

> ---
>  drivers/firmware/efi/cper.c | 10 ++---
>  include/linux/cper.h| 94 
> ++---
>  include/linux/efi.h |  4 +-
>  3 files changed, 54 insertions(+), 54 deletions(-)
>
> diff --git a/drivers/firmware/efi/cper.c b/drivers/firmware/efi/cper.c
> index 48a8f69da42a..684e65c11dde 100644
> --- a/drivers/firmware/efi/cper.c
> +++ b/drivers/firmware/efi/cper.c
> @@ -534,7 +534,7 @@ static void
>  cper_estatus_print_section(const char *pfx, struct acpi_hest_generic_data 
> *gdata,
>int sec_no)
>  {
> -   uuid_le *sec_type = (uuid_le *)gdata->section_type;
> +   guid_t *sec_type = (guid_t *)gdata->section_type;
> __u16 severity;
> char newpfx[64];
>
> @@ -545,12 +545,12 @@ cper_estatus_print_section(const char *pfx, struct 
> acpi_hest_generic_data *gdata
> printk("%s""Error %d, type: %s\n", pfx, sec_no,
>cper_severity_str(severity));
> if (gdata->validation_bits & CPER_SEC_VALID_FRU_ID)
> -   printk("%s""fru_id: %pUl\n", pfx, (uuid_le *)gdata->fru_id);
> +   printk("%s""fru_id: %pUl\n", pfx, gdata->fru_id);
> if (gdata->validation_bits & CPER_SEC_VALID_FRU_TEXT)
> printk("%s""fru_text: %.20s\n", pfx, gdata->fru_text);
>
> snprintf(newpfx, sizeof(newpfx), "%s%s", pfx, INDENT_SP);
> -   if (!uuid_le_cmp(*sec_type, CPER_SEC_PROC_GENERIC)) {
> +   if (guid_equal(sec_type, _SEC_PROC_GENERIC)) {
> struct cper_sec_proc_generic *proc_err = 
> acpi_hest_get_payload(gdata);
>
> printk("%s""section_type: general processor error\n", newpfx);
> @@ -558,7 +558,7 @@ cper_estatus_print_section(const char *pfx, struct 
> acpi_hest_generic_data *gdata
> cper_print_proc_generic(newpfx, proc_err);
> else
> goto err_section_too_small;
> -   } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PLATFORM_MEM)) {
> +   } else if (guid_equal(sec_type, _SEC_PLATFORM_MEM)) {
> struct cper_sec_mem_err *mem_err = 
> acpi_hest_get_payload(gdata);
>
> printk("%s""section_type: memory error\n", newpfx);
> @@ -568,7 +568,7 @@ cper_estatus_print_section(const char *pfx, struct 
> acpi_hest_generic_data *gdata
>gdata->error_data_length);
> else
> goto err_section_too_small;
> -   } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PCIE)) {
> +   } else if (guid_equal(sec_type, _SEC_PCIE)) {
> struct cper_sec_pcie *pcie = acpi_hest_get_payload(gdata);
>
> printk("%s""section_type: PCIe error\n", newpfx);
> diff --git a/include/linux/cper.h b/include/linux/cper.h
> index 4c671fc2081e..723e952fde0d 100644
> --- a/include/linux/cper.h
> +++ b/include/linux/cper.h
> @@ -74,36 +74,36 @@ enum {
>   * Corrected Machine Check
>   */
>  #define CPER_NOTIFY_CMC  
>   \
> -   UUID_LE(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C, 0xF4, \
> -   0xEB, 0xD4, 0xF8, 0x90)
> +   GUID_INIT(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C, 0xF4,   \
> + 0xEB, 0xD4, 0xF8, 0x90)
>  /* Corrected Platform Error */
>  #define CPER_NOTIFY_CPE  
>   \
> -   UUID_LE(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4, 0x81, \
> -   0xF2, 0x7E, 0xBE, 0xEE)
> +   GUID_INIT(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4, 0x81,   \
> + 0xF2, 0x7E, 0xBE, 0xEE)
>  /* Machine Check Exception */
>  #define CPER_NOTIFY_MCE  
>   \
> -   UUID_LE(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65, 0xAB, \
> -   0xE1, 0x49, 0x13, 0xBB)
> +   GUID_INIT(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65, 0xAB,   \
> + 0xE1, 0x49, 0x13, 0xBB)
>  /* PCI Express Error */
>  #define CPER_NOTIFY_PCIE   \
> -   UUID_LE(0xCF93C01F, 0x1A16, 0x4dfc, 0xB8, 0xBC, 0x9C, 0x4D, \
> -   0xAF, 0x67, 0xC1, 0x04)
> +   GUID_INIT(0xCF93C01F, 0x1A16, 0x4dfc, 0xB8, 0xBC, 0x9C, 0x4D,   \
> + 0xAF, 0x67, 0xC1, 0x04)
>  /* INIT Record (for IPF) */
>  #define CPER_NOTIFY_INIT   \
> -   UUID_LE(0xCC5263E8, 0x9308, 0x454a, 0x89, 0xD0,