Re: [PATCH ath-next 2/6] wifi: ath11k: Register debugfs for CFR configuration

[Yu Zhang(Yuriy)]

On 10/31/2025 10:43 AM, Baochen Qiang wrote:
> On 10/30/2025 12:31 PM, Yu Zhang(Yuriy) wrote:
>
> > @@ -88,6 +275,7 @@ void ath11k_cfr_deinit(struct ath11k_base *ab)
> >   			ar = ab->pdevs[i].ar;
> >   			cfr = &ar->cfr;
> >   
> > +			ath11k_cfr_debug_unregister(ar);
> >   			ath11k_cfr_ring_free(ar);
> >   
> >   			spin_lock_bh(&cfr->lut_lock);
> > @@ -140,6 +328,8 @@ int ath11k_cfr_init(struct ath11k_base *ab)
> >   		}
> >   
> >   		cfr->lut_num = num_lut_entries;
> > +
> > +		ath11k_cfr_debug_register(ar);
> >   	}
> >   
> >   	return 0;
>
> miss _debug_unregister() in the error handling?
The documention for debugfs_create_file() tells us:
 * NOTE: it's expected that most callers should _ignore_ the errors 
returned
 * by this function. Other debugfs functions handle the fact that the 
"dentry"
 * passed to them could be an error and they don't crash in that case.
 * Drivers should generally work fine even if debugfs fails to init anyway.

So IMO any failure to create the file should be ignored.