| Message ID | 20210122123303.41190-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v1] media: atomisp: Remove unused header | expand |
On Fri, Jan 22, 2021 at 02:33:03PM +0200, Andy Shevchenko wrote:
> sfi.h is not anyhow used by the driver. Remove it.
Mauro, can you give your Ack to be able to merge this via other tree?
On Fri, Jan 22, 2021 at 02:33:03PM +0200, Andy Shevchenko wrote: > sfi.h is not anyhow used by the driver. Remove it. Mario, Sakari, any comments on this? Can you give your Ack to get this thru another tree? > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Acked-by: Linus Walleij <linus.walleij@linaro.org> > --- > drivers/staging/media/atomisp/include/linux/atomisp_platform.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > index 5a5121d958ed..8c65733e0255 100644 > --- a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > +++ b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > @@ -22,7 +22,6 @@ > #include <asm/processor.h> > > #include <linux/i2c.h> > -#include <linux/sfi.h> > #include <media/v4l2-subdev.h> > #include "atomisp.h" > > -- > 2.29.2 >
Hi Andy, On Mon, Feb 08, 2021 at 04:19:27PM +0200, Andy Shevchenko wrote: > On Fri, Jan 22, 2021 at 02:33:03PM +0200, Andy Shevchenko wrote: > > sfi.h is not anyhow used by the driver. Remove it. > > Mario, Sakari, any comments on this? > Can you give your Ack to get this thru another tree? Feel free to go ahead on my behalf; I see no issue with that. Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Acked-by: Linus Walleij <linus.walleij@linaro.org> > > --- > > drivers/staging/media/atomisp/include/linux/atomisp_platform.h | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > > index 5a5121d958ed..8c65733e0255 100644 > > --- a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > > +++ b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h > > @@ -22,7 +22,6 @@ > > #include <asm/processor.h> > > > > #include <linux/i2c.h> > > -#include <linux/sfi.h> > > #include <media/v4l2-subdev.h> > > #include "atomisp.h" > > > > -- > > 2.29.2 > > > > -- > With Best Regards, > Andy Shevchenko > >
On Mon, Feb 08, 2021 at 04:46:59PM +0200, Sakari Ailus wrote: > Hi Andy, > > On Mon, Feb 08, 2021 at 04:19:27PM +0200, Andy Shevchenko wrote: > > On Fri, Jan 22, 2021 at 02:33:03PM +0200, Andy Shevchenko wrote: > > > sfi.h is not anyhow used by the driver. Remove it. > > > > Mario, Sakari, any comments on this? > > Can you give your Ack to get this thru another tree? > > Feel free to go ahead on my behalf; I see no issue with that. > > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Thank you, Sakari!
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h index 5a5121d958ed..8c65733e0255 100644 --- a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h +++ b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h @@ -22,7 +22,6 @@ #include <asm/processor.h> #include <linux/i2c.h> -#include <linux/sfi.h> #include <media/v4l2-subdev.h> #include "atomisp.h"