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 4B5EBC25B08 for ; Thu, 18 Aug 2022 02:39:38 +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:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=r+NYxhYdjEECe5tFJTiDzuzB/EPzUdBFKpQ7tlNwkHM=; b=KmldT2bCkfHKN0 qIFkMqtKgAd9Fpdgec09a30ImO46hYWLiV5Gz3uJGk1UDYiKPkY+cx4qB11SN64zbf1tIspd/3hxn zqFghkNJdnzxwvwmF6ZupPHaDaA/n9pIUDs7kjf/x9of0qKmne61WJ0kw0LqB9Xjd6v/zXrV4zD8i BPRKWScLrqcLpGjMi2ek9OK9aQN6GX4h98pHY+sX30sPPiyu47zHwUIDu9TSj/BX6SUMxL3dVZD3q iTgaPmlNUuGIQfqKbg8lYtQqVgBr3oVBSvmBo2uKf8qyoRzXg8KShfjBo6wdCS4H981nv9Fv6uLhy V4Uio3IFZL1PBtQXgh6w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oOVQT-00BhyL-H5; Thu, 18 Aug 2022 02:38:29 +0000 Received: from mga01.intel.com ([192.55.52.88]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oOVQQ-00BhuX-2Z for linux-arm-kernel@lists.infradead.org; Thu, 18 Aug 2022 02:38:27 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1660790306; x=1692326306; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=HeFoWCJ9GssholJv33MN3cVjscg4wW4frE7g9VETT9U=; b=QRQo+wEuHQYq435+8Ge4Rgh9MmC2N4lDQ1HBcRqX7TEVyX8Xbpca0cIQ VtJiUUvplSkkteIY45Zezzs9A5tanXb4pszIfb4I0so6CLvMMH6dSNRRN 4sNIrKkfP4jVBCM67Af7a0XgXryWJzhEmbQPY1QTKhlJfagf6YydxHiIv SK1G+T5rCyywJSfPvFHJAJ3vP35PsRA4CsKeyKRWZSbAGDC2pVA/xSH0j emkbx/CSCTgzw290jGZ+0kdk+emrXn3Z3p1UgooKsQep4xl1ruHS6dRI3 pAXE8L3bcczsohJ6CuntmoegGHKwAmZ5jctVfE68bBjif5mYON4+klAPJ Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10442"; a="318666885" X-IronPort-AV: E=Sophos;i="5.93,245,1654585200"; d="scan'208";a="318666885" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Aug 2022 19:38:21 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.93,245,1654585200"; d="scan'208";a="636632537" Received: from yilunxu-optiplex-7050.sh.intel.com (HELO localhost) ([10.239.159.165]) by orsmga008.jf.intel.com with ESMTP; 17 Aug 2022 19:38:17 -0700 Date: Thu, 18 Aug 2022 10:29:11 +0800 From: Xu Yilun To: "Manne, Nava kishore" Cc: Nava kishore Manne , "michal.simek@xilinx.com" , "hao.wu@intel.com" , "trix@redhat.com" , "mdf@kernel.org" , "gregkh@linuxfoundation.org" , "ronak.jain@xilinx.com" , "rajan.vaja@xilinx.com" , "abhyuday.godhasara@xilinx.com" , "piyush.mehta@xilinx.com" , "harsha.harsha@xilinx.com" , "lakshmi.sai.krishna.potthuri@xilinx.com" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-fpga@vger.kernel.org" , "git@xilinx.com" Subject: Re: [PATCH v2 1/3] fpga: manager: change status api prototype, don't use older Message-ID: References: <20220621092833.1057408-1-nava.manne@xilinx.com> <20220621092833.1057408-2-nava.manne@xilinx.com> <20220628083158.GA2442728@yilunxu-OptiPlex-7050> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220817_193826_203468_B1D62A7F X-CRM114-Status: GOOD ( 25.76 ) 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: , 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 2022-08-17 at 11:16:14 +0000, Manne, Nava kishore wrote: > Hi Yilun, > > Please find my response inline. > > > -----Original Message----- > > From: Xu Yilun > > Sent: Tuesday, June 28, 2022 2:02 PM > > To: Nava kishore Manne > > Cc: michal.simek@xilinx.com; hao.wu@intel.com; trix@redhat.com; > > mdf@kernel.org; gregkh@linuxfoundation.org; ronak.jain@xilinx.com; > > rajan.vaja@xilinx.com; abhyuday.godhasara@xilinx.com; > > piyush.mehta@xilinx.com; harsha.harsha@xilinx.com; > > lakshmi.sai.krishna.potthuri@xilinx.com; linux-arm- > > kernel@lists.infradead.org; linux-kernel@vger.kernel.org; linux- > > fpga@vger.kernel.org; git@xilinx.com > > Subject: Re: [PATCH v2 1/3] fpga: manager: change status api prototype, > > don't use older > > > > On Tue, Jun 21, 2022 at 02:58:31PM +0530, Nava kishore Manne wrote: > > > Different vendors have different error sets defined by Hardware. > > > If we always define the new bits when we cannot find an exact 1:1 > > > mapping in the core the 64 bits would soon be used out. Also, it's > > > hard to understand the mixture of different error sets. > > > > > > To address these issues updated the status interface to handle the > > > vendor-specific messages in a generic way. With the updated status > > > interface the vendor-specific driver files can independently handle > > > the error messages. > > > > I think we don't have to provide the vendor specific HW errors in a generic > > way, maybe the vendor specific drivers could handle them by its own device > > attributes. > > > > Since the output value set of the interface is specific to each driver, users > > should still interpret them in specific manners. So doesn't see much value for > > a class interface. > > > > Agree, vendor specific drivers could handle them by its own device attributes. > If it is the case, can we remove the existing status interface relevant changes from the core? We don't have to. Some *user* interfaces may become hard to use as time goes, but we still try to make them stable as there are exsiting users working on them once they are released. That also means we should be more careful when introducing new user interfaces. Thanks, Yilun > > Regards, > Navakishore _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel