[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20230731074300.6200-1-wangweidong.a@awinic.com>
Date: Mon, 31 Jul 2023 15:43:00 +0800
From: wangweidong.a@...nic.com
To: krzysztof.kozlowski@...aro.org
Cc: 13916275206@....com, alsa-devel@...a-project.org,
broonie@...nel.org, ckeepax@...nsource.cirrus.com,
colin.i.king@...il.com, conor+dt@...nel.org,
devicetree@...r.kernel.org, doug@...morgal.com, fido_max@...ox.ru,
herve.codina@...tlin.com, krzysztof.kozlowski+dt@...aro.org,
lgirdwood@...il.com, linux-kernel@...r.kernel.org,
liweilei@...nic.com, perex@...ex.cz, povik+lin@...ebit.org,
rf@...nsource.cirrus.com, robh+dt@...nel.org, ryans.lee@...log.com,
shumingf@...ltek.com, tiwai@...e.com, trix@...hat.com,
wangweidong.a@...nic.com, yijiangtao@...nic.com,
zhangjianming@...nic.com
Subject: Re: [PATCH V3 4/5] ASoC: codecs: aw88261 device related operation functions
Thank you very much for your advice
On 31/07/2023 08:51, krzysztof.kozlowski@...aro.org wrote:
> On 31/07/2023 08:41, wangweidong.a@...nic.com wrote:
>>
>>>> +
>>>> + return 0;
>>>> +}
>>>> +
>>>> +int aw88261_dev_init(struct aw88261_device *aw_dev, struct aw_container *aw_cfg)
>>
>>> You already used this function in patch #3, so your order of patches is
>>> confusing.
>>
>> Do I need to change the order of patch?
>> Do I neeed to put aw88261_device.c aw88261_device.h in patch #3 and
>> put aw88261.c aw88261.h in patch #4?
>> Is that how you change the order?
> Your patchset should be logically ordered, so first you add bindings
> (because it must be before their users), then you one piece, then other
> etc. I understand that only the last patch will make everything
> buildable, but still code should be added before its user/caller.
Thank you very much for your suggestion.
Do I need to keep the Makefile and kconfig files in a separate patch?
...
>>
>>>> +
>>>> + switch (chip_id) {
>>>> + case AW88261_CHIP_ID:
>>>> + ret = aw_dev_init((*aw_dev));
>>>> + break;
>>>> + default:
>>>> + ret = -EINVAL;
>>>> + dev_err((*aw_dev)->dev, "unsupported device");
>>>> + break;
>>>> + }
>>>> +
>>>> + return ret;
>>>> +}
>>>> +
>>>> +MODULE_DESCRIPTION("AW88261 device");
>>>> +MODULE_LICENSE("GPL v2");
>>
>>> Wait, is this a module? Does not look complete. I already saw one
>>> module, so what is this for? For which module?
>>
>> Can it be changed to MODULE_DESCRIPTION("AW88261 device lib")?
> If this is a module, then it can. If this is not a module, then why
> would you ever like to do it?
>> The function in the aw88261_device.c file, which I used in the aw88261.c file.
> Functions are not modules.
Thank you very much for your suggestion.
I will delete MODULE_DESCRIPTION and MODULE_LICENSE
Best regards,
Weidong Wang
Powered by blists - more mailing lists