From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 49482C433DF for ; Tue, 19 May 2020 07:39:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 21DB9207E8 for ; Tue, 19 May 2020 07:39:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1589873966; bh=RETqHAhDRrLingBIMpuJT+qul6CJYmcFdXTR2ALOTm0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=RE1T4QxupuyWdufm64IZTveSRmrmb9o0IxK3J0ueIvnNpT2KXf0Wcne3vXCU2EEDr 2blHJEi3W2PGpHaAWNCWNMP6qbZmb25AHL+oLOmYT1n1T1dZVvH4mpsKec8AzoGIb8 X7W+i0+yF0gBRAKzwf6+XB+YONfAybURUmTzTfSg= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728159AbgESHjZ (ORCPT ); Tue, 19 May 2020 03:39:25 -0400 Received: from mail.kernel.org ([198.145.29.99]:33582 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725996AbgESHjZ (ORCPT ); Tue, 19 May 2020 03:39:25 -0400 Received: from coco.lan (ip5f5ad5c5.dynamic.kabel-deutschland.de [95.90.213.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id E2FB4207C4; Tue, 19 May 2020 07:39:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1589873964; bh=RETqHAhDRrLingBIMpuJT+qul6CJYmcFdXTR2ALOTm0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=jyGli1kvt+0jVfnz91gzq2Yx6AQ/41XI6gfVGCYLSQx8meMWObCQNi4D24G2bSDM6 Euf3R5IsShLx1+JubrZ+CXvVAjA39GehmgQK+0cnpLJoksd29dgtdbxMaOyqth5Bh7 IMZMn1gREvAp8YXDtzHEGhcHf0HFa9q04LMzdyYY= Date: Tue, 19 May 2020 09:39:20 +0200 From: Mauro Carvalho Chehab To: Hans de Goede Cc: Patrik Gfeller , Francescodario Cuzzocrea , Andy Shevchenko , "linux-media@vger.kernel.org" Subject: Re: [GIT PULL] Ressurect the atomisp staging driver Message-ID: <20200519093920.7bb22161@coco.lan> In-Reply-To: <20200515114245.266a6fc8@coco.lan> References: <20200501215741.3be05695@coco.lan> <3f551a8f87808ee7828dc03d41c7a23faac89f3c.camel@mail.polimi.it> <20200503173213.78ae6aaa@coco.lan> <20200503180751.0b1e29c4@ASUS> <20200504101628.0f632bf2@ASUS> <20200504104934.7873cee3@coco.lan> <20200504124539.77eac397@ASUS> <20200504140833.11dd5622@coco.lan> <20200504154420.5dcf505f@ASUS> <20200515103232.47b2a35e@coco.lan> <20200515114245.266a6fc8@coco.lan> X-Mailer: Claws Mail 3.17.5 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Em Fri, 15 May 2020 11:42:45 +0200 Mauro Carvalho Chehab escreveu: > Em Fri, 15 May 2020 11:10:21 +0200 > Hans de Goede escreveu: > > > Hi all, > > > > On 5/15/20 10:32 AM, Mauro Carvalho Chehab wrote: > > > Em Mon, 4 May 2020 15:44:20 +0200 > > > Patrik Gfeller escreveu: > > > > > >>>> Maybe this is related to my partial build (only the module to test if > > >>>> it builds). I'll now do a full build of the kernel. > > >>> > > >>> Yeah, either that or there are some config options with different values. > > >>> > > >>> I was able to do a full build here without the above errors, although > > >>> I had to turn of some CONFIG options that were otherwise failing. > > >> > > >> Strange - I do not get it :-/ ... can you send me the .config file and > > >> Makefile you use; probably I missed something. > > > > > > I posted an alternate version, with the driver from the Yocto Aero tree > > > ported to Kernel 5.7-rc2. The Kernel sources used are at this directory: > > > > > > https://download.01.org/aero/deb/pool/main/l/linux-4.4.76-aero-1.3/ > > > > > > Such port is at the atomisp_yocto_v1: > > > > > > https://git.linuxtv.org/mchehab/experimental.git/log/?h=atomisp_yocto_v1 > > > > > > The goal of this port is not for upstreaming it. Instead, it is meant > > > to be used just for testing, using a version that should work - at least > > > with the Intel Aero board with the Aero drone's camera (tests needed, as > > > I don't have myself such hardware). This version is ISP2401 only. > > > > > > The firmware for it is at: > > > https://download.01.org/aero/deb/pool/main/f/firmware-atomisp/firmware-atomisp_0.1.orig.tar.xz > > > > > > If the ISP revision is equal or bigger than rev 02, this file should be > > > renamed to: > > > > > > shisp_2401b0_v21.bin > > > > > > Before getting excited, this version didn't work so far on my hardware: > > > > > > atomisp: Loaded firmware version 'irci_stable_candrpv_0415_20150521_0458'. > > > atomisp-isp2 0000:00:03.0: css load fw failed. > > > atomisp-isp2 0000:00:03.0: Failed to init css. > > > > > > (I didn't try to check why it is failing yet - maybe it is just > > > some version port related issue) > > > > > > If someone has access to the Intel Aero board, I'd appreciate if you > > > could test it and send the results. If it fails too, it would be > > > nice to have the dmesg output of this version and the dmesg from > > > a version which works. > > > > Mauro, first of all thank you for working on this. > > > > I wonder if you have tried to build and run the linux-4.4.76-aero-1.3 > > kernel on your T101HA ? It would be interesting to see if that one > > does (not) have the "css load fw failed" errors and who knows you > > might even be able to get that to actually show some sort of picture > > from one of the 2 sensors on the T101HA ? > > Yes, I tried, but the Kernel doesn't even boot. Grub2 simply doesn't > show any messages when trying to boot a Kernel 4.4 Kernel on it. > > As this device doesn't have any physical serial ports, and the only > USB cables I have are based on a chipset not supported by GRUB[1], I > can't see where it is failing. > > [1] A side note to myself: I need to spend some time checking what > serial cables would work with grub2 and try to buy one such cable. Eureka! Looking at the Yocto's port, I noticed this comment: #ifdef CONFIG_INTEL_MID_ISP /* Moorefield lacks PCI PM, BYT advertises it but it's broken, use PUNIT */ #define ATOMISP_INTERNAL_PM (IS_MOFD || IS_BYT || IS_CHT) #endif And some other codes like this: #ifdef CONFIG_INTEL_MID_ISP if (ATOMISP_INTERNAL_PM) { ret = atomisp_mrfld_power_up(isp); if (ret) return ret; } #endif Basically, for some types of hardware, it has to disable the atomisp_mrfld_power_up() and atomisp_mrfld_power_down() calls. So, a hack like this: diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c index 8c7b42221659..5c1f8bcc24f4 100644 --- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c +++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c @@ -824,12 +824,15 @@ static int atomisp_mrfld_power(struct atomisp_device *isp, bool enable) /* Workaround for pmu_nc_set_power_state not ready in MRFLD */ int atomisp_mrfld_power_down(struct atomisp_device *isp) { +return 0; + return atomisp_mrfld_power(isp, false); } /* Workaround for pmu_nc_set_power_state not ready in MRFLD */ int atomisp_mrfld_power_up(struct atomisp_device *isp) { +return 0; return atomisp_mrfld_power(isp, true); } Made the firmware to properly load, and the driver now shows signs of life: v4l2-ctl --all Driver Info: Driver name : atomisp Card type : ATOM ISP Bus info : PCI:0000:00:03.0 Driver version : 5.7.0 Capabilities : 0xa4200003 Video Capture Video Output Streaming Extended Pix Format Device Capabilities Device Caps : 0x24200003 Video Capture Video Output Streaming Extended Pix Format Media Driver Info: Driver name : atomisp-isp2 Model : Intel Atom ISP Serial : Bus info : Media version : 5.7.0 Hardware revision: 0x00002010 (8208) Driver version : 5.7.0 Interface Info: ID : 0x03000016 Type : V4L Video Entity Info: ID : 0x00000014 (20) Name : ATOMISP ISP CAPTURE output Function : V4L2 I/O Pad 0x01000015 : 0: Sink Link 0x02000062: from remote pad 0x1000010 of entity 'ATOMISP_SUBDEV_0': Data Priority: 0 Video input : 0 (ATOMISP ISP CAPTURE output: ok) Format Video Capture: Width/Height : 0/0 Pixel Format : '' Field : Any Bytes per Line : 0 Size Image : 0 Colorspace : Default Transfer Function : Default (maps to Rec. 709) YCbCr/HSV Encoding: Default (maps to ITU-R 601) Quantization : Default (maps to Full Range) Flags : Streaming Parameters Video Capture: Frames per second: invalid (0/0) Read buffers : 0 error 22 getting ctrl Automatic White Balance error 22 getting ctrl Red Balance error 22 getting ctrl Blue Balance error 22 getting ctrl Gamma error 22 getting ctrl exposure error 22 getting ctrl Light frequency filter image_color_effect 0x0098091f (int) : min=0 max=9 step=1 default=0 value=0 error 22 getting ctrl Image Color Effect CbCr bad_pixel_correction 0x08000000 (int) : min=0 max=1 step=1 default=0 value=1 gdc_cac 0x08000001 (int) : min=0 max=1 step=1 default=0 value=0 video_stablization 0x08000002 (int) : min=0 max=1 step=1 default=0 value=0 fixed_pattern_noise_reduction 0x08000003 (int) : min=0 max=1 step=1 default=0 value=0 false_color_correction 0x08000004 (int) : min=0 max=1 step=1 default=0 value=0 low_light_mode 0x08000005 (bool) : default=1 value=0 It doesn't stream on yet: [ 1040.688130] atomisp-isp2 0000:00:03.0: sh_css_init success [ 1040.688748] BUG: unable to handle page fault for address: 0000000000004518 [ 1040.688754] #PF: supervisor read access in kernel mode [ 1040.688757] #PF: error_code(0x0000) - not-present page [ 1040.688760] PGD 0 P4D 0 But at least it is now showing some signals of life. There are still lots of work to do before it can run, but it sounds that we'll eventually get there. Thanks, Mauro