Received: by 2002:a05:6a10:af89:0:0:0:0 with SMTP id iu9csp685989pxb; Fri, 28 Jan 2022 08:00:00 -0800 (PST) X-Google-Smtp-Source: ABdhPJxB+IeZfxLA5wKZx7+HteDRM9mC22MmlmNTLPOBzGvh+qvxDUPpHFFmtvaF1muPsPXGMom/ X-Received: by 2002:a63:2507:: with SMTP id l7mr7020482pgl.339.1643385600148; Fri, 28 Jan 2022 08:00:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643385600; cv=none; d=google.com; s=arc-20160816; b=TWRjUypRKzeJcbEOKoYhAjXbqGS8Rnr9/3pebHphALqH5gRRt8BgHPtoVucVvDPA1X xAxPTT3g7HM0r+b9s1Teyfv1wuFulbUiq2DHlcUw0QDJaVE2aMfw4kh0OyCYismoktWm ilmrmG4xkJwnwvgDD+vPIL7o0syBvvP38UepJQKd4M1EhH4cnZdkS/XXtQ5QRX5NLUgi nTVO+k6H41WnToEdUGuTn2J2pI15SWTwAl1PxYuRNXfXj3IRbWtpoDb4EVbwAy1OfNmo Mq/sfj1PPJNkeIPb7dNsB7QsZXnyd0nAZTZkhdgCB9BtfhWEQwm13CBXRdDP7SVXP8LF hofg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=Kf+kdg2c1HJXuwMScXRqFZTXE8Hr5rEGSTqbhdf4ZY0=; b=LOYbAvQ4IyvxcDIxEkzZTjYsb26qhteRJFzoL8qiw0tFy8hEoygZ2ZnmWlj3q5Dz// koZr4yH877Dtj8dZW6pVozvalLMl8iJrYgVY6/yb12r6AZHzNAzThztimAmDvcZS+avX ARsG3mIL6Aodeizdn8o9QOHQBNuHArWeSQzrJBML/WHn2AmuBplFzbh4Xpy8KJfDU3nK pKBd68CH9G3NLoVFi8KpvIbyQmVKwHGMEvjAO2JC2pR8izVTOGXrbEP61c+Bjx1ZGITb knDoylqGTnfPX2xoqQSFjSke1X33rkawEEZGlVMJzjyoXTCrmK6RNZ9kktMFQm5ZrQw7 lI5w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=SHlJFbWQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id gf11si2632304pjb.71.2022.01.28.07.59.47; Fri, 28 Jan 2022 08:00:00 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=SHlJFbWQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239208AbiA0S7N (ORCPT + 99 others); Thu, 27 Jan 2022 13:59:13 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56934 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232813AbiA0S7M (ORCPT ); Thu, 27 Jan 2022 13:59:12 -0500 Received: from mail-pg1-x52b.google.com (mail-pg1-x52b.google.com [IPv6:2607:f8b0:4864:20::52b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 644B6C061714; Thu, 27 Jan 2022 10:59:12 -0800 (PST) Received: by mail-pg1-x52b.google.com with SMTP id h23so3001214pgk.11; Thu, 27 Jan 2022 10:59:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=Kf+kdg2c1HJXuwMScXRqFZTXE8Hr5rEGSTqbhdf4ZY0=; b=SHlJFbWQLWkZNDz26HSpcPryc7ewCVr0VBXuXoOwGPVHQBahF4J2x943moqrQdDYoj 8kuW8y0ZqhhE1yk9JXOwTYnVh4KKlv8YLahzGsI3YWjcalMGQocu13vFnJ78p6LR6zLU k63lc/02L/H2JfFHVTbU6rv5N5dV20+nLVdpu3PN448rTlS3jI6v9yrg0V4f3AQ2yepS kBkWQPvlmSSaxQQEZBFuD5J9EdTHBtBtB1lHBD3UecOTfMoZm0DzhVk4AC8flHg/Tt84 yg0RWNVvATOGq+qZPNypnZF1Dt+VU9SyCHCXt2FKPjjwJz3K0rbT2p9mcKaZWuFb3rLt yeZg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=Kf+kdg2c1HJXuwMScXRqFZTXE8Hr5rEGSTqbhdf4ZY0=; b=k/gk36kU13sOWshxkx+heNzAdoMZdgeOWedY8/j3tCYGGsaBI/RPdLBl9c4JKsjS2N neUFZTYFf3JizMb++WAkZogENlWJz0JjiaRc5BP+u0Xv6CcRxtwE0QtU1yG+91jcjtTO qoAMXmA0aJHnDwcO8utD9AdCxUO3ICwsOloMqTNVUpe2CgFXm4Oq5SZLhzxWW0ODjmzH TQsfUGz22BqYLxFLyBj3zgZPAVGBblIuSxWslgPV4Ag6snyDD0oibNe92VkyoqeajnAV e4IMVW3rzJB5LOVya0bZa2Fwuk8bKyzWSwd5TH4szgBMVLtPQpW/PU9ZvAykJnHueS1v 6DeQ== X-Gm-Message-State: AOAM532kxYM82s2GxXCIvzS6BGFFQmbyP/gESKHAhonEUOCFGWAmieJn 9Oxq2P4oCQ/uaCJRmC76Zpg= X-Received: by 2002:a05:6a00:1992:: with SMTP id d18mr4030814pfl.85.1643309951730; Thu, 27 Jan 2022 10:59:11 -0800 (PST) Received: from [192.168.1.3] (ip72-194-116-95.oc.oc.cox.net. [72.194.116.95]) by smtp.gmail.com with ESMTPSA id p22sm6507491pfo.163.2022.01.27.10.59.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 27 Jan 2022 10:59:11 -0800 (PST) Message-ID: <01d73e56-346b-a175-6113-db98be6b0a53@gmail.com> Date: Thu, 27 Jan 2022 10:59:10 -0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH] spi: bcm-qspi: check for valid cs before applying chip select Content-Language: en-US To: Kamal Dasu , broonie@kernel.org, linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org Cc: bcm-kernel-feedback-list@broadcom.com References: <20220127185359.27322-1-kdasu.kdev@gmail.com> From: Florian Fainelli In-Reply-To: <20220127185359.27322-1-kdasu.kdev@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/27/2022 10:53 AM, Kamal Dasu wrote: > Apply only valid chip select value. This change fixes case where chip > select is set to initial value of '-1' during probe and PM supend and > subsequent resume can try to use the value with undefined behaviour. > Also in case where gpio based chip select, the check in > bcm_qspi_chip_select() shall prevent undefined behaviour on resume. > > Fixes: fa236a7ef240 ("spi: bcm-qspi: Add Broadcom MSPI driver") > Signed-off-by: Kamal Dasu Acked-by: Florian Fainelli Thanks! -- Florian