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 Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2076108

Title:
  Update IMX547 driver code

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


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to