powerpc: Add EXPORT_SYMBOL_GPL for symbols required by fs_enet and cpm_uart

fs_enet and cpm_uart need symbols from commproc.c (for CPM1) or cpm2_common.c. Add EXPORT_SYMBOL_GPL for cpmp, cpm_setbrg and cpm2_immr, so the drivers can be compiled as modules.

Building modules, stage 2.
MODPOST 5 modules
ERROR: "cpm2_immr" [drivers/net/fs_enet/fs_enet.ko] undefined!
ERROR: "cpmp" [drivers/net/fs_enet/fs_enet.ko] undefined! make[1]: *** [__modpost] Error 1
make: *** [modules] Error 2

---
 arch/powerpc/sysdev/commproc.c    |    3 +++
 arch/powerpc/sysdev/cpm2_common.c |    3 +++
 2 files changed, 6 insertions(+), 0 deletions(-)

index f6a6378..ddbe138 100644
@@ -51,6 +51,8 @@ static void m8xx_cpm_dpinit(void);
 static uint host_buffer; /* One page of host buffer */
 static uint host_end;    /* end + 1 */
 cpm8xx_t __iomem *cpmp;  /* Pointer to comm processor space */
+EXPORT_SYMBOL_GPL(cpmp);
+
 immap_t __iomem *mpc8xx_immr;
 static cpic8xx_t __iomem *cpic_reg;
 

@@ -302,6 +304,7 @@ cpm_setbrg(uint brg, uint rate)
 		out_be32(bp, (((BRG_UART_CLK_DIV16 / rate) - 1) << 1) |
 		             CPM_BRG_EN | CPM_BRG_DIV16);
 }
+EXPORT_SYMBOL_GPL(cpm_setbrg);
 
 #ifndef CONFIG_PPC_CPM_NEW_BINDING
 /*

index 859362f..b878a67 100644
@@ -51,11 +51,13 @@ static void cpm2_dpinit(void);
 #endif
 
 cpm_cpm2_t __iomem *cpmp; /* Pointer to comm processor space */
+EXPORT_SYMBOL_GPL(cpmp);
 
 /* We allocate this here because it is used almost exclusively for
  * the communication processor devices.
  */
 cpm2_map_t __iomem *cpm2_immr;
+EXPORT_SYMBOL_GPL(cpm2_immr);
 
 #define CPM_MAP_SIZE	(0x40000)	/* 256k - the PQ3 reserve this amount
 					   of space for CPM as it is larger

@@ -117,6 +119,7 @@ cpm_setbrg(uint brg, uint rate)
 
 	cpm2_unmap(bp);
 }
+EXPORT_SYMBOL_GPL(cpm_setbrg);
 
 /* This function is used to set high speed synchronous baud rate
  * clocks.

1.5.3.4

Maybe this is a stupid question, but why did you choose EXPORT_SYMBOL_GPL and not EXPORT_SYMBOL?

Timur Tabi
Linux Kernel Developer @ Freescale

By marking all new exports EXPORT_SYMBOL_GPL it stops new closed source device drivers from being built. We have to live the the existing ones, but we certainly don't want to encourage any more to be built.

Over on lkml there is a thread about moving all symbols of this type into private name spaces and removing the exports in the final kernel binary.

Jon Smirl
[email protected]

As I told replying to prev mail, this needs to be addressed in respective driver(s), since there is a way to retrieve such pointers without making them global.

This patch will exist in maillist as a workaround, but meanwhile I'll take a look at this problem.

To prevent using those pointers from within non-GPL modules. kind of policy now...

Sincerely, Vitaly

As the original copyright holder of nearly all of this of this code, I do not wish this be done.

Thanks.

	-- Dan

In this particular case this is not going to happen anyway, but I will take your opinion into account for the similar cases.

Sincerely, Vitaly

Hi Vitaly,

In particular in this case, as these pointers are currently not exported, at all. They are currently used by a few drivers and will disappear again once these drivers have been converted to use the propper accessors, which will also prevent access conflicts to the CPM registers.

Thanks,
Jochen