Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 093e5392 authored by Sarah Chin's avatar Sarah Chin Committed by Android (Google) Code Review
Browse files

Merge "Make PCC indication logic consistent between HIDL and AIDL" into tm-dev

parents 49afccb1 71af19fe
Loading
Loading
Loading
Loading
+12 −22
Original line number Diff line number Diff line
@@ -23,16 +23,12 @@ import android.os.Parcel;
import android.os.Parcelable;
import android.telephony.Annotation.NetworkType;

import com.android.telephony.Rlog;

import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.util.Arrays;
import java.util.Objects;

public final class PhysicalChannelConfig implements Parcelable {
    static final String TAG = "PhysicalChannelConfig";

    // TODO(b/72993578) consolidate these enums in a central location.
    /** @hide */
    @Retention(RetentionPolicy.SOURCE)
@@ -571,21 +567,19 @@ public final class PhysicalChannelConfig implements Parcelable {

        public @NonNull Builder setNetworkType(@NetworkType int networkType) {
            if (!TelephonyManager.isNetworkTypeValid(networkType)) {
                Rlog.e(TAG, "Builder.setNetworkType: Network type " + networkType + " is invalid.");
            } else {
                mNetworkType = networkType;
                throw new IllegalArgumentException("Network type " + networkType + " is invalid.");
            }
            mNetworkType = networkType;
            return this;
        }

        public @NonNull Builder setFrequencyRange(int frequencyRange) {
            if (!ServiceState.isFrequencyRangeValid(frequencyRange)
                    && frequencyRange != ServiceState.FREQUENCY_RANGE_UNKNOWN) {
                Rlog.e(TAG, "Builder.setFrequencyRange: Frequency range " + frequencyRange
                throw new IllegalArgumentException("Frequency range " + frequencyRange
                        + " is invalid.");
            } else {
                mFrequencyRange = frequencyRange;
            }
            mFrequencyRange = frequencyRange;
            return this;
        }

@@ -601,21 +595,19 @@ public final class PhysicalChannelConfig implements Parcelable {

        public @NonNull Builder setCellBandwidthDownlinkKhz(int cellBandwidthDownlinkKhz) {
            if (cellBandwidthDownlinkKhz < CELL_BANDWIDTH_UNKNOWN) {
                Rlog.e(TAG, "Builder.setCellBandwidthDownlinkKhz: Cell downlink bandwidth(kHz) "
                throw new IllegalArgumentException("Cell downlink bandwidth(kHz) "
                        + cellBandwidthDownlinkKhz + " is invalid.");
            } else {
                mCellBandwidthDownlinkKhz = cellBandwidthDownlinkKhz;
            }
            mCellBandwidthDownlinkKhz = cellBandwidthDownlinkKhz;
            return this;
        }

        public @NonNull Builder setCellBandwidthUplinkKhz(int cellBandwidthUplinkKhz) {
            if (cellBandwidthUplinkKhz < CELL_BANDWIDTH_UNKNOWN) {
                Rlog.e(TAG, "Builder.setCellBandwidthUplinkKhz: Cell uplink bandwidth(kHz) "
                throw new IllegalArgumentException("Cell uplink bandwidth(kHz) "
                        + cellBandwidthUplinkKhz + " is invalid.");
            } else {
                mCellBandwidthUplinkKhz = cellBandwidthUplinkKhz;
            }
            mCellBandwidthUplinkKhz = cellBandwidthUplinkKhz;
            return this;
        }

@@ -632,20 +624,18 @@ public final class PhysicalChannelConfig implements Parcelable {

        public @NonNull Builder setPhysicalCellId(int physicalCellId) {
            if (physicalCellId > PHYSICAL_CELL_ID_MAXIMUM_VALUE) {
                Rlog.e(TAG, "Builder.setPhysicalCellId: Physical cell ID " + physicalCellId
                throw new IllegalArgumentException("Physical cell ID " + physicalCellId
                        + " is over limit.");
            } else {
                mPhysicalCellId = physicalCellId;
            }
            mPhysicalCellId = physicalCellId;
            return this;
        }

        public @NonNull Builder setBand(int band) {
            if (band <= BAND_UNKNOWN) {
                Rlog.e(TAG, "Builder.setBand: Band " + band + " is invalid.");
            } else {
                mBand = band;
                throw new IllegalArgumentException("Band " + band + " is invalid.");
            }
            mBand = band;
            return this;
        }
    }