| Message ID | 20241013154056.12532-1-hdegoede@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | media: atomisp: mt9m114: Make it work on Asus T100TA | expand |
On Sun, Oct 13, 2024 at 05:40:52PM +0200, Hans de Goede wrote: > Hi All, > > Since there is some interest in getting the mt9m114 camera to work on > the Asus T100TA: > > https://github.com/jfwells/linux-asus-t100ta/issues/4 > https://lore.kernel.org/platform-driver-x86/e0c8c98f-64ec-4297-bbc8-de489414515c@zonnet.nl/ > > I have spend some time this weekend poking at this resulting in this > series which makes things work somewhat. > > Note that for now this will only work if you modify the BIOS settings > to change the ISP to be listed as a PCI device in lspci: > > 00:03.0 Multimedia controller: Intel Corporation Atom Processor Z36xxx/Z37xxx Series Camera ISP (rev 09) > > If you do not have this in lspci then things will not work. The BIOS > does not allow changing this setting from its menu, so we need to > use the grub setup_var_3 command form a patched grub. > > Note this requires your Asus T100TA to be at the latest v314 BIOS version, > this has not been tested with other versions! > > Download grubia32.efi from here: > > https://fedorapeople.org/~jwrdegoede/grub-efi-setup_var/grubia32.efi > > Boot this once and then run: > > setup_var_3 6f > > If the ISP is not visible in lspci this should show 01 as value > for the 0x6f offset. > > Now change this to 2: > > setup_var_3 6f 02 > > and then reboot. After this the ISP should be visible in lspci. > > For more details on how this works see: > https://hansdegoede.dreamwidth.org/25589.html > > Note this series is just a quick hack, the proper fix would be to get > the new drivers/media/i2c/mt9m114 standard v4l2 driver to work and > atomisp does support standard v4l2 drivers for a while now. > > I plan to take a shot at this as time permits, hopefully sometime > within a month but no promises. LGTM, some nit-picks in the individual replies. Reviewed-by: Andy Shevchenko <andy@kernel.org> P.S. Can you consider my patch to apply (that removes GPL text in the top comments)?
Hi, On 14-Oct-24 1:32 PM, Andy Shevchenko wrote: > On Sun, Oct 13, 2024 at 05:40:52PM +0200, Hans de Goede wrote: >> Hi All, >> >> Since there is some interest in getting the mt9m114 camera to work on >> the Asus T100TA: >> >> https://github.com/jfwells/linux-asus-t100ta/issues/4 >> https://lore.kernel.org/platform-driver-x86/e0c8c98f-64ec-4297-bbc8-de489414515c@zonnet.nl/ >> >> I have spend some time this weekend poking at this resulting in this >> series which makes things work somewhat. >> >> Note that for now this will only work if you modify the BIOS settings >> to change the ISP to be listed as a PCI device in lspci: >> >> 00:03.0 Multimedia controller: Intel Corporation Atom Processor Z36xxx/Z37xxx Series Camera ISP (rev 09) >> >> If you do not have this in lspci then things will not work. The BIOS >> does not allow changing this setting from its menu, so we need to >> use the grub setup_var_3 command form a patched grub. >> >> Note this requires your Asus T100TA to be at the latest v314 BIOS version, >> this has not been tested with other versions! >> >> Download grubia32.efi from here: >> >> https://fedorapeople.org/~jwrdegoede/grub-efi-setup_var/grubia32.efi >> >> Boot this once and then run: >> >> setup_var_3 6f >> >> If the ISP is not visible in lspci this should show 01 as value >> for the 0x6f offset. >> >> Now change this to 2: >> >> setup_var_3 6f 02 >> >> and then reboot. After this the ISP should be visible in lspci. >> >> For more details on how this works see: >> https://hansdegoede.dreamwidth.org/25589.html >> >> Note this series is just a quick hack, the proper fix would be to get >> the new drivers/media/i2c/mt9m114 standard v4l2 driver to work and >> atomisp does support standard v4l2 drivers for a while now. >> >> I plan to take a shot at this as time permits, hopefully sometime >> within a month but no promises. > > LGTM, some nit-picks in the individual replies. > Reviewed-by: Andy Shevchenko <andy@kernel.org> Thank you. > P.S. Can you consider my patch to apply (that removes GPL text > in the top comments)? Yes I'll pick this up together with other atomisp patches the next time I do a round of atomisp patch merging. Regards, Hans