[PATCH 0/1] UBUNTU: [Hardy SRU][SAUCE] Add signatures to airprime driver to support newer Novatel devices
Stefan Bader
stefan.bader at canonical.com
Mon May 4 13:24:48 UTC 2009
Tim Gardner wrote:
> Brad Figg wrote:
>> Stefan Bader wrote:
>>> Brad Figg wrote:
>>>> SRU Justification:
>>>>
>>>> Impact: Addition of new device signatures should have no impact to
>>>> existing
>>>> devices.
>>>> Fix: Simple addition of two device signatures to the device id table.
>>>>
>>>> Testcase: See bug report
>>>>
>>>> Please pull from :
>>>> git://kernel.ubuntu.com/bradf/ubuntu-hardy master
>>>>
>>>> Bug: #365291
>>>>
>>>> Brad Figg (1):
>>>> UBUNTU: [Hardy SRU] Add signatures to airprime driver to support
>>>> newer Novatel devices
>>>>
>>>> drivers/usb/serial/airprime.c | 2 ++
>>>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>>>
>>>>
>>> Are you positive that those should go to airprime? Most of the other
>>> Novatel are in option.c
>>>
>> I was following what was in the bug report which seemed to indicate it
>> worked for the submitter. I also looked at airprime.c and saw that
>> it had the signatures for the U760 and this is adding a newer U760.
>>
>> Brad
>>
>
> Stefan - I think it's correct to add new the USB IDs to
> drivers/usb/serial/airprime.c. After all, there is already an existing
> Novatel device supported by this driver, user tests indicate its a
> functional patch, and its not likely gonna cause rebase conflicts (this
> _is_ Hardy).
>
> rtg
Ok, if that works I don't want to argue over it. I just stumbled over it as I
had just checked airprime while looking at another bug and found it was removed
completely (post 2.6.24). Also the duplicate ID seems to have caused no trouble.
ACK and applied (with some header modification. Brad, for future patches, can
you have the subject lines in the following form? Thanks
UBUNTU: SAUCE: ...
Stefan
--
When all other means of communication fail, try words!
More information about the kernel-team
mailing list