This discussion has been locked.
You can no longer post new replies to this discussion. If you have a question you can start a new discussion

doubts using sfr

hi i wanted to create a genric function led_on(sfr) so that i can use it in at any part of the program as i wish .but i am not able to do it because when i pass the sfr the results are not as expected. i read that sfr cannot be indirectly addressed.what are the other methods to do so.
this is my code.

 #include "Main.H"


//macros

 #define LedOff (bit) 0
 #define LedOn (bit) 1


  //function prototype
   void Led_On(tByte);

   //special function registers
   sbit Port_Led = P1^7;


 void main()
{
 Led_On(Port_Led);
 while(1);
}

//function to turn on the led

        void Led_On(tByte LEDPORT)
        {
        LEDPORT =LedOn;
        }


how do i modify the code?please help

Parents
  • Maybe you could create a separate LED driver module and only use "logical" LED's in your code?

    pseudo:

    led.h:
    ------
    
    typdef enum { LED1, LED2, etc } tLEDs;
    void Led_On(tLEDs ledID, onOff);
    
    led.c:
    ------
    define SFR's here, not publicly available
    
    void Led_On(enum ledID, onOff)
    {
       switch (ledID)
       {
           use SFR's to switch each LED
       }
    }
    
    

    This way your application does not have to know anything about the underlying hardware and you don't have to pass SFR's. All hardware related things are in the driver. Makes it easier to modify the driver without having to change anything in the rest of the app.

    Regards,
    Joost Leeuwesteijn

Reply
  • Maybe you could create a separate LED driver module and only use "logical" LED's in your code?

    pseudo:

    led.h:
    ------
    
    typdef enum { LED1, LED2, etc } tLEDs;
    void Led_On(tLEDs ledID, onOff);
    
    led.c:
    ------
    define SFR's here, not publicly available
    
    void Led_On(enum ledID, onOff)
    {
       switch (ledID)
       {
           use SFR's to switch each LED
       }
    }
    
    

    This way your application does not have to know anything about the underlying hardware and you don't have to pass SFR's. All hardware related things are in the driver. Makes it easier to modify the driver without having to change anything in the rest of the app.

    Regards,
    Joost Leeuwesteijn

Children
No data