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;
Date:   Thu, 5 Jan 2023 12:23:57 +0100
From:   Henning Schild <henning.schild@siemens.com>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc:     Lee Jones <lee@kernel.org>, Pavel Machek <pavel@ucw.cz>,
        linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4] leds: simatic-ipc-leds-gpio: make sure we have the
 GPIO providing driver
Message-ID: <20230105122357.6711ad0c@md1za8fc.ad001.siemens.net>
In-Reply-To: <Y7aZ9Mmm16HVUnnu@smile.fi.intel.com>
References: <20221007153323.1326-1-henning.schild@siemens.com>
        <Y6WX1Y9GZmvxqlCc@google.com>
        <20230102162227.523d2a73@md1za8fc.ad001.siemens.net>
        <20230103212059.5c80fecb@md1za8fc.ad001.siemens.net>
        <Y7WMHl1Mv1alXadG@google.com>
        <20230104153924.0b92c52c@md1za8fc.ad001.siemens.net>
        <Y7WghcaWPpCHh6Wz@smile.fi.intel.com>
        <20230104203005.5654f3bc@md1za8fc.ad001.siemens.net>
        <Y7aZ9Mmm16HVUnnu@smile.fi.intel.com>
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?RtGiyx4q/AtzpM/62pjUVPLjTL57sbtbDSnyJJPZVVQgC8xU5q6/5v8IWey6?=
 =?us-ascii?Q?d3tKS8EIDXTIoJPzJ1I7C6cnfLlfELi++v7ocLCSPjkmyNSBrJ5bkKLweWtW?=
 =?us-ascii?Q?E/2LvOL3noJCDKATIOXdPvJ6/yvkRkps28tkl+wJNBUhX+ocdyIdZuyTS87q?=
 =?us-ascii?Q?VfsXGYqi1gtXJCvXNbqJaIVE6YkuIbl/a6zmCCpgweRe9qkpWHWY2UMnC9M0?=
 =?us-ascii?Q?N2VGePC8LQFmC3yGYGzSxbfez3uWwdXapK6uM3SD8ErzHeaRwWidiBs8Q8cR?=
 =?us-ascii?Q?kGj30lmZvv8bkMa/EYSWw7Q9a1C8TzXE78UkXohwhsUsxHVI7pt7BBfNNClu?=
 =?us-ascii?Q?VaGRn3cUUqFVACfUUZtssu1oDBesIrGb5V2zMLBPECeCHDvJya7uvelDNMfg?=
 =?us-ascii?Q?8DccfmNjpIWB+0BbbgVKrP8sA6OYhffPg3wsczSUKRnqq65XMfuVqz2+dKIm?=
 =?us-ascii?Q?tfOCWxzcX1JvZQ/Ne4z/6sgOU4cqEbxD/3ukfhdfnGIXYXIxgkiTDks2qWSX?=
 =?us-ascii?Q?RMr6/LZRmfwrKdSy8k4ReYyaRZ+UzQZJsgRaXGYoIi7H2PpGoXwM6oawAgJb?=
 =?us-ascii?Q?4JJmEN4DbmcDsoL7DxXBfBkSkSBwvnfHmX+bFqEcs8oFr6XLzUi+naR6Ffu8?=
 =?us-ascii?Q?HiSoZWLXAIi3HpdbjWjW/ndB+82V5fkmI1SvBMMEfha1oj4X+dD1W8Jw1DG4?=
 =?us-ascii?Q?mSAeD4Z43A9d/iWwVzADB0DDAk8GVUXM9nFNPz+G0gz/6JZiRgWNuVpYlFdy?=
 =?us-ascii?Q?Jgl1Tpqhp+bLYQsNdX9GQi9wt+ERRuk08WpDcFvFkqwdX3U0UxP2vgx/TcK6?=
 =?us-ascii?Q?c1VFwQ0wwkAy7bGaytoDn4wktZ/5YTv2hEFhVpeSZksQWAcR8nGbmE4C36dF?=
 =?us-ascii?Q?mCQYQKKHAl47W9UIKG2WzvJV2dVBSKHTEqkyYh1x0ljihUZD5MUkHJJAzI9L?=
 =?us-ascii?Q?86QwB0Mszlu5N7pese4i+HHZ77RCZsQ73EzoVqGtyf2Twd8ahGvYy6n5XYXC?=
 =?us-ascii?Q?twylFQVkfwqfNpa89W7pC7QjnrMIyU7jslHjEcDEwG4WFrmvTAtzlLENbAC2?=
 =?us-ascii?Q?S7e3sU3up0e5TlxbvWr8GMZ1N/95sixK4HZr+D9jhVwhh0xDkimig2CtFWr9?=
 =?us-ascii?Q?UmWW8Iy2Yiay3nAr6M+0NPe6zxwgsQcIJe8cVzzn7t+2S6K55ZZin1xr46Si?=
 =?us-ascii?Q?GhedWVE5oEH6gJl8DQvlYnwMhm+56i2FKqeodo9CN68Gw/65lVjCCixdZwvP?=
 =?us-ascii?Q?G5pejkLpTKyvStFZA9uoQW52bXfEcjgT6RVItgnESOknclypyENt3QJKMBXk?=
 =?us-ascii?Q?B1CaER+UZmvBqvl90Z/NK/k7Iop92rDdAWCmpZlg8dyLNeS8sDoB3KjQkWvD?=
 =?us-ascii?Q?yQZXpYYSz07w2QVUtRoUvfrWJNeIlOcIiJqWmWEDtQm436PLSqTXCgJ3pNYa?=
 =?us-ascii?Q?ugEorzmZmm/hBb/IpQxTgaF0Vn56kQJzbAC8K1MtmKyZPtZVgmXdWeZh0QFO?=
 =?us-ascii?Q?ClFdG14SIRz6eRNAyc+6hZRkCpW10kYJwoNbRvWJxsgL4UFmWNBVLwMwg++p?=
 =?us-ascii?Q?svBJFYc7i7dIQaJ5qJaxvaSuzYlsOIYTf2Mt94NooFFcKib1vbHKNSmepoxE?=
 =?us-ascii?Q?AtYKr3QbHw0KeetdG0N+sEYg2jbFjmUn2qeABqyvdS3jDI4AAaiw9DI7Eg6Y?=
 =?us-ascii?Q?ECCSbw=3D=3D?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 25ee246a-4158-4a46-a9a4-08daef0f58bb
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5780.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jan 2023 11:24:02.0775
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: OnRLx67uUwpJUjsV83+ZTy6CByzEFH3GInhucObmE9Ubd1XoK12SgMQ/RwEmI1MX2+1oLOhgMKVGhokWg5uItmkfXojryczGKEXl+MP5Yss=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV2PR10MB6041
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_MED,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE
        autolearn=no 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am Thu, 5 Jan 2023 11:35:48 +0200
schrieb Andy Shevchenko <andriy.shevchenko@linux.intel.com>:

> On Wed, Jan 04, 2023 at 08:30:05PM +0100, Henning Schild wrote:
> > Am Wed, 4 Jan 2023 17:51:33 +0200
> > schrieb Andy Shevchenko <andriy.shevchenko@linux.intel.com>:  
> > > On Wed, Jan 04, 2023 at 03:39:24PM +0100, Henning Schild wrote:  
> > > > Am Wed, 4 Jan 2023 14:24:30 +0000
> > > > schrieb Lee Jones <lee@kernel.org>:    
> 
> ...
> 
> > > > As we speak i already have the third box to eventually support,
> > > > which will likely be similar but this time around with
> > > > PINCTRL_ELKHARTLAKE    
> > > 
> > > A bit of offtopic here.
> > > 
> > > Are you able to get / fix / ... the firmware to work with the
> > > upstreamed version of pin control driver for Intel Elkhart Lake?
> > > 
> > > (I'm asking this in terms of the
> > > https://bugzilla.kernel.org/show_bug.cgi?id=213365)
> > >   
> > 
> > I can not tell. At the moment i am in a Siemens internal review
> > where i see code that is not even close to being ready for
> > upstream. Somewhat open-coded again from what it looks like.
> > 
> > And i do not have the machine the code is for.
> > 
> > Let me say "it is complicated" but some point in time a device with
> > LEDs attached to PINCTRL_ELKHARTLAKE will be proposed. Likely by me,
> > when i hopefully have such a device on my desk.  
> 
> Thanks for the information.
> 
> Consider above just as a point to be aware of when you come to
> the productization, so we won't need another pin control driver for
> the same chip.

IIRC we talked about this before in some other thread and the solution
was taking a newer BIOS base version. And since i never heard about
this again i hope people did the right thing.

Henning