Invalid config of device mrf12: Property "port" is not a valid property

[TehEbil]

Hello,
just copied your text to device section and I keep getting the stated error.

Port is "/dev/ttyUSB1", but changing it to ttyUSB1, USB1 and 1 doesn't change anything, what am I doing wrong?

Edit:
I can edit the following under devices in the config file as the following:

    {
      "id": "mrf24",
      "name": "mrf24",
      "class": "MilightRF24",
      "zones": [
        {
          "addr": "test",
          "port": "/dev/ttyUSB1",
          "zone": 0
        }
      ]
    },

but I neither can edit it with the pimatic GUI nor it seems to work (getting undefined port).

[mwittig]

@Xento can you please have a look at this?

[mwittig]

@TehEbil As far as I can see you have misplaced the port property which should be one the same level as zones. See the example I have copied from README EDIT: Actually, the example given in README does not match the current device schema.

[TehEbil]

Thank you for your effort & testing.

If you do that this way, pimatic automatically changes it to port being in zone and you get the following error when restarting pimatic:
Error loading device "mrf24": Invalid port specified: undefined

Btw. what is the plugin doing? Does it only send x, r for enabling henryks listening mode (for debugging example), or can you set channels, too? Because for me, henryk isn't working somehow, do not receive anything. But with nRF24 Scanner, I can detect my remote.

[Xento]

In my opinion it's the best to define the port outside the zones because my version of henryks sketch accepts as many zones / remotes as you like.
So its a kind of gateway and one port definition for all zones is enough because you only need one arduino for everything.

My version can receive the commands from the remotes and sets the state of the devices in pimatic.
And of course you can control your lights over pimatic.

[TehEbil]

Hello,
I think I tried yours before, too.

It gave me an compiler error because packet[] is declared locally in the if but is then used outside (see l.43 -> l.59ff), so I tried to move it before the second if but then I got spammed with ID:1001 Group 6 (or second time: ID:0002 Group 6) without pushing any button.

And doesn't one have to change
static const uint8_t CHANNELS[] = {9, 40, 71};
? Because idk, mine worked on channel 5, and I've seen henryk increased those numbers by two somewhere, so it'd be 11, 42, 73.

[Xento]

Mine version just build fine with this RF24 lib: https://github.com/TMRh20/RF24
9, 40, 71 are the correct channels.
Which bulbs do you have? This version only works for RGBW bulbs.
Group 6 can't be correct. You can only have 4 groups!

[TehEbil]

I dont get it, how is that even possible that it works for you? Look at the following:

      if (receiving) {
            if (mlr.available()) {
                  printf("\n");
                  uint8_t packet[7];
                  size_t packet_length = sizeof(packet);
                  mlr.read(packet, packet_length);

                  for (int i = 0; i < packet_length; i++) {
                        printf("%02X ", packet[i]);
                  }
            }

            int dupesReceived = mlr.dupesReceived();
            for (; dupesPrinted < dupesReceived; dupesPrinted++) {
            printf(".");
      }
    
      printf("\n");
      // Here, packet is clearly not defined, lol, it's defined in an if
      printf("ID %02X%02X;", packet[1], packet[2]);

I am just wondering why this would work for you, am I doing something wrong?

Yeah, I know, it's strange. I think it's because different packets are received and because of byte shifting, the outcome are some wrong groups, because I don't use RGBW bulbs but got a RGBW Lamp instead. So it's not possible to control it right now? I thought that this protocol is for any milight devices >.<

[Xento]

Ahh, I had multiple versions of my repo.
I think I fixed the bug.

[TehEbil]

Oh I see, now it makes sense, lol.
Unfortunately, I don't have any output. it fails at mlr.available() -> _pl1167.receive(CHANNELS[0])

btw.: do you also get these warnings?

sketch\MiLightRadio.cpp: In member function 'bool MiLightRadio::available()':

sketch\MiLightRadio.cpp:10:50: warning: left shift count >= width of type

 #define PACKET_ID(packet) ( ((packet[1] & 0xF0)<<24) | (packet[2]<<16) | (packet[3]<<8) | (packet[7]) )

                                                  ^

sketch\MiLightRadio.cpp:75:28: note: in expansion of macro 'PACKET_ID'

       uint32_t packet_id = PACKET_ID(_packet);

I played a bit with that code, replaced channels with looping from i=0 to i = 9 and finally I at least get some output:

Packet received: 6D6D6D6D6D6D6D6D6D6D6D6D (unreversed)
Packet received: B6B6B6B6B6B6B6B6B6B6B6B6 (reversed)
Sync 3h fail (0: B6)

But I don't know what that B6 means, I only know that my remote / lamp communicate over a channel between 0 and 9 (I think 5, 6, 7 and 8 are all possible). Do you guys know what else I can try?

And btw. sorry for spamming that thread

[Xento]

No I don't get this warnings.
Is this lamp from milight?
If not I think you have to completely reverse engineer the protocol.
Milight has some new lights with a different protocol.

[TehEbil]

I actually think it is, at least it looks like one:

click me for photo

Well, so there's still hope that the warnings are the reason for wrong input. Do you know why I get those, if I cloned (downloaded zip) your project, lol?

Which Arduino IDE are you using, or do you compile via linux, maybe another gcc version or something x.x

I also wonder why it is int packet[7] and on the other hand, on the makro define, there is packet[7] (although packet only goes from 0 - 6):

#define PACKET_ID(packet) ( ((packet[1] & 0xF0)<<24) | (packet[2]<<16) | (packet[3]<<8) | (packet[7]) )

As far as I know, ID consists of packet[0] first nib, packet[1] and packet[2], it's kinda confusing for me why 1) he uses 1, 2 & 3 within an usual array and 2) why he xor's it with the last byte (sequence one).

[Xento]

Hmm this lamp should work.
The remote looks like mine.
I'm using Arduino IDE 1.6.11
I don't know what henryk is doing there ;)

[TehEbil]

hmm, should try it too with 1.6 then, using 1.8.1 right now.

I used another syncword and different channel now from here:
http://arduino-projects4u.com/milight-rf-control/
the CW / WW one (although mine is RGBW)

and now I get the following output:

ID 5122;Group 7;All on;Short press;
68 51 21 18 0F 82 01 ....
ID 5121;Group 7;All off;Short press;
68 51 22 18 0F 8E 01 ..................................................................................
ID 5122;Group 7;Brightnes 15;Short press;
68 51 21 18 0F 94 01 ..........................................
ID 5121;Group 7;Group 1 off;Long press;

But unfortunately, it's not what I really pressed on the remote, but for now the one which gives me the least errors (or at least seem to work, lol). And unfortunately, only the M button and the slider work, the other 1-0 buttons work if I e.g. first press in 1 of group one and then 0 of group 2 and so one, rgb slider doesnt work at all x.x

Could you please give me your (hex)code for turning white on?
Maybe I can take my id and replace yours, if I am lucky at least that'll work.