On Thu, Feb 20, 2025 at 12:26:25PM +0100, Cezary Rojewski wrote: > There shall be no comma after the terminator entry and initializing > fields with 0 for statically defined structs is redundant. ... > + { } > + {} I think you want a bit of consistency in the style of the terminator. Preferable (by majority) is with space inside, but some are okay without it. I have no preferences, just one style for all in the same driver. -- With Best Regards, Andy Shevchenko
- Follow-Ups:
- Re: [PATCH 06/24] ASoC: codecs: rt1308: Fix definition of device_id tables
- From: Cezary Rojewski
- Re: [PATCH 06/24] ASoC: codecs: rt1308: Fix definition of device_id tables
- References:
- [PATCH 00/24] ASoC: codecs: Update device_id tables for Realtek drivers
- From: Cezary Rojewski
- [PATCH 06/24] ASoC: codecs: rt1308: Fix definition of device_id tables
- From: Cezary Rojewski
- [PATCH 00/24] ASoC: codecs: Update device_id tables for Realtek drivers
- Prev by Date: Re: [PATCH 00/24] ASoC: codecs: Update device_id tables for Realtek drivers
- Next by Date: Re: [PATCH 00/24] ASoC: codecs: Update device_id tables for Realtek drivers
- Previous by thread: [PATCH 06/24] ASoC: codecs: rt1308: Fix definition of device_id tables
- Next by thread: Re: [PATCH 06/24] ASoC: codecs: rt1308: Fix definition of device_id tables
- Index(es):
![]() |