Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-12 Thread Santosh Sivaraj
Santosh Sivaraj writes: > "Verma, Vishal L" writes: > > Hi Vishal, > >> On Sat, 2021-05-01 at 11:57 +0530, Santosh Sivaraj wrote: >>> "Verma, Vishal L" writes: >>> >>> Hi Vishal, >>> >>> > On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: >>> > > For NFIT to be available ACPI is a

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-12 Thread Santosh Sivaraj
"Verma, Vishal L" writes: Hi Vishal, > On Sat, 2021-05-01 at 11:57 +0530, Santosh Sivaraj wrote: >> "Verma, Vishal L" writes: >> >> Hi Vishal, >> >> > On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: >> > > For NFIT to be available ACPI is a must, so don't fail when nfit modules >>

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-12 Thread Verma, Vishal L
On Wed, 2021-05-12 at 21:00 +, Verma, Vishal L wrote: > > Did you mean for the errno check to be if (errno != ENOENT) ? > This is what was causing the unit test failure for me. This patch on > top fixes it for me: > > diff --git a/test/core.c b/test/core.c > index 44cb277..698bb66 100644 >

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-12 Thread Verma, Vishal L
On Sat, 2021-05-01 at 11:57 +0530, Santosh Sivaraj wrote: > "Verma, Vishal L" writes: > > Hi Vishal, > > > On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: > > > For NFIT to be available ACPI is a must, so don't fail when nfit modules > > > are missing on a platform that doesn't

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-01 Thread Santosh Sivaraj
Santosh Sivaraj writes: > "Verma, Vishal L" writes: > > Hi Vishal, > >> On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: >>> For NFIT to be available ACPI is a must, so don't fail when nfit modules >>> are missing on a platform that doesn't support ACPI. >>> >>> Signed-off-by: Santosh

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-05-01 Thread Santosh Sivaraj
"Verma, Vishal L" writes: Hi Vishal, > On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: >> For NFIT to be available ACPI is a must, so don't fail when nfit modules >> are missing on a platform that doesn't support ACPI. >> >> Signed-off-by: Santosh Sivaraj >> --- >>  test.h

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-04-30 Thread Verma, Vishal L
On Sun, 2021-03-28 at 07:39 +0530, Santosh Sivaraj wrote: > For NFIT to be available ACPI is a must, so don't fail when nfit modules > are missing on a platform that doesn't support ACPI. > > Signed-off-by: Santosh Sivaraj > --- >  test.h| 2 +- >  

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-04-06 Thread Santosh Sivaraj
"Aneesh Kumar K.V" writes: > Santosh Sivaraj writes: > >> For NFIT to be available ACPI is a must, so don't fail when nfit modules >> are missing on a platform that doesn't support ACPI. >> >> Signed-off-by: Santosh Sivaraj >> --- >> test.h| 2 +- >>

Re: [PATCH 2/4] test: Don't skip tests if nfit modules are missing

2021-04-05 Thread Aneesh Kumar K.V
Santosh Sivaraj writes: > For NFIT to be available ACPI is a must, so don't fail when nfit modules > are missing on a platform that doesn't support ACPI. > > Signed-off-by: Santosh Sivaraj > --- > test.h| 2 +- > test/ack-shutdown-count-set.c | 2 +- >