Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp317660pxv; Thu, 8 Jul 2021 03:17:45 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwGO+RfYDE56WP5Vf2dlnyfHE6ppanPdAua7UJAKvfsgxVeDLbZs8nSnP8zdxbtd9YWXNBL X-Received: by 2002:a92:8e04:: with SMTP id c4mr21707851ild.219.1625739465571; Thu, 08 Jul 2021 03:17:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1625739465; cv=none; d=google.com; s=arc-20160816; b=ZaBwvE3jFbdrVro3OztH1pMbcp92lGL38yzcBJ/YEc6xCtNgT9AxlnL/0ME9hOlIQ0 EcFaRSUy95mtwcHh51n/uKMVfOJr3CKYa8oRFsvWT4cjgjXzf7t0b7Vy1yD6MIoVMN6Z S9Y7CFCgHjvzBA2mfVTgD/YcOOmZOIVvk5cxjTfxneNi//PR53xybWQqtteSD1mJcByr CK068Mnhg378cb8HXmTaZUdzx95jNL+yNcvW7n30HPi40SmOIu4wAQOKez6xlZ9Blgpr VTTSluVjYuC7WpjAr2YM3D7G8OSQg1+2Llpmhyz2akXl+Sp2il0o87XMlhsEMFnC8j5K rPXQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=gURO19KDbC38rqtfjSvdzvXKUxH11RDcABZuk0H1SY4=; b=o8D82OlQjwm1o+02PXm0atWw+99GBHO+WJWi4lBGxv+aJBN9DUSO/4dISUVjlOvhaX Aoh5vhjNh/EYZEogWOAG42NA4XniZrEOy7J2ewaELWJLOz7yfuTb/8gsU1sMJMIgiqIA DHyHrKPnewTEgrvmq/NxROJhX9wmyri+n0cHNjbq9+TicWbkUoeX9R32rknXtMCJ/Uoz OLPdHqActXipWIRnUTtcRVzKUOaWGAYzMvx9X/d1ZLmpIt/4x2T+7vrGsNsmBUHpIxur qsK3BfmgeogLKgB8ZoOntY6Hpz4RoIKf4aWGOVMDADm5hCut78upcHkZ+LFnFj5Hf1gh cnsQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@mess.org header.s=2020 header.b=j5BMW73c; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a14si1999729jat.22.2021.07.08.03.17.33; Thu, 08 Jul 2021 03:17:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@mess.org header.s=2020 header.b=j5BMW73c; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231455AbhGHKTm (ORCPT + 99 others); Thu, 8 Jul 2021 06:19:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231372AbhGHKTl (ORCPT ); Thu, 8 Jul 2021 06:19:41 -0400 Received: from gofer.mess.org (gofer.mess.org [IPv6:2a02:8011:d000:212::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 316C9C061574; Thu, 8 Jul 2021 03:17:00 -0700 (PDT) Received: by gofer.mess.org (Postfix, from userid 1000) id 0991AC63D9; Thu, 8 Jul 2021 11:16:55 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=mess.org; s=2020; t=1625739416; bh=MbXs+ruuP/V2aD0zCVsEa0ovVcbtksoOixwDqbUj10k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=j5BMW73ccO0splqSY6ZoWSwJlxfiBa50Y8V8W72SU2mH/PiX9ZtJsontLvZDvG90Q WyBVLtiS/UQMqlyBrjHtytUzAJU5Kiv4zwMIqRYU59PAf8haViUA6xJfNXQ/JoHKdZ WOpsDMu4ZyhgOhzQ5KnWY5TvsXp8XJHmOnhvQzgYAOjS8hV9LCWWjs8J40scchNpDh u9ABjgT4VHeKXM0lPavYPUpF/Ote7sVZwuF91mCs2HX10/+Nj4DBgwxDmBIwEa0sWf r3aGX4LvAQAX9kqnndY3yTWT1lAOjt5GLgRmz44HUD7Oz92L4TdFvlkyaT0O4OQmpw 1N8jlZslvbELg== Date: Thu, 8 Jul 2021 11:16:55 +0100 From: Sean Young To: Viktor Prutyanov Cc: mchehab@kernel.org, robh+dt@kernel.org, khilman@baylibre.com, narmstrong@baylibre.com, jbrunet@baylibre.com, martin.blumenstingl@googlemail.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, rockosov@gmail.com Subject: Re: [PATCH v2 2/2] media: rc: introduce Meson IR blaster driver Message-ID: <20210708101655.GA28899@gofer.mess.org> References: <20210707141323.20757-1-viktor.prutyanov@phystech.edu> <20210707141323.20757-3-viktor.prutyanov@phystech.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210707141323.20757-3-viktor.prutyanov@phystech.edu> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 07, 2021 at 05:13:23PM +0300, Viktor Prutyanov wrote: > This patch adds the driver for Amlogic Meson IR blaster. > > Some Amlogic SoCs such as A311D and T950D4 have IR transmitter > (blaster) controller onboard. It is capable of sending IR > signals with arbitrary carrier frequency and duty cycle. > > The driver supports 3 modulation clock sources: > - sysclk > - xtal3 clock (xtal divided by 3) > - 1us clock > > Signed-off-by: Viktor Prutyanov > --- > changes in v2: > - threaded IRQ removed, all stuff done in IRQ handler > - DIV_ROUND_CLOSEST_ULL replaced with DIV_ROUND_CLOSEST > - compatible changed to "amlogic,meson-g12a-irblaster" > - 'debug' parameter removed > - dprintk() replaced with dev_dbg()/dev_info() > - carrier frequency checked against 0 > - device_name added > > drivers/media/rc/Kconfig | 10 + > drivers/media/rc/Makefile | 1 + > drivers/media/rc/meson-irblaster.c | 439 +++++++++++++++++++++++++++++ > 3 files changed, 450 insertions(+) > create mode 100644 drivers/media/rc/meson-irblaster.c > > diff --git a/drivers/media/rc/Kconfig b/drivers/media/rc/Kconfig > index d0a8326b75c2..6e60348e1bcf 100644 > --- a/drivers/media/rc/Kconfig > +++ b/drivers/media/rc/Kconfig > @@ -246,6 +246,16 @@ config IR_MESON > To compile this driver as a module, choose M here: the > module will be called meson-ir. > > +config IR_MESON_IRBLASTER > + tristate "Amlogic Meson IR blaster" > + depends on ARCH_MESON || COMPILE_TEST > + help > + Say Y if you want to use the IR blaster available on > + Amlogic Meson SoCs. > + > + To compile this driver as a module, choose M here: the > + module will be called meson-irblaster. > + > config IR_MTK > tristate "Mediatek IR remote receiver" > depends on ARCH_MEDIATEK || COMPILE_TEST > diff --git a/drivers/media/rc/Makefile b/drivers/media/rc/Makefile > index 692e9b6b203f..b108f2b0420c 100644 > --- a/drivers/media/rc/Makefile > +++ b/drivers/media/rc/Makefile > @@ -28,6 +28,7 @@ obj-$(CONFIG_IR_ITE_CIR) += ite-cir.o > obj-$(CONFIG_IR_MCEUSB) += mceusb.o > obj-$(CONFIG_IR_FINTEK) += fintek-cir.o > obj-$(CONFIG_IR_MESON) += meson-ir.o > +obj-$(CONFIG_IR_MESON_IRBLASTER) += meson-irblaster.o > obj-$(CONFIG_IR_NUVOTON) += nuvoton-cir.o > obj-$(CONFIG_IR_ENE) += ene_ir.o > obj-$(CONFIG_IR_REDRAT3) += redrat3.o > diff --git a/drivers/media/rc/meson-irblaster.c b/drivers/media/rc/meson-irblaster.c > new file mode 100644 > index 000000000000..bfcdf47e2100 > --- /dev/null > +++ b/drivers/media/rc/meson-irblaster.c > @@ -0,0 +1,439 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/** > + * meson-irblaster.c - Amlogic Meson IR blaster driver > + * > + * Copyright (c) 2021, SberDevices. All Rights Reserved. > + * > + * Author: Viktor Prutyanov > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; version 2 of the License and no later version. > + * > + * This program is distributed in the hope that it will be useful, but > + * WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or > + * NON INFRINGEMENT. See the GNU General Public License for more > + * details. > + * > + * The full GNU General Public License is included in this distribution in > + * the file called "COPYING". > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define DEVICE_NAME "Meson IR blaster" > +#define DRIVER_NAME "meson-irblaster" > + > +#define IRB_DEFAULT_CARRIER 38000 > +#define IRB_DEFAULT_DUTY_CYCLE 50 > + > +#define IRB_FIFO_LEN 128 > +#define IRB_DEFAULT_MAX_FIFO_LEVEL 96 > + > +#define IRB_ADDR0 0x0 > +#define IRB_ADDR1 0x4 > +#define IRB_ADDR2 0x8 > +#define IRB_ADDR3 0xc > + > +#define IRB_MAX_DELAY (1 << 10) > +#define IRB_DELAY_MASK (IRB_MAX_DELAY - 1) > + > +/* IRCTRL_IR_BLASTER_ADDR0 */ > +#define IRB_MOD_CLK(x) ((x) << 12) > +#define IRB_MOD_SYS_CLK 0 > +#define IRB_MOD_XTAL3_CLK 1 > +#define IRB_MOD_1US_CLK 2 > +#define IRB_MOD_10US_CLK 3 > +#define IRB_INIT_HIGH BIT(2) > +#define IRB_ENABLE BIT(0) > + > +/* IRCTRL_IR_BLASTER_ADDR2 */ > +#define IRB_MOD_COUNT(lo, hi) ((((lo) - 1) << 16) | ((hi) - 1)) > + > +/* IRCTRL_IR_BLASTER_ADDR2 */ > +#define IRB_WRITE_FIFO BIT(16) > +#define IRB_MOD_ENABLE BIT(12) > +#define IRB_TB_1US (0x0 << 10) > +#define IRB_TB_10US (0x1 << 10) > +#define IRB_TB_100US (0x2 << 10) > +#define IRB_TB_MOD_CLK (0x3 << 10) > + > +/* IRCTRL_IR_BLASTER_ADDR3 */ > +#define IRB_FIFO_THD_PENDING BIT(16) > +#define IRB_FIFO_IRQ_ENABLE BIT(8) > + > +static unsigned int max_fifo_level = IRB_DEFAULT_MAX_FIFO_LEVEL; > +module_param(max_fifo_level, uint, 0444); > +MODULE_PARM_DESC(max_fifo_level, "Max blaster FIFO filling level"); As discussed device tree might be a better place for this. > + > +struct irblaster_dev { > + struct device *dev; > + unsigned int irq; irq is not used after probe, no need for it here. > + void __iomem *reg_base; > + u32 *buf; > + unsigned int buf_len; > + unsigned int buf_head; > + unsigned int carrier; > + unsigned int duty_cycle; > + spinlock_t lock; > + struct completion completion; > + unsigned int max_fifo_level; > + unsigned int clk_nr; clk_nr is not used after probe, no need for it here. > + unsigned long clk_rate; > +}; > + > +static void irb_set_mod(struct irblaster_dev *irb) > +{ > + unsigned int cnt = irb->clk_rate / irb->carrier; > + unsigned int pulse_cnt = cnt * irb->duty_cycle / 100; Should these two be DIV_ROUND_CLOSEST()? > + unsigned int space_cnt = cnt - pulse_cnt; > + > + dev_dbg(irb->dev, "F_mod = %uHz, T_mod = %luns, duty_cycle = %u%%\n", > + irb->carrier, NSEC_PER_SEC / irb->clk_rate * cnt, > + 100 * pulse_cnt / cnt); > + > + writel(IRB_MOD_COUNT(pulse_cnt, space_cnt), > + irb->reg_base + IRB_ADDR1); > +} > + > +static void irb_setup(struct irblaster_dev *irb) > +{ > + unsigned int fifo_irq_threshold = IRB_FIFO_LEN - irb->max_fifo_level; > + > + /* > + * Disable the blaster, set modulator clock tick and set initialize > + * output to be high. Set up carrier frequency and duty cycle. Then > + * unset initialize output. Enable FIFO interrupt, set FIFO interrupt > + * threshold. Finally, enable the blaster back. > + */ > + writel(~IRB_ENABLE & (IRB_MOD_CLK(irb->clk_nr) | IRB_INIT_HIGH), > + irb->reg_base + IRB_ADDR0); > + irb_set_mod(irb); > + writel(readl(irb->reg_base + IRB_ADDR0) & ~IRB_INIT_HIGH, > + irb->reg_base + IRB_ADDR0); > + writel(IRB_FIFO_IRQ_ENABLE | fifo_irq_threshold, > + irb->reg_base + IRB_ADDR3); > + writel(readl(irb->reg_base + IRB_ADDR0) | IRB_ENABLE, > + irb->reg_base + IRB_ADDR0); > +} > + > +static u32 irb_prepare_pulse(struct irblaster_dev *irb, unsigned int time) > +{ > + unsigned int delay; > + unsigned int tb = IRB_TB_MOD_CLK; > + unsigned int tb_us = USEC_PER_SEC / irb->carrier; DIV_ROUND_CLOSEST()? > + > + delay = (DIV_ROUND_CLOSEST(time, tb_us) - 1) & IRB_DELAY_MASK; > + > + return ((IRB_WRITE_FIFO | IRB_MOD_ENABLE) | tb | delay); > +} > + > +static u32 irb_prepare_space(struct irblaster_dev *irb, unsigned int time) > +{ > + unsigned int delay; > + unsigned int tb = IRB_TB_100US; > + unsigned int tb_us = 100; > + > + if (time <= IRB_MAX_DELAY) { > + tb = IRB_TB_1US; > + tb_us = 1; > + } else if (time <= 10 * IRB_MAX_DELAY) { > + tb = IRB_TB_10US; > + tb_us = 10; > + } else if (time <= 100 * IRB_MAX_DELAY) { > + tb = IRB_TB_100US; > + tb_us = 100; > + } > + > + delay = (DIV_ROUND_CLOSEST(time, tb_us) - 1) & IRB_DELAY_MASK; > + > + return ((IRB_WRITE_FIFO & ~IRB_MOD_ENABLE) | tb | delay); > +} > + > +static void irb_send_buffer(struct irblaster_dev *irb) > +{ > + unsigned long flags; > + unsigned int nr = 0; > + > + spin_lock_irqsave(&irb->lock, flags); > + while (irb->buf_head < irb->buf_len && nr < irb->max_fifo_level) { > + writel(irb->buf[irb->buf_head], irb->reg_base + IRB_ADDR2); > + > + irb->buf_head++; > + nr++; > + } > + spin_unlock_irqrestore(&irb->lock, flags); > +} > + > +static bool irb_check_buf(struct irblaster_dev *irb, > + unsigned int *buf, unsigned int len) > +{ > + unsigned int i; > + > + for (i = 0; i < len; i++) { > + unsigned int max_tb_us; > + /* > + * Max space timebase is 100 us. > + * Pulse timebase equals to carrier period. > + */ > + if (i % 2 == 0) > + max_tb_us = USEC_PER_SEC / irb->carrier; > + else > + max_tb_us = 100; > + > + if (buf[i] >= max_tb_us * IRB_MAX_DELAY) > + return false; > + } > + > + return true; > +} > + > +static void irb_fill_buf(struct irblaster_dev *irb, unsigned int *buf) > +{ > + unsigned int i; > + > + for (i = 0; i < irb->buf_len; i++) { > + if (i % 2 == 0) > + irb->buf[i] = irb_prepare_pulse(irb, buf[i]); > + else > + irb->buf[i] = irb_prepare_space(irb, buf[i]); > + } > +} > + > +static void irb_send(struct irblaster_dev *irb) > +{ > + reinit_completion(&irb->completion); > + > + dev_dbg(irb->dev, "tx started, buffer length = %u\n", len); > + irb_send_buffer(irb); > + wait_for_completion_interruptible(&irb->completion); > + dev_dbg(irb->dev, "tx completed\n"); > +} This function has only one call site and it is not very long. Might as well inline it. > + > +static irqreturn_t irb_irqhandler(int irq, void *data) > +{ > + struct irblaster_dev *irb = data; > + > + writel(readl(irb->reg_base + IRB_ADDR3) & ~IRB_FIFO_THD_PENDING, > + irb->reg_base + IRB_ADDR3); > + > + if (irb->buf_head < irb->buf_len) > + irb_send_buffer(irb); > + > + complete(&irb->completion); > + > + return IRQ_HANDLED; > +} > + > +static int irb_set_tx_carrier(struct rc_dev *rc, u32 carrier) > +{ > + struct irblaster_dev *irb = rc->priv; > + > + if (carrier == 0) > + return -EINVAL; > + > + irb->carrier = carrier; > + irb_set_mod(irb); > + > + return 0; > +} > + > +static int irb_set_tx_duty_cycle(struct rc_dev *rc, u32 duty_cycle) > +{ > + struct irblaster_dev *irb = rc->priv; > + > + irb->duty_cycle = duty_cycle; > + irb_set_mod(irb); > + > + return 0; > +} > + > +static int irb_tx_ir(struct rc_dev *rc, unsigned int *buf, unsigned int len) > +{ > + struct irblaster_dev *irb = rc->priv; > + > + if (!irb_check_buf(irb, buf, len)) > + return -EINVAL; > + > + irb->buf = kmalloc_array(len, sizeof(u32), GFP_KERNEL); > + if (!irb->buf) > + return -ENOMEM; > + > + irb->buf_len = len; > + irb->buf_head = 0; > + irb_fill_buf(irb, buf); > + > + irb_send(irb); > + irb_send() does a wait interruptable, so it can be interrupted with a simple signal .. > + kfree(irb->buf); .. and then we free memory which is being sent. I think you need to use the spinlock here, free it and set the buf to NULL. > + > + return len; > +} > + > +static int irb_mod_clock_probe(struct irblaster_dev *irb) > +{ > + struct device_node *np = irb->dev->of_node; > + struct clk *clock; > + const char *clock_name; > + > + if (!np) > + return -ENODEV; > + > + if (!of_property_read_string(np, "mod-clock", &clock_name)) { > + if (!strcmp(clock_name, "sysclk")) > + irb->clk_nr = IRB_MOD_SYS_CLK; > + else if (!strcmp(clock_name, "xtal")) > + irb->clk_nr = IRB_MOD_XTAL3_CLK; > + else > + return -EINVAL; > + > + clock = devm_clk_get(irb->dev, clock_name); > + if (IS_ERR(clock) || clk_prepare_enable(clock)) > + return -ENODEV; > + } else { > + irb->clk_nr = IRB_MOD_1US_CLK; > + } > + > + switch (irb->clk_nr) { > + case IRB_MOD_SYS_CLK: > + irb->clk_rate = clk_get_rate(clock); > + break; > + case IRB_MOD_XTAL3_CLK: > + irb->clk_rate = clk_get_rate(clock) / 3; > + break; > + case IRB_MOD_1US_CLK: > + irb->clk_rate = 1000000; > + break; > + } > + > + dev_info(irb->dev, "F_clk = %luHz\n", irb->clk_rate); > + > + return 0; > +} > + > +static int __init irblaster_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct irblaster_dev *irb; > + struct rc_dev *rc; > + struct resource *range; > + int ret; > + > + irb = devm_kzalloc(dev, sizeof(*irb), GFP_KERNEL); > + if (!irb) > + return -ENOMEM; > + > + range = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + if (!range) { > + dev_err(dev, "no memory resource found\n"); > + return -ENODEV; > + } > + > + irb->reg_base = devm_ioremap_resource(dev, range); > + if (IS_ERR(irb->reg_base)) { > + dev_err(dev, "ioremap failed\n"); > + return PTR_ERR(irb->reg_base); > + } > + > + irb->irq = platform_get_irq(pdev, 0); > + if (irb->irq < 0) { > + dev_err(dev, "no irq resource found\n"); > + return -ENODEV; > + } > + > + if (max_fifo_level <= IRB_FIFO_LEN) > + irb->max_fifo_level = max_fifo_level; > + else { > + irb->max_fifo_level = IRB_FIFO_LEN; > + dev_warn(dev, "max FIFO level param truncated to %u", > + IRB_FIFO_LEN); > + } > + > + irb->dev = dev; > + irb->carrier = IRB_DEFAULT_CARRIER; > + irb->duty_cycle = IRB_DEFAULT_DUTY_CYCLE; > + init_completion(&irb->completion); > + spin_lock_init(&irb->lock); > + > + ret = irb_mod_clock_probe(irb); > + if (ret) { > + dev_err(dev, "modulator clock setup failed\n"); > + return ret; > + } > + irb_setup(irb); > + > + ret = devm_request_irq(dev, irb->irq, > + irb_irqhandler, > + IRQF_TRIGGER_RISING, > + DRIVER_NAME, irb); > + if (ret) { > + dev_err(dev, "irq request failed\n"); > + return ret; > + } > + > + rc = rc_allocate_device(RC_DRIVER_IR_RAW_TX); > + if (!rc) > + return -ENOMEM; > + > + rc->driver_name = DRIVER_NAME; > + rc->device_name = DEVICE_NAME; > + rc->priv = irb; > + > + rc->tx_ir = irb_tx_ir; > + rc->s_tx_carrier = irb_set_tx_carrier; > + rc->s_tx_duty_cycle = irb_set_tx_duty_cycle; > + > + ret = rc_register_device(rc); > + if (ret < 0) { > + dev_err(dev, "rc_dev registration failed\n"); > + rc_free_device(rc); > + return ret; > + } > + > + platform_set_drvdata(pdev, rc); > + > + return 0; > +} > + > +static int irblaster_remove(struct platform_device *pdev) > +{ > + struct rc_dev *rc = platform_get_drvdata(pdev); > + > + rc_unregister_device(rc); > + > + return 0; > +} > + > +static const struct of_device_id irblaster_dt_match[] = { > + { > + .compatible = "amlogic,meson-g12a-irblaster", > + }, > + {}, > +}; > +MODULE_DEVICE_TABLE(of, irblaster_dt_match); > + > +static struct platform_driver irblaster_pd = { > + .remove = irblaster_remove, > + .driver = { > + .name = DRIVER_NAME, > + .owner = THIS_MODULE, > + .of_match_table = irblaster_dt_match, > + }, > +}; > + > +module_platform_driver_probe(irblaster_pd, irblaster_probe); > + > +MODULE_DESCRIPTION("Meson IR blaster driver"); > +MODULE_AUTHOR("Viktor Prutyanov "); > +MODULE_LICENSE("GPL"); > -- > 2.21.0 Thanks Sean