| Message ID | 20240222-mhi-const-bus-mhi-host-pci_generic-v1-1-d4c9b0b0a7a5@quicinc.com |
|---|---|
| State | Accepted |
| Commit | 2ec11b5d6d900b17c7d001fbee4751fd2aa58917 |
| Headers | show |
| Series | bus: mhi: host: pci_generic: constify modem_telit_fn980_hw_v1_config | expand |
On Thu, Feb 22, 2024 at 06:00:23PM -0800, Jeff Johnson wrote: > MHI expects the controller configs to be const, and all of the other ones > in this file already are, so constify modem_telit_fn980_hw_v1_config. > > Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> - Mani > --- > drivers/bus/mhi/host/pci_generic.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/bus/mhi/host/pci_generic.c b/drivers/bus/mhi/host/pci_generic.c > index cd6cd14b3d29..51639bfcfec7 100644 > --- a/drivers/bus/mhi/host/pci_generic.c > +++ b/drivers/bus/mhi/host/pci_generic.c > @@ -538,7 +538,7 @@ static struct mhi_event_config mhi_telit_fn980_hw_v1_events[] = { > MHI_EVENT_CONFIG_HW_DATA(2, 2048, 101) > }; > > -static struct mhi_controller_config modem_telit_fn980_hw_v1_config = { > +static const struct mhi_controller_config modem_telit_fn980_hw_v1_config = { > .max_channels = 128, > .timeout_ms = 20000, > .num_channels = ARRAY_SIZE(mhi_telit_fn980_hw_v1_channels), > > --- > base-commit: 3ab6aff5793c3c7bdf6535d9b0024544a4abbdd5 > change-id: 20240222-mhi-const-bus-mhi-host-pci_generic-5ddf8f13d35c >
On Thu, Feb 22, 2024 at 06:00:23PM -0800, Jeff Johnson wrote: > MHI expects the controller configs to be const, and all of the other ones > in this file already are, so constify modem_telit_fn980_hw_v1_config. > > Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> Applied to mhi-next! - Mani > --- > drivers/bus/mhi/host/pci_generic.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/bus/mhi/host/pci_generic.c b/drivers/bus/mhi/host/pci_generic.c > index cd6cd14b3d29..51639bfcfec7 100644 > --- a/drivers/bus/mhi/host/pci_generic.c > +++ b/drivers/bus/mhi/host/pci_generic.c > @@ -538,7 +538,7 @@ static struct mhi_event_config mhi_telit_fn980_hw_v1_events[] = { > MHI_EVENT_CONFIG_HW_DATA(2, 2048, 101) > }; > > -static struct mhi_controller_config modem_telit_fn980_hw_v1_config = { > +static const struct mhi_controller_config modem_telit_fn980_hw_v1_config = { > .max_channels = 128, > .timeout_ms = 20000, > .num_channels = ARRAY_SIZE(mhi_telit_fn980_hw_v1_channels), > > --- > base-commit: 3ab6aff5793c3c7bdf6535d9b0024544a4abbdd5 > change-id: 20240222-mhi-const-bus-mhi-host-pci_generic-5ddf8f13d35c >
diff --git a/drivers/bus/mhi/host/pci_generic.c b/drivers/bus/mhi/host/pci_generic.c index cd6cd14b3d29..51639bfcfec7 100644 --- a/drivers/bus/mhi/host/pci_generic.c +++ b/drivers/bus/mhi/host/pci_generic.c @@ -538,7 +538,7 @@ static struct mhi_event_config mhi_telit_fn980_hw_v1_events[] = { MHI_EVENT_CONFIG_HW_DATA(2, 2048, 101) }; -static struct mhi_controller_config modem_telit_fn980_hw_v1_config = { +static const struct mhi_controller_config modem_telit_fn980_hw_v1_config = { .max_channels = 128, .timeout_ms = 20000, .num_channels = ARRAY_SIZE(mhi_telit_fn980_hw_v1_channels),
MHI expects the controller configs to be const, and all of the other ones in this file already are, so constify modem_telit_fn980_hw_v1_config. Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com> --- drivers/bus/mhi/host/pci_generic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- base-commit: 3ab6aff5793c3c7bdf6535d9b0024544a4abbdd5 change-id: 20240222-mhi-const-bus-mhi-host-pci_generic-5ddf8f13d35c