Nicholas Krause wrote:

This changes the return type of the function,shift_bits to unsigned long in order to avoid overflowing issues with this function when called as found when running sparse on the kernel source tree.

sparse is just a tool to help finding problems. The goal is not to do something to shut it up, but the remove the actual problem.

In this case, if the return type of shift_bits() were a problem, you would have just moved the problem to its caller where the value is then returned as unsigned int anyway.

Stop spamming us with patches for code that you do not understand.

Regards, Clemens

Signed-off-by: Nicholas Krause xerofoify@gmail.com

sound/pci/oxygen/xonar_dg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/pci/oxygen/xonar_dg.c b/sound/pci/oxygen/xonar_dg.c index 4cf3200..2337a76 100644 --- a/sound/pci/oxygen/xonar_dg.c +++ b/sound/pci/oxygen/xonar_dg.c @@ -238,7 +238,7 @@ void set_cs4245_adc_params(struct oxygen *chip, cs4245_write_spi(chip, CS4245_MCLK_FREQ); }

-static inline unsigned int shift_bits(unsigned int value, +static inline unsigned long shift_bits(unsigned int value, unsigned int shift_from, unsigned int shift_to, unsigned int mask)