[{"id":3688666,"web_url":"http://patchwork.ozlabs.org/comment/3688666/","msgid":"<177826412637.8043.3507805822778132637.b4-ty@linux.intel.com>","list_archive_url":null,"date":"2026-05-08T18:15:26","subject":"Re: [PATCH v5 0/2] platform/x86: x86-android-tablets: use real\n firmware node references with intel drivers","submitter":{"id":83553,"url":"http://patchwork.ozlabs.org/api/people/83553/","name":"Ilpo Järvinen","email":"ilpo.jarvinen@linux.intel.com"},"content":"On Mon, 04 May 2026 12:58:55 +0200, Bartosz Golaszewski wrote:\n\n> First let me repeat the problem statement:\n> \n> ==\n> \n> GPIO software node lookup should rely exclusively on matching the\n> addresses of the referenced firmware nodes. I tried to enforce it with\n> commit e5d527be7e69 (\"gpio: swnode: don't use the swnode's name as the\n> key for GPIO lookup\") but it broke existing users who abuse the software\n> node mechanism by creating \"dummy\" software nodes named after the device\n> they want to get GPIOs from but never attaching them to the actual GPIO\n> devices. They rely on the current behavior of GPIOLIB where it will match\n> the label of the GPIO controller against the name of the software node\n> and does not require a true link.\n> \n> [...]\n\n\nThank you for your contribution, it has been applied to my local\nreview-ilpo-next branch. Note it will show up in the public\nplatform-drivers-x86/review-ilpo-next branch only once I've pushed my\nlocal branch there, which might take a while.\n\nThe list of commits applied:\n[1/2] ACPI: provide acpi_bus_find_device_by_name()\n commit: 083a59e05db9203c346e87d822c0c12a0f7a7c85\n[2/2] platform/x86: x86-android-tablets: enable fwnode matching of GPIO chips\n commit: 1448c2d2ca5cc7f4ea6694e6bc809946de0a751c\n\n--\n i.","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-gpio@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256\n header.s=Intel header.b=ZoVtHq3L;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c0a:e001:db::12fc:5321; helo=sea.lore.kernel.org;\n envelope-from=linux-gpio+bounces-36476-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=\"ZoVtHq3L\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=198.175.65.10","smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=linux.intel.com"],"Received":["from sea.lore.kernel.org (sea.lore.kernel.org\n [IPv6:2600:3c0a:e001:db::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4gBy466cw6z1yJq\n\tfor ; Sat, 09 May 2026 04:16:22 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id AD88B3034A88\n\tfor ; Fri, 8 May 2026 18:15:41 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id E6FBC37BE71;\n\tFri, 8 May 2026 18:15:39 +0000 (UTC)","from mgamail.intel.com (mgamail.intel.com [198.175.65.10])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 7612837AA7A;\n\tFri, 8 May 2026 18:15:38 +0000 (UTC)","from fmviesa006.fm.intel.com ([10.60.135.146])\n by orvoesa102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 08 May 2026 11:15:38 -0700","from ijarvine-mobl1.ger.corp.intel.com (HELO localhost)\n ([10.245.244.100])\n by fmviesa006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 08 May 2026 11:15:31 -0700"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1778264139; cv=none;\n b=UlTtCndByExgJpesYfWAsIF3GqaS5antQK0HCk/MuZhbVDQIsO7S9X/xwINUytF8oDm92Q4WReWo7ktJjnasleawCZIh7Fd4kEsq/hxYxEDjaZ6fChiqpxOUN/b3tu2JYZIinkMsYcC8qQ4d/0frMkoXzPQf3WZFHS8alEnxivE=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1778264139; c=relaxed/simple;\n\tbh=VXdNdUVDlNnrZMEfP58I/g5Qk2zen5k91cmmz+Eb7dU=;\n\th=From:To:Cc:In-Reply-To:References:Subject:Message-Id:Date:\n\t MIME-Version:Content-Type;\n b=oWbusas8Yvf6OFJD46ArPpF87s88vel775OnfpxQTqRcf2N5vV4hAgFrU8BrhGt5cw2lhkGYgRl1RcajvKwZL/MnXroP/cmsbmkemMBzsHPNquhJxegtyMQTI6EmzRf4FcCytSahDmvWzK53NDQZZNfxHnWCC2pfgksblxLOSxg=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com;\n spf=pass smtp.mailfrom=linux.intel.com;\n dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=ZoVtHq3L; arc=none smtp.client-ip=198.175.65.10","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1778264138; x=1809800138;\n h=from:to:cc:in-reply-to:references:subject:message-id:\n date:mime-version:content-transfer-encoding;\n bh=VXdNdUVDlNnrZMEfP58I/g5Qk2zen5k91cmmz+Eb7dU=;\n b=ZoVtHq3LVX7dacbIK1MgIgBBAFv1PNlBGb3o4xsiVwCISbFTJaED6rBL\n xnU/h5k7NJTatOF7FxrCvBGcWwviRMAIzCFvseEBKMpLjXTOGacmoNi2v\n lpD7IBdNTWqdQV2JngB6bvogTVRe1Q2QCcz4fAaFHcyLtogeiKAN785r8\n Z+QHVOKk4/AZr63VeBsJ9uoNakW6GCT2YYXiJsHHz3YmlTKEh9EprpRTS\n 457fshUrP6mhNzBla5lgDU4PzP8ceidpPt+v3mRYK/M8x1XDcKvEaBmdt\n vT2REkvB9Uwr6VT0+m4/dZ/AQvgaq5BwKSlNUpKUeJPtxi748Qcx51SlQ\n g==;","X-CSE-ConnectionGUID":["x4EsYHAxR9GQO49WBglRUA==","vFA22ba1Qma2f4jms0kdEw=="],"X-CSE-MsgGUID":["zmcNmp++QdaWfDP3wtD7+w==","gUkmPBOORTG+RDk/G1VHvg=="],"X-IronPort-AV":["E=McAfee;i=\"6800,10657,11780\"; a=\"96665841\"","E=Sophos;i=\"6.23,224,1770624000\";\n d=\"scan'208\";a=\"96665841\"","E=Sophos;i=\"6.23,224,1770624000\";\n d=\"scan'208\";a=\"232320521\""],"X-ExtLoop1":"1","From":"=?utf-8?q?Ilpo_J=C3=A4rvinen?= ","To":"Andy Shevchenko ,\n Daniel Scally ,\n Heikki Krogerus ,\n Sakari Ailus ,\n Greg Kroah-Hartman ,\n \"Rafael J. Wysocki\" , Danilo Krummrich ,\n Mika Westerberg ,\n Andy Shevchenko , Linus Walleij ,\n Hans de Goede ,\n Dmitry Torokhov , Len Brown ,\n Bartosz Golaszewski ","Cc":"linux-acpi@vger.kernel.org, driver-core@lists.linux.dev,\n linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,\n platform-driver-x86@vger.kernel.org, brgl@kernel.org","In-Reply-To":"\n <20260504-baytrail-real-swnode-v5-0-c7878b69e383@oss.qualcomm.com>","References":"<20260504-baytrail-real-swnode-v5-0-c7878b69e383@oss.qualcomm.com>","Subject":"Re: [PATCH v5 0/2] platform/x86: x86-android-tablets: use real\n firmware node references with intel drivers","Message-Id":"<177826412637.8043.3507805822778132637.b4-ty@linux.intel.com>","Date":"Fri, 08 May 2026 21:15:26 +0300","Precedence":"bulk","X-Mailing-List":"linux-gpio@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"7bit","X-Mailer":"b4 0.13.0"}}]