diff mbox series

[v1,1/1] usb: fotg210: Use *-y instead of *-objs in Makefile

Message ID 20240508150406.1378672-1-andriy.shevchenko@linux.intel.com
State New
Headers show
Series [v1,1/1] usb: fotg210: Use *-y instead of *-objs in Makefile | expand

Commit Message

Andy Shevchenko May 8, 2024, 3:04 p.m. UTC
*-objs suffix is reserved rather for (user-space) host programs while
usually *-y suffix is used for kernel drivers (although *-objs works
for that purpose for now).

Let's correct the old usages of *-objs in Makefiles.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/usb/fotg210/Makefile | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

Comments

Andy Shevchenko May 8, 2024, 3:06 p.m. UTC | #1
On Wed, May 08, 2024 at 06:04:06PM +0300, Andy Shevchenko wrote:
> *-objs suffix is reserved rather for (user-space) host programs while
> usually *-y suffix is used for kernel drivers (although *-objs works
> for that purpose for now).
> 
> Let's correct the old usages of *-objs in Makefiles.

Note, the original approach is weirdest from existing. Only a few
drivers use this (-objs-y) one most likely by mistake.
Linus Walleij May 27, 2024, 1:18 p.m. UTC | #2
On Wed, May 8, 2024 at 5:04 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:

> *-objs suffix is reserved rather for (user-space) host programs while
> usually *-y suffix is used for kernel drivers (although *-objs works
> for that purpose for now).
>
> Let's correct the old usages of *-objs in Makefiles.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

I haven't examined the claim in detail but I trust you so:
Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/drivers/usb/fotg210/Makefile b/drivers/usb/fotg210/Makefile
index 5aecff21f24b..8f5b0fb9b988 100644
--- a/drivers/usb/fotg210/Makefile
+++ b/drivers/usb/fotg210/Makefile
@@ -1,10 +1,6 @@ 
 # SPDX-License-Identifier: GPL-2.0
 
-# This setup links the different object files into one single
-# module so we don't have to EXPORT() a lot of internal symbols
-# or create unnecessary submodules.
-fotg210-objs-y				+= fotg210-core.o
-fotg210-objs-$(CONFIG_USB_FOTG210_HCD)	+= fotg210-hcd.o
-fotg210-objs-$(CONFIG_USB_FOTG210_UDC)	+= fotg210-udc.o
-fotg210-objs				:= $(fotg210-objs-y)
 obj-$(CONFIG_USB_FOTG210)		+= fotg210.o
+fotg210-y				:= fotg210-core.o
+fotg210-$(CONFIG_USB_FOTG210_HCD)	+= fotg210-hcd.o
+fotg210-$(CONFIG_USB_FOTG210_UDC)	+= fotg210-udc.o