Public bug reported:

[ Impact ]

* With current use of non-devm functions the user is responsible for
freeing the resources and since this is not handled currently the driver
does not probe the sensor on the second load, since resources are not
freed.

* Here is a dmesg snippet:

On first load
ubuntu@kria:~$ sudo dmesg | grep imx547
[ 72.621037] imx547: module is from the staging directory, the quality is 
unknown, you have been
warned.
[ 72.628992] imx547 7-001a: imx547 : imx547 probe success !
On second load:
ubuntu@kria:~$ sudo dmesg | grep imx547
[ 312.319586] imx547 7-001a: GT TRX Reset GPIO not setup in DT
[ 312.325487] imx547: probe of 7-001a failed with error -16

* New implementation of using device managed ie devm_* functions ensures that 
resource is
automatically removed when driver is unloaded.

[ Test Plan ]

* Xilinx will verify the functionality when testing kernel is provided.

[ Where problems could occur ]

* Since freeing of the GPIO resource is not explicitly handled in current 
implementation
the driver fails to probe the sensor and fails to add it to media subsystem.

[ Other Info ]

* Patch/Tag to be included : https://github.com/Xilinx/mv-camera-sensor-
module/releases/tag/v0.3

** Affects: linux-xilinx-zynqmp (Ubuntu)
     Importance: Undecided
         Status: Confirmed

** Affects: linux-xilinx-zynqmp (Ubuntu Jammy)
     Importance: Medium
     Assignee: Portia Stephens (portias)
         Status: In Progress

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux-xilinx-zynqmp in Ubuntu.
https://bugs.launchpad.net/bugs/2076108

Title:
  Update IMX547 driver code

Status in linux-xilinx-zynqmp package in Ubuntu:
  Confirmed
Status in linux-xilinx-zynqmp source package in Jammy:
  In Progress

Bug description:
  [ Impact ]

  * With current use of non-devm functions the user is responsible for
  freeing the resources and since this is not handled currently the
  driver does not probe the sensor on the second load, since resources
  are not freed.

  * Here is a dmesg snippet:

  On first load
  ubuntu@kria:~$ sudo dmesg | grep imx547
  [ 72.621037] imx547: module is from the staging directory, the quality is 
unknown, you have been
  warned.
  [ 72.628992] imx547 7-001a: imx547 : imx547 probe success !
  On second load:
  ubuntu@kria:~$ sudo dmesg | grep imx547
  [ 312.319586] imx547 7-001a: GT TRX Reset GPIO not setup in DT
  [ 312.325487] imx547: probe of 7-001a failed with error -16

  * New implementation of using device managed ie devm_* functions ensures that 
resource is
  automatically removed when driver is unloaded.

  [ Test Plan ]

  * Xilinx will verify the functionality when testing kernel is
  provided.

  [ Where problems could occur ]

  * Since freeing of the GPIO resource is not explicitly handled in current 
implementation
  the driver fails to probe the sensor and fails to add it to media subsystem.

  [ Other Info ]

  * Patch/Tag to be included : https://github.com/Xilinx/mv-camera-
  sensor-module/releases/tag/v0.3

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux-xilinx-zynqmp/+bug/2076108/+subscriptions


-- 
Mailing list: https://launchpad.net/~kernel-packages
Post to     : kernel-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kernel-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to