Received: by 2002:a05:7412:2a8c:b0:e2:908c:2ebd with SMTP id u12csp1179152rdh; Mon, 25 Sep 2023 05:45:11 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHGz9qgvViCifE1Cvlb0zUSx/ebYB4KCCeE1pZe9XEAxfOJNHxYma9Gfcq6jdgDeJ68Ase+ X-Received: by 2002:a9d:73d0:0:b0:6be:e553:334 with SMTP id m16-20020a9d73d0000000b006bee5530334mr7458686otk.7.1695645911252; Mon, 25 Sep 2023 05:45:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1695645911; cv=none; d=google.com; s=arc-20160816; b=v/lGaZnl0Ven98pw6/LyiRYjWvzsqYYYFsP0ACdbTnslIz2W6v1+nS9UO2WrrfgCIA GMK3dlZ264+f8qj1DKXPKPR/mHlOk5gU/eH45cI1JAyf8UZpQWathie62bnSh7IJRfZJ kg4wPlWrj+6C0ftBxM+1CSNSzUzdmIibNg5cP3JU1g7FvJen5NF3BX/wk8fJLZkyWVZP t67pl8AYafYf0nfon3cWBKBkUWiVeBQbc2FNc5JlrSxbUS4ULcyQsFJZV4Vv2WB4LGRp cPRaZCaN2emLbjw3rr3uSofyWpfLVlN8VeMpA3lyCX/VNr09KiWpsKA2QBkJgtXfKvNM fi/A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:organization:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=F5gUu9kz44VlRg8RVqisaiUdClbe8sKMG4ZwRaT3wPM=; fh=rvrCdUXPNX/0NBHNqh3KQo35MUEnsBNzYZ9DEy8ERJ0=; b=MiYBVgXqixDYOTYMmbarj3AYJy60x37dK90E5/de7FhpTxrZMTSbb4PqDavE5iAZgA g5+CFMIgQiY3fZ8tdQapfwi3/CjWkE1qHTBhiCMhWbc5kBT9X2cAusppTO6yH6DES2pX dF+znCr7bb2eO3c30O2+JHqzvKxBx6Kkm7NzkDXoQU+TeH8UkP607PDGVkGl46kbpef4 xZd31V4VuUanKzd3Gj2gqPtmEzqBS92EzN5QfmhJStDTnjFBhnU32yQmNuIc9zE4DrUZ Dg6zBzcqfYy05OayOaZ9Zrjyh96JyBzPLhMZxreryU4eXm2PoSwJ0QzCY/X02QCQCdOX /MeA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=j9oiRVS4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from morse.vger.email (morse.vger.email. [2620:137:e000::3:1]) by mx.google.com with ESMTPS id bz33-20020a056a02062100b0056531bfc660si11166811pgb.143.2023.09.25.05.45.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Sep 2023 05:45:11 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) client-ip=2620:137:e000::3:1; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=j9oiRVS4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id 3D907807C744; Mon, 25 Sep 2023 00:21:37 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229836AbjIYHVe (ORCPT + 99 others); Mon, 25 Sep 2023 03:21:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58884 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229714AbjIYHVd (ORCPT ); Mon, 25 Sep 2023 03:21:33 -0400 Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.93]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B136C0; Mon, 25 Sep 2023 00:21:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1695626487; x=1727162487; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=iwm0ztNtqQCxIpKMtplcbblkEbTF1amu+3/nhNLIbIQ=; b=j9oiRVS41sZd4cG/paNUPhh4xsm6WjVz0xsace+w9WxcBus96bYjhQj1 R9N6IzBZbtZ1NafDUQlbA8Ir6ezToZ1sBjRTLwQFjwBDpXe3ON03+GsZL jMjAgZiBO9Rr+qDcU+U4+jTO3xUb3JnHAcikkMUjm1xI9eE9sNqPW9++a 8IF6DpGupK4WyS5tPyLjTq2zIWoT0FN0GJ+XeFdvBC6AcKrHjrnQbT9eg FYicMCaUaJOAMM+ehRyOTCrFiqsYHMxU3zbEVPz6YMzupXO19EI11lXSy DyUZdRodh7nlbDcXnTS8uf6Qt5HvYkyfeCDw/v9N/uWBpwBoHlpqYUHBu Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10843"; a="378455991" X-IronPort-AV: E=Sophos;i="6.03,174,1694761200"; d="scan'208";a="378455991" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Sep 2023 00:21:26 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10843"; a="813835171" X-IronPort-AV: E=Sophos;i="6.03,174,1694761200"; d="scan'208";a="813835171" Received: from smile.fi.intel.com ([10.237.72.54]) by fmsmga008.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Sep 2023 00:21:24 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.97-RC0) (envelope-from ) id 1qkfuE-00000000HGa-0Dhf; Mon, 25 Sep 2023 10:21:22 +0300 Date: Mon, 25 Sep 2023 10:21:21 +0300 From: Andy Shevchenko To: Biju Das Cc: Pavel Machek , Lee Jones , linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org, Biju Das Subject: Re: [PATCH v2 1/2] leds: pca955x: Convert enum->pointer for data in the match tables Message-ID: References: <20230923171921.53503-1-biju.das.jz@bp.renesas.com> <20230923171921.53503-2-biju.das.jz@bp.renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230923171921.53503-2-biju.das.jz@bp.renesas.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Mon, 25 Sep 2023 00:21:37 -0700 (PDT) On Sat, Sep 23, 2023 at 06:19:20PM +0100, Biju Das wrote: > Convert enum->pointer for data in the match tables, so that > device_get_match_data() can do match against OF/ACPI/I2C tables, once i2c > bus type match support added to it. > > Replace enum->struct *pca955x_chipdefs for data in the match table. > Simplify the probe() by replacing device_get_match_data() and ID lookup > for retrieving data by i2c_get_match_data(). > > While at it, add const definition to pca955x_chipdefs[]. ... > - struct pca955x_chipdef *chipdef; > + const struct pca955x_chipdef *chipdef; No need to preserve TAB(s) here. Otherwise looks good to me, Reviewed-by: Andy Shevchenko -- With Best Regards, Andy Shevchenko