[bootlin/training-materials updates] master: Kernel slides: interrupts: suggest to use pdev->name in platform drivers (d55e64f8)

Michael Opdenacker michael.opdenacker at bootlin.com
Tue Apr 27 15:53:02 CEST 2021


Repository : https://github.com/bootlin/training-materials
On branch  : master
Link       : https://github.com/bootlin/training-materials/commit/d55e64f8354b7a80ec8902146fe69d1ef7e9ee1f

>---------------------------------------------------------------

commit d55e64f8354b7a80ec8902146fe69d1ef7e9ee1f
Author: Michael Opdenacker <michael.opdenacker at bootlin.com>
Date:   Tue Apr 27 15:53:02 2021 +0200

    Kernel slides: interrupts: suggest to use pdev->name in platform drivers
    
    - Something that's already very common
    - Allows to distinguish the entries in /proc/devices for different devices
    
    Signed-off-by: Michael Opdenacker <michael.opdenacker at bootlin.com>


>---------------------------------------------------------------

d55e64f8354b7a80ec8902146fe69d1ef7e9ee1f
 .../kernel-driver-development-interrupts.tex                 | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/slides/kernel-driver-development-interrupts/kernel-driver-development-interrupts.tex b/slides/kernel-driver-development-interrupts/kernel-driver-development-interrupts.tex
index f25a209d..f858563a 100644
--- a/slides/kernel-driver-development-interrupts/kernel-driver-development-interrupts.tex
+++ b/slides/kernel-driver-development-interrupts/kernel-driver-development-interrupts.tex
@@ -4,12 +4,8 @@
   \frametitle{Registering an interrupt handler 1/2}
   The {\em managed} API is recommended:
   \begin{minted}[fontsize=\small]{c}
-int devm_request_irq(struct device *dev,
-                     unsigned int irq,
-                     irq_handler_t handler,
-                     unsigned long irq_flags,
-                     const char *devname,
-                     void *dev_id);
+int devm_request_irq(struct device *dev, unsigned int irq, irq_handler_t handler,
+                     unsigned long irq_flags, const char *devname, void *dev_id);
   \end{minted}
   \begin{itemize}
   \item \code{device} for automatic freeing at device or module
@@ -20,7 +16,9 @@ int devm_request_irq(struct device *dev,
   \item \code{handler} is a pointer to the IRQ handler function
   \item \code{irq_flags} are option masks (see next slide)
   \item \code{devname} is the registered name (for
-	\code{/proc/interrupts})
+	\code{/proc/interrupts}). For platform drivers, good idea
+	to use \code{pdev->name} which allows to distinguish
+	devices managed by the same driver (example: \code{44e0b000.i2c}).
   \item \code{dev_id} is an opaque pointer. It can typically
         be used to pass a pointer to a per-device data structure.
         It cannot be \code{NULL} as it is used as an identifier for




More information about the training-materials-updates mailing list