[coreboot-gerrit] Change in coreboot[master]: src/driver/wifi: Add _DSM method for DDRRFIM

[Varshit B Pandya (Code Review)]

Attention is currently required from: Subrata Banik, Maulik V Vaghela, Tim Wawrzynczak, Paul Menzel, Sugnan Prabhu S.
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61020 )

Change subject: src/driver/wifi: Add _DSM method for DDRRFIM
......................................................................

Patch Set 19:

(5 comments)

File src/drivers/wifi/generic/acpi.c:

https://review.coreboot.org/c/coreboot/+/61020/comment/36d34608_6b060812 
PS18, Line 40: //bool is_cnviDdrRfim_enabled = false;
> delete this

Done

https://review.coreboot.org/c/coreboot/+/61020/comment/6aa2ee6b_a21a8ce4 
PS18, Line 160: bool *is_cnvi_ddr_rfim_enabled = false;
> This is creating a pointer that points to the value `false`, presumably 0...  […]

Done

https://review.coreboot.org/c/coreboot/+/61020/comment/b2a9086e_b5fc4616 
PS18, Line 542: bool *is_cnviDdrRfim_enabled = false;
              : 	*is_cnviDdrRfim_enabled = config->enable_cnvi_ddr_rfim;
> This would look cleaner if we just use a regular bool here, e.g. […]

Done

https://review.coreboot.org/c/coreboot/+/61020/comment/9a257114_094a8707 
PS18, Line 606: if (*is_cnviDdrRfim_enabled)
> then […]

Done

https://review.coreboot.org/c/coreboot/+/61020/comment/a7c2bc4e_82e711f0 
PS18, Line 610: dsm_ids[dsm_count].arg = (void *)is_cnviDdrRfim_enabled;
> `dsm_ids[dsm_count]. […]

Done