Re: Need help determining if the change is warranted.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This may be a resend. My first msg may be stuck in moderation,
because I sent w a new email addr.

On Mon, Mar 27, 2023 at 02:34:33PM +0300, Anton Gusev wrote:
> In the file drivers/leds/flash/leds-lm3601x.c, function lm3601x_strobe_set,
> the calls to regmap_update_bits aren't checked for errors.
> 
> I am unsure whether adding the checks is warranted, since
> lm3601x_read_faults might cover the conditions that can cause
> regmap_update_bits to fail there. On the other hand, if this is not true,
> then lm3601x_strobe_set can fail silently. Also, all other calls to
> regmap_update_bit in the driver are checked or directly returned.
> -- 
> Anton Gusev

Hi Anton, 

I don't see a patch posted for this, so I'll go ahead and respond.

It seems *something* can be improved here. Either examine those ret values
as they roll in, or stop assigning those ret values. Maybe, as you guess,
lm3601x_read_faults() is doing the needed checks. That's something you
could dig into further before posting the patch. Also, note the answer
might not be the same for all 3 of those calls.

Alison

> 
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies@xxxxxxxxxxxxxxxxx
> https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]

  Powered by Linux