[reviewing akmod] request for reviewing intel-ipu6 akmod package

[Kate Hsuan]

Hi,

Recently, we are working on getting IPU6 MIPI camera to work for the
laptops. We also made a akmod package for Intel opensource drivers and
the package will live in RPM fusion. The details can be found here.
https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469

Could a reviewer who is familiar with akmod package take a look into it?

Thank you

[Nicolas Chauvet]

Le ven. 10 févr. 2023 à 10:24, Kate Hsuan  a écrit :
> Hi,
>
> Recently, we are working on getting IPU6 MIPI camera to work for the
> laptops. We also made a akmod package for Intel opensource drivers and
> the package will live in RPM fusion. The details can be found here.
> https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469
>
> Could a reviewer who is familiar with akmod package take a look into it?

Hello Kate,
I can do the review, but I will be on vacation next monday for the next 2 weeks.
Hopefully the userspace part can be sorted out by Hans...

Thanks for working on this.

[Kate Hsuan]

Hi Nicolas,

On Fri, Feb 10, 2023 at 7:10 PM Nicolas Chauvet  wrote:
> Le ven. 10 févr. 2023 à 10:24, Kate Hsuan  a écrit :
> > Hi,
> >
> > Recently, we are working on getting IPU6 MIPI camera to work for the
> > laptops. We also made a akmod package for Intel opensource drivers and
> > the package will live in RPM fusion. The details can be found here.
> > https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469
> >
> > Could a reviewer who is familiar with akmod package take a look into it?
>
> Hello Kate,
> I can do the review, but I will be on vacation next monday for the next 2 weeks.

Thank you very much.
No worries and enjoy your vacation :D
> Hopefully the userspace part can be sorted out by Hans...

Right, we are working on them. I also can discuss the order of the
package review sequence.

Thanks again :)
> Thanks for working on this.
> _______________________________________________
> devel mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
> Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: https://lists.fedoraproject.org/archives/list/[e-mail]
> Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue

[Demi Marie Obenour]

On 2/10/23 04:24, Kate Hsuan wrote:
> Hi,
>
> Recently, we are working on getting IPU6 MIPI camera to work for the
> laptops. We also made a akmod package for Intel opensource drivers and
> the package will live in RPM fusion. The details can be found here.
> https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469
>
> Could a reviewer who is familiar with akmod package take a look into it?
>
> Thank you

This is great!  That said, are you working on getting this driver
upstreamed, with corresponding working, open userspace?

[Kate Hsuan]

Hi,

On Sat, Feb 11, 2023 at 4:54 AM Demi Marie Obenour
 wrote:
> On 2/10/23 04:24, Kate Hsuan wrote:
> > Hi,
> >
> > Recently, we are working on getting IPU6 MIPI camera to work for the
> > laptops. We also made a akmod package for Intel opensource drivers and
> > the package will live in RPM fusion. The details can be found here.
> > https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469
> >
> > Could a reviewer who is familiar with akmod package take a look into it?
> >
> > Thank you
>
> This is great!  That said, are you working on getting this driver
> upstreamed, with corresponding working, open userspace?

Hans already got the important part (INT3472 LED) of IPU6 controls
upstream. However, Intel IPU6 and IVSC drivers had already opensource.
The entire solution is based on the Intel proprietary application
libraries to make the MIPI camera work. Simply speaking, For now, we
need Hans' upstream work, Intel driver, and Intel binary library files
to make it work. Now, we are working on packaging the v4l2loopback
solution for the users. :)
> --
> Sincerely,
> Demi Marie Obenour (she/her/hers)
> _______________________________________________
> devel mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
> Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: https://lists.fedoraproject.org/archives/list/[e-mail]
> Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue

[Demi Marie Obenour]

On 2/12/23 22:06, Kate Hsuan wrote:
> Hi,
>
> On Sat, Feb 11, 2023 at 4:54 AM Demi Marie Obenour
>  wrote:
> > On 2/10/23 04:24, Kate Hsuan wrote:
> > > Hi,
> > >
> > > Recently, we are working on getting IPU6 MIPI camera to work for the
> > > laptops. We also made a akmod package for Intel opensource drivers and
> > > the package will live in RPM fusion. The details can be found here.
> > > https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469
> > >
> > > Could a reviewer who is familiar with akmod package take a look into it?
> > >
> > > Thank you
> >
> > This is great!  That said, are you working on getting this driver
> > upstreamed, with corresponding working, open userspace?
>
> Hans already got the important part (INT3472 LED) of IPU6 controls
> upstream. However, Intel IPU6 and IVSC drivers had already opensource.

How is the LED driver the important part?  Are the IPU6 and IVSC
drivers being upstreamed?

Also, the software-controlled LED was a bad design for obvious security
reasons.  The hardware should have been designed such that there
is no way to power on the image sensor without turning on the LED.
This would have been very easy to implement.
> The entire solution is based on the Intel proprietary application
> libraries to make the MIPI camera work. Simply speaking, For now, we
> need Hans' upstream work, Intel driver, and Intel binary library files
> to make it work. Now, we are working on packaging the v4l2loopback
> solution for the users. :)

Are there plans to replace the proprietary userspace with open
source code?  I don’t care if the open source userspace is less
optimized than the proprietary binary.  I do care that distributions
can ship with IPU6 support out of the box, and that requires open
source userspace and drivers that are either upstream or themselves
running in userspace via VFIO.