[v1,0/5] media: ir-spi: A few cleanups

Message

Andy Shevchenko March 5, 2024, 5:48 p.m. UTC

While removing of_gpio.h where it's being unused, this driver seems to 
deserve more love. Hence this series.

Andy Shevchenko (5):
  media: ir-spi: Don't use "proxy" headers
  media: ir-spi: Make use of device properties
  media: ir-spi: Utilise temporary variable for struct device
  media: ir-spi: Remove trailing comma in the terminator entry
  media: ir-spi: Unify indentation and comment style

 drivers/media/rc/ir-spi.c | 41 +++++++++++++++++++++------------------
 1 file changed, 22 insertions(+), 19 deletions(-)

Comments

Andi Shyti March 5, 2024, 10:43 p.m. UTC | #1

Hi Andy,

On Tue, Mar 05, 2024 at 07:48:25PM +0200, Andy Shevchenko wrote:
> While removing of_gpio.h where it's being unused, this driver seems to 
> deserve more love. Hence this series.

ehehe... thanks for the cleanup!

Andi

Andy Shevchenko March 11, 2024, 2:25 p.m. UTC | #2

On Tue, Mar 05, 2024 at 07:48:25PM +0200, Andy Shevchenko wrote:
> While removing of_gpio.h where it's being unused, this driver seems to
> deserve more love. Hence this series.

Sean, any comments on this series?

Sean Young March 11, 2024, 2:50 p.m. UTC | #3

On Mon, Mar 11, 2024 at 04:25:19PM +0200, Andy Shevchenko wrote:
> On Tue, Mar 05, 2024 at 07:48:25PM +0200, Andy Shevchenko wrote:
> > While removing of_gpio.h where it's being unused, this driver seems to
> > deserve more love. Hence this series.
> 
> Sean, any comments on this series?

Looks great, I've applied it to my branch.


Sean

Andy Shevchenko March 11, 2024, 2:50 p.m. UTC | #4

On Mon, Mar 11, 2024 at 04:25:19PM +0200, Andy Shevchenko wrote:
> On Tue, Mar 05, 2024 at 07:48:25PM +0200, Andy Shevchenko wrote:
> > While removing of_gpio.h where it's being unused, this driver seems to
> > deserve more love. Hence this series.
> 
> Sean, any comments on this series?

Since previous reply, I found an answer in your repository :-)
Thanks!