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:   Wed, 19 Jul 2023 12:54:03 +0200
From:   Henning Schild <henning.schild@siemens.com>
To:     Lee Jones <lee@kernel.org>
Cc:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Hans de Goede <hdegoede@redhat.com>,
        Wim Van Sebroeck <wim@linux-watchdog.org>,
        Guenter Roeck <linux@roeck-us.net>,
        linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org,
        platform-driver-x86@vger.kernel.org,
        linux-watchdog@vger.kernel.org, Pavel Machek <pavel@ucw.cz>,
        Mark Gross <markgross@kernel.org>,
        Tobias Schaffner <tobias.schaffner@siemens.com>
Subject: Re: [PATCH 2/3] leds: simatic-ipc-leds: default config switch to
 platform switch
Message-ID: <20230719125403.61ce47e2@md1za8fc.ad001.siemens.net>
In-Reply-To: <20230719084301.GG1082701@google.com>
References: <20230718105213.1275-1-henning.schild@siemens.com>
        <20230718105213.1275-3-henning.schild@siemens.com>
        <ZLaf0Cn2kfb0qxJJ@smile.fi.intel.com>
        <20230718164308.6b7d875a@md1za8fc.ad001.siemens.net>
        <20230719084301.GG1082701@google.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?o4Kn1UUVpBnnEqnIbatkOsN+CACnBfvsj98tGru5P69UHXLyL4ewtNEYqN4x?=
 =?us-ascii?Q?6Wp2IGbR2C2CtwieMWWjjM2z8OcNADQvdVlo+eQnT8N5Tx9A0+1CVokR6WME?=
 =?us-ascii?Q?5GnkkwPfyFUKNL+4QI3vaDURM2dN+iZOV4H8MFZMqfkpGXPAe4fSBWsNg34+?=
 =?us-ascii?Q?zURYBBGgsdzft1pMHJ3GowQZ5OfVRy/ZLK7U6bnFRsYlLWVynAnGA0iMKNSm?=
 =?us-ascii?Q?kUAer+zXvLvp8ajb5bmTzj6LD20QbOtRraVNngZ5SUyihALvESAam6g8KC4Q?=
 =?us-ascii?Q?ufscF2W6xWc/olXWNg0erLbhtH8EnLJQPT/zWXf7aQN3LAUFUtcgX3n3pgr0?=
 =?us-ascii?Q?0rJJcDhplfyHU9hb722k+Ko/s6hjT0M5FCtnmL26nBySZjF5aGU1E4e4aunJ?=
 =?us-ascii?Q?kA3DCCfmiH1lUKZKn4G0majkjV1EBr9NOVjIXLdfGY0RTVQ+8Koa1Ki5vkSc?=
 =?us-ascii?Q?rfXFfONKjVnpZqP6NDzn48HzoZxJa3fKVEaYv+sBfUredaGfxPBHq+QaOAxM?=
 =?us-ascii?Q?8HukABxI++Elv+Vq7mzbdFPY7rEl9zHEVxsgca0Cpz64YwuqQWKFbmGNmItm?=
 =?us-ascii?Q?ild33z9KDXbvmJPHGFUPCXw2ugcxw5Hf1C6gTs+Iskond6baBWCiKtEjcUDI?=
 =?us-ascii?Q?r5kjMuINfXU5QN13LNF16QFeDS898j3IXdFYftQIrij2CE5Wp+fE6WQX4LHV?=
 =?us-ascii?Q?q3dWtK9OhpMrsDeoNe6D0kh5ewtWL4kb2Da9uWHddGFUUcXagtC2Pir+AmdV?=
 =?us-ascii?Q?7ngoXcPTyMK5d3CpW2ksPdbP4HzNHKtkhM21xvHb6jtGvySRNT1CeR6bI7zY?=
 =?us-ascii?Q?IyIWhWt6VOChPPCUXWlq+OYOKaFBudJUfTeqBGOy+JVJmpRqRfTnElDuRKDo?=
 =?us-ascii?Q?dA5ufJYQn1248sD4+ZZHlq8t+WDSYhALeY4CQhjA4IQF1Wm2w1FQ6LhAzZ0/?=
 =?us-ascii?Q?eq2fPXqRqd1WW5rvDnWfIaMm7d7CUnPCDftBxuMiyVl2XIQCYWVg48qHM14g?=
 =?us-ascii?Q?Ta1aB0jqTQoE5tX+tqDj/r1YCc5YjkdtZpJ/SgzRmY00uWK9lebjQTifxRBv?=
 =?us-ascii?Q?zpxsJkjD+1UP0a4qOx5961dIt6EGlIPuLPrL4+JsxvwUMgp4OFstjn9Z3m4v?=
 =?us-ascii?Q?3bnHS1pcxhHd9uSsMyP9NibCSa4kqnzMUN9MzkDusY4/iYZWkB0HQoxYXM2A?=
 =?us-ascii?Q?cdjIanLvIkmP4dbXLfpkHS1vPLjKpDLwW2qrT3TsO2URjSgICpM8NXyszidK?=
 =?us-ascii?Q?gINTHBqtVTHTykr9muK47fF906XCCO6Mack8lqVmcLwcpYJs/6PT+M7NYutk?=
 =?us-ascii?Q?wXR4Tjn9RVgROAb/k9oJnRl3S9Ldvc9L2T2wGdbrh+IpcD3LpVvMrSH5fGr1?=
 =?us-ascii?Q?i/aUnsgHXhudmU+WUaEDsWsj8zIzOz41lszdWApA4dvieThM8CkDKZDBktzi?=
 =?us-ascii?Q?eqPsoNl8q4Imgpm7dlZRhf1hdtyv3qnIDplxO+PJj+4fh501SqqKfL1WoFNr?=
 =?us-ascii?Q?SZFGqCBHPaE1+zhapO+gPxGVduP8WQFD1oxymiwJkLMfCttUhKm5LVIVRQGB?=
 =?us-ascii?Q?jBJjM0rA9/+0YvuAY1MDcNOWHLWNXvFgkBw+7HI3IOo9y2g7RFW73sky/auX?=
 =?us-ascii?Q?MmavqZ84Ci12+f8sTragZyYhOB0I3fhztch4QXD7ils9RzwEoTAAQmms554N?=
 =?us-ascii?Q?JszJyg=3D=3D?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 315be135-9610-4606-2b73-08db8846795c
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5780.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jul 2023 10:54:07.0696
 (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: JYUaFFQMCmssE2epWZidB5JN5e9BpM9tHL3X7le4DZu1X4FF+/s7QBTWt/xqYXNUacg13753gxrnhkzx9KBxDFincRB5e15Z5tbR4am5yk0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR10MB5618
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_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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 Wed, 19 Jul 2023 09:43:01 +0100
schrieb Lee Jones <lee@kernel.org>:

> On Tue, 18 Jul 2023, Henning Schild wrote:
> 
> > Am Tue, 18 Jul 2023 17:21:04 +0300
> > schrieb Andy Shevchenko <andriy.shevchenko@linux.intel.com>:
> >   
> > > On Tue, Jul 18, 2023 at 12:52:12PM +0200, Henning Schild wrote:  
> > > > If a user did choose to enable Siemens Simatic platform support
> > > > they likely want the LED drivers to be enabled without having
> > > > to flip more config switches. So we make the LED drivers config
> > > > switch default to the platform driver switches value.    
> > > 
> > > Same as per previous patch.
> > >   
> > 
> > Same as per previous patch.  
> 
> Same as per previous patch.
> 

I will assume that you are asking me to "default y", as discussed on
that other patch. And will send a v2 accordingly.

In case the ordering of default, depends etc matters to you, please say
so.

Henning