Re: [PATCH 2/2] media: intel: ipu-bridge: add blank line after declarations

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Hungyu,

On Sun, Jun 07, 2026 at 01:08:53PM +0000, Hungyu Lin wrote:
> Add a blank line between the local variable declarations and
> the following statement in ipu_bridge_get_ivsc_acpi_dev().
> 
> This addresses a checkpatch.pl warning.
> 
> No functional change intended.
> 
> Signed-off-by: Hungyu Lin <dennylin0707@gmail.com>
> ---
>  drivers/media/pci/intel/ipu-bridge.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/pci/intel/ipu-bridge.c b/drivers/media/pci/intel/ipu-bridge.c
> index 471a449f11f6..28d309646c1e 100644
> --- a/drivers/media/pci/intel/ipu-bridge.c
> +++ b/drivers/media/pci/intel/ipu-bridge.c
> @@ -180,6 +180,7 @@ static struct acpi_device *ipu_bridge_get_ivsc_acpi_dev(struct acpi_device *adev
>  		struct acpi_device *consumer, *ivsc_adev;
>  
>  		acpi_handle handle = acpi_device_handle(ACPI_PTR(adev));
> +

This newline indeed is added where appropriate, but there's an extra one
just a few lines above. Please remove that one as well.

>  		for_each_acpi_dev_match(ivsc_adev, acpi_id->id, NULL, -1)
>  			/* camera sensor depends on IVSC in DSDT if exist */
>  			for_each_acpi_consumer_dev(ivsc_adev, consumer)

-- 
Sakari Ailus