Skip to content

Commit

Permalink
Staging: crystalhd: Get rid of unecessary BCMLOG_ENTER macro
Browse files Browse the repository at this point in the history
The BCMLOG_ENTER macro is used only in five functions, perhaps
it is remainder of debugging some specific problem,
now, this macro don't seems to be useful, so it should be removed.

Signed-off-by: Jorgyano Vieira <jorgyano@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Jorgyano Vieira authored and Greg Kroah-Hartman committed Feb 24, 2012
1 parent 15fd62b commit eb6cfa5
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 14 deletions.
3 changes: 0 additions & 3 deletions drivers/staging/crystalhd/crystalhd_hw.c
Original file line number Diff line number Diff line change
Expand Up @@ -1627,7 +1627,6 @@ enum BC_STATUS crystalhd_download_fw(struct crystalhd_adp *adp, void *buffer, ui
uint32_t fw_sig_len = 36;
uint32_t dram_offset = BC_FWIMG_ST_ADDR, sig_reg;

BCMLOG_ENTER;

if (!adp || !buffer || !sz) {
BCMLOG_ERR("Invalid Params.\n");
Expand Down Expand Up @@ -1724,8 +1723,6 @@ enum BC_STATUS crystalhd_do_fw_cmd(struct crystalhd_hw *hw,

crystalhd_create_event(&fw_cmd_event);

BCMLOG_ENTER;

if (!hw || !fw_cmd) {
BCMLOG_ERR("Invalid Arguments\n");
return BC_STS_INV_ARG;
Expand Down
4 changes: 0 additions & 4 deletions drivers/staging/crystalhd/crystalhd_lnx.c
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,6 @@ static int chd_dec_open(struct inode *in, struct file *fd)
enum BC_STATUS sts = BC_STS_SUCCESS;
struct crystalhd_user *uc = NULL;

BCMLOG_ENTER;
if (!adp) {
BCMLOG_ERR("Invalid adp\n");
return -EINVAL;
Expand Down Expand Up @@ -327,7 +326,6 @@ static int chd_dec_close(struct inode *in, struct file *fd)
struct crystalhd_adp *adp = chd_get_adp();
struct crystalhd_user *uc;

BCMLOG_ENTER;
if (!adp) {
BCMLOG_ERR("Invalid adp\n");
return -EINVAL;
Expand Down Expand Up @@ -513,8 +511,6 @@ static void __devexit chd_dec_pci_remove(struct pci_dev *pdev)
struct crystalhd_adp *pinfo;
enum BC_STATUS sts = BC_STS_SUCCESS;

BCMLOG_ENTER;

pinfo = pci_get_drvdata(pdev);
if (!pinfo) {
BCMLOG_ERR("could not get adp\n");
Expand Down
7 changes: 0 additions & 7 deletions drivers/staging/crystalhd/crystalhd_misc.h
Original file line number Diff line number Diff line change
Expand Up @@ -200,15 +200,8 @@ enum _chd_log_levels {
BCMLOG_INFO = 0x00000001, /* Generic informational */
BCMLOG_DBG = 0x00000002, /* First level Debug info */
BCMLOG_SSTEP = 0x00000004, /* Stepping information */
BCMLOG_ENTER_LEAVE = 0x00000008, /* stack tracking */
};

#define BCMLOG_ENTER \
do { \
if (g_linklog_level & BCMLOG_ENTER_LEAVE) \
printk(KERN_DEBUG "Entered %s\n", __func__); \
} while (0)


#define BCMLOG(trace, fmt, args...) \
do { \
Expand Down

0 comments on commit eb6cfa5

Please sign in to comment.