Re: [RFC PATCH 02/11] ARM: OMAP: expose control.h to mach area

* Valentin, Eduardo [off-list ref] [120528 03:34]:

Hello,

On Mon, May 28, 2012 at 12:25 PM, Shilimkar, Santosh
[off-list ref] wrote:

quoted

On Fri, May 25, 2012 at 1:55 PM, Eduardo Valentin
[off-list ref] wrote:

quoted

This patch exposes the definitions under control.h to
drivers outside the machine code.

Signed-off-by: Eduardo Valentin <redacted>
---

After second thought, this complete header movement needs to avoided.
Drivers should not anyway include something like <mach/control.h>

May be split the control.h header file data into ..
- defines used by mach-omap2/* files which can remain in "control.h"
in existing location.

OK..

quoted

- common functions/defines used across drivers/*, mach-omap2/*,
plat-omap/*, should
go to include/linux/omap_control.h

Right..

quoted

- Driver specific defines like thermal, usb etc, should go to
respective drivers file.

Indeed.

quoted

What do you think ?

I think we are in line. And I believe I saw a similar comment by
Benoit in other email thread.

Having a better thinking of this, it makes sense to have the
definition specific to drivers in the driver scope only, as they are
going to be used only there anyway.

I will drop this patch off and update the remaining changes
accordingly (drop the change in control.h for thermal specific and
move it to omap_bandgap.h).

Yes good, we should not expose these to the other drivers, that will
lead into misuse of these defines from various other drivers that we
may not notice until it's too late.

Regards,

Tony