On 28/02/2022 21:06, Bjorn Helgaas wrote: > On Sun, Feb 27, 2022 at 02:52:08PM +0100, Krzysztof Kozlowski wrote: >> Use a helper for seting driver_override to reduce amount of duplicated >> code. Make the driver_override field const char, because it is not >> modified by the core and it matches other subsystems. > > s/seting/setting/ > or even better, s/for seting/to set/ > >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx> > > Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > >> - char *driver_override; /* Driver name to force a match */ >> + /* >> + * Driver name to force a match. >> + * Do not set directly, because core frees it. >> + * Use driver_set_override() to set or clear it. > > Wrap this comment to fill 78 columns or so. Thanks, I'll fix both. Best regards, Krzysztof
