Re: [PATCH] drivers: staging: iio: frequency: ad9832.h: Fixed TODO note.
From: Dan Carpenter
Date: Fri Jun 27 2025 - 15:45:49 EST
On Fri, Jun 27, 2025 at 07:29:36PM +0000, Abdalla Ahmad wrote:
> Hi
>
> > Nothing of the above explains "why you are doing this".
>
> The original TODO in drivers/staging/iio/frequency/ad9832.h was:
> > TODO: struct ad9832_platform_data needs to go into include/linux/iio
> So I guess if it really needs to go into include/linux/iio and ad9832 being a DAC, then include/linux/iio/dac/ is the appropriate place. Otherwise, the TODO note needs to be removed.
>
Please, change your email client to line wrap at 74 characters.
The way you quoted Andy's email it weird as well. You'll want to
configure your email client to do it properly.
regards,
dan carpenter