Re: [PATCH 2/7] Subscribes for USB resources for TI DM644x EVM platform.

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

 



Hello.

Subbrathnam, Swaminathan wrote:

Sergei,
	usb.c contains definitions which are more applicable to DA8xx platform and is not shared in any way with other DaVinci platform devices (DMxxxx).

So what? This is due to Kevin's decision to have everything in the single directory.

That is the reason I had created usb_davinci.h file.  The functionality in this file contains represents the capability of DMxxx devices as compared to DA8xx.  I would like to recommend that the usb.h file be changed to usb_da8xx.h in this way there will be clear representation.

I'm totally against creating 2 separate files for DA8xx and DMxxx but let's hear what Kevin says.

Regards
swami

-----Original Message-----
From: Sergei Shtylyov [mailto:sshtylyov@xxxxxxxxxxxxx] Sent: Wednesday, October 07, 2009 11:11 PM
To: Subbrathnam, Swaminathan
Cc: davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx
Subject: Re: [PATCH 2/7] Subscribes for USB resources for TI DM644x EVM platform.

Hello.

Swaminathan S wrote:

This patch implements the support for USB VBUS, PHY control, DM644x USB memory
mapsi,

   Maps?

IRQ, USB Mode.  It initializes for a single instance of MUSB controller.

Signed-off-by: Swaminathan S <swami.iyer@xxxxxx>
[...]
diff --git a/arch/arm/mach-davinci/include/mach/usb_davinci.h b/arch/arm/mach-davinci/include/mach/usb_davinci.h
new file mode 100644
index 0000000..02f8af7
--- /dev/null
+++ b/arch/arm/mach-davinci/include/mach/usb_davinci.h

Please don't create yet another header where <mach/usb.h> was intended to be used...

@@ -0,0 +1,41 @@
+/*
+ * This file contains the processor specific USB definitions
+ * of the TI DaVinci platforms.
+ *
+ * Copyright (C) 2009 Texas Instruments.

WBR, Sergei



--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux