Re: [PATCH] media: ipu6: Remove workaround for Meteor Lake ES2

[Hao Yao <hao.yao@intel.com>]

Hi Sakari, Bingbu,

On 2025/4/25 13:40, Sakari Ailus wrote:
> Hi Bingbu,
> 
> On Fri, Apr 25, 2025 at 11:05:54AM +0800, Bingbu Cao wrote:
>> Sakari,
>>
>> On 4/24/25 7:55 PM, Sakari Ailus wrote:
>>> Hi Bingbu,
>>>
>>> On Thu, Apr 24, 2025 at 12:18:06PM +0800, Bingbu Cao wrote:
>>>> Hao,
>>>>
>>>> On 3/11/25 4:41 PM, Hao Yao wrote:
>>>>> There was a hardware bug which need IPU6 driver to disable the ATS.
>>>>> This workaround is not needed anymore as the bug was fixed in
>>>>> hardware level.
>>>>>
>>>>
>>>> It is better to send v2 with fixes tag here:
>>>>
>>>> Fixes: 25fedc021985 ("media: intel/ipu6: add Intel IPU6 PCI device driver")
>>>
>>> Is this worth backporting? Nothing is broken because of this, is it?
>>
>> It is necessary. IPU6 on Raptor Lake share same device ID with
>> Meteor Lake, this workaround will break camera on Raptor Lake.
>>
>> And Meteor Lake ES2 is not used in product.
> 
> Ok, this should be visible in the commit message.
> 
> How about adding:
> 
> Additionally, Raptor Lake has the same IPU6 PCI ID and x86 stepping and but
> does have the bug. Removing the Meteor Lake workaround is also required for
> the driver to function on Raptor Lake.
> 
In fact it should be Arrow Lake. I will add below to next version:
---
  Additionally, Arrow Lake has the same IPU6 PCI ID and x86 stepping 

  but does not have the bug. Removing the Meteor Lake workaround is 

  also required for the driver to function on Arrow Lake. 

  

  Fixes: 25fedc021985 ("media: intel/ipu6: add Intel IPU6 PCI device 

  driver")
---