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

From: Sakari Ailus

Date: Sun Jun 07 2026 - 16:37:11 EST


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@xxxxxxxxx>
> ---
> 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