Message-ID: <4a334087-9c72-2d86-e1d6-5d08ed5adc45@kontron.de>
Date:   Mon, 13 Feb 2023 17:23:33 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.2
Subject: Re: [PATCH 3/6] Revert "regulator: pca9450: Add SD_VSEL GPIO for
 LDO5"
Content-Language: en-US
To:     Marek Vasut <marex@denx.de>, Frieder Schrempf <frieder@fris.de>,
        Liam Girdwood <lgirdwood@gmail.com>,
        linux-kernel@vger.kernel.org, Mark Brown <broonie@kernel.org>
Cc:     Per-Daniel Olsson <perdo@axis.com>,
        Rickard x Andersson <rickaran@axis.com>,
        =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= <u.kleine-koenig@pengutronix.de>
References: <20230213155833.1644366-1-frieder@fris.de>
 <20230213155833.1644366-4-frieder@fris.de>
 <ee7d0f98-3ae9-7e66-785e-65c9e11a7232@denx.de>
From:   Frieder Schrempf <frieder.schrempf@kontron.de>
In-Reply-To: <ee7d0f98-3ae9-7e66-785e-65c9e11a7232@denx.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YUIwYk4yUGxYTTVUTDNVR0hrQW81RTJhS0Z6dXVFc29JTldSaStMNXhZeEJX?=
 =?utf-8?B?NXhYcWZRb2lJTnozM0Z1YXJFRVF0aXhRMUM1eFVUMjRrRFI0cEZZemJjUHdV?=
 =?utf-8?B?NkhMdHNYTERHYnFqSUJMMGZvSHJZSERJeDQyT05pTHdVeFNSWGZQazRyUmhk?=
 =?utf-8?B?TCtZVGU5czZuTFAvakNqbmdIRDVGY0VtNjVDTkdaZzVWS0VUak0wZHRLVlZE?=
 =?utf-8?B?MWVmWTZqTitSVVBGU25WNGxkY2xucUs3aVplUUpSMVdrRHNLbmpNNHBsNjZ3?=
 =?utf-8?B?NnI1SEc5dk5ld21hRTlwQ3F6WmdteHNZZ2tYcm0yTWppZWRCbGFRSUw2bnY2?=
 =?utf-8?B?eWFNZzdMUnBsL1hVTk43cmdlQWdLTXg3cXFTTTg1RHFLTDBVRzNROTdQbzFv?=
 =?utf-8?B?WGV6QzRyT0V6cUxPWlNRUEJId3FoUmFMM0J6angzaEFEb0FvRWtUS0F4UXRm?=
 =?utf-8?B?T3N2amNCYWFlWlFsUW1zUTY1RzB5NHg2QXUwTWhyQlYyTk5YWjVnZlJaNTRl?=
 =?utf-8?B?R1ZpL1Q2d1ZPWkRDRE9DaEtiQjlKZDNUNHdvaGxzbzFEUkttbDRha1lVVjR3?=
 =?utf-8?B?eDJIM2VPOEZ0NU12VGFTMmtWQVVOREEyOFVuUEsyWFZxR2VVNSszUWpiaTc1?=
 =?utf-8?B?c0VRc0V2M0cvQnpia0V3Zm1NOGpNYjE0eHFXb1BwS0Uwb0dyNS9iMVJlWUs1?=
 =?utf-8?B?aUZEVXk5YkpyTmF5NzlsNDRwcW9KUkhvWHBDMTE5SHUxN0h5T1VWekttakhy?=
 =?utf-8?B?cE5MN1V4eEhGT2ovNDJQZTdDY215VU5oYUNxcUlMS0FaUnZCN2Q3cklKOTNW?=
 =?utf-8?B?SnlWM0FMRHNmNHZ0RE9HanhWNnUrU0ptTHJ1d3VybGJ0VnlrUE1SaG81ZkRM?=
 =?utf-8?B?TlFITzFHcW9oN0paWGRqd0djVXJkZ09iOFBSYjIxQVhnSDV4V3JPazF5WFlB?=
 =?utf-8?B?VjBQNldBT25id1g2b3dIS1BITStURm1pK0NZdUhPNUNpcEg2bGh4dGQ1Z1dk?=
 =?utf-8?B?TklLakhYeTBQOVdkRDJTM29OeGVma1hQTytaWWw2UU1mMG1QK3Jmc1hWU1kw?=
 =?utf-8?B?SW5nSml5TEdHWDJaWk1qWmRWZzRyNzJDejAwN29BK0RlN0pmcFM4allFMW80?=
 =?utf-8?B?Z0RxZndZRUVzNGRvYkxPYk95L1lEMG1LVUlLRTc1cVlIZUNPOWRKdk9FQU9n?=
 =?utf-8?B?dlJCQ0RRZjJ2bWNMMExDT2pGUE5sYTdNb3piVk4rcEJvTnRQSFVOWlBRMHhD?=
 =?utf-8?B?dm1QQ05EYVRSZlhodUtMWGR2Q3dhNmlHSGN0VUNQUGpDWEJUd0Q2WEpxMEhm?=
 =?utf-8?B?WktyakhNZlNlTGdiaUNGM0drRmtXamFmMUg3ZEhWR0FwS0RmcWlMR3UrdUJB?=
 =?utf-8?B?OXhMbjl0cnVqbVhGNC9WSVdKM253aEpWMnFLeU41NmdDRGFPVDhsWEp6bWZH?=
 =?utf-8?B?SXFFNlBsVFBZYXpPbEVSb0VIRXZYSW96WkM1cGJTZWVHclRPZmowNEFjWjR0?=
 =?utf-8?B?ZjArK3NZWHZnM1N3LzgxalVWSm1SUjlFRUlYVGptcmg3UVJNNFEwOHI3WE5x?=
 =?utf-8?B?NFZrMDVXQkR1R25heUQraHpmR085bCszenl3SHNmU3ZLQklzUEhDV1FkdS9V?=
 =?utf-8?B?M1ViUURTZDAwV3JtYU9TRHcxeEpEZWU2UGF0bDE4eXVBZ29FRFpKQXdGbHVW?=
 =?utf-8?B?eEFrMjVVeHF0UW9kZzFQTkFXTmoxVHR2TFNXUjdza3AzbzZvVlhTYUtiNFZs?=
 =?utf-8?B?WmZkQUNZbW5pY0NpOW9NaXgxSThLdmNpcGxHcmVZak4zQ1ZGT0VhYmtKcy9G?=
 =?utf-8?B?Qzk2dzUzSmh5bzdxenZQckc4cTdtUzdoU2tQekZLclppSThCa1NvY1RISEpK?=
 =?utf-8?B?MjY2bjR2SlNnNXc1U0ZJNHpHaTNQYm5kUUxUYXRFbFkvUXhta0haVzV4bnFr?=
 =?utf-8?B?amRwL3diNGpJcHd5ckR2R2g5c1ZoUVVPdnk2WExBSGIzT1NVc3puZURTMG80?=
 =?utf-8?B?WnllQ0puMU1OUjZ4eFQycWRjcmw4RjBuSXVpaVlJSkdBaHZ1QURFcnBhdE1k?=
 =?utf-8?B?cGtTbzdtQXR4YkhBUjV4dzlxVG5DT0t3NHl3TlhhVUlucExxcUxDclJqSXpx?=
 =?utf-8?B?S0VsWExieXo2dVRKZEFVYlpOd2plYmhiNGZyTG5nRmFSVE1EMzhGWnRLSGI3?=
 =?utf-8?B?Y0E9PQ==?=
X-OriginatorOrg: kontron.de
X-MS-Exchange-CrossTenant-Network-Message-Id: 2e51dc8a-2c2b-47db-19c2-08db0ddea82d
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5681.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2023 16:23:35.9579
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 8c9d3c97-3fd9-41c8-a2b1-646f3942daf1
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: pEWoKpJOXWRL21TJouV1W2Q3hLjfsoeSG0KVBVUzztryQVA5vBF7FzDPyUdIhYiTeIeibAKPLrLlBDtws1gJ9v63YysHTiJJGu0KDD7BPYI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR10MB5572
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 13.02.23 17:16, Marek Vasut wrote:
> On 2/13/23 16:58, Frieder Schrempf wrote:
>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>
>> This reverts commit 8c67a11bae889f51fe5054364c3c789dfae3ad73.
>>
>> It turns out that all boards using the PCA9450 actually have the
>> SD_VSEL input conencted to the VSELECT signal of the SoCs SD/MMC
> 
> 'connected' , typo .

Thanks!

> btw would it make sense to squash 3..5 patches into a single patch ?

Hm, don't know. I think the changes are easier to understand with the
current separation between "revert", "core changes" and "pca9450 changes".