RE: [PATCH Bluez v2 1/4] AVRCP: Register for VolumeChanged Notification

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

> >
> >  size_t avrcp_handle_vendor_reject(uint8_t *code, uint8_t *operands);
> > +void register_volume_notification ( struct avctp *session);
> > --
> > 1.7.5.4
> 
> I had to change it completely to make it work, Im afraid you never
> really tested this as it did not compile for me, were you using
> bootstrap-configure?
> 

The code is compiling perfectly at my end and I have tested it several times with PTS before pushing it. I follow below steps to compile my code - 

 autoreconf --install
./configure --prefix=/usr --mandir=/usr/share/man --sysconfdir=/etc --localstatedir=/var --libexecdir=/lib
 make
 make install

I also compiled and tested my code using bootstrap-configure by following below steps.

./bootstrap
./bootstrap-configure
./configure --prefix=/usr --mandir=/usr/share/man --sysconfdir=/etc --localstatedir=/var --libexecdir=/lib
make
make install

Had I known that you were facing compiling issues with my code earlier, I would have published the procedure how I have been compiling the code.
It would be really nice if you could point any issues with my procedure of compiling & testing the code, so that I can adapt to the correct procedure and follow for any future updates. 

> Anyway, I send a new series that should take care of subscribing to
> VolumeChanged event which should be working with Sony MW600.
> 
> Now the only part missing is to send SetAbsoluteVolume when the volume
> has changed on our side.

> --
> Luiz Augusto von Dentz

Thanks & Regards,
Vani Patel
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux