Received: by 2002:a05:6a10:413:0:0:0:0 with SMTP id 19csp623889pxp; Wed, 9 Mar 2022 09:17:37 -0800 (PST) X-Google-Smtp-Source: ABdhPJzneXiHA1no8K3CqOcDT/Dj0EaYpkrkcj5TThlQsZMR/maZFI6a1hOhpr3PPyak8e2VkcDg X-Received: by 2002:a63:e747:0:b0:372:c757:c569 with SMTP id j7-20020a63e747000000b00372c757c569mr604786pgk.516.1646846256811; Wed, 09 Mar 2022 09:17:36 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1646846256; cv=none; d=google.com; s=arc-20160816; b=niUZXzNAQbNLWzkjB8ebmSgDhniEmsde4AP+AtI5Ir9Q+1JQ1sVtTUZ6+Idt39+Z/M Ym53k87AuMiwt6/z+4pT9J/RFMcsZd2JIbZ7r/9Q6LTCWgpC6C+dGMjv5FzPLeKvy/ps 9PNgrsSED0RnDByXHO8aVLlCpELN83ZjfQEgeMMvo/rfkqzxMa4+DZYVE3q9Nn5hPEiT cERwp/qpjxpJxaARgqD5e8aETX9iibP1KH+3YcmCXzNk3DV9knTT3BxXVmiURyqVEY+1 el9a/8N/ThEb8khIJs0Qppm24FBqTBTFo6nsQmWh+99eLIHqyN8Cbu84NeU0LhOVR5mN pR5w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=+XmXyDdzXTmYSkLqHU9R3AoYt+Mv+MfQmKLG3KLVNn4=; b=FsRtAAT82mfhrx9puu5i8sIou3R1aueeojm1Y9+KKZT2hIIWYwrz0hMqDFqm1Xk4lE lCu+HORSRDQxEbkdLg+1nCIde0zvSZXQhrDuWeOK7J5Pz+dNfy9/sIjqedmQ5M2HE5uG YkBxGWTyHGK5FFLGQ5LdQkHmfnVIykfPX4lFdfuZ6P7RTg3ts6IMYtdcwKKDDyIO5mNn mC9uGSfED/zyKNhQ1CJVhJfroosb609pEqb0ZdFc5cEvKwgVjaz0W9nh2iGnmH+3b/QG MaDDQXFCgggQBc4pcllVpBpNTTGYLi3asr6Zxko6DhCRmPtqWaMI7kfETl1QiQRlPQvn 8jEw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@microchip.com header.s=mchp header.b=DCXCjW4X; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=microchip.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p8-20020a056a000a0800b004f2a6ab0ce4si2767857pfh.5.2022.03.09.09.17.20; Wed, 09 Mar 2022 09:17:36 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=fail header.i=@microchip.com header.s=mchp header.b=DCXCjW4X; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=microchip.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237546AbiCIQxX (ORCPT + 99 others); Wed, 9 Mar 2022 11:53:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235843AbiCIQpo (ORCPT ); Wed, 9 Mar 2022 11:45:44 -0500 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.153.233]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C04266D87C; Wed, 9 Mar 2022 08:39:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1646843947; x=1678379947; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=Iu9Y9l00LK8qD2HMPVyxqsR2snkJn5NMx606dNlvBmQ=; b=DCXCjW4X9eNeWUeN95Pxe6Fn+nFfrzhlzUqTx0v9zrSLiaJecYxxY3FK cUB+CkTo6lApXeoo+PAim3P4dVsA229ZeftnlSnLrYxTSj1RV4LJYUr3O QFCkjwMkcF8Z5QS+gJd4+tZtnJpGXMMi5OPcqL1TmKc2YWLyVW4fHgQS3 KsStjbRdv2fMMTijX3dQkKUMiEL55DXnP5bGFxSo5VjC77ADuPmtcFRUZ 2qekBx7sGufJ4OrefQFRqGoxHCgkNPf4eymxEtETEL80ovzTWMjFnefns 1H+t18HZaxxQhVgsv3RJW4H6bPbZuRC70Xwh6hOzBFpecaz2rf8c7KxJF Q==; X-IronPort-AV: E=Sophos;i="5.90,167,1643698800"; d="scan'208";a="165122975" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa1.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 09 Mar 2022 09:39:07 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) 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, 9 Mar 2022 09:39:06 -0700 Received: from ROB-ULT-M18282.microchip.com (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Wed, 9 Mar 2022 09:38:57 -0700 From: Eugen Hristev To: , CC: , , , , , , Eugen Hristev Subject: [PATCH v8 05/13] media: atmel: atmel-sama5d2-isc: fix wrong mask in YUYV format check Date: Wed, 9 Mar 2022 18:37:50 +0200 Message-ID: <20220309163758.2672727-6-eugen.hristev@microchip.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220309163758.2672727-1-eugen.hristev@microchip.com> References: <20220309163758.2672727-1-eugen.hristev@microchip.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,T_SCC_BODY_TEXT_LINE, T_SPF_PERMERROR autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org While this does not happen in production, this check should be done versus the mask, as checking with the YCYC value may not include some bits that may be set. Is it correct and safe to check the whole mask. Fixes: 123aaf816b95 ("media: atmel: atmel-sama5d2-isc: fix YUYV format") Signed-off-by: Eugen Hristev Reviewed-by: Jacopo Mondi --- drivers/media/platform/atmel/atmel-sama5d2-isc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/atmel/atmel-sama5d2-isc.c b/drivers/media/platform/atmel/atmel-sama5d2-isc.c index c244682ea22f..025c3e8a7e95 100644 --- a/drivers/media/platform/atmel/atmel-sama5d2-isc.c +++ b/drivers/media/platform/atmel/atmel-sama5d2-isc.c @@ -291,7 +291,7 @@ static void isc_sama5d2_config_rlp(struct isc_device *isc) * Thus, if the YCYC mode is selected, replace it with the * sama5d2-compliant mode which is YYCC . */ - if ((rlp_mode & ISC_RLP_CFG_MODE_YCYC) == ISC_RLP_CFG_MODE_YCYC) { + if ((rlp_mode & ISC_RLP_CFG_MODE_MASK) == ISC_RLP_CFG_MODE_YCYC) { rlp_mode &= ~ISC_RLP_CFG_MODE_MASK; rlp_mode |= ISC_RLP_CFG_MODE_YYCC; } -- 2.25.1