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;
Message-ID: <b56cdcf1-459b-63a2-a060-49564e11ce9f@wolfvision.net>
Date:   Mon, 28 Nov 2022 15:26:51 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.0
Subject: Re: [PATCH v3 1/3] iio: add struct declarations for iio types
Content-Language: en-US
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc:     Gerald Loacker <gerald.loacker@wolfvision.net>,
        linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, Jonathan Cameron <jic23@kernel.org>,
        Lars-Peter Clausen <lars@metafoo.de>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Jakob Hauser <jahau@rocketmail.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Nikita Yushchenko <nikita.yoush@cogentembedded.com>
References: <20221125083526.2422900-1-gerald.loacker@wolfvision.net>
 <20221125083526.2422900-2-gerald.loacker@wolfvision.net>
 <Y4CcspD1xkmhmWbh@smile.fi.intel.com> <Y4CgiMd4XQMV4KFV@smile.fi.intel.com>
 <a55e73f7-4daf-6892-34dc-61c6f6581d8e@wolfvision.net>
 <Y4S3WnYWVnmiVFc+@smile.fi.intel.com>
 <4d1b0054-efd4-e10e-17a6-d236052afa49@wolfvision.net>
 <Y4TAF1hn0l1CziUh@smile.fi.intel.com>
From:   Michael Riesch <michael.riesch@wolfvision.net>
In-Reply-To: <Y4TAF1hn0l1CziUh@smile.fi.intel.com>
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?RlFuNG4xWjJidU5XNUIrSnZnb0lkbG5JNmNqdXhYeGdIWk5DZ0RueTFncjZa?=
 =?utf-8?B?RGRJK3BCVDF5TUVVWU52Q0JGS016TWdPQ2RHM2tjNHBXWEtzd3Z0YTRuMTRO?=
 =?utf-8?B?bllEQktYV0tKdko1a0R5a2R4LzlHQkUydVNtS0lMQUdnSERtaE8vUUdrMEtC?=
 =?utf-8?B?S3hua0Evcm9EbUt2YlJMZDdUNi9vbUR6UnRHK1VpREhkTW0rWnQwVFBaQ1JV?=
 =?utf-8?B?SFZZTXRMSU5BbDFmOUU2VmZJUXUyYUM5VHo2YUtRdXVWMHVvTUEvNG00djRi?=
 =?utf-8?B?eVNtNTk5aklWemN6Y1d1ZWwrYVZrMkN3NXVQUkVsQTZSTVhGVXBFejIvN05R?=
 =?utf-8?B?V29pbERrSXlhQ1FhQ2loUUtreDR2NkR2MUIxMmc4RE5UTDhWVDc3SFppOVFH?=
 =?utf-8?B?YS9GV2FTNWw0OStxbEw2bnZoNTRwTFpCaDlrMGFwSGloMEZJbzBQQWsvRGRE?=
 =?utf-8?B?cVJSNTE2SHRRSFkzSGwyVVBpUGJIU0owaEplZmdkaHhsQ1A0SmNqMjhGVzNF?=
 =?utf-8?B?blZOb2hneE5hM0tXeWJjNkVQdmNxd0Q2eUVGaWdDWUExb0ovMkhvTDFlc1Iz?=
 =?utf-8?B?K3BsTkVWbGh1RlVGZ25hUklVN0VmMEZGZm5EMlFiWnc4VWRwRUNXekVaQmhO?=
 =?utf-8?B?YU5YK3lpaVdSVkRRSVlLRmVIczFHd2dXZnI4YmtiM1Z6SGFzWWZXUWszMW5s?=
 =?utf-8?B?NkZMbG9sRW1GV3o1ZFNYanRoQW5ZZ1hqd0UxTm4wc1A5TG9KRUxZTUs1by9V?=
 =?utf-8?B?RDV3L2dKQko1dWIxbXdaa1ZGU3B0alVpeDRsbkF2Y3hOMUxQeGlJaTNvVmE3?=
 =?utf-8?B?MUQvWnEyM0VkbjJLdTVxWmwrZEwwRVNSSGVuTkI3a2VwOFE4eGVBZlpTclpY?=
 =?utf-8?B?TjYxTkJQRDZZSWFORktrdWxaZ1FLa2ZrcFNnNmswT3huS1BRMzVyWDgvaHll?=
 =?utf-8?B?UjEwSFExdzBGdkxtVElOKzMyUmRKcVhtZnBGWldUOHZYQlJKM0NGS28yQmNJ?=
 =?utf-8?B?djJKa0pZVW1mc2x3YlJkN3ZhR3dkMmZoOFRNVXY3QmZsWjhVdnRUWThXVEpD?=
 =?utf-8?B?UkF6RGZlRk42QjRHZUJBby8xTG9nOFdzdXNkVkN5ZUF5TGZMcnhBeDdHakJ1?=
 =?utf-8?B?RXlSUzRNS0dnY3dvc21EVkRWdkU0cDJueFF5V05oY0s1Tm9kKzN1Vlc4bTJh?=
 =?utf-8?B?aHJuRC9rR1hMS2ZLMmU0dk92TkdITkFmQ2YzM2JnbU42T0hKcXdWeUoxNThz?=
 =?utf-8?B?VlJCUjZHK3FIdjBxYlJGVUp2Y0RKekNsQ2drU3FJRXFvekx4b09QYWJKRW4w?=
 =?utf-8?B?NUFnQ21GQThSMFNoNWhSR0ZZY1JMdEVwK2NCTVQrOWxjQ1BaNzlDS0Jib2NC?=
 =?utf-8?B?SEdoaFRJbHdwL0tSOERubnBOMk5iMlhTdUd2aHd0UkpjcmdDWHdNM2trMXFY?=
 =?utf-8?B?dTMrM0k3V0o4elhqVU1ZVms2QjVmS0dPMW1ITU83TDRiSDdqRzB1Ry81TFRn?=
 =?utf-8?B?OXFpcUVvSE1qbjg4UDh3OHdLNjFVblVZR1Q1cmlITVF0SExCQWlPakJRM0x6?=
 =?utf-8?B?RUhPWUkvTDZYN3ZIMnpVWEhldUkrQ3oxUjVqeXpLN3V6QVJPY1Iwd0RuZlNH?=
 =?utf-8?B?eFBlc2ZBbzdyU0Rydms1N3A0SllpWHFxTHJrNk1mNlR2UlpyMlFEQmU5WmUr?=
 =?utf-8?B?elhmTm1oWVcwbzVEbGtRWC9vS1NyT29mT01vS3JqNVRHUjJHYUJUV3ZINDVX?=
 =?utf-8?B?NkJGU3VYRGgzRXdhS1l5U1NBYjZhNjZBTFlwZFI5THMwUHBFeitSeUNsbmJJ?=
 =?utf-8?B?aWFPQUlrZi9rMytyWXNUdFliemJCWjY4cFV5ckw1ejhGb2pDOEtPZXpwblln?=
 =?utf-8?B?OWhWU1NFdzRXV0ZGRldLVm54Q2F0Slpjd3JYZTNxQkVrVlVMMVpnWkUwY21r?=
 =?utf-8?B?bHdWUnBFdGtYbGRyL3dnMWM2Y1VxT0hUa3NtaTdpQXR1NXBkZWttc1UwWjkv?=
 =?utf-8?B?WXdsb1dyOWwxZjJEUG1GWFlZd3llSVVaTHhoWHJPUHVYbUxITHZaWU5qNDdo?=
 =?utf-8?B?Z3JFbktmYldtVGM3a2gzU0F6UHNESktoQVNES2JMQWJWajh4eng1M2U0M2J4?=
 =?utf-8?B?ZjRoUm5DczRMSG10UkhnUmFYNFR4Q0hwME8yTGtlQU9VUHFWVW9iaGx5cXQr?=
 =?utf-8?B?M2tIa0l0VjdkMmhCclhwMlNXd29oMjdwdjMrdWorcW95eGFyRk9BcUh2MlFu?=
 =?utf-8?B?RUt1cU5ZRUVjWC90bk9VUzNBQ2pRPT0=?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: fec0a83e-4070-419e-9954-08dad14c9866
X-MS-Exchange-CrossTenant-AuthSource: DU0PR08MB9155.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Nov 2022 14:26:53.2332
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: e94ec9da-9183-471e-83b3-51baa8eb804f
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: XRCgVyA5lqrinfIpFVNFDXIg/ME01I+ABVTtL88DwhHLlNbZWZDJCd9YJ+xr2NhtJwUUyPdWPq5DeGPSeNLbVxdCMhnN3e+B4l1WslAPSEg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR08MB7959
X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        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

Hi Andy,

On 11/28/22 15:05, Andy Shevchenko wrote:
> On Mon, Nov 28, 2022 at 02:48:48PM +0100, Michael Riesch wrote:
>> On 11/28/22 14:27, Andy Shevchenko wrote:
>>> On Mon, Nov 28, 2022 at 01:18:04PM +0100, Gerald Loacker wrote:
>>>> Am 25.11.2022 um 12:01 schrieb Andy Shevchenko:
> 
> ...
> 
>>> It's a rule to use _t for typedef:s in the kernel. That's why
>>> I suggested to leave struct definition and only typedef the same structures
>>> (existing) to new names (if needed).
>>
>> Andy, excuse our ignorance but we are not sure how this typedef approach
>> is supposed to look like...
>>
>>>> or
>>>
>>>> 	typedef iio_val_int_plus_micro iio_val_int_plus_micro_db;
>>
>> ... because
>>
>> #include <stdio.h>
>>
>> struct iio_val_int_plus_micro {
>> 	int integer;
>> 	int micro;
>> };
>>
>> typedef iio_val_int_plus_micro iio_val_int_plus_micro_db;
>>
>> int main()
>> {
>>   struct iio_val_int_plus_micro a = { .integer = 100, .micro = 10, };
>>   struct iio_val_int_plus_micro_db b = { .integer = 20, .micro = 10, };
>>   return 0;
>> }
>>
>> won't compile.
> 
> I see. Thanks for pointing this out.
> 
> Then the question is why do we need the two same structures with different
> names?

Most probably we don't need "struct iio_val_int_plus_micro_db" at all
since IIO_VAL_INT_PLUS_MICRO_DB and IIO_VAL_INT_PLUS_MICRO get the same
treatment in industrialio-core.c. At least it should not be introduced
in the scope of this series. In the end this is up to whoever writes the
first driver using the common data structures and IIO_VAL_INT_PLUS_MICRO_DB.

Best regards,
Michael