[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <36b382da-dc1c-4aeb-add0-a96082ea71d9@suswa.mountain>
Date: Fri, 27 Jun 2025 22:45:36 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: Abdalla Ahmad <Abdalla.Ahmad@...ame.org.jo>
Cc: Andy Shevchenko <andriy.shevchenko@...el.com>,
Lars-Peter Clausen <lars@...afoo.de>,
Michael Hennerich <Michael.Hennerich@...log.com>,
Jonathan Cameron <jic23@...nel.org>,
David Lechner <dlechner@...libre.com>,
Nuno Sá <nuno.sa@...log.com>,
Andy Shevchenko <andy@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Gabriel Shahrouzi <gshahrouzi@...il.com>,
open list <linux-kernel@...r.kernel.org>,
"open list:IIO SUBSYSTEM AND DRIVERS" <linux-iio@...r.kernel.org>,
"open list:STAGING SUBSYSTEM" <linux-staging@...ts.linux.dev>
Subject: Re: [PATCH] drivers: staging: iio: frequency: ad9832.h: Fixed TODO
note.
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
Powered by blists - more mailing lists