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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E628AC433EF for ; Wed, 22 Jun 2022 13:54:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zO6nW7VlO0HqnG5VfSjJxm+hCSt1HF+uOHOAfhtzMYc=; b=wqhUyFfQQPd/oY sGNBieIOq9zrhQKtgqivrKswBDHDUAq9z4IoXwJ9tKzE8PVzgsQhqJb8unzJkjhTXJ41ZdEkcPFvq KqiGBGHSWq2EC5HhFNPGOObSPFshbaeYZWMd70HtC7PmgF6ChcVdVwzF1TvWvib8OftHcnX03/4zJ e6SM2Nm8MgjWsN2vW2gapy4Z7KlFmcq61PdVPxyT2Ok9fJBOBA5DkuDTfbH51oKaxwk0ePLiv9uMU PO8+CxovvzvKAjeFD/qKVejgRudxoPhZO/gi7YctXjdw49+512VLEegwXfXs0P+nSC04jL59nsm// +f38PTditwcnsiwQRU2A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o40mp-00Aivy-FJ; Wed, 22 Jun 2022 13:52:51 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o40mk-00Ait0-Js for linux-arm-kernel@lists.infradead.org; Wed, 22 Jun 2022 13:52:49 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1655905966; x=1687441966; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=/dGSUwIfXQDhMWzryhdC635Q77eZu7olT6rsBE7VZPA=; b=vYdsxr6E7JVKABm3VfMZBCCvBRYcGKCZuvwGvMg4aERQ0R2d8H7uUBAq kKK3ab3Wq8GQRqtj6qnfVyHKX2rfXvvpitdm1TDHBluNFzplR8vZs0xW2 OGh/PD59dZMOoAGH2Rd1HDvFMgWjtnP+mGkzAX1zyxaysmCv2CXRy4uBp +orqtHMcM9zWLV0/0LCcgx6L18LabAiyPmwLmxw2UuBsRt0Yu0oqKoVCD oKnVWEA3R3ogFSBsVVp4prICZ/WrQZPfnXBv2WtBWT50Y60qPCKtgsHNu m37VWiPX9CBV5DcvjbMiFqKIX7aysXSqiJOVJN1ULwsdNa3AaiQ3jwa9k Q==; X-IronPort-AV: E=Sophos;i="5.92,212,1650956400"; d="scan'208";a="161544392" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa4.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 22 Jun 2022 06:52:43 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Wed, 22 Jun 2022 06:52:41 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (10.10.215.89) by email.microchip.com (10.10.87.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17 via Frontend Transport; Wed, 22 Jun 2022 06:52:41 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZjPz4sIUvkWtsJD0VTJYFKjahy9asw76sSp8/u2Hgp2hcxoFaJ/nfJfqP3ue21MS7SLALJOnfq75O21MjtclxaUnBhm+aegcBsJOChT+eptABZzyAe5loD8Xsy6ys62k9hDPiuKTzUHq+sdy6lzd5TYu7TTFRMGrKbuMdk4vlcwPOQYvdOblWoQCxKCAbpKVBVgxJ8lMux14RlSV1WSKS9PVvEZzuQKIjT/Rj1dFvE1S3K+uebXy/z/g6VyfcA34XzHEicnaNatInPcEBFMauoIteQ7IBkGOdn546vdsZ2GHqruIBWzoO3Xykyubb89AvfEq/5EBYyXh5eN1KSStFQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=/dGSUwIfXQDhMWzryhdC635Q77eZu7olT6rsBE7VZPA=; b=j3V126WFZgfn9Iq6nXUtTjBoneb53sOyYuPru7GeKALGaG4Fk96YWhmcVGwbBAJ5qx2XekCh4qDLEDnCJsTNbGyH1InyHEUs+hnFAntXEFBj55poejArMgeLHUaPKHDJfxiBXZzQ5HeqFYuxIEAtQ/cdLUxYdqBnVPlzePyA1j0q5h+g60jzzonioI3WwjP75Z5B5klRhLxBZeIP8m6I+F261AfNA2ZdJ8VL5a9ro70kAosaBouEscpnHR4psf1kVHWTWy2vBEI9h1TFIrn0C4yrRtCFlfYDuIt5ztceC62chL+ciSDyPFf7QDJgPk3LJdLGakxEzPX8vUdIZ/Pm4g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchiptechnology.onmicrosoft.com; s=selector2-microchiptechnology-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/dGSUwIfXQDhMWzryhdC635Q77eZu7olT6rsBE7VZPA=; b=LMAIjzJnwvlSIFvIBm9jI6LAs6xC8o3VNUqoUrlQLh/uRxje58mUdW91cSqJhhGPjDPj+AfXObUyKao1HxtO4NxRwC8G2E8Kxb4QdKUm5xaAtgcoXPPrgZtBxelQH7qZFZ8X/KHHi/fMTd7uE/uZEI94Blo9dylOTP8V1cR53jU= Received: from BL1PR11MB5384.namprd11.prod.outlook.com (2603:10b6:208:311::14) by BY5PR11MB4184.namprd11.prod.outlook.com (2603:10b6:a03:192::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5373.15; Wed, 22 Jun 2022 13:52:38 +0000 Received: from BL1PR11MB5384.namprd11.prod.outlook.com ([fe80::c9c8:3bab:6b8f:5376]) by BL1PR11MB5384.namprd11.prod.outlook.com ([fe80::c9c8:3bab:6b8f:5376%6]) with mapi id 15.20.5353.022; Wed, 22 Jun 2022 13:52:38 +0000 From: To: , , Subject: Re: [PATCH v10 0/5] media: atmel: atmel-isc: implement media controller Thread-Topic: [PATCH v10 0/5] media: atmel: atmel-isc: implement media controller Thread-Index: AQHYXtNy0DBb1d5tjEaFV5GKOhbPdK1bn80AgAAIywCAABOpAIAABMaA Date: Wed, 22 Jun 2022 13:52:38 +0000 Message-ID: <5f7e6025-295f-55d6-067c-e0d341e58293@microchip.com> References: <20220503095127.48710-1-eugen.hristev@microchip.com> <1da61f9c-0605-dc9d-63a3-21c18fcb74c7@xs4all.nl> <0455d962-d13e-9d88-c513-282defe07dd2@xs4all.nl> In-Reply-To: <0455d962-d13e-9d88-c513-282defe07dd2@xs4all.nl> Accept-Language: en-US, ro-RO Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 518a381d-bd9d-4db9-660a-08da5456781f x-ms-traffictypediagnostic: BY5PR11MB4184:EE_ x-microsoft-antispam-prvs: x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: bZgHgpeYSVfxTp31fRaoiW9vA0SUt1xxI7S2TSWAJI7ptucEfrtRgg1TlCqthhmX4/oFVHvO+AhrroodL+ML4mJJ/CiXvX9N3g4GrZqGGB/UWxmLDB5Vvrpd3EgaxL6QUfU/nRC2/mhkodHiPXm8kc61N43gVn3lSeqrpBCFpq/xM1Ma7d9B7EdwK7bbAd4mckPhgasEuN4w9tN5gk+HwCB78V63nqFmIsAeGVCrUfq6QliDlaSV8CWYnk41Y43Uh3lap8BIDidh/LH4hsC3jCQNkW6u7J9AaK1+22jo4QZsNmIhqCI0J8bHYrOVBSn7alkpGCxkpB2bm2IBz7YNUM4yZvCESv6Y57nVczQIvWKhqYxcg4VA2uppfKLTtGFE9/TYlzXZPNDzgh1i0cOPxD49WSKG8fGFY+m/8i5BmoARfGI1WYjU3VJEJez3XugK0NqTAmZNPa5sSuHT7vzndhlmEi0NZBItUW8rwjZfGX4NV4/BrLicDCT+fn9PfLb8mlpH4wjryvXdjFXCX/94TKsYWBwDtu3NVciUT4E/BQZ61iiTwRjOmw+xyXoumoMBu/3Bdp4cwPyqRk2BQ3I/Ew3Jmf0RjvTLLjJSXQsCunF3kD5kCH33Fckvam0bSKgtomf3PqEqW896mbiZAKdcd9+gIwn3MuQdOpwc3WuE52XE60iVeK/2iRoBzXdIq5hMTGKTBimcUkxyuAu4LEQ3i2spB6cDeepCo4/1759+U5THh3pv8lYU/m4v1TIfHTde3CRuRS2+SZRcxP0Iq56RTatwbJt9kVoG8Myw13+D4pxOa3H+KLmUqZr0smkM0GS592l0StYgUp9h24m9GcESVrrnrZnKGlE5VNnj0ys0pQs= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL1PR11MB5384.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230016)(136003)(39860400002)(396003)(346002)(376002)(366004)(5660300002)(966005)(6486002)(8936002)(478600001)(6506007)(6512007)(26005)(86362001)(31696002)(2906002)(122000001)(53546011)(38100700002)(2616005)(41300700001)(186003)(38070700005)(83380400001)(4326008)(76116006)(66946007)(66556008)(66476007)(64756008)(8676002)(66446008)(316002)(36756003)(31686004)(110136005)(54906003)(91956017)(71200400001)(43740500002)(45980500001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?ZEpzRUFQd2lnZVN3SGwvNUNJS0FxejROZUtIanJINzNIMXZWQ241b3lNdGV2?= =?utf-8?B?SVlwT1BRcWt6VVdqVThyYWlyNEg0blNnemtkU1lxYlByQjA1eUxhTGI5MTY5?= =?utf-8?B?dm9mMWNkVkptcy9uUjl0OFU1VHh4NmRobUNGb3lkbUZwQzliNWVDT3RsS0Jj?= =?utf-8?B?b2pDQ2lpYVZJSVhGUUdsTWlSM3podWlZTGFMTDdhWkVNckRsQ0p6V0VwcG9Y?= =?utf-8?B?ZVVoanJ0c28xVXZnYWp1YjJHT1duTExhdVJ4YWsyTUp4SEVmRDYzeGFpaUhU?= =?utf-8?B?cjVHa0JSWnBKaFUxSVlCcnIxcFpCQnNwVHBXdFhtWFA3UmRkUk9HM2ZlNXlC?= =?utf-8?B?T2dXSjdFSWxJRjNwQThyMnVWcytqRGhnUU1jdjd3c1lpNnlxMTlTUGZWRFNB?= =?utf-8?B?c0dWQ0pBOEFFU3BBbm9BaHRud0VDWjJrNUI0UXZaQnNNdk12cGhVU0tkWXFI?= =?utf-8?B?UC9ONTVyRk45S3lWQjYyb29wZEprN0NHTTE4aXQvcVlIZVNETFFUSnFvSjdz?= =?utf-8?B?QXN0S3VXRThJVmFBdEhLdS83Q1Bhc2RXZXk5RUd2U0JHT01EdW1mQWJKRkVu?= =?utf-8?B?aWEwZ2hleVFOOGdpd010NnhURTJWVndwTFoxMzM2RzV3dDUxbThDKzJmdmxl?= =?utf-8?B?YWxRNDVtaGtRRDYzbFE0NzBTRnZqQ3EzajhYUmVRUTBrM3dteHZZcWRYaGo0?= =?utf-8?B?OVFyN2FOUFEzVHc5LzdyYS9tZFVzME1ZMjFTY3Nwd29sZm5SUGVRUlNTWXVv?= =?utf-8?B?cFVFTFBKcVIxYUZ4aDZmVktNN2FDcnFzbDQzL2UzQVBWb1h2ekthckx2eSsz?= =?utf-8?B?TitSNGlJVmJXWGFLVjNTNE5PY0NZcXByTG56U25yNmJKVVprd0pyZlh4K3FP?= =?utf-8?B?ZktlMHFuZDBPMi9oOTlNdUNxL0diVHUvbk9RQ0E0aVhJRllQSWQxZ1RlQUVJ?= =?utf-8?B?VXNCd01odURxOW5OVUVlajczNCtVcGN3empBa012aThzVHBVb2JyUXYzdHoy?= =?utf-8?B?UTB3RjM3c3NhaEorNXVZeTNNaU9STERSSHlLdXU4dGtwUmdqWXVhemhuTlRG?= =?utf-8?B?V2NtSkErdzZzNWN1d3ZnT0FJQWFYVEJ4ZStWTlZlODV2cUNJL1VNR2Y5ZjRM?= =?utf-8?B?RC9FWE00cEpiVzRFSFhQU0RpNW5kQnIzTHdGc0VlQjQvU0c3anNBVWtIVm12?= =?utf-8?B?WjZkMFg2dGFvYUFpamRtZ2xuSDV4WXBXZnpTVmdkOEdMcWtVZ0sxNlVMWER1?= =?utf-8?B?d2lnekt1Z1Q3UG03ZWhoN201cm44MExUdXdjYzJ0RGdmNDVpTU1QVmxTVmZu?= =?utf-8?B?T0dCRU5EMXh6bno4MndHNGxsczZQV1VmcCtHbjdUc0J1c09sMWRSK1lzYXFi?= =?utf-8?B?NVFsZENyaklNNTlZTnJSam5BNy9vN0NJaE1XODVUelJmdFVWd3UxZE1hVWJJ?= =?utf-8?B?UkhhVHkzSDZCSVhrRFJDMFdaUks1bVVEWkdiWi9vRjd2dnpPL2pac0FFSjhZ?= =?utf-8?B?UW5rY0tINHNWdTRYK0RuQmVzaEhzQnZmRmxUSU9pc1B1ZkNlazM1aWNpRnlo?= =?utf-8?B?L0xsdDNtdFNpSkdndTdjenp4bWNkdndWRnA4L09XWHhrcldFMUhGMHJXRzgv?= =?utf-8?B?NnZYNGc1WEJ3a1dDVlhna0wxVFlYZHo1ZzZBaXBKMFk4RnpscjZRdVNtSG1Q?= =?utf-8?B?NTVsRndIUlF3NlZmRURxUWd0Vk9PT0xpR3FOcWVBd2Fwc0doN3M0UncxM2gx?= =?utf-8?B?QklwTFF1ZlRXLzIzRXBZcTlNTnlJUnFtY3lwRkJIcmpzYURNb1ZZeWhpaGZu?= =?utf-8?B?OVh5VFJaR09KYThkTnYweHpabjFLZW9JVVg4UWNtaE5tYzdHUUNpUzhRa2F5?= =?utf-8?B?VTlzLzllamtCNUtWRkpkc0p0anN3MFc0UUZQOHo5OU1aVk9GYUh4RThCeWt6?= =?utf-8?B?VFJCZFFGMUJvL0lteVVvMlQrZ1VKclZlRVA4cXB4My9qV0xiYkl1VHZucFAr?= =?utf-8?B?NWpwZVZFNXlvQTNzMFVmUTJJZC9CQUdNWWhGaExyWm80U3RYWmo3Z1lwckZJ?= =?utf-8?B?TExGSEx1RTI2YVBkTDY1K1F1a3JvbnJHcWtyZTAvMjFwdGpka3FQUkZudTll?= =?utf-8?B?Q0xWZU5WTVgrK2NoaStFc0RaUHJ1eXBpbkhGUENZaUgwN0liZjhMQk1Ud3RV?= =?utf-8?B?elJNSlF1ME1YejMzVXNQUndQOVRabU5xblBmaVpYZndJSWZZaHRUUkY4TGtt?= =?utf-8?B?V3QycVV3Y3hoNXNGeDg5MmExMnpjaEE5SzNSOWF5b1UwQWFMbG00R0pWWG82?= =?utf-8?B?a1J2QVdQdUVZR3lzQ3gyRlBOWjREV2hsYW5JdGRLdkxoZFIrekhFazVpdTlC?= =?utf-8?Q?URe7gYPh3rYbD1VM=3D?= Content-ID: <78510C5CE63DFF4897B8C10E2E99DC2B@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BL1PR11MB5384.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 518a381d-bd9d-4db9-660a-08da5456781f X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Jun 2022 13:52:38.4965 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: yW8yZEQu5vti+VIszxBoAu4J5ExleLtMMZh8jj5kqM6K+XqMMQ8J+qH6nqO9zDY8ApHvK05fo6oWn1iD/4RsHnEZHE6JSkWXoWdQv0NS66M= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB4184 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220622_065246_796761_7435CC87 X-CRM114-Status: GOOD ( 34.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, jacopo@jmondi.org, linux-kernel@vger.kernel.org, Claudiu.Beznea@microchip.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 6/22/22 4:35 PM, Hans Verkuil wrote: > Hi Eugen, > > On 22/06/2022 14:25, Eugen.Hristev@microchip.com wrote: >> On 6/22/22 2:53 PM, Hans Verkuil wrote: >>> Hi Eugen, >>> >>> On 03/05/2022 11:51, Eugen Hristev wrote: >>>> This series is a split from the series : >>>> [PATCH v9 00/13] media: atmel: atmel-isc: implement media controller >>>> and it includes the media controller part. >>>> previous fixes were sent on a different patch series. >>>> >>>> As discussed on the ML, moving forward with having the media link validate at >>>> start/stop streaming call. >>>> I will test the patch : >>>> [RFC PATCHv2] vb2: add support for (un)prepare_streaming queue ops >>>> afterwards, but that patch requires moving my logic to the new vb2 callbacks. >>> >>> I'm looking at merging this series, but I would like to have the output of >>> 'v4l2-compliance -m /dev/mediaX' to verify that the MC links etc. is all >>> correct. >> >> Hello Hans, >> >> Please have a look at attached file . Unless you want me to add the >> whole output to the e-mail ? > > No, this is fine, thank you! > >> >> I also added output of media-ctl -p for your convenience. >> the subdev2 is a device and driver that is not upstream and has some >> compliance issues, they are reported by the v4l2-compliance tool, but >> they should not affect this series, it's a synopsys driver that was >> rejected on mainline a few years ago, I took it for internal usage, but >> it's not cleaned up nor worked a lot upon. > > OK, good to know. > > From the compliance output: > > v4l2-compliance 1.22.1, 32 bits, 32-bit time_t > > This is an old v4l2-compliance version. Compile it directly from the > v4l-utils git repo and check the output again. Okay, I will start a build. I have to get my full environment ready, fire up a Buildroot. It will be ready tomorrow. > > Compliance test for atmel_isc_commo device /dev/media0: > > As you can see, the driver name is cut off. Isn't 'atmel-isc' > a better name? Maybe, but this name is the name of the entity, and is taken from the name of the kernel module. Since the split of the isc driver into atmel_isc_common and atmel-sama5d2-isc , atmel-sama7g5-isc , this name has been set for the module here: https://git.linuxtv.org/hverkuil/media_tree.git/tree/drivers/media/platform/atmel/Makefile?h=for-v5.20e#n7 And the ISC driver takes the module name and uses it for video name here: https://git.linuxtv.org/hverkuil/media_tree.git/tree/drivers/media/platform/atmel/atmel-isc-base.c?h=for-v5.20e#n1999 > >> >>> >>> And one more question which may have been answered already in the past: >>> >>> Changing to the MC will break existing applications, doesn't it? Or did I >>> miss something? >>> >> >> The existing applications will have to configure the pipeline now. It >> will no longer work by configuring just the top video node /dev/video0 . >> They would have to use media-ctl for it, something similar with this set >> of commands: >> >> media-ctl -d /dev/media0 --set-v4l2 '"imx219 >> 1-0010":0[fmt:SRGGB10_1X10/1920x1080]' >> media-ctl -d /dev/media0 --set-v4l2 >> '"dw-csi.0":0[fmt:SRGGB10_1X10/1920x1080]' >> media-ctl -d /dev/media0 --set-v4l2 '"csi2dc":0[fmt:SRGGB10_1X10/1920x1080]' >> media-ctl -d /dev/media0 --set-v4l2 >> '"atmel_isc_scaler":0[fmt:SRGGB10_1X10/1920x1080]' > > I'd like to see this documented in a new > Documentation/admin-guide/media/atmel-isc.rst file. That can be a new patch. Allright, I can do that. > > Regards, > > Hans > >> >> Thank you for taking care of this ! >> >> Eugen >> >>> Regards, >>> >>> Hans >>> >>>> >>>> Full series history: >>>> >>>> Changes in v10: >>>> -> split the series into this first fixes part. >>>> -> moved IO_MC addition from first patch to the second patch on the driver changes >>>> -> edited commit messages >>>> -> DT nodes now disabled by default. >>>> >>>> Changes in v9: >>>> -> kernel robot reported isc_link_validate is not static, changed to static. >>>> >>>> Changes in v8: >>>> -> scaler: modified crop bounds to have the exact source size >>>> >>>> Changes in v7: >>>> -> scaler: modified crop bounds to have maximum isc size >>>> -> format propagation: did small changes as per Jacopo review >>>> >>>> >>>> Changes in v6: >>>> -> worked a bit on scaler, added try crop and other changes as per Jacopo review >>>> -> worked on isc-base enum_fmt , reworked as per Jacopo review >>>> >>>> Changes in v5: >>>> -> removed patch that removed the 'stop' variable as it was still required >>>> -> added two new trivial patches >>>> -> reworked some parts of the scaler and format propagation after discussions with Jacopo >>>> >>>> >>>> Changes in v4: >>>> -> as reviewed by Hans, added new patch to remove the 'stop' variable and reworked >>>> one patch that was using it >>>> -> as reviewed by Jacopo, reworked some parts of the media controller implementation >>>> >>>> >>>> Changes in v3: >>>> - change in bindings, small fixes in csi2dc driver and conversion to mc >>>> for the isc-base. >>>> - removed some MAINTAINERS patches and used patterns in MAINTAINERS >>>> >>>> Changes in v2: >>>> - integrated many changes suggested by Jacopo in the review of the v1 series. >>>> - add a few new patches >>>> >>>> Eugen Hristev (5): >>>> media: atmel: atmel-isc: prepare for media controller support >>>> media: atmel: atmel-isc: implement media controller >>>> ARM: dts: at91: sama7g5: add nodes for video capture >>>> ARM: configs: at91: sama7: add xisc and csi2dc >>>> ARM: multi_v7_defconfig: add atmel video pipeline modules >>>> >>>> arch/arm/boot/dts/sama7g5.dtsi | 51 ++ >>>> arch/arm/configs/multi_v7_defconfig | 3 + >>>> arch/arm/configs/sama7_defconfig | 2 + >>>> drivers/media/platform/atmel/Makefile | 2 +- >>>> drivers/media/platform/atmel/atmel-isc-base.c | 485 +++++++++--------- >>>> .../media/platform/atmel/atmel-isc-scaler.c | 267 ++++++++++ >>>> drivers/media/platform/atmel/atmel-isc.h | 50 +- >>>> .../media/platform/atmel/atmel-sama5d2-isc.c | 34 +- >>>> .../media/platform/atmel/atmel-sama7g5-isc.c | 32 +- >>>> 9 files changed, 685 insertions(+), 241 deletions(-) >>>> create mode 100644 drivers/media/platform/atmel/atmel-isc-scaler.c >>>> >>> >> > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel