Re: i915 build issue

2023-07-11 Thread Jani Nikula
On Tue, 11 Jul 2023, John Garry  wrote:
> On 11/07/2023 10:58, Jani Nikula wrote:
>>> I didn't notice anything on dri-devel mailing list about this.
>> I presume you're using CONFIG_WERROR=y or W=e.
>
> I'm just using whatever vanilla x86_64_defconfig gives.
>
>> 
>> See [1] and [2]. I'm undecided how we should treat this.

Seeing that the x86 defconfigs enable CONFIG_WERROR=y, we're probably
going to have to roll this back.

https://patchwork.freedesktop.org/patch/msgid/2023070214.25093-1-jani.nik...@intel.com

BR,
Jani.


>
> Ok.
>
> Thanks,
> John
>

-- 
Jani Nikula, Intel Open Source Graphics Center


Re: i915 build issue

2023-07-11 Thread John Garry

On 11/07/2023 10:58, Jani Nikula wrote:

I didn't notice anything on dri-devel mailing list about this.

I presume you're using CONFIG_WERROR=y or W=e.


I'm just using whatever vanilla x86_64_defconfig gives.



See [1] and [2]. I'm undecided how we should treat this.


Ok.

Thanks,
John



i915 build issue

2023-07-11 Thread John Garry

Hi guys,

Did anyone else notice this build issue on v6.5-rc1:

drivers/gpu/drm/i915/i915_pci.c:143:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:151:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:159:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:166:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:174:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:180:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:186:15: error: expected expression
before ‘,’ token
  GEN3_FEATURES,
   ^
drivers/gpu/drm/i915/i915_pci.c:209:15: error: initialized field
overwritten [-Werror=override-init]
  .has_snoop = false,
   ^
...

I'm using following gcc:

john@localhost:~/mnt_sda4/john/linux> gcc --version
gcc (SUSE Linux) 7.5.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I didn't notice anything on dri-devel mailing list about this.

Cheers,
John


Re: i915 build issue

2023-07-11 Thread Jani Nikula
On Tue, 11 Jul 2023, John Garry  wrote:
> Hi guys,
>
> Did anyone else notice this build issue on v6.5-rc1:
>
> drivers/gpu/drm/i915/i915_pci.c:143:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:151:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:159:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:166:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:174:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:180:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:186:15: error: expected expression
> before ‘,’ token
>GEN3_FEATURES,
> ^
> drivers/gpu/drm/i915/i915_pci.c:209:15: error: initialized field
> overwritten [-Werror=override-init]
>.has_snoop = false,
> ^
> ...
>
> I'm using following gcc:
>
> john@localhost:~/mnt_sda4/john/linux> gcc --version
> gcc (SUSE Linux) 7.5.0
> Copyright (C) 2017 Free Software Foundation, Inc.
> This is free software; see the source for copying conditions.  There is NO
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
>
> I didn't notice anything on dri-devel mailing list about this.

I presume you're using CONFIG_WERROR=y or W=e.

See [1] and [2]. I'm undecided how we should treat this.


BR,
Jani.


[1] https://lore.kernel.org/r/87wmzezns4@intel.com
[2] https://gitlab.freedesktop.org/drm/intel/-/issues/8768


-- 
Jani Nikula, Intel Open Source Graphics Center